goto を使わずに

質問

ＶＣ＋＋でプログラムを作ったんですが、上司からgotoは使うなとの指示があり困ってます。
なにせ、まだまだＶＣ＋＋になれてないし、かなりの初心者です。
Ｃ言語は学生のころ習いました。
if(a==0x0030){ //0による除算を回避するために0は入力不可とする(0ディバイト)
int ret; //エラー画面出力
ret=AfxMessageBox("0は入力不可です。",MB_OK|MB_ICONEXCLAMATION);//入力が0の場合エラーを表示

goto end;

}

・
・
・
end:
//エラーがあった場合ここにとんで最初からやり直される
;
}

こんなかんじなのですが、なにか解決法ありませんか？

回答No.9

BLK314

AfxMessageBox()使っているってことはＭＦＣですね。
だったら、例外もありではないでしょうか？

たとえば、MFCでCDialog派生で作って
入力値を0以上99以下に制限するなんてよくやりますよね。
１００を入力しＯＫを押すと、メッセージボックスが表示される。

今回の処理と似ていませんか？

似通っているのだったら、似たように実装してもいいのでは？

例外でCException 派生を使えば、
ReportError()にメッセージボックス表示を任せることが可能です

goto文は便利ですし、時には使った方がプログラムがすっきりすることもあります。

ただ、むやみに使いすぎるといわゆる"スパゲティ"ができかねませんので、使うときは慎重さが求められます。
(1重のループだったら、gotoよりもcontinueやbreakを使うべき。
１重のループでもswitchがあり、特定のcaseだけ脱出したい場合、
もしくは2重以上のループの場合、
フラグを用いる方法もあるが、一般にgotoで一気に脱出するほうがわかりやすいと思います）

回答No.8

jacta

他の方も指摘されていますが、gotoを使う方がよい可能性がかなりあります。
省略部分次第ですね。

回答No.7

chie65536（@chie65535）

基本中の基本

　for (;;) [
　　printf("数値を入力して下さい：");
　　scanf(…（略）…);
　　if (○○エラー判定) {
　　　printf("○○エラーです。\n";
　　} else if (△△エラー判定) {
　　　printf("△△エラーです。\n";
　　} else if (□□エラー判定) {
　　　printf("□□エラーです。\n";
　　} else {
　　　break;
　　}
　　printf("もう一度、");
　}
　（入力後の処理）

回答No.6

pepe_clift

こんな感じ？
for(;;){
多分aに何かからデータを代入する。
if(a=='0'){
ret=AfxMessageBox("0は入力不可です。",MB_OK|MB_ICONEXCLAMATION);
break;
　else｛
　 aが'0'では無い時の処理
}
}

回答No.5

Tacosan

同じく全体像が見えないので解決法も分からないんだけど, ループ + continue とか break なんか, かなぁ. 「実は goto が最善」という可能性もないわけじゃない.
もっとも, 「C を学んだ」人間が
a==0x0030
なんて書いちゃだめだと思う. もっと素直に
a=='0'
と書くべき. あと, AfxMessageBox の返り値が不要なら変数に代入しなくていい.

お礼

回答有り難うございます。
やはりわかりにくいですよね・・・
新しく書き直したんで見てみてください。

回答No.4

titokani

省略されている部分がわからないとなんともいえないけど、
if(a==0x0030){ //0による除算を回避するために0は入力不可とする(0ディバイト)
int ret; //エラー画面出力
ret=AfxMessageBox("0は入力不可です。",MB_OK|MB_ICONEXCLAMATION);//入力が0の場合エラーを表示
}else{
・
・
・
}
//エラーがあった場合ここにとんで最初からやり直される
;
}
とか。

回答No.3

D-Matsu

外側が見えないのでどうやって「最初からやり直し」ているのかにもよりますが、

if(a==0x0030){
// エラー処理
} else {
// 通常処理
}

とか。
初心者だというのならなおさら「安易にgotoを使わない」癖を今のうちに徹底して叩き込んでおくべきです。

回答No.2

Alice022

ソースコードの全体が良く分からないので、参考にならないかもしれませんが。
勉強課題のようなので、恐らくサブルーチンは考えずに、一つの関数内に処理を記述しているのでしょうか。
そうであれば、While文などを使用して判定し、エラーであれば最初の処理を繰り返させるロジックを考えた方が良いかもしれません。

これを考えるのに頭の中で良く分からないようでしたら、フローチャートで、この部分のロジックを考えてみると整理しやすいと思います。

回答No.1

zwi

プログラムの全体像が良くわかりませんが、ループ構造内ならcontinue;で良いのでは？あるいは、入力と判定部分を関数にしてＯＫになるまで呼び出し側でループにするとか。gotoを使ったほうが分りやすい場合は良いですが、このぐらいの処理構造で使うのは止めましょう。

ＶＣ＋＋以前にＣ言語の構造化プログラミングを勉強されたほうが良いですよ。