C言語のGOTO文（組み込み系）

質問

私は組み込みでソフト開発を行っているものです。
(アセンブラでの経験は長いが、Ｃは短い)
基本的な質問になってしまいますがご了承ください。
Ｃ言語で"goto文は使うな、スパゲティプログラムになりやすい。
使うなら多重ループからの脱出のみ"　ということを良く聞きます。
本当ににそうでしょうか？・・・

例えば、下記サイトの図６をＣ言語で書いてみました。
もちろんgotoを使いました。
gotoを使わないで、誰でもわかりやすく書くことなどできるのでしょうか？
よろしくお願いします。

http://techon.nikkeibp.co.jp/article/NEWS/20071119/142670/?ST=lsi&P=2

/* 開始 */

KAISHI:
p74 = High;

if(p70 == High){
register = 1;
}
if(p71 == High){
register = register + 1;
if(register != 1){
goto KAISHI;
}
if(p72 == High){

回答No.13

zwi · ベストアンサー

>初めの１０秒間でｓｗ１が押されなかったらエラー処理１を実施して終了。
>無事にｓｗ１が押されると次の１０秒がスタートする。
>次の１０秒間でｓｗ２が押されなかったら""エラー処理２を行った後エラー処理１を""実施して終了。
>無事にｓｗ２が押されると次の次の１０秒がスタートする。
>次の次の１０秒間でｓｗ３が押されなかったら""エラー処理３を行った後エラー処理２を行いエラー処理１""を実施して終了。
>です。

それなら、こうすればお望み通りになります。
switch( timerCheck() ) {
case ERR_SYORI3:
____エラー処理３
case ERR_SYORI2:
____エラー処理２
case ERR_SYORI1:
____エラー処理１
____break;
}

・ERR_SYORI3→ERR_SYORI2→ERR_SYORI1
・ERR_SYORI2→ERR_SYORI1
・ERR_SYORI1
ですので、これでＯＫですか？

お礼

OKです。

回答No.5

TreatMeGently

No.4回答者さんが正解です。構造化プログラミングです。再帰呼び出しも出来るように（自分から自分を呼び出す。）なります。
　ただメモリーマップドI/Oと思われますので切り替えてからある程度のディレイ時間が必要かも知れません。
　読み込みポート指定出力してからの時間をWRITE_DELAY_TIME,
　読み込んで次のポートを読み出す為の時間READ_DELAY_TIMEが必要かも知れませんので
　万が一に備えて#defineで宣言して変更し易くしておくのが良いかと思われます。
　Sleep()はスレッド停止時間が使用出来ればそのまま、使えないならForNextLoopでSleep()を新規に作成します。
　組み込みのハードウェアのタイミングについては質問者さんが一番ご存知だと思われますが。

#define WRITE_DELAY_TIME 0
#define READ_DELAY_TIME 0

int checkButton(){
int register=0;
p74 = High;
Sleep(WRITE_DELAY_TIME);
if(p70 == High){register ++;}
Sleep(READ_DELAY_TIME);
if(p71 == High){ register ++;}
Sleep(READ_DELAY_TIME);
if(p72 == High){ register ++;}
Sleep(READ_DELAY_TIME);
...
return (register == 1) ;
}

補足

今回の場合は、DO-WHILE文で代替ができそうです。
では、下記のような場合はどうでしょう。
N0.8さんが言われる「出口を一か所に集中させるにはgoto文を使う」です。
果たして、goto文を使わないでできるのでしょうか。

説明：
初めの１０秒間でｓｗ１が押されなかったらエラー処理１を実施して終了。
無事にｓｗ１が押されると次の１０秒がスタートする。
次の１０秒間でｓｗ２が押されなかったらエラー処理２を実施して終了。
無事にｓｗ２が押されると次の次の１０秒がスタートする。
次の次の１０秒間でｓｗ３が押されなかったらエラー処理３を実施して終了。

/*開始*/

int timer //型宣言 timerは１秒割込みで０になるまで１秒置きに-1されるものとする

timer = 10; // 10秒をセット
do{
____if(timer == 0){goto ERR_SYORI1;}
}while(sw1 == off)

timer = 10; // 10秒を再セット
do{
____if(timer == 0){goto ERR_SYORI2;}
}while(sw2 == off)

timer = 10; // 10秒を再セット
do{
____if(timer == 0){
________エラー処理３
ERR_SYORI2:
________エラー処理２
ERR_SYORI1:
________エラー処理１
________break;
____}
}while(sw3 == off)

/*終了*/

回答No.4

kmee

私なら、こんな感じにすると思います

while(継続条件) {
if( checkButton() ) {
playSound();
}
}

そして、その流れ図のようなチェックをして、「音を鳴らしてはならない」なら0,「音を鳴らす」なら1を返す checkButton() 関数を作ります。
コードは、あなたが書いたのと似たのになりますが、
int checkButton(){
int register=0;
p74 = High;
if(p70 == High){register ++;}
if(p71 == High){ register ++;}
if(p72 == High){ register ++;}
...
return (register == 1) ;
}
と、gotoは使いませんし、KAISIへ戻ったりもしません。

さて、このプログラム、「わかりづらい」でしょうか?

gotoが害、というのは、構造化プログラミングという考え方から来たものです。
http://ja.wikipedia.org/wiki/%E6%A7%8B%E9%80%A0%E5%8C%96%E3%83%97%E3%83%AD%E3%82%B0%E3%83%A9%E3%83%9F%E3%83%B3%E3%82%B0
現在主流のオブジェクト指向は、この構造化を発展させたものです。
現在主流の言語の多くは、goto文がありません。「ループからの脱出」等の特定の使い方に限定されたジャンプ命令(continue,break等)があるだけです。

では、どうするのか?
最初からgotoを使わない設計をします。

その記事をざっと斜め読みしましたが、初級者がアセンブリでプログラムしているようです。
Cでのif+gotoにあたる条件付分岐命令を使うことを念頭にフローチャートを作っているように思われます。
命令の少ないアセンブリでは、それが正しいやりかたかもしれません。
それをそのままCへ持ってこようとすれば、どうしてもgoto文を使いたくなります。

ですが、最初から、例えば構造化プログラミングを念頭に設計すれば、フローチャート自体が別なものになるでしょう。
goto文を使うことも無いはずです。

/* フローチャート自体も、最近のプログラミング設計には向かないと言われています */

補足

回答で書かれた文において、
最後の文の

return (register == 1) ;

はどういう意味でしょうか？
教えてください。
宜しくお願いします。

回答No.3

zwi

私もdo～while()で書いちゃいますね。

やりたいことからすると、無駄が有るけどこんなかんじで。
____do {
________doremi = 0;
________register = 0;
________p74 = High;
________if(p70 == High) {
____________register++;
____________doremi = 1;
________}
________if(p71 == High) {
____________register++;
____________doremi = 2;
________}
________if(p72 == High) {
____________register++;
____________doremi = 3;
________}
________if(p73 == High) {
____________register++;
____________doremi = 4;
________}
________
________・
________
________・
________
________
____} while( register!=1 );

回答No.2

jk39

私ならこう書きます。
まあ、do-while文も結構敬遠される書き方で、
do-whileにせざるを得ない設計は
他人のことを考えない悪い設計と私の職場なら言われかねません。

do{
　p74 = High;

　if(p70 == High){
　　register = 1;
　}
　if(p71 != High){
　　break;
　}
　register = register + 1;
}while(register != 1);

if(p72 == High){
　・
　・

回答No.1

noname#214300

組み込み系という事で、企業などでPGをされている方だと仮定しますが、余所の人間がどうこう言おうと、職場の宗教に合わせるのが良いと思います。
（個人であれば、明日の自分の事を考えて使用頻度を見極めれば良いかと）

GOTOを使わないというのは、単に他人がそのソースをメンテするときに処理を追いにくくなるからであって、要はその他人がGOTOを追い切れる土壌にあるならばそれなりに使って構わないと思います。

ちなみに自分の職場ではGOTOの使用は『例え何があっても』御法度といった宗教で動いていますが（GOTOを使わなければ書けないアルゴリズムはそれ自体が間違っているとのこと）、自分でしかメンテしないモジュールには適度にGOTOを使っています。神の領域に逝ってしまった諸先輩の技量に合わせるために残業はしたくありませんので……。

GOTOを使えばステップ数もそれなりに減るのに、使わない為にIF文を8重くらいにネストしたコードを弄ったことがありますが、そこまで来ると別の弊害を感じました。そのくせやたらめったらグローバルに変数置いてたりして、一体どうなってるのかと。

要は、何事も適度がいいって事だと思います。