Il pulsante programmatico fa scattare l'eccezione 'System.StackOverflowException'

Question

Ho scritto un programma WinForms in C# .Net per fare clic su un pulsante a livello di codice all'interno di un modulo di password.

Form1 carica e mostra Form2 come finestra di dialogo.

L'applicazione si chiuderà se DialogResult è altro che DialogResult.OK.

Finora ho un evento click del pulsante, che è codificato come segue:

if (txtpass.Text == "") 
      { 
       MessageBox.Show("You need to enter a password", "Password", MessageBoxButtons.OK, MessageBoxIcon.Asterisk); 
       txtpass.Focus(); 
      } 
      else 
      { 
       if (txtpass.Text == "1234") 
       { 
        radButton1.DialogResult = DialogResult.OK; 
        radButton1.PerformClick(); 
       } 
       else 
       { 
        MessageBox.Show("Password Incorrect", "Password", MessageBoxButtons.OK, MessageBoxIcon.Error); 
        txtpass.Text = ""; 
        txtpass.Focus(); 
       } 
      } 

Io uso radButton1.PerformClick();, ma in esecuzione il programma mi dà il seguente messaggio:

An unhandled exception of type 'System.StackOverflowException' occurred in mscorlib.dll 

Sono non so cosa stia causando questa eccezione.

Answer 1

Modifica Non immagino. Dire al pulsante di fare clic su se stesso da se stesso sta sicuramente causando un ciclo infinito. Questo fa sì che il metodo venga chiamato ripetutamente, riempiendo lo stack e causandone l'overflow.

La mia ipotesi è che chiamare il PerformClick() stia causando di nuovo il metodo corrente che è stato postato, causando così un ciclo di chiamata infinito e risultante in uno StackOverflowException.

Per evitare questo, è necessario fissare la logica da qualche parte nel codice in modo che:

if (txtpass.Text == "1234") 

viene valutato come false e il metodo di scatto non viene chiamato più e più volte. È possibile ottenere questo risultato impostando txtpass.Text = "" giusto prima di fare in modo che faccia nuovamente clic su di esso.

Answer 2

Per chiamare il gestore di eventi ancora una volta da dentro è possibile utilizzare il seguente codice:

if (txtpass.Text) 
{ 
    case "1234": 
     radButton1.DialogResult = DialogResult.OK; 

     txtpass.Text = "12345"; 

     radButton1.PerformClick(); 

     break; 

    default: 
     case "12345": 
     break; 

} 

Answer 3

Normalmente si dovrebbe chiamare manualmente l'evento che si sta tentando di eseguire.

E.g. se si dispone di un metodo

button1_Click(object sender, ButtonEventArgs e) 
{ 
} 

Poi si dovrebbe chiamare il seguente nel codice:

button1_Click(this, new ButtonEventArgs()); 

Penso che forse è necessario spiegare un po 'di logica nel codice anche se, come non è chiaro quello che si' sto cercando di fare. StackOverflow probabilmente perché stai facendo

PerformClick() -> PerformClick() -> PerformClick() perché il testo "1234" non cambia mai tra le chiamate.

Answer 4

PerformClick() all'interno dell'evento click del pulsante? Se è così, è lì che stai andando male perché stai lanciando la tua applicazione in un ciclo infinito. tasto scatta

utente,
.NET gestisce Click() handler,
Button scatta PerformClick(),
.corre NET Click) gestore (,
Button scatta PerformClick(),
.NET gestisce Click() handler,
Button scatta PerformClick(),

ecc

È form1 sicuramente chiamando ShowDialog() su form2, e non solo Show()?

Invece di radButton1.DialogResult, provare a impostare this.DialogResult == DialogResult.OK.

La DialogResult proprietà su un pulsante dice NET che DialogResult da assegnare al Form quando il Button viene cliccato.

Answer 5

Un sovraccarico di stack si verifica in genere perché il metodo si chiama in modo indefinito, poiché ogni volta che viene chiamato un metodo, una voce viene aggiunta allo stack nel punto in cui non vi è più alcuna pila rimasta.

Per fermare la ricorsione, rimuovere la riga radButton1.PerformClick();