値を設定し、nullで動作していないことを確認してください

Question

私は本当に混乱しています。シリアルが値を持っているかどうかをチェックし、そうでない場合はそのメッセージを表示するためにこのコードが必要です。

public class Calculator { 

    public static void main(String[] args) { 
     Product p2 = new Product("1st", 1.15); 
     p2.setSerial("1655"); 
     System.out.println("done"); 
    } 
} 

class Product { 

    private String name; 
    private String serial; 
    private double price; 

    public Product(String name, String serial, double price) { 
     this.name = name; 
     this.serial = serial; 
     this.price = price; 
    } 

    public Product(String name, double price) { 
     this.name = name; 
     serial = null; 
     this.price = price; 
    } 

    public void setSerial(String serial) { 
     if (serial == null) 
      this.serial = serial; 

     else 
      JOptionPane.showMessageDialog(null, "This item has a serial number"); 

    } 

    public String getSerial() { 
     return serial; 
    } 
} 

Answer 1

if (serial == null) 

あなたがクラスのメンバーをチェックするために持っているようにメソッドに渡されたパラメータをチェックしている珍しいことではありません。あなたはそれを変更する必要があります

if (this.serial == null) 

あなたのパラメータとインスタンスメンバーが同じ名前を持っているので混乱しています。

GUIとモデルクラスを混在させるのはなぜですか？あなたがテストしているなら、モデルクラスとGUIを混ぜるのは良い考えではありません。

これ以外にも、if elseの{}を省略していることがわかりますが、そのようにしないでください。それらを持っています。このような混乱を避けるために。