マジックナンバーの削除：「いいえ」と言うタイミングはいつですか？

マジックナンバー（ハードコードされた値）がプログラムに大混乱をもたらす可能性があることは誰もが知っています。特にコメントのないコードセクションを変更するときは、どこに線を引きますか？

たとえば、2日間の秒数を計算する関数がある場合、置き換えますか

seconds = num_days * 24 * 60 * 60

と

seconds = num_days * HOURS_PER_DAY * MINUTES_PER_HOUR * SECONDS_PER_MINUTE

どの時点で、ハードコーディングされた値の意味が完全に明らかであると判断し、そのままにしておくのですか？

数値リテラルの代わりにシンボリック定数を使用する理由は2つあります。

1. マジックナンバーが変更された場合のメンテナンスを簡素化するため。これはあなたの例には当てはまりません。1時間の秒数や1日の時間数が変わることはほとんどありません。

2. 読みやすさを向上させるため。「24 * 60 * 60」という表現は、ほとんどの人にとって明らかです。「SECONDS_PER_DAY」もありますが、バグを探している場合は、SECONDS_PER_DAYが正しく定義されていることを確認する必要があります。簡潔さには価値があります。

一度だけ出現し、プログラムの残りの部分から独立しているマジックナンバーの場合、そのナンバーのシンボルを作成するかどうかは好みの問題です。疑問がある場合は、先に進み、シンボルを作成してください。

こんなことしないで：

public static final int THREE = 3;

私は魔法の数字を持たないというルールを守ります。

ながら

seconds = num_days * 24 * 60 * 60

クランチモードで3〜4週間、1日10時間コーディングした後、ほとんどの場合、完全に読み取り可能

seconds = num_days * HOURS_PER_DAY * MINUTES_PER_HOUR * SECONDS_PER_MINUTE

読みやすくなっています。

FrustratedWithFormsDesignerの提案が優れています：

seconds = num_days * DAYS_TO_SECOND_FACTOR

またはさらに良い

seconds = CONVERT_DAYS_TO_SECONDS(num_days)

あなたが非常に疲れているとき、物事は明らかでなくなります。防御的にコードします。

ノーと言う時はほとんどいつもです。ハードコードされた数字を使用する方が簡単だと思うときは、UIレイアウトなどの場所にあります-フォーム上のすべてのコントロールの配置に定数を作成すると、非常に手間がかかり疲れます。そのコードは通常UIデザイナーによって処理されます大した問題ではありません。... UIが動的にレイアウトされる場合、またはアンカーに対する相対位置を使用する場合、または手動で作成される場合を除きます。その場合、レイアウトに意味のある定数を定義する方が良いと思います。そして、何かを「ちょうどいい」ように配置/配置するために、あちこちでファッジファクターが必要な場合は、それも定義する必要があります。

しかし、あなたの例では、置換24 * 60 * 60するDAYS_TO_SECONDS_FACTOR方が良いと思います。

コンテキストと使用法が完全に明確な場合、ハードコードされた値でも問題ないことを認めます。これは、しかし、判断の呼び出しです...

例：

@rmxが指摘したように、0または1を使用してリストが空であるか、ループの境界にあるかどうかを確認することは、定数の目的が非常に明確な場合の例です。

番号の意味や目的を特定できない場合は停止してください。

seconds = num_days * HOURS_PER_DAY * MINUTES_PER_HOUR * SECONDS_PER_MINUTE

数字を使用するよりも読みやすくなっています。（単一のSECONDS_PER_DAY定数を持つことで読みやすくすることもできますが、それは完全に別の問題です。）

コードを見ている開発者がそのコードの動作を確認できると仮定します。しかし、彼らもその理由を知っていると仮定しないでください。あなたの定数がその理由を理解するのを助けてくれるなら、それを選んでください。そうでない場合は、しないでください。

1つの回答で示唆されたように、定数が多すぎる場合は、代わりに外部構成ファイルを使用することを検討してください。ファイルに数十個の定数があると、読みやすさが改善されません。

私はおそらく次のようなことに対して「いいえ」と言うでしょう。

#define HTML_END_TAG "</html>"

そして間違いなく「いいえ」と言うでしょう：

#define QUADRATIC_DISCRIMINANT_COEF 4
#define QUADRATIC_DENOMINATOR_COEF  2

次のような明らかなことのために定数の使用を促進するために見つけた最高の例の1つは、HOURS_PER_DAY次のとおりです。

私たちは、物事が人の仕事の列にどれくらい座っているかを計算していました。要件は大まかに定義され、プログラマ24は多くの場所でハードコーディングされました。最終的には、実際に1日8時間しか働かないのに、ユーザーが24時間問題を抱えていることを罰するのは公平ではないことがわかりました。タスクがこれを修正し、他のレポートに同じ問題があるかもしれないものを見ると、24のコードをgrep /検索することはかなり困難でした。HOURS_PER_DAY

数が完全に一定で、変更の可能性がない限り、それは完全に受け入れられると思います。したがって、あなたの場合は、seconds = num_days * 24 * 60 * 60（もちろん、ループ内でこの種の計算を行うような愚かなことをしないと仮定して）うまくいき、間違いなく読みやすさよりも優れていseconds = num_days * HOURS_PER_DAY * MINUTES_PER_HOUR * SECONDS_PER_MINUTEます。

このようなことをすると悪いのです。

lineOffset += 24; // 24 lines to a page

ページにこれ以上行を挿入できなかった場合や、変更する意図がない場合でも、代わりに定数変数を使用してください。最終的に、ポイントは可読性であり、CPUで2サイクルの計算を節約しません。貴重なバイトがすべての価値のために絞られたとき、これはもはや1978年ではありません。

seconds = num_days * 24 * 60 * 60

まったく問題ありません。これらは変更されることはないため、実際には魔法の数字ではありません。

合理的に変更できる、または明確な意味を持たない数値は、変数に入れる必要があります。これはほとんどすべてを意味します。

ある単位から別の単位に値を変換するための定数（マジック値）を作成することは避けます。変換する場合、話すメソッド名を好みます。この例では、これはたとえばDayToSeconds(num_days)内部的なものになります。「24」と「60」の意味が明確であるため、メソッドはマジック値を必要としません。

この場合、秒/分/時間を使用することはありません。TimeSpan / DateTimeのみを使用します。

決定するパラメータとしてコンテキストを使用します

たとえば、「calculateSecondsBetween：aDay and：anotherDay」という関数がある場合、関数名が非常に代表的であるため、これらの数値が何をするかについてあまり説明する必要はありません。

そして別の質問は、それを異なる方法で計算する可能性はどれですか？同じことをする方法がたくさんあることがあるので、将来のプログラマーをガイドし、使用した方法を示すために、定数を定義することでそれを理解することができます。