Tidy First? ―個人で実践する経験主義的ソフトウェア設計

9章説明定数

さて、コードを読んでいると、理解できない数字を見ることがある。もしくは、コード中に繰り返し現れる文字列の固定値を見ることがある。その固定値が何を表しているのかを知ろうとする。

シンボリック定数を作ろう。リテラル定数を使うのをやめ、シンボリック定数で置き換えよう。

まあ、見てほしい。私がまだ駆け出しのプログラマーだったころからよく目にしたアドバイスだ。いまだにこんなコードが許されると思っている人も多いらしい。

if response.code = 404
    ...ほげほげ...

すまない。ちょっと嫌味ったらしかった。とっ散らかす人たち（ヒント：自分かも）を裁くことが目的ではないのだ。何かを変更する前に自らが先に整頓できる人間になることが目的なのだ。

PAGE_NOT_FOUND := 404
if response.code = PAGE_NOT_FOUND
    ...ほげほげ...

ここで注意してほしいことがある。同じリテラルが2箇所に現れ、別の意味を持つことがあるのだ。これでは整頓しても役に立たない。

ONE = 1
...ONE... # 1が必要な箇所すべて

コードを読んでいく。理解する。その理解をコードに落とせば、頭の隅に置いておく必要がなくなる。

これに関連して、他にも整頓がある。まとめて理解あるいは変更する必要がある定数を1箇所に集めたり、何らかの理由でひとかたまりになっている定数を分離したりだ。それは自分で考えてほしい。結合、凝集、お好きにどうぞ。