単一文字の定数はリテラルよりも優れていますか？

私は最近、ほとんどすべての単一文字を定数として提供するクラスに遭遇しました。からCOMMAまでのすべてBRACKET_OPEN。これが必要かどうか疑問に思います。私は、単一文字のリテラルを定数に取り込むことが役立つと示唆する「記事」を読みました。だから、私は懐疑的です。

定数を使用する主な魅力は、変更が必要なときにメンテナンスを最小限に抑えることです。しかし、いつコンマを表すために「、」とは異なるシンボルを使用し始めますか？

リテラルの代わりに定数を使用する唯一の理由は、コードを読みやすくすることです。しかし、city + CharacterClass.COMMA + state（たとえば）本当に読みやすいですcity + ',' + stateか？

私にとって、主にあなたが別のクラスと別のインポートを導入するという短所は長所を上回ります。そして、可能な限り少ないコードを信じています。だから、私は一般的なコンセンサスがここにあるのだろうかと思っています。

トートロジー：

質問の最初の文を読んだ場合、この質問は マジックナンバーの削除 などの適切な使用に関するものではなく、せいぜいひどい愚かな愚かな一貫性に関するものであることは非常に明白です。これはこの答えが対処するものです

常識はあなたにそれを伝えるconst char UPPER_CASE_A = 'A';か、const char A = 'A'お使いのシステムにメンテナンスや複雑さが、何も追加しません。const char STATUS_CODE.ARRIVED = 'A'別のケースです。

定数は、実行時には不変なものを表すものと想定されていますが、将来コンパイル時に変更する必要がある場合があります。いつconst char A =以外に正しく等しくなるのでしょうAか？

public static final char COLON = ':'Javaコードで見た場合、それを書いた人を見つけてキーボードを壊してください。あなたCOLONからの表現がこれまでに変更された:場合、メンテナンスの悪夢が生じます。

難読化：

誰かがそれをCOLON = '-'使用している-場所でどこでも代わりにそれを必要とするためにそれを変更するとどうなりますか？基本的assertThat(':' == COLON)にすべての単一のconst参照に対して、それらが変更されないことを確認するように言う単体テストを書くつもりですか？テストを変更したときに誰かにテストを修正してもらうだけですか？

誰かがそれpublic static final String EMPTY_STRING = "";が有用で有益であると実際に主張する場合、あなたは彼らの知識を修飾し、他のすべてについてそれらを安全に無視します。

名前付きバージョンですべての印刷可能な文字を使用できるようにすることは、誰がそれを行ったとしても、監視なしでコードを書く資格がないことを示すだけです。

凝集：

また、人工的に凝集力を低下させます。なぜなら、それを使用し、関連しているものから物を遠ざけるからです。

コンピュータプログラミングでは、結合とは、モジュールの要素がどの程度一緒に属しているかを指します。したがって、凝集度は、特定のモジュール内の機能の断片間の関係の強さを測定します。たとえば、高度に凝集したシステムでは、機能は強く関連しています。

カップリング：

また、関係のない多くのクラスも一緒に結合されます。それらはすべて、実際に実行する内容に実際には関係のないファイルを参照するためです。

密結合とは、クラスのグループが互いに強く依存している場合です。このシナリオは、クラスが多くの責任を引き受ける場合、または1つの懸念が独自のクラスを持つのではなく、多くのクラスに広がっている場合に発生します。

あなたが使用した場合より良い名前のようなDELIMITER = ','名前が総称で、何のセマンティックを運ばないので、あなたはまだ、同じ問題を抱えているだろう。値を再割り当てすることは、リテラルを検索して置換することよりも、影響分析を行うのに役立ちます','。いくつかのコードはそれを使用し、,他のコードは使用する必要がありますが、;今は必要ですか？それでも、すべての使用を手動で確認して変更する必要があります。

野生で：

私は最近、1,000,000+ LOC18歳のアプリケーションをリファクタリングしました。のようなものがありましたpublic static final COMMA = SPACE + "," + SPACE;。それは、" , "それが必要な場所を単にインライン化するよりも良い方法ではありません。

読みやすさを議論したい場合は、whitespace文字を表示できるようにIDEを構成することを学ぶ必要があります。これは、エントロピーをシステムに導入する非常に怠zyな理由です。

また、複数のパッケージおよびクラスの,単語のスペルミスを複数回定義していたCOMMA。コード内で混在するすべてのバリエーションへの参照。完全に無関係なものを壊すことなく、何かを修正しようとすることは悪夢にほかなりません。

アルファベットと同じ、複数のがありましたUPPER_CASE_A、A、UPPER_A、A_UPPERほとんどの時間に等しいだったことA が、いくつかのケースではありませんでした。ほとんどすべての文字についてですが、すべての文字についてではありません。

そして、編集履歴から、これらの単一のものが18年間にわたって編集または変更されたようには見えませんでした、今明らかな理由のために、追跡できないものがあまりにも多くの方法で壊れてしまうため、新しい変数同じ理由で変更できない同じものを指す名前。

健全な現実では、このプラクティスは何もせずに最大エントロピーから始めると主張することはできません。

私はこの混乱をすべてリファクタリングし、すべてのトートロジーをインライン化し、新しい大学の採用は、これらのconst参照が実際に指し示していた複数のレベルの間接参照を追いかける必要がないため、はるかに生産的でしたvs含まれるもの。

定数を使用する主な魅力は、変更が必要なときにメンテナンスを最小限に抑えることです。

絶対違う。定数は定義により変化しないため、これは定数を使用する理由ではありません。定数が変更された場合、それは定数ではありませんでしたか？

定数を使用することの魅力は、変更管理や、プログラムを人々が記述、理解、および保守しやすくすることとは関係ありません。コロンがURLセパレータとして使用されているプログラムのすべての場所を知りたい場合、定数URLSeparatorを定義する規律があれば簡単に知ることができます。:そして:、基本クラス、?:演算子、その他を示すために使用されるコード内のすべての場所を取得します。

私はこれが時間の無駄な無駄だと述べる他の答えには完全に反対します。名前付き定数はプログラムに意味を追加します。これらのセマンティクスは、人間と機械の両方がプログラムをより深く理解し、より効果的に維持するために使用できます。

ここでのコツは、定数を避けることではなく、構文上のプロパティではなく、セマンティックなプロパティで名前を付けることです。使用されている定数は何ですか？プログラムのビジネスドメインがタイポグラフィ、英語の構文解析などである場合を除いて、これを呼び出さないでください。モノのセマンティクスを明確にするために、それまたはそのようなものを呼び出します。CommaListSeparator

いいえ、それは愚かです。

必ずしも馬鹿げているわけではないのは、ローカライズのためにそのようなものを名前付きラベルに引き込むことです。たとえば、アメリカでは千の区切り記号はカンマ（1,000,000）ですが、他のロケールではコンマではありません。それを名前付きラベル（適切な非コンマ名）に入れると、プログラマーはそれらの詳細を無視/抽象化できます。

しかし、「マジックストリングが悪い」という理由で定数を作成することは、単に貨物を栽培することです。

あいまいになるか、いくつかの異なる目的に使用される文字がいくつかあります。たとえば'-'、ハイフン、マイナス記号、またはダッシュとしても使用します。次のように別々の名前を作成できます。

static const wchar_t HYPHEN = '-';
static const wchar_t MINUS = '-';
static const wchar_t EM_DASH = '-';

後で、コードを次のように再定義することにより、明確にするためにコードを変更することを選択できます。

static const wchar_t HYPHEN = '-';
static const wchar_t MINUS = '\u2122';
static const wchar_t EM_DASH = '\u2014';

これが、特定の単一文字に定数を定義することを検討する理由かもしれません。 ただし、この方法であいまいな文字の数は少ないです。せいぜい、あなたはそれらのためだけにそれをするだろうと思われます。また、この方法でコードを分解する前に、実際にあいまいな文字を区別する必要があるまで待つことができると主張します。

表記規則は言語や地域によって異なる可能性があるため、翻訳テーブルからこのようなあいまいな句読点をロードする方がよいでしょう。

定数は意味を追加する必要があります。

COMMAをコンマとして定義しても意味はありません。コンマはコンマであることがわかっているからです。代わりに、意味を破棄します。これは、実際にはCOMMAが実際にはコンマではなくなった可能性があるためです。

目的にコンマを使用し、名前付き定数を使用する場合は、目的に応じて名前を付けます。例：

• city + CharacterClass.COMMA + state =悪い
• city + CITY_STATE_DELIMITER + state =良い

書式設定に関数を使用する

私は個人的には好んでFormatCityState(city, state)おり、短くてテストケースに合格する限り、その関数の本体がどのように見えるかを気にしません。

定数のCOMMAは、デバンクよりも優れている','か","、かなり簡単であるという考え。もちろん、それが理にかなっている場合があります。たとえば、final String QUOTE = "\"";すべてのスラッシュなしで読みやすさを大幅に節約しますが、言語制御文字のようなものを除き、私はそれらが非常に有用である\ 'とは"思いません。

使用するのfinal String COMMA = ","は悪い形だけでなく、危険です！誰かがから、セパレータを変更したいとき","に";"、彼らは定数にファイルを変更行くかもしれないCOMMA = ";"、それは彼らがそうするために高速ですので、それだけで動作します。ただし、COMMAを使用していた他のすべてのものは、外部の消費者に送信されるものも含めてセミコロンです。したがって、すべてのテストに合格します（マーシャリングとアンマーシャリングのコードもすべてCOMMAを使用したため）が、外部テストは失敗します。

便利なのは、便利な名前を付けることです。そして、はい、時には複数の定数が同じ内容で異なる名前を持つことがあります。たとえばfinal String LIST_SEPARATOR = ","。

したがって、あなたの質問は「リテラルよりも単一の文字定数である」であり、答えは明確にいいえであり、そうではありません。しかし、これらの両方よりも優れているのは、その目的が明示的に示されている狭いスコープの変数名です。確かに、これらの追加の参照にいくつかの余分なバイトを費やします（おそらくそれらはコンパイルされないと仮定します）が、長期的なメンテナンスでは、アプリケーションのコストの大部分があります時間をかける価値があります。

字句解析器と構文解析器を作成し、整数定数を使用して端末を表現しました。単純にするために、たまたま単一文字端末はASCIIコードを数値として持っていましたが、コードはまったく別のものであった可能性があります。そのため、定数値として '、'のASCIIコードが割り当てられたT_COMMAがあります。ただし、ASCIIセットを超える整数が割り当てられた非端末の定数もありました。yaccやbisonなどのパーサージェネレーター、またはこれらのツールを使用して作成されたパーサーを見ると、基本的に誰もがそれを行ったという印象を受けました。

したがって、他のすべての人と同様に、コード全体でリテラルの代わりに定数を使用するという明確な目的のために定数を定義することは無意味であると思いますが、あなたが説明するような定数。パーサーの場合、定数は文字リテラルを表すためだけのものではないことに注意してください。それらは、たまたま文字リテラルである可能性があるエンティティを表します。

対応するリテラルの代わりに定数を使用することが理にかなっている、いくつかの孤立したケースを考えることができます。たとえば、UNIXボックスではNEWLINEをリテラル「\ n」に定義しますが、WindowsまたはMacボックスでは「\ r \ n」または「\ n \ r」を定義します。表形式のデータを表すファイルの解析についても同じことが言えます。FIELDSEPARATORおよびRECORDSEPARATOR定数を定義できます。これらの場合、特定の機能を果たす文字を表す定数を実際に定義しています。それでも、あなたが初心者プログラマーである場合、フィールドセパレーター定数をCOMMAと名付け、FIELDSEPARATORと呼ぶべきであることに気付かないかもしれません。そして、気付くまでにコードは生産中であり、次の事業、

最後に、あなたが記述の練習かもしれないあなたは、特定の文字エンコーディングでエンコードされたデータを処理するコードを記述し、いくつかのケースでは意味をなすが、ISO-8859-1を言うが、エンコーディングは後で変更することが予想されます。もちろん、このような場合、ローカライズまたはエンコードおよびデコードライブラリを使用して処理する方がはるかに理にかなっていますが、何らかの理由でそのようなライブラリを使用してエンコードの問題を処理できなかった場合は、定数を使用してソースコード全体に散らばっているハードコーディングされたリテラルの代わりに、単一のファイルで再定義する必要がある場合は、方法があります。

リンク先の記事について：文字リテラルを定数に置き換えることを主張しようとは思わない。インターフェイスを使用して定数をコードベースの他の部分にプルする方法を説明しようとしていると思います。これを説明するために使用される定数の例は、非常にひどく選択されていますが、私はそれらがどんな意味でも重要ではないと思います。

ここでのすべての素晴らしい答えに加えて、良いプログラミングは、同じコードを何度も繰り返すことなく、自分自身や他の人によって構築できる適切な抽象化を提供することだと考えてください。

優れた抽象化により、コードは使いやすく、保守も簡単になります。

私DELIMITER=':'は、それ自体が貧弱な抽象概念であり、それよりも優れているだけであることに完全に同意しますCOLON=':'（後者は完全に貧しいため）。

文字列と区切り文字を含む適切な抽象化には、1つ以上の個々のコンテンツアイテムを文字列にパックする方法と、パックされた文字列からもアンパックする方法が含まれます。このような抽象化は、ほとんどの言語ではクラスとして概念としてバンドルされます。たとえば、このクラスが使用されているすべての場所を検索し、いくつかの抽象化が使用される各ケースでパックされた文字列の形式に関するプログラマの意図を確信できるという点で、その使用は実質的に自己文書化されます。

このような抽象化が提供されると、DELIMITERorまたはCOLONisの値を確認することなく簡単に使用でき、実装の詳細の変更は一般に実装に限定されます。つまり、要するに、これらの定数は、適切な抽象化の中に隠された実装の詳細でなければなりません。

定数を使用する主な魅力は、変更が必要なときにメンテナンスを最小限に抑えることです。

通常、いくつかの関連する機能を組み合わせた優れた抽象化は、メンテナンスを最小限に抑えるのに適しています。まず、プロバイダーをコンシューマーから明確に分離します。第二に、実装の詳細を隠し、代わりに直接便利な機能を提供します。第三に、いつどこで使用されているかを高レベルで文書化します。

このような定数が効果的に使用されているのを一度目にしたのは、既存のAPIまたはドキュメントを一致させることです。COMMA特定のソフトウェアがCOMMA抽象構文ツリーのタグとして使用されるパーサーに直接接続されていたため、使用されているようなシンボルを見てきました。また、正式な仕様に一致するために使用されることも確認しました。正式な仕様では、可能な限り完全に明確にしたいという理由でCOMMAはなく、シンボルが表示さ','れることがあります。

どちらの場合も、名前の付いたシンボルを使用COMMAすると、別の方法でばらばらになった製品に凝集性を提供できます。その値は、過度に冗長な表記法のコストを上回る場合があります。

リストを作成しようとしていることに注意してください。

したがって、次のようにリファクタリングします。 String makeList(String[] items)

つまり、データの代わりにロジックを除外します。
言語はリストの表現方法が異なる場合がありますが、コンマは常にコンマです（トートロジーです）。したがって、言語が変更された場合、コンマ文字を変更しても役に立ちませんが、これは役立ちます。

これが仲間の開発者によってアプリケーションの一部として記述されたクラスである場合、これはほぼ間違いなく悪い考えです。他の人が既に指摘したようにSEPARATOR = ','、値を変更できる場所などの定数を定義することは理にかなっています。

ただし、名前がその内容を正確に記述する定数を宣言するのが理にかなっている場合と、定数の名前を適切に変更しないと値を変更できない場合が少なくとも2つあります。

• 数学定数または物理定数PI = 3.14159。ここで、定数の役割は、記号名PIが表す値よりもはるかに短く、読みやすいため、ニーモニックとして機能することです。
• パーサー内の記号の完全なリストまたはキーボード上のキー。ほとんどまたはすべてのUnicode文字を含む定数のリストを用意するのも理にかなっているかもしれません。などの一部の文字Aは明白であり、はっきりと認識できます。しかし、あなたは簡単に伝えることができますАし、A離れて？最初の文字はキリル文字Аで、後者はラテン文字Aです。それらはグラフィカルにほとんど同じですが、異なるUnicodeコードポイントで表される異なる文字です。私はむしろ定数CYRILLIC_CAPITAL_Aを持っていて、LATIN_CAPITAL_A私のコードでは、ほぼ同一に見える2つの文字よりも。もちろん、キリル文字を含まないASCII文字のみで作業することがわかっている場合、これは無意味です。同様に、私は毎日ラテンアルファベットを使用しているため、中国語の文字を必要とするプログラムを作成する場合は、理解できない文字を挿入するよりも定数を使用することをお勧めします。日常的に漢字を使用している人にとっては、漢字は明白かもしれませんが、ラテン文字は名前付き定数として表現する方が簡単かもしれません。したがって、ご覧のとおり、それはコンテキストに依存します。それでも、作成者はライブラリの使用方法や特定のアプリケーションで読みやすさを向上させるために定数が必要な文字を事前に知ることができないため、ライブラリにはすべての文字の記号定数が含まれる場合があります。

ただし、このような場合は通常、システムクラスまたは専用ライブラリによって処理され、アプリケーション開発者によって記述されたコード内で発生することは、非常に特別なプロジェクトに取り組んでいない限り非常にまれです。

多分。

単一文字の定数を区別するのは比較的困難です。したがって、カンマではなくピリオドを追加しているという事実を見逃すのはかなり簡単です

city + '.' + state

一方、それは比較的難しい間違いです

city + Const.PERIOD + state

国際化およびグローバリゼーション環境によっては、ASCIIアポストロフィとWindows-1252のオープンおよびクローズアポストロフィ（またはASCIIの二重引用符とWindows-1252のオープンおよびクローズの二重引用符）の違いが大きく、視覚化が難しいことで有名ですコードで。

おそらく、コンマではなくピリオドを誤って入力することが重要な機能上の問題であった場合、タイプミスを見つける自動テストが行​​われるでしょう。ソフトウェアがCSVファイルを生成している場合、テストスイートが、市と州の間に期間があることを非常に迅速に発見すると期待しています。ソフトウェアがさまざまな国際化構成を持つクライアントで実行されることになっている場合、おそらくテストスイートは各環境で実行され、アポストロフィが必要な場合はMicrosoftのオープンクォートがある場合に取得されます。

特に包括的なテストスイートがない古いコードを使用している場合でも、これらの問題を回避できる詳細なコードを選択する方が理にかなっているプロジェクトを想像できます。グリーンフィールド開発プロジェクト。また、特定のアプリケーションで問題となる可能性がある句読点文字だけでなく、すべての句読点文字に定数を追加することは、おそらく過度の過剰です。

単一文字の定数はリテラルよりも優れていますか？

この辺りには多くの混乱が浮かんでいます。それらをバラバラにいじめることができるかどうか見てみましょう。

定数は以下を提供します：

• 意味論
• 変更、開発中
• 間接

単一の文字名に移動しても、セマンティクスにのみ影響します。名前はコメントとして有用で、文脈が明確でなければなりません。値ではなく意味を表現する必要があります。それがすべて単一の文字でうまくできるなら。できない場合は、しないでください。

リテラルと定数はどちらも開発中に変更できます。これがマジックナンバーの問題を引き起こします。文字列もマジックナンバーにすることができます。

セマンティックな意味が存在し、両方が定数である場合、定数がリテラルよりも大きな値を持っているかどうかは間接になります。

インダイレクションは、多くのインダイレクション以外の問題を解決できます。

インダイレクションは、1つの場所でアイデアの値を決定できるため、マジックナンバーの問題を解決できます。意味的には、それが価値があるためには、その名前がそのアイデアが明確であるようにしなければなりません。名前は価値ではなく、アイデアに関するものでなければなりません。

インダイレクションはやり直すことができます。リテラルを検索および置換して変更を加えることを好む人もいます。42が明らかに生命の意味であり、42のモリブデン原子番号と混同されない限り、それは問題ありません。

単一の文字でそのような有用な区別を行うことができる場所は、コンテキストに大きく依存します。しかし、私はそれを習慣にするつもりはありません。

大多数の意見に対する哲学的な矛盾として、私は、洗練されていない19世紀のフランスの農民プログラマーと

単調で永遠に続く明快さ、すべてに対する驚くほど賢明な見方、真実であるという理由だけでの自閉症との膨大な満足を思い出しました。「それをすべて混乱させます！」「彼が亡命中であれば、外に誰もいられない」とターンブルを叫んだ。

GKチェスタトン、ボールと十字架

真実を正しく評価することには何の問題もありません。特にコンピューターと話すときは、真実を述べることに何の問題もありません。

コンピューターに嘘をつくと、

ペリー・ファラー-メリーランド州ジャーマンタウン（More Programming Pearlsより）

しかし、ほとんどの場合、私はそれが馬鹿だと言う人々に同意します。FORTRANのプログラミングを学ぶには若すぎますが'A' = 'Q'、あらゆる種類のすばらしい暗号を再定義して思い付くことができると聞いたことがあります。あなたはこれをしていません。

以前に提起されたi18nの問題を超えて（これはグリフ "COMMA"を再定義するのではなく、DECIMAL_POINTのグリフを本当に再定義する）。人間に意味を伝えるためにフランスのニンジン引用符または英国の単一引用符を作成することは重要であり、それらは実際には定数ではなく変数であるべきです。定数は次のようになりますAMERICAN_COMMA := ','し、comma := AMERICAN_COMMA

また、ビルダーパターンを使用してSQLクエリを構築している場合は、

sb.append("insert into ")
 .append(table_name)
 .append(" values ")
 .append(" ( ")
 .append(val_1)
 .append(",")
 .append(val_2)
 .append(" ); ")

他の何よりも、しかしあなたが定数を追加しようとするなら、それは

INSERT_VALUES_START = " ( "
INSERT_VALUES_END = " ) "
INSERT_VALUES_SEPARATOR = " , "
QUERY_TERMINATOR = ";"

sb.append("insert into ")
 .append(table_name)
 .append(" values ")
 .append(INSERT_VALUES_START)
 .append(val_1)
 .append(INSERT_VALUES_SEPARATOR)
 .append(val_2)
 .append(INSERT_VALUES_END)
 .append(QUERY_TERMINATOR)

ただし、他の誰かがプログラム（またはタイプ）を視聴したことがある場合は、いくつかの興味深い癖に気付くかもしれません。私たち全員が恒星タイピストではありません。私たちの多くは遅くプログラミングに参加したか、ソビエトのキーボード（キーが入力される場所）で育ちました。キーボードで文字を見つけたりオートコンプリートに頼ったりする代わりに、個々の文字をカットアンドペーストしたいのです。

文字列を自動補完するものは何もないので、「con」、alt-space、down、down、down、Enterを押してコンマを取得し、「con」、alt-space、down、ダウン、入力します。私はそれをするかもしれません。

文字列リテラルについて覚えておくべきもう1つのことは、それらのコンパイル方法です。少なくともDelphiでは（これが私がスタックに取りつかれている唯一の言語です）、各関数のスタックにポップされたリテラルを作成します。したがって、多くのリテラル=多くの関数オーバーヘッド; function_Aの「、」は、function_Bの「、」と同じメモリビットではありません。これに対処するために、横に構築およびリンクできる「リソース文字列」があります。 Pythonでは、すべての文字列リテラルはオブジェクトであり、実際に使用するのutils.constants.COMMA.join(["some","happy","array","strings"])が良いように思えるかもしれませんが、このページで何度も繰り返される点については素晴らしい考えではありません。

しかし、いつコンマを表すために「、」とは異なるシンボルを使用し始めますか？

ローカリゼーション用。

英語圏の国では、小数部の整数部と小数部を分ける記号は「。」であり、これを「小数点」と呼びます。他の多くの国では、記号は「、」であり、通常は現地語で「コンマ」と同等と呼ばれます。同様に、英語圏の国が「、」を使用して3桁のグループを大きな数字で区切る場合（100万分の1,000,000など）、カンマを小数点として使用する国ではドット（1.000.000）を使用します。

したがって、グローバリゼーションを行っている場合は、DECIMAL_POINTおよびCOMMA定数を作成する場合があります。