Java-完全に静的なクラスを持つことは悪い考えですか？

私は現在、より大きなソロプロジェクトに取り組んでおり、インスタンスを作成する理由がわからないクラスがいくつかあります。

たとえば、現在のダイスクラスはすべてのデータを静的に格納し、そのメソッドもすべて静的です。サイコロを転がして新しい値を取得したいときは、を使用するだけなので、初期化する必要はありませんDice.roll()。

私はこのような主な機能を1つだけ持っているいくつかの類似したクラスを持ち、すべてのイベントを担当する一種の「コントローラー」クラスで作業を開始しようとしています（プレイヤーが移動したとき、そうです）、このクラスでも同じ考えに従うことができることがわかりました。これらの特定のクラスに対して複数のオブジェクトを作成する予定はないので、それらを完全に静的にすることは悪い考えでしょうか？

Javaに関しては、これが「悪い習慣」と見なされるのかと思っていました。私が見たものから、コミュニティはこのトピックに関して一種の分裂のように思われますか？とにかく、私はこれに関するいくつかの議論が大好きだし、リソースへのリンクも素晴らしいでしょう！

本当に静的な静的クラスには何も問題はありません。つまり、メソッドの出力が変化する原因となる内部状態はありません。

Dice.roll()1から6までの新しい乱数を単に返す場合、状態は変化しません。確かに、あなたはRandomインスタンスを共有しているかもしれませんが、定義による状態の変化は、出力が常に適切でランダムになるとは考えません。また、スレッドセーフであるため、問題はありません。

多くの場合、最終的な「ヘルパー」またはプライベートコンストラクターと静的メンバーを持つ他のユーティリティクラスが表示されます。プライベートコンストラクターにはロジックが含まれておらず、誰かがクラスをインスタンス化するのを防ぐためだけに使用されます。最後の修飾子は、これが派生したいクラスではないという考えを持ち帰ります。単なるユーティリティクラスです。適切に行われれば、静的で最終的なものではないシングルトンや他のクラスメンバが存在しないはずです。

これらのガイドラインに従い、シングルトンを作成していない限り、これにはまったく問題はありません。コントローラークラスについて言及しますが、これにはほぼ確実に状態の変更が必要になるため、静的メソッドのみを使用することはお勧めしません。あなたは、静的なユーティリティクラスに大きく依存していることはできますが、できません作ること静的なユーティリティクラス。

クラスの状態の変化とは何ですか？乱数は定義上非決定的であるため、戻り値は頻繁に変更されるため、1秒間、乱数を除外します。

純粋な関数とは、決定論的な関数のことです。つまり、特定の入力に対して、1つだけの出力が得られます。静的メソッドを純粋な関数にする必要があります。Javaには、状態を保持するために静的メソッドの動作を微調整する方法がありますが、それらはほとんど良いアイデアではありません。メソッドをstaticとして宣言すると、典型的なプログラマーは、すぐにそれが純粋な関数であると想定します。予想される動作から逸脱することは、一般的に言えばプログラムでバグを作成する傾向があるため、避けるべきです。

シングルトンは、できる限り「純粋な関数」とは反対の静的メソッドを含むクラスです。単一の静的なプライベートメンバーは、クラスに対して内部的に保持されます。これは、インスタンスが1つだけであることを保証するために使用されます。これはベストプラクティスではなく、いくつかの理由で後で問題が発生する可能性があります。私たちが話していることを知るために、シングルトンの簡単な例を以下に示します。

// DON'T DO THIS!
class Singleton {
  private String name; 
  private static Singleton instance = null;

  private Singleton(String name) {
    this.name = name;
  }

  public static Singleton getInstance() {
    if(instance == null) {
      instance = new Singleton("George");
    }
    return instance;
  }

  public getName() {
    return name;
  }
}

assert Singleton.getInstance().getName() == "George"

staticクラスの制限の例を挙げるために、あなたのゲーマーの何人かが彼らのサイコロにわずかなボーナスを得たいとしたらどうでしょうか？そして、彼らは大金を払っても構わないと思っています！:-)

はい、別のパラメーターを追加できます。そのためDice.roll(bonus)、

後でD20が必要になります。

Dice.roll(bonus, sides)

ええ、しかし、一部のプレーヤーは「最高に能力のある」特技を持っているので、決して「手探り」することはできません（1をロール）。

Dice.roll(bonus, sides, isFumbleAllowed)。

これは面倒になりつつありますか？

Diceクラスの特定のケースでは、静的メソッドではなくインスタンスメソッドを使用すると、テストが大幅に簡単になると思います。

Diceのインスタンス（Gameクラスなど）を使用する何かを単体テストする場合、テストから、常に固定値のシーケンスを返すDiceのテストダブルのフォームを挿入できます。あなたのテストは、それらのサイコロを振るのに適切な結果がゲームにあることを確認できます。

私はJava開発者ではありませんが、完全に静的なDiceクラスでこれを行うのはかなり難しいと思います。/programming/4482315/why-does-mockito-not-mock-static-methodsを参照してください

これは実際にはMonostateパターンとして知られており、すべてのインスタンス（および「インスタンスなし」）がステータスを共有しています。すべてのメンバーはクラスメンバーです（つまり、インスタンスメンバーはありません）。それらは一般に、単一の責任または要件に関連するメソッドと定数のセットをバンドルする「ツールキット」クラスを実装するために使用されますが、動作するために状態を必要としません（純粋に機能します）。実際、Javaにはそれらの一部がバンドルされています（たとえば、Math）。

オフトピックビット：私はめったにVisualBasicの中のキーワードの命名に同意していないが、この場合には、私が考えてshared（それがさ、確かに明確で、意味的に優れた共有よりも、クラス自体とそのすべてのインスタンス間で）staticライフサイクルの後に残ります（で宣言されているスコープの）。