Dotaz: [C#] foreach, delete textbox a label

[Lanny2013]

Příloha:

• cod.txt (739 bytů)

Dobrý den, snažím se odstranit určité Texboxy a Labely které jsou dynamicky vytvořené v Groupboxu. Chtěl bych, ale odstranit jenom některé. Problém je v tom, že se mi vždy podaří odstranit všechny Labely ale Textbox pouze ten první. Můžete mi prosím poradit kde mám chybu. předem děkuji.

---

---

Odpovědi

[Jamie]

Chtělo by to sem dát i ten kód, který ty prvky vytváří. Navíc by se kolekce, kterou procházíš přez foreach, neměla měnit.

[skunkOS]

Přesně tak. Kdykoliv dochází k nějakým destrukcím v kolekci, kterou iteruješ přes foreach, je to náběh na chybu. Když se něco v kolekci mění, použij for.

[Jamie]

Btw. které "některé" chceš odstraňovat. Neměl by v každém foreach být ještě for (místo použití h a i)?

[Lanny2013]

Ahoj posílám čás kodu, který vkládá prvky. Zde si ale myslím, že chyba není.

Label label2 = new Label();
label2.Name = "labelg" + TxtInt;
label2.Top = LabelTop + 30;
label2.Width = 10;
label2.Left = LabelLeft + 101;
label2.Text = "-";
Form3.form4.groupBox3.Controls.Add(label2);

TextBox txt2 = new TextBox();
txt2.Name = "txtbox" + TxtInt;
txt2.Top = TextboxTop + 30;
txt2.Left = TextboxLeft + 52;
txt2.Width = 114;
txt2.Height = 20;
txt2.Text = StandartUcetAdd.cislo;
Form3.form4.groupBox3.Controls.Add(txt2);

TxtInt++;

Co se téče cyklu for tak ten jsem nechtěl použít, jelikož neznám konečný počet prvků dopředu. Nicméně jsem to upravil tak, že nejprve zjistím celkový pořet prvků.

int count =  groupbox.Controls.Count;

potom přes for vybrat jenom určité prvky do určitého čísla int count. I tak to ale nefunguje chová se to úplně stejně jako bez for. A to tak, že když mám jeden foreach třeba na textbox tak je vše v pořádku. Jakmile za to přidám druhý foreach na vymazání lablelu tak se provede jenom ten první foreach a ten druhý se neprovede. Když je prohodím tedy prvně chci vymazat label a potom textbox. Tak je to přesně naopak label vymaže, ale textboxy už ne.

[MadCatX]

A nevyhodí ti to nějakou vyjímku? Upravovat si „pod rukama“ vektor, který procházíš cyklem není dobrý nápad. Dej si do toho kódu breakpoint a vykrokuj si to debuggerem.

[pavlix]

Mimochodem, jak se to správně dělá v C#? V pythonu je to triviální.

[kaaja]

Správně jde například použít .ToList() z LINQ, které prakticky klonuje kolekci do listu. A tu už při procházení neměníš.

[pavlix]

Používám list(something) v Pythonu, takže je to zjevně ekvivalentní.

[Lanny2013]

Omlouvám se za pozdní odpověď. Přesně jak píšeš nakonec jsem ten kod změnil úplně. Použil jsem jak jsi psal LINQ a dosáhl jsem požadovaného výsledku. Děkuji