currentShop

Question

翔仔哥，我有一个问题就是修改产品信息控制判断语句

if(product != null && product.getShop() != null && product.getShop().getShopId() != null)
我们这里的shopId是从会话域中查询出来的然后在将它赋值给Product对象
Shop currentShop = (Shop) request.getSession().getAttribute(“currentShop”);
product.setShop(currentShop);
那么这个currenShop是不是只有店铺Id值，其它什么店铺名称，位置都是空值，那上面的判断语句中&&product.getShop() != null这个是不是多余的？或则说我有什么地方没搞懂

Answer 1

翔仔 2019-04-16 00:22:50

同学好，

if(product != null && product.getShop() != null && product.getShop().getShopId() != null)

是在service层里，而service层是假设不知道controller传入的东西是否靠谱的基础上去弄的，如果你不判断product.getShop() != null，而直接product.getShop().getShopId() != null，有可能因为shop为空而报错。

• 提问者 Joker4411416 #1

  我以为shop里面没有值就是空，我运行了这段代码
  @Test
  	public void test() {
  		Shop shop = new Shop();
  		if(shop != null) {
  			System.out.println("shop对象不为空");
  		}
  		shop.setShopId(1L);
  		if(shop != null) {
  			System.out.println("赋值shopId的shop不为空");
  		}
  	}
  发现只要实例化了shop对象，它就不为空，但是product.getShop().getShopId() 的值只要大于0，那么shop可能是空吗？

• 翔仔 回复 提问者 Joker4411416 #2

  只要new出实例就不会为空，即便ShopId为空。但是service层先判断product.getShop()不为空是为了防止万一product.getShop()为空 product.getShop().getShopId报错

• 提问者 Joker4411416 回复 翔仔 #3

  好的，知道了