- 今までみた絶望的なソースコード [転載禁止]©2ch.net
404 :デフォルトの名無しさん[sage]:2015/10/26(月) 23:04:21.65 ID:nye5D6dJ - マジックナンバーは必要悪でしょ。
周知されてて、全体で統一されてたら何も文句無いけどな。 日時未定を29991231で持ってるシステムに関わってたけど。 特定のビットが立ってないことと、ファイルの有無と、ディレクトリであるか否かはまた別問題ではないのだろうか。 isExistなのに、ディレクトリ除外しちゃうの?ライブラリなのに外から見た挙動変えちゃうの?って不安のほうが大きい。 そんなもんで判定せずに、判定箇所で切り分けるか、旧関数自体をassertで殺す方がマシ。 その度に多数の修正は躊躇なく入れるべき。 挙動を変えるのに、変えた意識が無い方が、 あり得ないくらい誰も原因に気づかない、どハマりする障害を起こす元。
|
|