haraChan's Blog

お久しぶりです・・・
何ヶ月ぶりの更新だか、すでにわからないですがｗ
今日の作業中に見つけた（というか、解析中に見つかった）意味不明なコードが・・・

  If 0 = "1" Then
      処理１
  ElseIf 0 = "2" Then
      処理２
  Else
      処理３
  End If

Elseしか通らないですよね！？

んー...ゼロとオーの書きまつがいによるバグでなければ
処理1,処理2を通らぬよう(かつ実害のない範囲で元コード
を残す方向で)意図的に潰したように見受けられますがー

>Elseしか通らないですよね！？
多分これを作成した方は、処理３しかやりたく
なかったんですよ。きっと...

もともとは処理１や２も使われていたのが、なんらかの仕様変更により３だけが残った、というドラマがあったに１票

みんながみんな同じツッコミをしていてﾜﾗﾀ。
というワタクシも同じことを脳内で思ってました。

意外とあるんですよ、こういうコード。
# みんなそういう経験があるからつっこんでいる？

むしろ「そういうドラマがあって処理３しか通さないようにしたんだったら
その旨をコメントか何かで書いておこうぜ。な！」と思ったわけで。
3ヶ月後の自分がメンテナンスした時点で忘れてるかもしれないですしね。
昨日のズボンはもうはけない、ってNHKでも言ってたです。
（かなり年齢絞られるネタ）

自分にとってタイムリーな話で笑えない．．．
ついこないだ暇だったので、findBugsを入れてみたら
そこら中にこんなソースが、ちりばめられていました。

Mr.Tです、こんにちは。

処理3をELSEで処理するんじゃなくて、最初のIF判定Trueの部分に書きたいと思うのは私だけ？

>昨日のズボンはもうはけない、ってNHKでも言ってたです。
やべ、○学生日記ww

なんか、うちの近所でも、
if (n =! 0)
とかあったらしい。
え？これはただのバグ？

リファクタリングが必要、とのことで、見せてもらったコードに同じようなのが入ってたことがあります。
「なんでこんなことしてんの？」と聞いたら、
・処理１、処理２のコードを、新しく処理３で置き換えた。
・処理１と処理２は履歴として置いておきたい。
・コメントにしないのは、何か問題が発生したときにすぐ以前のコードに戻せるから。
とかのたまってました。
要するに、デバッグ用に(確信なく)試しにやってみたコードがたまたまうまくいったので、そのまま放置、というパターンらしい。
だから、わけわかんなくなってリファクタリングが必要になるんだろうがー！

別の人のコードを見て作ったんじゃ？
元々は「O」と書いてあったのに
「0」と間違えて書いたのでは？

皆さん同じ突っ込みされていますね＾＾；
皆さんのコメント読んでから「なるほど～」と思ったのですが、コメントにしておいてくれれば良いのに・・
やまださんが書かれていたような理由でそのままになっているのでしょうか？？
もともとのプログラムの仕様書も無いとの事だったので、もう作り直したほうが早いぞ・・・ということになりました。

Hi! Everyone who reads this blog - Happy Reconciliation and Accord..!

JQolOY Thanks for the article.Much thanks again. Keep writing.

I haven't checked in here for a while because I thought it was getting boring, but the last few posts are good quality so I guess I will add you back to my daily bloglist. You deserve it friend :)

Wow, incredible weblog layout! How long have you ever been blogging for? you made blogging glance easy. The whole glance of your web site is magnificent, as smartly as the content!

8x4e6P I loved your article post. Will read on...

Would you be all for exchanging links?