特定の値を文字列として保存するのは悪い習慣ですか？

それは非常に曖昧なタイトルですが、私はそれを表現するより良い方法を考えることができませんでした。しかし、ちょうど一例として、ゲーム内のキャラクターが動いている方向について考えてください。文字列を使用してからのようなことをするのはちょっと間違っているように感じif(character.direction == "left")ます。の代わりにを誤って使用しLeftたりl、他の何かを使用したりするなど、愚かなエラーの余地が多すぎるように思えleftます。私の疑いは正しいですか？もしそうなら、このような何かを達成するための好ましい方法は何ですか？

使用している言語が列挙型の使用をサポートしている場合、それらを使用します。特定のタイプで使用可能なオプションの数を制限できます。例：Javaの場合：

public enum Direction {
    LEFT,
    RIGHT,
    UP,
    DOWN
}

コードでリテラル文字列（またはマジックナンバー）を使用するのはひどい習慣です。

列挙型は優れているか、少なくとも定数を使用します（もちろん、列挙型は1つ以上の関連する定数の単なるラッパー型です）。

const string LEFT = "left"
if (someThing == LEFT) { doStuff(); }

この単純なスイッチには多くの利点があり、それらの説明をどこから始めればよいかさえわかりません（少なくともコーヒーがなければ）。マジックナンバーを読んでください。これはまったく同じ概念で、文字列値ではなく数値にのみ適用されます。

ここに簡単な参考文献があります、もっとたくさんあると確信しています：https : //stackoverflow.com/questions/47882/what-is-a-magic-number-and-why-is-it-bad

簡単な答え：はい、文字列は一連のテキスト文字の保存とアクセス以外のタスクには理想的ではありません。抽象化の基礎となるビットが文字列であっても、変数または定数として参照することには利点があります。

長い答え：ほとんどの言語は、問題のドメインに近い型を提供します。そうでない場合でも、おそらく（etcなど）leftとは異なるエンティティとして定義できる方法がありますright。を使用して文字列に変換すること"left"は、単に言語およびエラーチェックなどのIDEの便利な機能を失うことです。使用する必要がある場合でも

left = "left";

または何か同等のものを使用すると、コード全体で文字列を参照するときに文字列を使用する利点があります。例えば、

if (player.direction == Left)

コンパイル時エラー（ベストケース、javaなど）として検出されるか、またはそのビットに相当するIDEによってフラグが付けられます（最悪の場合、基本など）。

さらにleft、参照可能なエンティティとしての概念を持つことは、方向のタイプに応じて決定する方向に対応するのに役立ちます。を使用することにより"left"、方向はになりStringます。型のより良い抽象化を見つけたり作成したりする場合は、コード本体全体を探索して、のすべてのインスタンスを変更する必要があります"left"。型が作成されている場合、定数または変数を変更する必要はありません。

本当に必要なタイプは、ドメインに固有のものです。あなたのコードの計画は何leftですか？おそらく、左を向いている、または前方に移動しているテクスチャまたはスプライトのx軸をミラーリングしたいでしょう。その場合、leftその動作を反映するプロパティまたはメソッドを使用してrightオブジェクトを作成し、オブジェクトは反対のミラー化されていない結果になる場合があります。スプライトまたはテクスチャを表すオブジェクトでそのロジックを実行できます。この場合も、上記の理由では"left"なく使用することに苦しむでしょうleft。

Java（およびおそらく私がまだ知らない他の言語）ではenum、Java の場合はインスタンスの有限セットとenum同様に有用であるため、優れた代替手段ですfinal class。必要に応じてビヘイビアを定義できます。またenum、を宣言するファイルの外部で手間をかけずにオープンクラスに切り替えることができますが、多くの言語はをenumプリミティブタイプと見なすか、使用するために特別な構文を必要とします。それenumは、それと関係のないコードにフードをリークし、後で修正する必要があるかもしれません。enumオプションを検討する前に、言語の概念を理解してください。

言語を指定しませんでしたが、一般的な答えを出すには、何かを表すのではなく、実際にテキストが必要なときに文字列を使用する必要があります。たとえば、あなたが与えた例は、実稼働ソフトウェアでは受け入れられません。

すべての言語にはさまざまな基本データ型があり、タスクに最も適したデータ型を使用する必要があります。この例を使用するには、数値定数を使用してさまざまな状態を表すことができます。したがって、leftの値は0で、rightの値は1です。あなたが言及した理由に加えて、数値はより少ないスペース（メモリ）を占有し、複数の文字列を比較するよりも数値を比較する方が常に高速です。

提案されているように、言語で許可されている場合は列挙を使用します。あなたの特定の例では、明らかにそれがより良い選択です。

状況に合ったデータ型を使用してください。

string基になる内部/アプリケーション内通信としての型の使用は、本質的に問題ではありません。実際には、時にはそれがだ望ましい文字列を使用する：あなたは、パブリックAPIを使用している場合、それは人間が読めるようにAPIの中に出入り値のために望ましい可能性があります。APIを使用している人が学習やデバッグを簡単に行えるようになります。

コードの本当の問題は、値を繰り返すことにあります。

これをしないでください：

if (direction == 'left') {
  goLeft();
}

これらの条件のいずれかまたは「左」のその他の使用方法のいずれかでタイプミスをした場合、タイプミスしたため、コードベース全体の検索を効果的に実行できない可能性があります。

代わりに、使用している言語で必要なデータ型をサポートしているものに応じて、列挙または定数に対するコードを作成します。

手段はそれLEFTに、割り当てる必要があります一度と一度だけ、アプリケーションインチ そして、残りのコードはこれらの列挙型または定数を活用する必要があります。

const string LEFT = "left";

if (direction == LEFT) {
  goLeft();
}

これにより、必要に応じて、後でルートに使用するデータ型をより簡単に変更することもできます。

それは悪い習慣ですか？

おそらくはい、それは依存正確にあなたが何をしているか、また、使用しているプログラミング言語。

あなたの例では、方向が取ることができる値のセットが小さく固定されていることが推測できます。この場合、文字列を使用する利点はなく、明確な欠点もあります。この例では：

• enumあなたのプログラミング言語がこれをサポートしている場合のタイプは、望ましいだろう。
• それ以外の場合は、名前付き整数定数を使用して、定数を1つ定義します。

しかし、値のセットが動的に変更可能である場合、または時間とともに進化する必要がある場合、答えは異なる場合があります。ほとんどの言語では、enumタイプの変更（値の追加または削除など）を行うには、完全な再コンパイルが必要です。（たとえば、Javaで列挙値を削除すると、バイナリの互換性が失われます。）名前付き整数定数にも同じことが当てはまります。

このようなシナリオでは、別のソリューションが必要になる場合があり、文字列の使用はオプションの1つです。（そして、シナリオが動的拡張機能を備えた固定コアを必要とする場合は、文字列リテラルと名前付き定数を組み合わせることができます。）

Javaで実装している場合character.direction == "left"、別の理由で悪い習慣です。==文字列の同等ではなくオブジェクトのIDをテストしているため、文字列の比較には使用しないでください。（"left"文字列のすべてのインスタンスがインターンされたことを保証できれば機能しますが、それにはアプリケーション全体の分析が必要です。）

提案されているように、Enumを使用する必要があります。ただし、Strignsの代わりにEnumを使用するだけでは不十分です。あなたの特定の例では、プレイヤーの速度は実際には物理学によるベクトルでなければなりません：

class Vector {
  final double x;
  final double y;
}

次に、このベクトルを使用して、ティックごとにプレーヤーの位置を更新する必要があります。

次に、それを列挙型と組み合わせて読みやすくします。

enum Direction {
  UP(new Vector(0, 1)),
  RIGHT(new Vector(1, 0)),
  DOWN(new Vector(0, -1)),
  LEFT(new Vector(-1, 0));

  private Vector direction;

  Direction(Vector v) {
    this.direction = v;
  }

  public Vector getVector() { return this.direction; }
}