Vždy uloži pouze poslední hodnoty do seznamu (ArrayListu) a minulé se přepíšou (C# aplikace) – .NET – Fórum – Programujte.com

Mám za domácí úkol vytvořit obchod, který bude data ukládat do Seznamu (ArrayListu) a z něho také zobrazovat. Ale vždy to uloži do seznamu pouze poslední hodnoty a ty minulé se přepíšou.

Dle mého za to může řádek:

obchod = new Obchod(textBox1.Text, textBox2.Text, Convert.ToDouble(textBox3.Text), Convert.ToInt32(textBox4.Text));

is never assigned to, and will always have its default value null

        Obchod obchod;



        private void button1_Click(object sender, EventArgs e)

        {

             obchod = new Obchod(textBox1.Text, textBox2.Text, Convert.ToDouble(textBox3.Text), Convert.ToInt32(textBox4.Text));



            obchod.pridejZboziNaSklad(textBox1.Text, textBox2.Text, Convert.ToDouble(textBox3.Text), Convert.ToInt32(textBox4.Text));

        }

        



        private void button2_Click(object sender, EventArgs e)

        {

            richTextBox1.Text = "";

            richTextBox1.Text += "" + obchod.vypisSklad();

        }



    }



        class Zbozi



        {

            private string nazev, popis;

            private int mnozstvi;

            private double cena;



            public Zbozi(string nazev, string popis, double cena, int mnozstvi)

            {

                this.nazev = nazev;

                this.popis = popis;

                this.cena = cena;

                this.mnozstvi = mnozstvi;

            }



            public string vypisInfo(string nazev, string popis, double cena, int mnozstvi)

            {

                return string.Format(" Název produktu: {0} \n Popis Produktu: {1}\n Cena produktu: {2} Kč\n Množství produktu: {3} ks\n\n", nazev, popis, cena, mnozstvi);

            }

        }



        class Obchod

        {

            Zbozi zbozi;

            List<Zbozi> sklad = new List<Zbozi>();



            private string nazev, popis;

            private int mnozstvi, id;

            private double cena;//, castkaZProdeje;



            //Konstruktor pro pridani zbozi na sklad

            public Obchod(string nazev, string popis, double cena, int mnozstvi)

            {

                this.nazev = nazev;

                this.popis = popis;

                this.cena = cena;

                this.mnozstvi = mnozstvi;



                MessageBox.Show(" Produkt byl úspěšně přidán do databáze");

            }



            //Prida zbozi na sklad

            public void pridejZboziNaSklad(string nazev, string popis, double cena, int mnozstvi)

            {

                sklad.Add(zbozi = new Zbozi(nazev, popis, cena, mnozstvi));

            }



            //Vypise na obrazovku sklad

            string vystup;



            public string vypisSklad()

            {

                foreach (Zbozi go in sklad)

                {

                    vystup = string.Format("" + go.vypisInfo(nazev, popis, cena, mnozstvi));

                }



                return vystup;

            }

        }

Zdravím,

dotyčný řádek vytváří pokaždé novou instanci třídy Obchod, která má vždy stejné výchozí hodnoty. Předchozí položky se tedy nepřepisujou, protože tam žádné nejsou.

if (obchod == null)

{

	obchod = new Obchod(textBox1.Text, textBox2.Text, Convert.ToDouble(textBox3.Text), Convert.ToInt32(textBox4.Text));

}

...

Hm, pozdě :smile1:

Díky moc, vyzkouším zítra.

Bohužel, já chodím na střední a dálkově, takže hodin strávených s učitelem jsou dvě až tři za měsíc.

djanosik: a co tam je teda ještě ne tak úplně správně ať se poučím?

To djanosik :
Takze bohuzel vas napad nevysel, stale se to chova stejne. Cely projekt (Microsoft Visual C# 2008) prikladam.

Kromě toho neustálého vytváření nového obchodu s jednou položkou, ještě pořád přiřazuješ (místo přidávání) 'vystup' ve vypisSklad(). A ten konstruktor Obchod(...........) je k ničemu. Všechny tři věci jsem opravil v souboru, který jsem postnul, stačilo by se na to podívat :smile1:

To nervak : Pravda, to jsem přehlédl.

Díky moc oboum pánům, již to mám opraveno.

Co jsem koukal, největší problém asi bylo že oboje třídy nebyly veřejné.