最近在做一个项目优化与重构,测试同学在回归功能测试时,指出一个页面功能有Bug,并扔出日志:
NPE嘛,这还不简单,根据日志记录的行数,定位到代码行数,都几乎不需要执行接口测试单步调试,几乎就可以确定修改方案如下:
于是,代码push,并提交Merge Request。
过了几分钟,测试再次反馈,还是有Bug:
此时,如果死死分析上面的错误信息,就不难知道解决方案。奈何,我并没有参与之前的项目研发。对于这里面的业务逻辑并不清楚。另一方面,通过postman接口单步调试,发现这个方法超500行代码(这也是项目亟需优化与重构的原因),一时半会也不会很快理解里面的完整业务逻辑。这是后话。。
话说回来,我坚持跟测试同学说,这是NPE问题,日志里写的清清楚楚。但是测试坚持说,数据库明明有这一条数据:
- 证据1:
- 证据2,重新拿出文章开头的日志(修复NPE问题后,没有下面的错误stacktrace):
Total: 1
并且记录的SQL语句,和参数Parameters,和postman调试时,RequestBody一模一样。
此时才发现问题不仅仅是一个简简单单的NPE异常。
诸般死锁,加上leader的提示,此处可能 是mybatis的配置问题。
给出简化版的mybatis-config.xml
配置文件:
<configuration>
<settings>
<!-- 数据库下划线写法自动改为java驼峰写法 -->
<setting name="mapUnderscoreToCamelCase" value="true"/>
</settings>
</configuration>
出错的代码片段:
ChannelAdset adset = channelJobMapper.getAdsetById(map.getLong("adset_id"));
if (adset != null) {
map.put("facebook_adset_id", adset.getFacebook_adset_id());
}
对应的mybatis mapper查询文件:ChannelJobMapper.xml
<select id="getAdsetById" parameterType="Long" resultType="com.ppdai.cbd.platform.po.ChannelAdset">
select adset_id, facebook_adset_id
from channel_ad_adset
where adset_id = #{id}
</select>
对应的PO实体类:
@Data
@NoArgsConstructor
public class ChannelAdset implements Serializable {
private Long adset_id;
private String facebook_adset_id, account_id, status;
}
发现没有,实体类PO的定义,极其不规范:
- 没有使用驼峰命名;
- 多个字段使用一个private修饰符。
mapUnderscoreToCamelCase
配置为true,就是把数据表里面查询出来的下划线命名的字段映射到实体类的驼峰命名的属性字段。然后我们的PO实体类属性字段命名是下划线。导致实体类是一个空类,及仅仅只是一个new Object,Object里面的全部属性都是null。
最后总结一下:
日志记录不出问题来,select 查出数据,但是在mybatis转换层,就没有数据;之前没有空指针判断,报错NPE;加了空指针判断,条件不为空那行代码没有执行,导致出现
版权声明:本文内容由互联网用户自发贡献,该文观点仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 举报,一经查实,本站将立刻删除。
文章由极客之音整理,本文链接:https://www.bmabk.com/index.php/post/142189.html