设置一个值并检查null是否不起作用

-1

我真的很困惑。我需要这段代码来检查serial是否有值，如果不设置值则显示该消息。设置一个值并检查null是否不起作用

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; 
    } 
}

来源

2017-08-09 Christosmag

使用调试器。如果您传入null，则只设置该值。你可能是指'if（this.serial == null）'。 –

请注意，您不应在setter中使用消息框。理想情况下，你根本不应该使用setter：只在构造函数中初始化它。但是如果你必须拥有setter，则抛出'IllegalStateException'。 –

您应该抛出异常而不是提出一个对话框，并让调用者决定如何处理它。 – Tezra

if (serial == null)

这是不寻常你检查过的地方，你必须检查类的成员方法的参数。您必须将其更改为

if (this.serial == null)

您感到困惑，因为您的参数和实例成员具有相同的名称。

你为什么要将GUI与模型类混合？如果你测试的很好，但是混合模型类和GUI不是一个好主意。

除此之外，我可以看到您为{if else}省略{}，不这样做。拥有他们。只是为了避免这样的混乱。

来源

2017-08-09 14:29:16

它是值得一提的是，“你的参数和实例成员具有相同的名字”本身并不是问题。这很常见，特别是在构造函数中。你必须小心你指的是哪一个。 –

设置一个值并检查null是否不起作用

回答

相关问题