「goto使ってますか?」記事へのコメント

コーディング規約で仕方なく

[Anonymous Coward]

コーディング規約で仕方なく、っていうのは
「コーディング規約で仕方なく使ってる」って意味でしょうか？
質問文からして、そういう意味になるのかなと思ったのですが･･･

で、もしそうだとした場合、コーディング規約でgotoを禁止するのではなく
逆にgotoの仕様を強いるような理由が何かあるってことでしょうか

Re:コーディング規約で仕方なく

[Anonymous Coward]

うちの課では「1関数内にreturnを複数書くのは(見難くなるから)禁止。(出口は一ヶ所であるべき)」というルールがあり、goto 使いまくりです…

int func(char *arg) {
  int n;
  if (arg == NULL) {
    n = -1;
    goto END;
  }
  /* なんか処理 */

  if (なんかエラー) {
    n = 1;
    goto END;
  }
  /* なんか処理 */

  n = 0;
END:
  return n;
}

「こーゆー(↓)場合は積極的に使え」程度なら、賛同できるんですが…

int func() {
  char *p1 = NULL, *p2 = NULL;

  p1 =

Re:コーディング規約で仕方なく

[Anonymous Coward]

それがCだったとしたら、freeにNULLポインタを渡すのは完全に合法ですので

if (p1 != NULL)
  free(p1);
if (p2 != NULL)
  free(p2);

は

  free(p1);
  free(p2);

と書いてかまいません。

Re:コーディング規約で仕方なく

[Anonymous Coward]

free()呼び出しに要するCPUのコストか、NULLポインタチェックに掛かるコードのサイズか、どちらを問題視するかということでしょうね。