異なるクラスのオブジェクトのJavaリフレッシュインスタンス

Question

私は3つの異なるクラスを持っていますが、すべて1つのクラスのインスタンスを使用する必要があります。その1つのクラスは、player1、player2などの複数のインスタンスを独自に作成することもできますし、staticは古い名前を上書きするため、静的に動作しません。

私は最後のクラスで "new"キーワードを使用すべきではないことを知っていますが、それ以外の方法はわかりません。

public class Test{ 
    public static void main(String[] args){ 
     Player player1 = new Player("bob"); 
     Player player2 = new Player("Hank"); 
     System.out.println("Original Name: " + player1.getName()); 
     System.out.println("Original Name 2: " + player2.getName()); 
     Display dis = new Display(); 
     dis.disp(); 

     System.out.println("Changed Name: " + player1.getName()); 
     System.out.println("Changed Name 2: " + player2.getName()); 
    } 
} 

class Player{ 
    private String pName = ""; 
    public Player(){} 
    public Player(String name){ 
     pName = name; 
    } 

    public void setName(String inName){ 
     pName = inName; 
    } 

    public String getName(){ 
     return pName; 
    } 
} 

class Display{ 
    public void disp(){ 
     Player player1 = new Player(), player2 = new Player(); //Unneeded 
     System.out.println("Player name: " + player1.getName()); 
     System.out.println("Player name 2: " + player2.getName()); 

     player1.setName("Joe?"); 
     player2.setName("Billy?"); 

     System.out.println("Player new name: " + player1.getName()); 
     System.out.println("Player new name 2: " + player2.getName()); 
    } 
} 

Answer 1

あなたがDisplayにあなたがTestで作成しPlayerの同じインスタンスを変更しようとしているようです。それはケースが実際にある場合は、disp()にこれらのオブジェクトを渡す必要があります：メソッドを呼び出す

が

public void disp(Player player1, Player player2){ 
    System.out.println("Player name: " + player1.getName()); 
    System.out.println("Player name 2: " + player2.getName()); 

    player1.setName("Joe?"); 
    player2.setName("Billy?"); 

    System.out.println("Player new name: " + player1.getName()); 
    System.out.println("Player new name 2: " + player2.getName()); 
} 

次のようになります。dis.disp(player1,player2);

私はここでいくつかのコードの臭いがあることを示す必要がありますが、それはのように思えますそれがそうであれば、この段階でそれについてあまり心配するべきではありません。

Answer 2

Displayクラスには、必要に応じて名前を変更するなど、必要に応じて外部クラスがプレイヤーと対話できるようにする2つのプライベートPlayerフィールドとパブリックメソッドを与えます。クラスsetPlayer1(Player p)とsetPlayer2(Player p)のメソッドを与えたり、クラスがコンストラクターパラメーターを介して2つのプレーヤーを受け入れるようにすることができます。

非常にの理由がない限り、静的変数の使用は避けてください。

Answer 3

さて、なぜあなたは、クラスにしない：

class Display{ 
    public static void disp(Player player, String newName){ 
     System.out.println("Player name: " + player.getName()); 

     player.setName(newName); 

     System.out.println("Player new name: " + player.getName()); 
    } 
} 

をして呼び出し：テストクラスで

Display.disp(player1, "Joe?"); 
Display.disp(player2, "Hank?"); 

が

ので、基本的には、単に表示の静的メソッドを作りますパラメータとして適切なフィールドを入力します。