老师，我感觉saveOrUpdate方法这里判断应该加上查询条件

Question

老师的是：if(product.getId() != null){

我感觉应该是这样：if (productMapper.selectByPrimaryKey(product.getId()) !=null){

判断如果传入的参数id为数据库中已有的，而不只是判断id是否为空
不知道是不是这样，希望老师帮忙解答，谢谢

Answer 1

Geely 2018-01-09 23:51:05

你好，同学，if里面可别放sql...比较危险。正如@星辰_兼信同学所说。。

你的db思路是对的，但是别放在if里。

• 提问者 wendy_z #1

  非常感谢！

• 慕桂英9447872 #2

  请问老师，为什么if里面放sql比较危险？

Answer 2

星辰Iron 2018-01-09 21:35:15

你好同学，像你这样频繁的执行SQL语句进行查询操作效率是非常低的

老师那种根据产品是否有ID来判断是更新商品还是新增商品的方式更好。