这段代码有什么问题吗？

Question

我正在从目录的文件中读取内容。我必须根据文件名称将文件分开，然后读取它们的内容。当我简单地运行代码而不读取内容时，所有文件都会以特定文件名列出，但是当我尝试读取内容时，它只从几个文件中读取内容，实际上只有 10 个文件。但该目录大约有 1000 个特定名称的文件。我在这里发布代码。

for (i = 0; i <= filenames.length; i++) {
    read = new FileReader("trainfiles/"+filenames[i]);          
    br = new BufferedReader(read);

    if (filenames[i].matches(".*ham.*")) {
        System.out.println("ham:" + filenames[i]);
        while ((lines = br.readLine()) != null) {
            st = new StringTokenizer(lines);
            while (st.hasMoreTokens()) {
                System.out.println(st.nextToken());
            }
        }
        br.close();
    }
}

谁能告诉我我哪里做错了！？
谢谢

编辑＃1我做了一些修改，我在这里被告知，但问题仍然存在，这是代码。

for(i=0;i<=filenames.length;i++){
            read = new FileReader("trainfiles/"+filenames[i]);

            br = new BufferedReader(read);

            if(filenames[i].matches(".*ham.*")){
                System.out.println("ham:"+filenames[i]);

                        while((lines = br.readLine())!= null){
                            st = new StringTokenizer(lines);
                            while(st.hasMoreTokens()){
                                System.out.println(st.nextToken());
                            }

                        }

            }
            br.close();
            read.close();




                        }

编辑 #2 现在代码看起来像这样，但同样......它没有给我我想要的结果。

for (i = 0; i < filenames.length; i++) {
               try {


                if (filenames[i].matches(".*ham.*")) {
                     read = new FileReader("trainfiles/"+filenames[i]);          
                        br = new BufferedReader(read);
                    System.out.println("ham:" + filenames[i]);
                    while ((lines = br.readLine()) != null) {
                        st = new StringTokenizer(lines);
                        while (st.hasMoreTokens()) {
                            System.out.println(st.nextToken());
                        }
                    }
                }
               } finally {

                read.close();
                br.close();
               }
            }

Answer 1

我会像这样重写您的代码，然后看看您得到什么输出：

for (filename : filenames) {
   if (filename.matches(".*ham.*")) {
      System.out.println("ham:" + filename);

      // reset these to null (where are they declared?)
      read = null;   
      br = null;   
      try {
         read = new FileReader("trainfiles/"+filename);          
         br = new BufferedReader(read);

         while ((lines = br.readLine()) != null) {
            System.out.println(lines);
            // st = new StringTokenizer(lines);
            // while (st.hasMoreTokens()) {
            //    System.out.println(st.nextToken());
            // }
         }
      } catch (Exception e) {
         e.printStackTrace();
      } finally {
         if (br != null) br.close();
         if (read != null) read.close();
      }
   } 
}

对原始代码的一些一般注释：

1. 如果您确实需要数组索引，则仅使用 for 循环。首选 for-each 循环（即 for (filename : filenames) ...）。

2. 在尽可能小的范围内声明变量。在这种情况下，您应该声明您的 read 和 br 变量，我将它们初始化为 null。

3. 除非要使用文件，否则切勿打开它。在这里，这意味着在条件块内打开它。

4. 由于打开文件可能会引发异常，因此br可能无法初始化，在这种情况下您无法关闭它。您需要首先检查 null。

Answer 2

首先，您应该使用i。其次，matches 需要正则表达式，而不是 *-glob。您使用的表达式是 [something]ham[something] 的有效正则表达式 - 这就是您的意思吗？

我认为您不需要关闭 Filereader - 我认为 BR 的 close 会向上传播。但这值得检查。 编辑 正如前面提到的，您需要始终在 if 之外关闭文件。

Answer 3

您还应该关闭您的 FileReader 对象 read。

除非这是家庭作业，否则我还建议您看看 commons-io。

编辑＃1：我建议在finally块中执行两个关闭操作。

编辑#2：你尝试过这个吗？

for (i = 0; i <= filenames.length; i++) {
   try {
    read = new FileReader("trainfiles/"+filenames[i]);          
    br = new BufferedReader(read);

    if (filenames[i].matches(".*ham.*")) {
        System.out.println("ham:" + filenames[i]);
        while ((lines = br.readLine()) != null) {
            st = new StringTokenizer(lines);
            while (st.hasMoreTokens()) {
                System.out.println(st.nextToken());
            }
        }
    }
   } finally {
    br.close();
    read.close();
   }
}

Answer 4

1000 多个文件需要读取的文件量很大。如果它无法读取文件，它应该抛出异常（具体来说是 IOException）。也许打印 catch 块中的异常消息并将其粘贴到此处。

我不知道 StringTokenizer 类，但是当您仅打印没有 StringTokenizer 的行时，代码是否会出错？

另一种选择是使用线程。您拥有文件数组，然后启动一些读取文件的线程（生产者/消费者问题）。

顺便说一句，您可以使用 FileFilter 类来过滤文件。

http ://download.oracle.com/javase/1.4.2/docs/api/java/io/File.html#listFiles%28java.io.FileFilter%29