forループが正しく動作していないのはなぜですか？このコードのように、名前が "mubeen"と表示される必要がある場合は、何度も言及する必要があります。

Question

Thread t; 
    string mystring = ""; 
    private void button2_Click(object sender, EventArgs e) 
    { 

     t = new Thread(write); 
     t.Start(); 
     while (t.IsAlive) ; 
     textBox1.Text = mystring; 

    } 

    void write() 
    { 
     int i; 
     for (i=0; i<1000 ;i++) ; 
     mystring+= "mubeen" +i.ToString() + "\r\n"; 


    } 

Answer 1

forループを書く方法が間違っています。必要な命令が複数ある場合は、コードを{セミコロンを行うと、削除するために、あなたは、このように

のための次に置く：

for (i=0; i<1000 ;i++) 
{ 
    mystring+= "mubeen" +i.ToString() + "\r\n"; 
} 

と私はちょうどあなたが同じ理由でこのようにそれを変更する必要がありますので、あなたがあなたの中で同じ間違いをした実現しました：

あなたはmubeen1000として出力を持っている0

Answer 2

Thread t; 
string mystring = ""; 
private void button2_Click(object sender, EventArgs e) 
{ 

    t = new Thread(write); 
    t.Start(); 
    while (t.IsAlive) ; 
    textBox1.Text = mystring; 

} 

void write() 
{ 
    int i; 
    for (i=0; i<1000 ;i++) { 
     mystring+= "mubeen" +i.ToString() + "\r\n"; 
    } 


} 

Answer 3

理由があるため、あなたのループ構文

int i; 
// since you don't specify the method to loop through, 
// this is basically looping through doing nothing other than incrementing i, resulted in i = 1000, in which therefore exiting the loop 
for (i=0; i<1000; i++); 

// this is the next statement which is execute, which is setting mystring to mubeen1000 
mystring+= "mubeen" +i.ToString() + "\r\n"; 

であるあなたは、次のようにforループを設定する必要があります。また

// since you expect it starts with mubeen1, start with i=1 and the condition i<=1000 (instead of i<1000) 
int i; 
for (i=1; i<=1000; i++) 
// specify what you want to do in the body of the loop 
{ 
    mystring+= "mubeen" + i.ToString() + "\r\n"; 
} 

、あなたのwhile文を問題があるように見える場合もあります。

while (t.IsAlive) 
{ 
    textBox1.Text = mystring; 
} 

Answer 4

問題は、forループの最後に誤ったセミコロン;が含まれていることです。これは、ループの本体が実際には必要な文字列連結ではなく、空文であることを意味します。

ループ変数は、ループの外部からアクセスできるのではなく、ループ自体で宣言することが望ましいです。私はあなたがを行ったと思う。これを最初にやるが、それは後の使用にアクセスできないときに移動した。ループ内でiが宣言されたときにiがそのコードで使用できなかったという事実は、そのコード行がのループではないことを示す手がかりになっていたはずです。

次に、busyloopを一般的に（while (t.IsAlive) ;）持っているのは本当に悪い考えです。UIスレッドではそれ以上に悪いことがあり、作業中に何もしないようにUIスレッドをブロックします。 を非同期でにして、その間にUIスレッドを他のUI関連のタスクを処理したままにしておき、完了したらそれを更新する必要があります。また、スレッド間で共有されているフィールドが存在することを避けることが望ましいです。このフィールドは、このアクティビティとは無関係のコードにアクセスしてアクセス可能です。それはのように狭くできるだけスコープの状態を維持するのが最善です：

private async void button2_Click(object sender, EventArgs e) 
{ 
    string myString = await Task.Run(() => Write()); 
    textBox1.Text = myString; 
} 

private string Write() 
{ 
    StringBuilder output = new StringBuilder(); 
    for (int i = 0; i < 1000; i++) 
     output.AppendLine("mubeen" + i); 

    return output.ToString(); 
} 

注ループ内の文字列を連結することも非常によくスケールアップするつもりはないという。大きな文字列連結のために設計されたStringBuilderまたは別の同様のツールを使用する方が良いでしょう。