前言:今天在改完代码后进行接口测试时,突然发现自己写的一个接口一直提示错误,检查代码查发现这个坑。
逻辑是这样的,我在controller层方法里面判断service方法返回的结果code如果是1000,则表示逻辑处理成功,否则处理失败。处理逻辑大概如下所示
public class Test {
//定义表示成功的代码
public static final Integer SUCCESS_CODE = 1000;
//定义了响应结果
class Response{
private Integer code;
public Integer getCode() {
return code;
}
public void setCode(Integer code) {
this.code = code;
}
public Response(Integer code){
this.code = code;
}
}
//业务逻辑处理返回成功
public Response bizHandler(){
return new Response(1000); //问题代码处一
}
public static void main(String[] args) {
Response response = new Test().bizHandler();
if(SUCCESS_CODE == response.getCode()){
System.out.println("处理成功了");
}else{
System.out.println("处理失败了");
}
}
}
竟然提示我处理失败了???
1000 != 1000吗?
我突然想到可能是 == 符号的比较在对象之间是判断引用地址是否一致的,很明显,这里一个地址指向在方法区,一个指向在堆中,两者地址并不一样所以结果是false。所以我只需要把问题代码处一的1000改为静态常量SUCCESS_CODE一致即可,如下:
public Response bizHandler(){
//return new Response(1000);
return new Response(SUCCESS_CODE);
}
好了问题暂时解决。
但是我以前代码这样写是没有问题的,于是我就查看以前的代码,发现逻辑大概一致,只不过返回的code是100。换了个值就可以了吗?我于是尝试了下改动:
public class Test {
//定义表示成功的代码
public static final Integer SUCCESS_CODE = 100;//这里原来是1000,现在换成100
//业务逻辑处理返回成功
public Response bizHandler(){
return new Response(100);//这里原来是1000,现在换成100
}
}
结果是这样:
竟然成功了???
说实在的,我是知道有装箱,拆箱这么一说的,装箱调用valueOf方法,拆装调用intValue方法,但是换了个数你就不一样我就很奇怪了,没办法,看下源码吧。
发现有个IntegerCache这么个东西,cache是缓存的意思吧。原因竟然在这。
原来Java为了提升性能,搞了个常量池,如果你的数值是在常量池里面的,他就直接返回了,这个常量池的范围在-128和127(默认)之间
所以刚才的code=100结果是true这也就解释得通了。
总结:我们一般判断两个Integer类型的数值是否相等会显示的调用intValue进行判断,这样就可以避免出现这种神坑的问题啦。我是阿雷,一个求赞的程序员。