Как реализовать вывод товаров из List у которых закончился срок годности?

Question

[neitoo]

Есть список товаров, для примера

List<Product> tovar = new List<Product>()
            {
                new Product(){id = 2,name = "Яблоки",dataproizv = "11.11.2021",cpisanie = "Свежие яблоки.",cena = 100,skidka = 5, kategorya = "Овощи"}
                new Product(){id = 2,name = "Молоко",dataproizv = "16.10.2021",cpisanie = "Свежее молоко.",cena = 80,skidka = 10, kategorya = "Кисломолочные"}
            };

Необходимо вывести товары у которых закончился срок годности (кисломолочным дано 3 месяца на хранение)
Я думал сделать через метод, но как в него передать значение из списка или весь список не понимаю.. (Если требуется скину метод)
Либо сделать перебор списка с условием, например:

for (int i = 0; i < tovar.Count; i++)
            {
                //Тут написать условие
                Console.WriteLine("ID " + tovar[i].id + ": \"" + tovar[i].name + "\"" +" | Категория: "+ tovar[i].kategorya + " | Дата производства: " + tovar[i].dataproizv + " | " + tovar[i].cena + " RUB | Скидка " + tovar[i].skidka + " %  " + "\nОписание товара: " + tovar[i].cpisanie);
            }

но опять же даты у меня хранятся в строках...
Может есть другие решения, или подскажите что сделать с данными вариантами?

Comments

[edward_freedom]

Я думал сделать через метод, но как в него передать значение из списка или весь список не понимаю

как обычный аргумент
У тебя есть дата производства, сколько хранится этот товар где узнать

[neitoo]

До этого я работал без списка, по этому в методе я просто прибавлял месяц к дате производства, а теперь надо сделать в списке это всё

public class Product
    {
        public int daays;
        public int mes;
        public int year;
        public int id;
        public string name;
        public string dataproizv;
        public string datagodn;
        public string cpisanie;
        public int cena;
        public int skidka;
        public string kategorya;


        public void GodenDo()
        {
            var dd = int.Parse(dataproizv.Replace(".", String.Empty));
            var dd2 = int.Parse(datagodn.Replace(".", String.Empty));
            int sum = dd + dd2;
            string sums = Convert.ToString(sum);
            Console.WriteLine($"Годен до: {sums[0]}{sums[1]}.{sums[2]}{sums[3]}.{sums[4]}{sums[5]}{sums[6]}{sums[7]}");

        }
    }

Метод конечно так себе

[Myclass]

Всё надо и хранить, и организовывать через базу данных. И продукты, и их сроки действия и актуальный статус. И показ в зависимости от статуса. Но, если исторические какие показы происходят, то да, в этом случае и неактивные элементы в списках показываются, но уже не для изменения а именно для того, как это было например год назад - с каким продуктом например был тот или иной ордер.

[neitoo]

Myclass, я только начал изучать c#, понятия не имею как работать с базами данных к сожалению

Answer 1

[Владимир Коротенко]

using System;
using System.Collections.Generic;
using System.Globalization;
using System.Linq;

namespace ConsoleApp2
{
    /// <summary>
    /// Не используйте транслит! Любой китаец или индус поймет английский, 
    /// а вот русский только поржет. 
    /// </summary>
    public class Product
    {
        public int Id { get; set; }
        public string Name { get; set; }
        public string ImportDate { get; set; }
        public string Description { get; set; }
        /// <summary>
        /// Рубли умноженные на 10000. помним об этом то есть у вас копейка имеет 2знака после запятой!!!!
        /// </summary>
        public Int64 Price { get; set; }
        public int Discount { get; set; }
        public string Category { get; set; }
        /// <summary>
        /// Дата списания
        /// </summary>
        public string DueDate { get; set; }
        public string ДляОсобоУпоротыхЭтоТакиРаботает { get; set; }
    }
    public static class Extensions
    {
        /// <summary>
        /// Convert string from format dd.MM.yyyy to DateTime
        /// </summary>
        /// <param name="inputDate"></param>
        /// <returns></returns>
        public static DateTime ParseDateDayMounthYear(this string inputDate) =>
            DateTime.ParseExact(inputDate, "dd.MM.yyyy", CultureInfo.InvariantCulture);
    }
    class Program
    {
        static void Main(string[] args)
        {
            #region заполняем список товаров
            List<Product> tovar = new List<Product>()
            {
                new Product(){Id = 2,
                    Name = "Яблоки",
                    ImportDate = "11.11.2022",
                    Description = "Свежие яблоки.",
                    Price = 1000000,
                    Discount = 5, 
                    DueDate="11.11.2022",
                    Category = "Овощи"},
                new Product(){Id = 2,Name = "Молоко",
                    ImportDate = "16.10.2021",
                    DueDate="16.10.2021",
                    Description = "Свежее молоко.",
                    Price = 800000,
                    Discount = 10, 
                    Category = "Кисломолочные"}
            };
            #endregion

            // число месяц год
            var data =  tovar.Where(item => item.DueDate.ParseDateDayMounthYear() < DateTime.Today).ToList();
        }
    }
}

Answer 2

[Сергей]

Сделайте дату датой, и потом такую выборку

tovar.Where(item=> (item.dataproizv.AddDays(21)) < DateTime.Today );

Comments

[neitoo]

а без DateTime возможно?

[Сергей]

А не, такое не совсем подойдёт, там же товары с разными сроками годности.

[edward_freedom]

neitoo, а зачем без него, с ним удобнее, та и логичнее

[Alexander]

neitoo, а почему не DateTime, т.к. это как минимум удобно.

Имхо, такое удобнее реализовывать через какую-то БД и sql запросы с условиями.

[neitoo]

edward_freedom, я понимаю что с ним удобнее, но до его изучения еще не дошел

[neitoo]

Alexander, edward_freedom, просто важно было понять возможно ли сделать через то что я написал, и главное как

[Сергей]

neitoo, с ним быстрее будет работать.

[neitoo]

werwolflg, я уже понял

[Alexander]

neitoo, можно, примерно как предложил werwolflg , только с более сложным условием. Нужно понять где и как хранится срок годности для каждого продукта или категории.

[neitoo]

Alexander, я показал код в комментарии к заданию как работал до списка, а теперь со списком уже так не получается, по этому где и как хранить срок годности не решил

[Сергей]

neitoo, а самым правильным способом будет зависти ещё одно поле expiration_date, которое заполнять при добавлении товара. И уже потом делать выборку по нему.

[Alexander]

neitoo, ну так а что мешает делать тоже самое в списке?

[Сергей]

neitoo, создаете ещё один класс Category, в нем поля id, title, expirate_date и при добавлении товара автоматом высчитываете дату до которой он годен и записываете в него.

[neitoo]

Alexander, то что в метод не понимаю как отправить элемент с датой производства, или весь список как в метод отправить

[Alexander]

neitoo, используйте этот класс, только GodenDo() переделайте, что бы он возвращал значение, либо дату до которого годен, а потом сравнивайте с тукущей, либо возвращал bool значение годности просчитывая его прямо в том методе.
А потом либо

tovar.Where(item=> item.GodenDo() < DateTime.Today);

либо
tovar.Where(item=> item.GodenDo() == false);

Думаю вам стоит детальнее разобраться как работать с List и Linq.

P.S. Не понял, почему два одинаково оформленных кода в теге отображаются по разному...