读文件，报java.lang.OutOfMemoryError

[justinavril]

大家看下这个代码：

import java.io.*;

public class FileToString {
	 public static String readFile(String fileName) {
		String output = ""; 
		
	    File file = new File(fileName);
	       
	    if(file.exists()){
	    	if(file.isFile()){
	    		try{
	    			BufferedReader input = new BufferedReader (new FileReader(file));
	    			StringBuffer buffer = new StringBuffer();
	    			String text;								//问题出在这2行，如果我这么写的话，就会报那个错误。
	    			   											//String text = input.readLine();
	    			while((text = input.readLine()) != null)	//while(text != null)
	    				buffer.append(text +"\n");
	    			   
	    			output = buffer.toString();	    			   
	    		}
	    		catch(IOException ioException){
	    			System.err.println("File Error!");

	    		}
	    	}
	    	else if(file.isDirectory()){
	    		String[] dir = file.list();
	    		output += "Directory contents:\n";
	    		
	    		for(int i=0; i<dir.length; i++){
	    			output += dir[i] +"\n";
	    		}
	    	}
	    }
	    else{
	    	System.err.println("Does not exist!");
	    }
	    return output;
	 }
	 
	 public static void main (String args[]){
		 String  str = readFile("C:/1.txt");
		 
		 System.out.print(str);
	 }
}

奥运加油！

北京加油！

中国加油！

我当时觉得按注释那样写，也没什么错误啊，结果运行就出错了 报java.lang.OutOfMemoryError的错误

[justinavril]

我觉得按照注释那么写 和 正常运行的代码 没有什么逻辑区别啊...
最多的不同就是读到最后一行 大不了text为空嘛...

[xuhaiyang]

注释那么写就死循环，当然报java.lang.OutOfMemoryError的错误

[justinavril]

[Quote=引用 2 楼 xuhaiyang 的回复:]
注释那么写就死循环，当然报java.lang.OutOfMemoryError的错误
[/Quote]
为什么是死循环啊...我没看出来啊...

[xuhaiyang]

因为你text从input里读了一行后就永远不变了，当然是死循环 。
也就是text说不可能变为null

[cydp007]

while((text = input.readLine()) != null)

你这样写是正确的的原因是 。 只有这样才能读完整个文件。

text不停的有新值.

String text = input.readLine();
           while(text != null)

这样写只是把你的文件第一行不停的读入而已. 所以会报 out错误.

[cydp007]

while((text = input.readLine()) != null)

你这样写是正确的的原因是 。 只有这样才能读完整个文件。

text不停的有新值.

String text = input.readLine();
           while(text != null)

这样写只是把你的文件第一行不停的读入而已. 所以会报 out错误.

[justinavril]

哦 对 我明白了 嗯 没转过弯来
[Quote=引用 5 楼 cydp007 的回复:]
Java codewhile((text = input.readLine()) != null)



你这样写是正确的的原因是 。 只有这样才能读完整个文件。

text不停的有新值.


Java codeString text = input.readLine();
while(text != null)




这样写只是把你的文件第一行不停的读入而已. 所以会报 out错误.
[/Quote]

[justinavril]

那其实要是按注释那么写 在while循环里 再加上text = input.readLine();就可以了

[zidasine]

//String text = input.readLine();
如果这个text不为null
那么//while(text != null)

就相当于while(true)
所死循环

while((text = input.readLine()) != null)
是每循环一次读取一行新的数据赋给text
所以每次text都是新的值
当读到最后的时候text==null
结束循环

[imasmallbird]

[Quote=引用 8 楼 justinavril 的回复:]
那其实要是按注释那么写 在while循环里 再加上text = input.readLine();就可以了
[/Quote]

对头~~

[dsn21]

死循环。。。。。。。。啊 帮顶

[小爽昵称已被占用]

1.
String text = input.readLine();
while(text!= null) {
buffer.append(text +"\n");
text = input.readLine();
}
---------------------
2.或者合成一句。。。
while ((text=input.readLine()) != null)
--------------------
如果按照注释那样写，readLine只执行了一次，所以text永远也不为null，所以死--循-环-

[KOOK_OKKO]

String text = input.readLine();
while((text = input.readLine()) != null)
这样写的话只要input流里一直有一行数据
所以就不为null，所以不停的循环

[qq82819313]

[Quote=引用 8 楼 justinavril 的回复:]
那其实要是按注释那么写 在while循环里 再加上text = input.readLine();就可以了
[/Quote]

不能这么改，这么改就错了，这样相当于第一行没有取到！
如果实在要改的话：

while((text!= null) {
buffer.append(text +"\n");
text = input.readLine();
}

[yeeshengwei]

原因: 死循环,迟早是耗尽buffer的已分配内存
给一个测试:

                    //test
                    String text = input.readLine();
                    int counter = 1;
                    long startTime = System.nanoTime();
                    while(text != null){
                    	buffer.append(text + "\n");
                    	long endTime = System.nanoTime();
                        System.out.println(counter + ": " + (endTime - startTime) + "ns");
                        counter++;
                    }

运行结果：

132917: 4396824736ns
132918: 4396842336ns
Exception in thread "main" java.lang.OutOfMemoryError: Java heap space
	at java.util.Arrays.copyOf(Unknown Source)
	at java.lang.AbstractStringBuilder.expandCapacity(Unknown Source)
	at java.lang.AbstractStringBuilder.append(Unknown Source)
	at java.lang.StringBuffer.append(Unknown Source)

[xuechenyoyo]

BufferedReader input = new BufferedReader (new FileReader(file));
StringBuffer buffer = new StringBuffer();
顾名思义，用缓存的方式来处理数据，如果数据较大肯定会OutOfMemoryError.
不要缓存,读一句写一句就不会OutOfMemoryError.