不小心犯的一个空指针错误

      今天碰到了一个空指针异常，代码看起来很普通，一点也看不出会空指针，后来debug才发现了问题所在。这是很容易忽略的地方，记一笔，不知道大家有没有碰到过。

      下面代码是根据这个问题写的。

import java.util.ArrayList;

public class Model{
	static ArrayList<String> list;
	static void read(ArrayList<String> list) {
		if(list == null) {
			list = new ArrayList<String>();
		}
		list.add("Hello");
	}
	
	public static void main(String[] args) {		
		Model.read(list);
		System.out.println(list.size());
	}
}

 会输出什么呢？

 

 大家可以先猜一猜。

评论

30 楼 kenvi 2010-07-17  

import java.util.ArrayList;   
  
public class Model{   
    static ArrayList<String> list;   
    static void read(ArrayList<String> list) {
    System.out.println(Model.list==list);//此次输出true;
    
        if(list == null) {   
            list = new ArrayList<String>(); 
            System.out.println(Model.list==list);
        }   
        list.add("Hello");  
	System.out.println(Model.list==list);//此次输出false;
	System.out.println(Model.list);
	System.out.println(list);
    }   
       
    public static void main(String[] args) {           
        Model.read(list); 
        System.out.println(list.size());   
    }   
}  

首先要了解一点，Java里的引用其实就是C里面的指针，而==比较的是两个引用是否指向同一块内存。
在以下代码中

System.out.println(Model.list==list);//此次输出true;
    
        if(list == null) {   
            list = new ArrayList<String>(); 
            System.out.println(Model.list==list);
        } 

两次结果不一样，第一次是true，第二次是false，很显然是list = new ArrayList<String>();这个语句导致的结果不同，而Model.list==list是判断两个引用是否指向同一内存，也就是说，list被重新指向了new ArrayList<String>()开辟的空间，所以就不同，但是Model.list没有改变。继续看下面的：

System.out.println(Model.list);//输出为null
System.out.println(list);//输出为//[hello]

显然，list没有被回收！！

29 楼 dk101 2010-07-13  

楼上好多兄弟姐妹都把这个问题阐述得很清楚了，我个人赞同楼上“xinchi21cool ”的观点：
    static void read(ArrayList<String> list) {
        if(list == null) {
            list = new ArrayList<String>();
        }
        list.add("Hello");
    }
在read方法中，通过关键字new出来的局部变量new ArrayList<String>();只在方法体内有效，方法执行完成后，JVM执行垃圾回收，变量被置成null，随之对象list也就再次被置成null（初始化是为null），所以会出现空指针异常。其实在方法read内部，对象list非空。所以导致空指针的根本原因是作用域的问题，而不是值传递或者引用传递的问题。

28 楼 lx281 2010-06-22  

king607267 写道

我在楼主的代码上添加了两行：

import java.util.ArrayList;   
  
public class Model{   
    static ArrayList<String> list;   
    static void read(ArrayList<String> list) {
    System.out.println(Model.list==list);//此次输出true;
        if(list == null) {   
            list = new ArrayList<String>(); 
        }   
        list.add("Hello");  
	System.out.println(Model.list==list);//此次输出false;
    }   
       
    public static void main(String[] args) {           
        Model.read(list); 
        System.out.println(list.size());   
    }   
} 

我比较同意yangyi 的说法，我是这样理解的，java只有值传递！
上面代码传入read方法的只是Model.list引用变量的副本
进入read方法后其实产生了两个引用只不过都是指向null所以第一次输出为true，然后后面的代码
new了一个新的对象赋给list，此时list指向已经发生了变化所以第二
次输出为false，然而Model.list依然指向null，这正是引发空指针的原因

import java.util.ArrayList;   
  
public class Model{   
    static ArrayList<String> list;   
    static ArrayList<String> list2;   

    static void read(ArrayList<String> list) {
        if(list == null) {   
            list = new ArrayList<String>(); 
        }   
        list.add("Hello");  
    }   
       
    public static void main(String[] args) {           
        System.out.println(list == list2); // true
    }   
} 

所有的null都是equal的，不管定义的是哪个变量

27 楼 chroya 2010-06-20  

shadowlin 写道

为什么我觉得主要是lz没分清方法局部变量和类的变量的的问题呢？
和值传递还是引用有关系么？

不是局部变量和类变量的问题，局部变量的名字和类变量的相同而已，下面这样你就明白了。

import java.util.ArrayList;

public class Model{
	static ArrayList<String> list;
	static void read(ArrayList<String> s) {
		if(s== null) {
			s= new ArrayList<String>();
		}
		list.add("Hello");
	}
	
	public static void main(String[] args) {		
		Model.read(list);
		System.out.println(list.size());
	}
}

26 楼 shadowlin 2010-06-20  

为什么我觉得主要是lz没分清方法局部变量和类的变量的的问题呢？
和值传递还是引用有关系么？

25 楼 wzju64676266 2010-06-20  

System.out.println(list.size());  

执行list.size();出现nullpointexception

24 楼 king607267 2010-06-20  

list为null所以空指针

23 楼 xinchi21cool 2010-06-20  

	static void read(ArrayList<String> list) {
		if(list == null) {
			list = new ArrayList<String>();
		}
		list.add("Hello");
	}

出了这个方法你那new出来的ArrayList就没有用了，你main方法里的list能有值吗？

22 楼 king607267 2010-06-20  

我在楼主的代码上添加了两行：

import java.util.ArrayList;   
  
public class Model{   
    static ArrayList<String> list;   
    static void read(ArrayList<String> list) {
    System.out.println(Model.list==list);//此次输出true;
        if(list == null) {   
            list = new ArrayList<String>(); 
        }   
        list.add("Hello");  
	System.out.println(Model.list==list);//此次输出false;
    }   
       
    public static void main(String[] args) {           
        Model.read(list); 
        System.out.println(list.size());   
    }   
} 

我比较同意yangyi 的说法，我是这样理解的，java只有值传递！
上面代码传入read方法的只是Model.list引用变量的副本
进入read方法后其实产生了两个引用只不过都是指向null所以第一次输出为true，然后后面的代码
new了一个新的对象赋给list，此时list指向已经发生了变化所以第二
次输出为false，然而Model.list依然指向null，这正是引发空指针的原因

21 楼 love_ai87 2010-06-20  

whaosoft 写道

”建议你换个IDE吧它就提示你有问题了“ 没明白啊  在Eclipse里肯定木错啊 符合java语法啊 不知道 上面指的是嘛意思
这是java基础问题 也算常见
其实 你传进方法里 new的 和你传一去的不是一个对象哦 ，其实就再上面说的 java就是值传递

eclipse里面类变量是蓝色的，而局部变量是黑色的

20 楼 yangyi 2010-06-20  

Java当然是值传递的，这个没错，不过这个问题应该这么理解：
这里传递的值本身就是对象的引用，而这个被引用的对象在这里是NULL
这就是问题的关键，Java没有传递list引用的引用，而是传递list值的引用，当list值是null时，可以理解方法传递时同时有两个指针指向这个null，即实例变量list和局部变量list参数。
因为局部变量优先性高于实例变量，遮蔽后，new的新对象赋予了局部变量。
从这里开始两个引用分别指向了不同的对象实体，实例变量list继续指向null，而局部变量list已经更改为新的Array，之后的所有操作，都和实例变量指向的对象没有关系。所以会产生空指针错误

19 楼 qianhd 2010-06-20  

这种艾斯比写法 明显就是懂啊不懂的水货瞎写

18 楼 Liuy.Silence 2010-06-20  

main中操作的是全局的list,而read中操作的是局部的list.这两个对象并不是同一个,必然会报错.

17 楼 冇心人 2010-06-20  

不就是调用静态方法时 ArrayList还没有实例化吗？就像THINK IN JAVA里讲的  你拿着遥控器，却没有电视机，你怎么用？  怎么扯到值传递上去了，再说JAVA只有值传递吗？
他这里的意思是...LIST是NULL的.因为JAVA是值传递的.方法中的LIST与传入的LIST(NULL)是完全搭不上关系的.因为传入的参数LIST本来就是NULL.方法内的LIST初始化不等于方法外的LIST初始化.

16 楼 honeyshit 2010-06-20  

不就是调用静态方法时 ArrayList还没有实例化吗？就像THINK IN JAVA里讲的  你拿着遥控器，却没有电视机，你怎么用？  怎么扯到值传递上去了，再说JAVA只有值传递吗？

15 楼 holly2k 2010-06-19  

说实在，写了这么久JAVA，还真是没碰到楼主说的这种问题，不过我是这么理解的，这首先是一个运行时的错误，因此没有编译上的问题，起码我的eclipse没有显示有啥错误的，运行时调用了类的静态方法，将参数传进去，此时传进去的对象域因为没有实例化，因此其实是传了NULL给调用的方法的，其实相当于调用了Model.read(null)，此时是否在方法里做实例化已与原来的变量引用没有关系了，因为再调用其方法就产生了空指针异常。
可以通过J-catTeam的例子或是在类中直接实例化该list：如下
public class Model {
static ArrayList<String> list = new ArrayList<String>();

static void read(ArrayList<String> lista) {
if (lista == null) {
lista = new ArrayList<String>();
}
lista.add("Hello");

}

public static void main(String[] args) {
Model.read(list);
System.out.println(list.size());
}
}
依然是运行是将对象的域传给方法，但此时因为该域有引用的实例，因此此时会将对象地址传给该方法，在方法中做了数据添加，再回来输出时也可正常看到显示数目为1。
另我似乎一直认为JAVA方法调用时传的是对象的地址，上面有挺多人认为是值传递的，可否举一下例子呢？？？
不过说实在我自己一般不会用楼主的这种编码方式，挺别扭的，其实我最近也遇到一个空指针问题，代码如下：
int var;
if(object != null) {
   var = object.getField();
}
这么一个语句出现空指针异常，看了半个小时及DEBUG才发现，方法调用的时候返回的是一个Integer对象，被赋予int变量时需要内部调用方法进行转换，但此时刚好返回的Integer对象为空，因此导致空指针异常，因为原代码var的声明与该异常出现的代码较远，不像现在我直接放在一起，因此当时也时看了挺久才发现的，呵呵

14 楼 freej 2010-06-19  

这种错误有一个专有名词：遮蔽（shadow）

13 楼 tianmo2008 2010-06-19  

以前搞c/c++的？java应该不提倡这种写法的

12 楼 J-catTeam 2010-06-19  

有些朋友理解错了
1.传递进去的时候就是一个null，在read的时候new出来的和那个成员变量没有任何关系。所以这里和值传递之类的没有太大的关系
2.把代码改成这样子就明了了
public class Model {
static ArrayList<String> list;

static void read(ArrayList<String> lista) {
if (lista == null) {
lista = new ArrayList<String>();
}
lista.add("Hello");

}

public static void main(String[] args) {
list = new ArrayList<String>();
list.add("hh");
Model.read(list);
System.out.println(list.size());
}
}

11 楼 wangdgsc 2010-06-19  

在方法内部修改外部变量的引用地址，又没有把外面的修改掉，当然会出现空指针异常了，这是正常的，