vol.17

楽をするために手をかける

update:2005/04/06


たとえばコード海の真ん中に、こんなカタマリがあったとしましょう

If bpni="1" Then
  CSV = CSV & "hoge"
Else
  CSV = CSV & "," & "foo"
End If
でもって、このカタマリをどうこうする日がやってきたとします。そのとき、このカタマリが何をしたいか、はっきりわかりますか?
ここで自分が書いたなら大丈夫という答えは禁止ですよ、ダンナ。
なぜ自分が書いたなら大丈夫なんでしょう?昨日のコードでも?一月なら?半年なら?一年なら??ね、無理でしょ。
自分で書いたソースでもこうなります。まして人のソースなんて言わずもがなでございます。

先ほどのソース、"bpni"のデータ型は?"2"とか"z"とか"@_@"とか入ってくる可能性は?そもそもなになの?これは?
"CSV"という変数名だからといって、CSVファイルの出力バッファという裏づけは?文字列型の処理に見えるけど本当?
しまいには、変数宣言に舞い戻って定義を調べる羽目になります。

ここで、ASPだったりすると全部Variantなので即死の危険性があります。
またOption Explicitが無いというのは論外です。

さらに、何処から何が代入されて、何処でどう使われているかを確かめて…

考えただけでぞっとする作業ですな。

だから、せいぜい後で楽することを考えて…
'---Boshの部品番号インデックスに特注とある場合には、特注車マーカーを付加する
If strBochPartsNumberIndex = BOSH_PARTS_INDEX_SPCECIAL
  strCarSequenceVolume = strCarSequenceVolume & SPECIAL_ORDER
Else
  strCarSequenceVolume = strCarSequenceVolume & NORMAL_ORDER
End If
ね、これならまだ解りやすいでしょ。
昔から言うじゃないですか。

備えよ、常にって。

あれ?ちょっとちがった?

戻る