文件名未从不同的类添加到ArrayList

时间:2013-09-10 14:37:33

标签: java list class arraylist filenames

我在课程 checksumFinder 中有一个方法 hexFinder 。在hexFinder中,每个文件都被读取并命名为 good 文件或 bad 文件。然后将其放入两个ArrayLists之一, listGoodFiles listBadfiles

这两个ArrayLists都属于 multiFile 方法中的 checksumGUI 类。此方法为所选文件夹中的每个文件调用 hexFinder 。它还会输出一条消息,显示该文件夹中有多少文件。

当我将文件名添加到其中一个ArrayLists时,不会添加任何内容。

以下是 checksumFinder 类中可能导致问题的一些代码

class checksumFinder {

checksumGUI cg = new checksumGUI();

String hexFinder(File currentFile,...){

.... // determine file is good/bad 

if (l1 == l2) {
    cg.listGoodFiles.add(currentFile.getName());
    } else{
      cg.listBadFiles.add(currentFile.getName());
    }

以下是 checksumGUI 类的 multiFile 方法

public void multiFile(JFileChooser inFileName) throws FileNotFoundException, IOException {

    checksumFinder cf = new checksumFinder();
    ArrayList<String> listTypeErrFiles = new ArrayList<String>();
    File folderFile = inFileName.getSelectedFile();
    File[] listAllFiles = folderFile.listFiles();

    for (int i = 0; i < listAllFiles.length; i++) {
        File currentFile = listAllFiles[i];
        if (currentFile.isFile() && currentFile.getName().endsWith(".pcf")) {

            cf.hexFinder(currentFile, null, null, null);
        } else {
            listTypeErrFiles.add(currentFile.getName());
            System.out.println("-----------------------------");
            System.out.println("Incorrect filetype:\n" + currentFile.getName());
            System.out.println("-----------------------------\n");
        }
    }

    JTextArea ta = new JTextArea(25, 25);
    ta.setText("Folder contains " + listAllFiles.length + " files\n\n"+ "Folder has " + 
            listGoodFiles.size() + " good files;\n" + listGoodFiles + "\nFolder has " +
            listGoodFiles.size() + " bad files;\n" + listBadFiles +"\nFolder has " + 
            listTypeErrFiles.size() + " file of wrong type;\n" + listTypeErrFiles);

            ta.setWrapStyleWord(true);
            ta.setLineWrap(true);
            ta.setCaretPosition(0);
            ta.setEditable(false);

            JOptionPane.showMessageDialog(null,
            new JScrollPane(ta),"Folder Contents", JOptionPane.INFORMATION_MESSAGE);
}

从我制作这个程序的方式来看,这两个类之间有很多动作。 (我确信它可以做得更好,但这是我知道的唯一方法)

编辑:

我已经查看过像this这样的类似问题,但我发现没有任何有用的问题

编辑:

可以找到 checksumFinder 类的(最新)代码HERE

非常感谢任何帮助,谢谢。

1 个答案:

答案 0 :(得分:3)

您正在checksumFinder中创建一个新的checksumGUI实例,而不是访问现有实例:

cg = new checksumGUI();

您应该将现有实例作为构造函数的参数:

class checksumFinder {
    private final checksumGUI cg;

    checksumFinder(checksumGUI cg) {
        this.cg = cg;
    }

    ...
}

而不是checksumFinder cf = new checksumFinder();中的multiFile(),将当前实例传递给checksumFinder构造函数:

checksumFinder cf = new checksumFinder(this);

(另外,使用java编码约定,例如大写的类名,会使代码更容易阅读。)