Получить данные из списка объектов в C #

Question

Я хочу получить идентификатор, имя и валюту из списка товаров: 1 - TID - 13,2

namespace Test
{
    class Product
    {
        private int id;
        private string name;
        private double currency;
        List<Product> products = new List<Product>();
        public Product(int id,string name,double currency)
        {
            this.id = id;
            this.name = name;
            this.currency = currency;
        }
        public void addProduct(Product product)
        {
            products.Add(product);
        }
        public void listOfProducts()
        {
            foreach (object o in products)
            {
                Console.WriteLine(o);
            }
        }
    }
    class Program
    {
        static void Main(string[] args)
        {
            Product p = new Product(1, "TID", 13.2);
            p.addProduct(p);
            p.listOfProducts();
        }
    }
}

но когда я выполняю этот код, я получаю Test.product Может кто-то сказал мне, что я должен получить данные из моего списка, а не имя списка

Answer 1

У вас есть некоторые программные проблемы, а не циклический просмотр списка продуктов.Вы должны понимать разницу между статическими и нестатическими типами.Статические методы принадлежат самому классу.Так что, когда вы пишете static перед объявлением, говорите, что эта переменная или метод для класса, а не для экземпляров.

Другая проблема с вашим кодом была, очевидно, для цикла.Вам следует перебирать тип Product, а не object, поскольку объект не имеет тех же членов, что и Product.

namespace Test
{
    class Product
    {
        static private List<Product> Products;

        private int id;
        private string name;
        private double currency;

        public Product(int id, string name, double currency)
        {
            this.id = id;
            this.name = name;
            this.currency = currency;
        }

        public static void AddProduct(Product product)
        {
            if (Products == null)
            {
                Products = new List<Product>();
            }
            Products.Add(product);
        }

        public static void GetListOfProducts()
        {
            foreach (Product product in Products)
            {
                Console.WriteLine(String.Format("id:{0} name:{1} currency:{2}", product.id, product.name, product.currency));
            }
        }
    }


    class Program
    {
        static void Main(string[] args)
        {
            Product p = new Product(1, "TID", 13.2);
            Product.AddProduct(p);
            Product.GetListOfProducts();
        }
    }
}

Answer 2

Я хотел показать вам некоторый код с лучшими практиками программирования C#.Некоторые из включенных функций:

• Неизменяемый объект Product со свойствами Quantity, Name и Currency.Валюта должна иметь тип decimal, если она предназначена для хранения значений типа валюты.Свойства имеют PascalCase соглашение об именах.
• Определите равенство при сравнении значений через .Equals(Product other) и используйте этот метод в .NET Framework путем реализации интерфейса IEquatable<Product>.
• Преобразование любогоProduct в string с помощью метода .ToString() и реализации обратного анализа текста в Product с помощью .Parse(text) статического метода
• Удалено List<Product> изнутри каждый экземпляр продукта, так как его рекомендуется определять локально относительно того, где он будет использоваться.
• Реализация GetHashCode() в соответствии с рекомендациями, когда Product используется в Dictionary collection.
• В конце у меня есть тестовый код, который тестирует весь этот код

Определение класса

public class Product : IEquatable<Product>
{
    public int ID { get; }
    public string Name { get; }
    public decimal Currency { get; }

    public Product(int id, string name, decimal currency)
    {
        this.ID= id;
        this.Name=name;
        this.Currency=currency;
    }

    /// <summary>
    /// Converts a Product into a string description of itself.
    /// </summary>
    /// <returns>
    /// A <see cref="string"/> of the form <![CDATA["123 - ABC - 3.21"]]> with quantity, name and currency.
    /// </returns>
    public override string ToString() => $"{ID} - {Name} - {Currency}";

    /// <summary>
    /// Parses text into a Product.
    /// </summary>
    /// <param name="description">The description of the Product. Expecting the description 
    /// to be of the form <![CDATA["123 - ABC - 3.21"]]> with quantity, name and currency.</param>
    /// <returns>A new Product or null</returns>
    public static Product Parse(string description)
    {
        string[] parts = description.Split('-');
        if(parts.Length==3)
        {
            if(int.TryParse(parts[0].Trim(), out int id))
            {
                string name = parts[1].Trim();
                if(decimal.TryParse(parts[2].Trim(), out decimal currency))
                {
                    return new Product(id, name, currency);
                }
            }
        }
        return null;
    }

    /// <summary>
    /// Determines whether the specified <see cref="System.Object" />, is equal to this instance.
    /// </summary>
    /// <param name="obj">The <see cref="System.Object" /> to compare with this instance.</param>
    public override bool Equals(object obj)
    {
        if(obj is Product product)
        {
            return Equals(product);
        }
        return false;
    }

    /// <summary>
    /// Indicates whether the current object is equal to another object of the same type.
    /// </summary>
    /// <param name="other">A Product to compare with this Product.</param>
    public bool Equals(Product other)
    {
        return other!=null
            && ID==other.ID
            && Name==other.Name
            && Currency==other.Currency;
    }

    public override int GetHashCode()
    {
        int hashCode = 1486216620;
        hashCode=hashCode*-1521134295+ID.GetHashCode();
        hashCode=hashCode*-1521134295+EqualityComparer<string>.Default.GetHashCode(Name);
        hashCode=hashCode*-1521134295+Currency.GetHashCode();
        return hashCode;
    }
}

Тестовый код

/// <summary>
/// Code for https://stackoverflow.com/q/53321654/380384
/// </summary>
class Program
{
    static void Main(string[] args)
    {
        var all = new List<Product>();
        all.Add(new Product(1, "TID", 13.2m));
        all.Add(new Product(2, "TJQ", 11.8m));
        all.Add(new Product(3, "UIZ", 15.7m));
        string description = "4 - UYA - 18.4";
        all.Add(Product.Parse(description));

        foreach(Product item in all)
        {
            Console.WriteLine(item);
        }
        //1 - TID - 13.2
        //2 - TJQ - 11.8
        //3 - UIZ - 15.7
        //4 - UYA - 18.4

        if(all[3].ToString().Equals(description))
        {
            Console.WriteLine(@"Product -> String is ok.");
        }
        if(Product.Parse(description).Equals(all[3]))
        {
            Console.Write(@"String -> Product is ok.");
        }
    }
}

Answer 3

Вы выводите сам объект, но если вы не переопределите ToString, он просто выводит имя типа. Вы также приводите Product к Object, поэтому вы не сможете использовать Product свойства. Вместо этого используйте:

foreach (Product p in products)
{
    Console.WriteLine($"{p.Id} - {p.Name} - {p.Currency}");
}

Я использовал (публичные) свойства, в настоящее время у вас есть только личные поля. Так что отредактируйте свой класс как здесь:

public class Product
{
    public int Id { get; set; }
    public string Name { get; set; }
    public double Currency { get; set; }
    List<Product> Products { get; set; }

    public Product(int id, string name, double currency)
    {
        this.Id = id;
        this.Name = name;
        this.Currency = currency;
        this.Products = new List<Product>();
    }
    // ...
}