c＃TextBoxの問題、引数を取得できません

Question

このコードについて何か怪しいものがあり、解決方法がわかりません。

基本的player.Nameが変更テキストボックスから引数を取得し、それに挿入するが、それはそのような何かをdoesntのようだ...それともイムはそれを間違った方法を確認する...

private void textBox1_TextChanged(object sender, EventArgs e) 
     { 
      Player player = new Player(); 
      player.Name = textBox1.Text; //Problematic line 
     } 

     private void Button_Start_Click(object sender, EventArgs e) 
     { 
      Player player = new Player(); 

      switch (player.Name.ToUpper()) 
      { 
       case "N/A": Label_Question.Text = "Please set your name!"; break; 
       default: Label_Question.Hide(); goto QuizStart; 
      } 
      QuizStart: 
      { 

      } 

     } 

必要があります

Answer 1

私はではなく、単一のButton_Start_Clickにすべてを詰め込む方法を抽出勧め：

private Boolean IsPlayerValid(Player player) { 
    if (null == player) 
     return false; 

    if (String.Equals("N/A", player.Name, StringComparison.OrdinalIgnoreCase)) { 
     Label_Question.Text = "Please set your name!"; 

     return false; 
    } 

    //TODO: you may want add up conditions here 

    return true; 
    } 

    private void Button_Start_Click(object sender, EventArgs e) { 
    Player player = new Player() { 
     player.Name = textBox1.Text // you may want this - assign a player's Name 
    }; 

    if (!IsPlayerValid(player)) 
     return; 

    Label_Question.Hide(); 

    // Quiz Start 
    } 

Answer 2

問題は、textchangedイベントとbutton pressイベントの両方で "Player"の新しいインスタンスを作成していることです。つまり、ボタンを押すプレーヤーは常に空です。このような

何かがあなたのために物事を修正する必要があります。

Player player; 

private void textBox1_TextChanged(object sender, EventArgs e) 
    { 
     player = new Player(); 
     player.Name = textBox1.Text; //Problematic line 
    } 

    private void Button_Start_Click(object sender, EventArgs e) 
    { 
     switch (player.Name.ToUpper()) 
     { 
      case "N/A": 
       Label_Question.Text = "Please set your name!"; 
       return; 
      default: 
       Label_Question.Hide(); 
       break; 
     } 
    } 

か、なぜ（あなたがここでは示されていないコードに他の何かをしたい場合を除き）すべて一緒にtextchangedスキップしない：

を

Player player; 

    private void Button_Start_Click(object sender, EventArgs e) 
    { 
     player = new Player(); 
     player.Name = textBox1.Text; 

     switch (player.Name.ToUpper()) 
     { 
      case "N/A": 
       Label_Question.Text = "Please set your name!"; 
       return; 
      default: 
       Label_Question.Hide(); 
       break; 
     } 
    } 

EDIT：厄介なGOTO文も取り除くためのコードを変更しました。