考虑循环的第一次迭代。的值
str是空的,价值
sCurrentLine就是
"<?xml version="1.0"?>"。
在字符串连接中,空引用将转换为字符串“空”。例如:
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-resourcesJava 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条)