プログラマーを悩ませる、命名の難しさについて

三井康宏 @koglachesis

@stealthinu tempは温度を扱うシステムに組み込まれると誤解されるかもよ。temporaryとtemperatureと。

Kiyoshi SATOH @stealthinu

@koglachesis ああ、そうだった…　その指摘はリーダブルコードでも書いてあったのに…

suno (すの) @suno88

プログラムを書く時に変数名に flag とか flg という名前を使わないよう強く意識するようになったきっかけは、藤原博文氏の名著『C プログラミング診断室』だ。/ Ｃプログラミング診断室／上司が問題／フラグ http://t.co/Zf0gIat1be

まっつん @mattun72

@suno88 意味のはっきりしないフラグの多用もそうだけど、ステートの変遷によって挙動が変わっていくステートマシンなプログラムって複雑になりがちですよね。

suno (すの) @suno88

@mattun72 そうですね。デバッグが難しくなります。保持しているステータスの数が増えてくると、組み合わせが増えて大変です。こう言う時、うまくテストを自動化できると心強いのですが。

Kiyoshi SATOH @stealthinu

@suno88 早速挫折したｗ parking_flagみたいなのをhas_parkingにしてみたらcard_flagみたいのはcard_availableにしないといけなくなってうおお…と。なのでやっぱflagだけ取ってお茶にごそうかと。うーむ…

suno (すの) @suno88

@stealthinu すみません、笑っちゃいけないのは分かりますが笑ってしまいましたｗ 昔書いたものは直したくなるものですよね。ここは _flag を取るだけにして、次からは今日の知見を生かした命名にするってことでひとつ。

Kiyoshi SATOH @stealthinu

@suno88 ですねえ…　なかなかむつかしいっす。

suno (すの) @suno88

変数名やカラム名に flag とつけることの是非について同業のフォロワーさんたちと語り合った後、自社のソースを見る。ソースには errFlg があり、データベースには delete_flag をはじめ旗があちこちにはためいている。うんざりだ。はた迷惑とはまさにこのこと。

suno (すの) @suno88

tmpErrMsg と tmpErrMsg2 があり、片方は空文字列、もう片方は要素数 0 の配列で初期化されている。最終的に allErrMsg 変数(文字列)に統合される。初心者が一生懸命書きました、という感じのソースで、微笑ましい。──微笑ましいわけないだろうが!!