前言
今天测试提交了一个bug,跟踪发现了一段代码,初看还没什么问题,简易后的代码:
private String getType(String loadBalancerPort) {
return "something" + loadBalancerPort == null ? "All" : loadBalancerPort + "";
}
这段代码想要实现的功能是返回
return "something"+(loadBalancerPort == null ? "All" : loadBalancerPort + "")
细看发现后面使用的是三元运算符,这就是造成这个bug的元凶。
解决方案:使用括号括起来。
延伸
在做这个例子的时候,使用ideal,习惯性的使用sonar扫描:
这里很明显的提醒这里表达式有问题。
总结
- 三元运算符格式:expression1 ? expression2 : expression3
- 三元运算符会将:右边的视为一个整体,其优先级是很低的,如果要先求三元运算符中的顺,必须先将其用括号括起来。
- 尽量别在三元运算里面使用复杂的表达式。
- 多多格式化代码。
- 完成编码,使用sonar扫描,清除里面的异味,漏掉,bug。