早期リターンを禁止されるつらさ

転職した会社で早期リターンが禁止されている。 正確に言うと、misraCを踏襲し、関数内ではreturnは末尾に１つだけ、という制約が設けられている。 この他にも関数ポインタが禁止等も色々あるが、早期リターンを禁止されるのは本当に困る。 早期リターンによってどれだけ気持ちよくコーディングできるか分かっていない。 もうこの後の行は読まなくていいんだ、という精神的安堵感。 これをもうこの会社では得られない。 関数のポインタ引数のconst禁止程の破壊力がある。 地獄のような１０重以上のネスト地獄・・・。 律儀に守ることによる可読性の低下の方が問題ではなかろうか。

[strow0343]

そのMISRA-Cルールは必要(required)なのでドキュメントに残すとか逸脱の手続きを取れば逸脱可能。ちなみにMISRA-Cでは関数ポインタは禁止してないはず / 未だにMISRA-C:2004なのがバレてしまった感

[punychan]

転職面接の最後の質問には御社のコーディングルールを教えてくださいってのがいるな

[sgo2]

コレは分かる。複数breakやcontinueも禁止(do〜whileと組み合わせて回避出来ない)だし、条件式が複雑になりすぎてもNG。あと暗黙の型変換がNG(コンパイラによる型チェックのメリットが潰されてる)も辛い。

[buhoho]

話には聞いたことあるけど実在する規約のなのか。10行以上の関数は作るなってことなんだろうか。個人的には細かい関数バラバラに作られるのもそれはそれで読みづらいことが多い

[y_hirano]

「早期returnを禁止しても複雑度が増さないくらいに関数化を徹底しろ」という風にも解釈できるけど、まああまり現実的ではないかも。まだ組み込みが牧歌的だった時代の名残とかかな。

[kknsd]

関数出口については、MISRA C:2012では必要”required”じゃなくて推奨”advisory”なので、守らない時の逸脱手続きは必須ではないっぽ　(MISRA C:2004ではrequired)

[sa-yama321]

ひどいルールだなー

[k_akashi]

こういう場合は良い、みたいな例外条件の教育とかもされず/改訂も出来ず、単にルール制定してれば良い/是非はわからんがルールだから従え、みたいな環境だと色々大変なんだろうなぁ…

[kiyo_hiko]

ガードの事かな？https://qiita.com/kouyan/items/7b8b456b626447a1e24e 俺は禁止されたら `goto <一番後ろ>;` 使う。gotoかわいいよgoto。ちなみにMISRA-CじゃなくてCERT C読むとgotoは一部の状況(08.MEM)で推奨すらされている。割と増田の状況向き

[door-s-dev]

これはかわいそう。転職する時に早期リターンが禁止されてるかなんて聞かないもんなぁ。 ちょろっとでもコードを見せてもらったほうがいいのかもしれん

[strawberryhunter]

①このコーディング規約を考えたやつ馬鹿じゃねーのと公言する　②わざと早期リターンするコードを書く。指摘されたら食い下がる。　③増田が先頭に立って会社のルールを変える／VisualBasicはそういうコードばっかり。

[nunulk]

目的や理由の書かれてないガイドラインを作らないでほしい

[honeybe]

「地獄のような１０重以上のネスト地獄」を解消することで早期リターンせずにすむコーディングに変えていくのが良さそう。それはそうとして「早期リターン禁止」はクソコーディング規約なのでクソですね！

[gm91]

逸脱の手続き取ったらいいだけでは。

[indication]

関数分離で対抗だ！、関数ポインタ無しってCの旨みが無くなった感がハンパない

[aceraceae]

そんな理不尽な会社は辞めたほうがいいい。

[toaruR]

誤解から生まれたローカル制約なんかな？(・ω・)

[sugawara1991]

早期リターン多用するけど戻り値実際何だったのか確認とかロギングなどを「その関数内で完結させる」場合の記述の簡潔さと排反なのが悩ましい。早期リターンの戻り値は妙な演算で求めない、等の規約を併用したい

[tybalt]

プリチェック用の補助関数を作って、早期リターンしていた処理はそちらに移動する感じかな。`if(pre_check()) ret = fn_body()` みたいな。

[kenzy_n]

祭祀における詔

[arket789]

増田氏かわいそすぎる… ディープネストの方がマシ扱いとか遭遇した事ない。このルール推してる人がモダンJavascriptのコードとか見たら発狂しそう。

[fops]

MISRA-Cは自動車システム用のガイドラインらしい。http://www.c-lang.org/detail/misra_c.html／これ？→「関数は、その最後に1つだけの出口を持たなけらばならない」

[sds-page]

転職しろというか転職後でこれか

[narwhal]

早期リタイアで対抗だ。