前几天在定位生产问题时，由于日志中的JSON格式的List集合是不去重的，想看看去重后的数量,太长不能直接拷贝为String类型的值，就想着放到一个文件中写个main()读取转换下，发现明明读取应该结束了，但是最后的结果总是比文件中多那么些。最后发现是忽略了read()返回的len的问题，做个记录

上Main()方法

public static void main(String[] args) {

        try (FileReader reader = new FileReader(new File("E:\\ideaWorkspace\\testProject\\src\\main\\java\\test\\crowdList.txt"))){
            char[] buf = new char[100];
            int len;
            StringBuilder sb = new StringBuilder();
            while ((len = reader.read(buf)) != -1) {

                String str = new String(buf);
                sb.append(str);

            }

            List<String> strings = JSON.parseArray(sb.toString(), String.class);
            System.out.println(strings.size());
            Set<String> deSet = new HashSet<>(strings);
            System.out.println(deSet.size());


        } catch (FileNotFoundException e) {
            e.printStackTrace();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

分析下

乍一看感觉没啥问题，但是出错了，错误的内容是

image

是阿里巴巴的fastJson报出来的说不是一个闭合的json字符串，但是文件中确实是个有开闭的符号的，我们把每一行都打出来看看

image

我们可以看到明明结束的打印出来之后，后边又打出来了一串，而且这一串和上边的一模一样

结论

到这边就能看出来了，最后一次read()肯定是没有把我们用来存储内容的数组装满就是代码中的buf，装了多少呢，其实就是从0开始len个长度，剩下的还是上一次读取存在里边的，那就有两种思路了

• 每次读取后清空缓存数组(每次初始化还是有性能损失)

while ((len = reader.read(buf)) != -1) {
    String str = new String(buf);
    sb.append(str);
    buf = new char[100];
}

• 存储读取结果时只取前len个长度(读多少用多少,这个比较好)

while ((len = reader.read(buf)) != -1) {
    String str = new String(buf,0,len);
    sb.append(str);
}