Simpeオブジェクトとクラスの問題 - Java

Question

コードソースを投稿するのは初めてです。私はObject &コンストラクターを練習していますが、このコードを作成することになりましたが、実行するとStringGeneratedがnullになり、「The Color Generated was：null」と表示されます。私はそれを変更して修正しようとしましたが、失敗しました。読んでくれてありがとう。

public class Color{ 
    int colorValue; 
    String colorGenerated1; 

    public Color(String colorGenerated){ 
     System.out.println("The Color Generated was: " + colorGenerated); 
    } 

    public void randomizeColor(){ 
     Random randomColor= new Random(); 
     colorValue = randomColor.nextInt(3); 
    } 

    public int getColor(){ 

     if(colorValue == 1){ 
      colorGenerated1 = "Red"; 
     } else if(colorValue == 2){ 
      colorGenerated1 = "Blue"; 
     } else if(colorValue == 3){ 
      colorGenerated1 = "Yellow"; 
     } 

     return colorValue; 
    } 

    public static void main(String[] args){ 
     Color color = new Color("colorGenerated"); 
     color.randomizeColor(); 
     color.getColor(); 
    } 
} 

Answer 1

ランダムな色で設定する前に生成された色を印刷しています。 Randomは（メソッド呼び出しごとに1回ではなく）1回だけ取得する必要があります。私は何かのようにします

public class Color { 
    private static Random random = new Random(); 
    int colorValue = random.nextInt(3) + 1; // nextInt excludes 3 and includes 0 

    @Override 
    public String toString() { 
     if (colorValue == 1) { 
      return "Red"; 
     } else if (colorValue == 2) { 
      return "Blue"; 
     } else if (colorValue == 3) { 
      return "Yellow"; 
     } 
     return "Unknown"; 
    } 

    public static void main(String[] args) { 
     Color color = new Color(); 
     System.out.println("The Color Generated was: " + color); 
    } 
} 

Answer 2

a readを持ってください。あなたのコードは正常に動作しています。

• コンストラクタにprintステートメントを追加しました。&オブジェクトを最初に初期化しました。
• 次に、色をランダム化するコードを追加しました&色を印刷する方法はありません。
• したがって、コンストラクタを呼び出すと、nullのみが返されます。

Answer 3

まず、クラスのインスタンスを作成します。実際にはコンストラクタを実行します。パラメータは文字列です。それが表示されます。 その後、colorgeneratedを初期化します。 color.getColor（）の後に、生成されたcolorを表示する必要があります。なぜなら、実際には初期化されているからです。