Entfernen von Textbox/Label bei Klick auf die Schaltfläche C#

Question

Ich versuche, Textfelder und Beschriftungen nacheinander auf Knopfdruck zu entfernen. Ich habe eine Liste von Textboxen namens inputTextBoxes. Hier

ist der Code für das Hinzufügen von:

private void onClickAdd(object sender, EventArgs e) 
    { 
     inputTextBoxes = new List<TextBox>(); 
     Label label1 = new Label(); 
     label1.Name = "label1"; 
     label1.Text = "w" + i; 
     label1.Location = new System.Drawing.Point(5, 10 + (20 * i)); 
     label1.Size = new System.Drawing.Size(30, 20); 
     this.Controls.Add(label1); 

      TextBox text1 = new TextBox(); 
      text1.Name = "text1"; 
      text1.Location = new System.Drawing.Point(35, 10 + (20 * i)); 
      text1.Size = new System.Drawing.Size(25, 20); 
      inputTextBoxes.Add(text1); 
      this.Controls.Add(text1); 
      i++; 
    } 

Zum Entfernen ich dies versuchen:

private void onClickRemove(object sender, EventArgs e) 
    { 
     foreach(TextBox text1 in inputTextBoxes) 
     { 
      this.Controls.Remove(text1); 
     } 

    } 

Aber es entfernt nur die letzte Textbox hinzugefügt, klicken againg auf die Schaltfläche nicht tut, etwas.

Answer 1

Sie möchten nur eine TextBox auf einmal entfernen, warum brauchen Sie eine foreach-Schleife? greifen nur die letzte oder erste TextBox und wenn sie es von den Kontrollen nicht null entfernen:

private void onClickRemove(object sender, EventArgs e) 
{ 
    var textBoxToRemove = inputTextBoxes.LastOrDefault(); 
    // or 
    // var textBoxToRemove = inputTextBoxes.FirstOrDefault(); 
    if (textBoxToRemove != null) 
    { 
     this.Controls.Remove(textBoxToRemove); 
     inputTextBoxes.Remove(textBoxToRemove); 
    } 
} 

Stellen Sie sicher, dass es von inputTextBoxes entfernen auch so das nächste Mal fragt einen Text zu entfernen, wird es nicht versuchen, entferne es wieder und gehe zum nächsten.

bearbeiten

@Piotr Nowak ein weiteres Problem hingewiesen hat Sie haben, Sie eine neue Liste für inputTextBox zuweisen jedes Mal, wenn eine neue TextBox hinzufügen, sollten Sie die Liste zuweisen nur einmal, wenn Sie Ihre Klasse erstellen.

entfernen diese aus onClickAdd Methode:

inputTextBoxes = new List<TextBox>(); 

Und diese verwenden, wenn Sie die Liste als Feld, um es Ihre Klasse deklarieren:

private readonly inputTextBoxes = new List<TextBox>(); 

Answer 2

Du eine neue Liste in Ihrer OnClickAdd ständig zu schaffen() Methode:

inputTextBoxes = new List<TextBox>(); 

Versuchen Sie zu überprüfen, ob die inputTextBoxes null ist nd dann nur diese Codezeile. Ansonsten lass einfach den Rest des Codes laufen. Denken Sie auch daran, die inputTextBoxes-Liste zu löschen, nachdem die onClickRemove() -Methode das Entfernen von Textfeldern/Beschriftungen beendet hat.