ボタン操作

質問

VC++2008ExpressEditionを使用してプログラムを作成しています。

Windowsフォームアプリケーションを作成し、そこに、TextBoxとButtonを放り込み、ボタンを押すと以下のようなコードが実行されるようにしました。

　　System::Void button1_Click(System::Object^ sender, System::EventArgs^ e) {
　　　　static bool flug=false;
　　　　if(flug)return;
　　　　flug=true;
　　　　button1->Enabled=false;
　　　　button1->Text=L"実行中";
　　　　button1->Update();

　　　　int i;
　　　　for(i=0;i<200;i++){
　　　　　　int t=clock();
　　　　　　while(10>clock()-t);
　　　　　　textBox4->Text+=L"aaraeaewa"+i+L"\r\n";
　　　　　　textBox4->SelectionStart = textBox4->Text->Length;
　　　　　　textBox4->ScrollToCaret();
　　　　}
　　　　button1->Enabled=true;
　　　　flug=false;
　　　　button1->Text=L"実行";
　　}

このように設定したボタンを何回も連続して押すと、この関数を実行中は多重実行されないことを想定しているのですが、やってみると多重実行されてしまいます。
どの様にすれば多重実行しないように出来るのでしょうか？
また、この関数を実行中は自分で作成したアイテムを操作することが、一切できなくなってしまうのですが、何か解決方法は無いでしょうか？

宜しくお願いします。

回答No.1

redfox63 · ベストアンサー

イベント内でループをしているため メッセージポンプが機能しないのが原因でしょう
そのため ボタンのClickメッセージがWindowsのメッセージキューに残ってしまい
Clickイベントが終了してからもう一度呼ばれるといったことが繰り返されます

forループの中で Application::DoEvents() を実行するようにしてみましょう

　　　　for(i=0;i<200;i++){
　　　　　　int t=clock();
　　　　　　while(10>clock()-t);
　　　　　　textBox4->Text+=L"aaraeaewa"+i+L"\r\n";
　　　　　　textBox4->SelectionStart = textBox4->Text->Length;
　　　　　　textBox4->ScrollToCaret();
　　　　　　// これを追加 ClickイベントなどのWindowsメッセージを処理
　　　　　　Application::DoEvents();
　　　　}

お礼

解答有り難うございました
確かにこのようにすると、再描画がかけられるのでできました。

回答No.3

chie65536（@chie65535）

蛇足な追記。

　　　static bool flug=false;

「何かの目印」と言う意味での変数なら、変数名は「flug」ではなく「flag」です。

回答No.2

chie65536（@chie65535）

＞この関数を実行中は多重実行されないことを想定しているのですが、やってみると多重実行されてしまいます。

余計なフラグ判定が原因。

　　　　static bool flug=false;
　　　　if(flug)return;
　　　　flug=true;
と
　　　　flug=false;
を削ってみましょう。

ボタン１が押された瞬間、関数の入り口でボタン１のEnabledをfalseにすれば、処理が終わるまで「押そうと思っても絶対に押せない」ので、多重実行はしない「筈」です。

もちろん、関数の出口でEnabledをtrueに戻すのも忘れないように。

要は、自分自身のEnabledプロパティが再入フラグの代わりになるのです。

お礼

解答有り難うございます。
このフラグは無くて出来なかったから付けたものなので付いていても消えていても、無理でした。