考虑循环的第一次迭代。的值str是零,和值
sCurrentLineIS "<?xml version="1.0"?>"。
在字符串连接中,空引用将转换为字符串“ null”。例如:
String x = null;String y = "a";String z = x + y; // "nulla"
因此,在您执行的第一个迭代中:
str += sCurrentLine;
的价值str将是
"null<?xml version="1.0"?>"
您可以初始化
str为
""…,但我不可以。
无论如何,没有理由为什么它应该是一个静态变量-
即使再次调用该方法,您是否真的想保持旧值不变?
那将是非常奇怪的。它应该是一个局部变量。
另外,您不应该在这样的循环中使用字符串连接-
在性能方面太糟糕了,因为它需要不断复制旧
数据。请
StringBuilder改用。
我也不会捕获这样的异常-您是否真的希望呼叫者
不知道有任何问题?只需声明您的方法可以抛出
IOException。
另外,我不会用FileReader自己的语言-它使您无法控制
用于读取文件的编码-它将始终使用平台
默认编码,这可能是不合适的。我会用一个FileInputStream
包裹在InputStreamReader。然后,您可以使用
try-with-resources
Java 7中的语句来FileInputStream自动关闭。
因此,经过所有这些更改,您的代码将变成这样:
public static String read(String file) throws IOException { StringBuilder builder = new StringBuilder(); try (InputStream input = new FileInputStream(file)) { BufferedReader reader = new BufferedReader( new InputStreamReader(input, "UTF-8")); String line; while ((line = reader.readLine()) != null) { builder.append(line); } } return builder.toString();}
请注意,这会将多行文件转换为单行文本-如果您要 保留行数(请不要在意行分隔符),请在循环中
使用
builder.append("n")或类似的方法。如果
要保留确切的行尾,请不要使用readLine-只需将字符读入缓冲区即可。
另外,考虑使用第三方库(例如Guava)来代替所有这些 *** 作-例如,您可以使用
Files.toString()。
欢迎分享,转载请注明来源:内存溢出
评论列表(0条)