Java - 访问内部类中的变量或为最终变量赋值

Question

我需要创建几个按钮并上传文件。所以我想创建一个函数来设置这些按钮。但是，我的 setNewButton 内部出现编译错误。

我的代码如下所示：

public class Solution extends JFrame {
    private static final String FILE_NAME_1 = "my file1";
    private File file1;
    private void setNewButton(Container contentPane, final String fileName, String format, File file) {
        contentPane.add(Box.createVerticalStrut(5));
        final Label label = new Label("Select " + fileName + " in ." + format +" format");
        contentPane.add(label);
        contentPane.add(Box.createVerticalStrut(10));
        Button selection = new Button("Select " + fileName);
        contentPane.add(selection);
        selection.addActionListener(new FileSelectionListener("Only " + format + " is allowed", format) {
            @Override
            protected void setSelection(File selectedFile) {
                file = selectedFile;  // compilation error here
                label.setText("Selected" + fileName + selectedFile.getAbsolutePath());
            }
        });
    }

    public uploadFiles() {
        Container contentPane = this.getContentPane();
        setNewButton(contentPane, FILE_NAME_1, "xls", file1);
    }
}

错误是：Variable file is accessed from within inner class, needs to be declared final

我在stackoverflow中检查了一些类似的问题。我知道file 必须像label 和fileName 一样是最终的。

但是这里的file 可能是final，因为我想将selectedFile 分配给它。

我想知道这个问题是否有任何解决方法。

任何帮助将不胜感激。 :)

感谢@M。普罗霍罗夫和@Chang Liu。 根据 JLS 8.1.3. Inner Classes and Enclosing Instances

任何使用但未在内部类中声明的局部变量、形参或异常参数必须声明为 final 或实际上是 final，否则在尝试使用时会发生编译时错误。

所以当我尝试在FileSlectionListener 内部发送参数file 时，会出现编译错误。但是，Solution 中的成员file1 不是局部变量，所以如果我从我的方法中删除file，就不会出错。所以@talex 的答案在这种情况下是正确的。

但是，由于我的问题是找到一种将File 传递给内部类并使用selectedFile 分配变量的方法，因此我找不到解决方法。我的解决方法是基于@Chang Liu 的回答。

我修改后的代码如下：

public class Solution extends JFrame {
    private static final String FILE_NAME_1 = "my file1";
    private File file1;
    private void setNewButton(Container contentPane, final String fileName, String format) {
        contentPane.add(Box.createVerticalStrut(5));
        final Label label = new Label("Select " + fileName + " in ." + format +" format");
        contentPane.add(label);
        contentPane.add(Box.createVerticalStrut(10));
        Button selection = new Button("Select " + fileName);
        contentPane.add(selection);
        selection.addActionListener(new FileSelectionListener("Only " + format + " is allowed", format) {
            @Override
            protected void setSelection(File selectedFile) {
                setFile(selectedFile, fileName);  // no compilation error here
                label.setText("Selected" + fileName + selectedFile.getAbsolutePath());
            }
        });
    }

    public uploadFiles() {
        Container contentPane = this.getContentPane();
        setNewButton(contentPane, FILE_NAME_1, "xls", file1);
    }

    private void setFile(File file, String fileName) {
        switch (fileName) {
            case FILE_NAME_1:
                sollFile = file;
                break;
            default:
                throw new AssertionError("Unknown File");
        }
    }
}

不过，如果您有更好的答案，欢迎给我任何建议。 :)

Answer 1

您有两个名称为 file 的变量。一个是类变量，另一个是方法参数。

只需从您的方法中删除参数file，一切都会正常工作。

Answer 2

正如M. Prokhorov's comment所说，如果你去JLS 8.1.3. Inner Classes and Enclosing Instances，你会看到它声明：

任何使用但未在内部类中声明的局部变量、形参或异常参数必须声明为 final 或 effectively final，否则在尝试使用时会发生编译时错误。 em>

关于变量使用的类似规则适用于 lambda 表达式的主体。

所以方法setNewButton作为变量的参数File file在你的内部类new FileSelectionListener的方法setSelection中不是effectively final，即你分配了一个新的值到这个变量，这使得它不是 实际上是最终的。

通过为file 定义一个设置器而不是传递参数来解决此编译时错误的一些解决方法（但我不确定这是否是最佳做法）：

public class Solution extends JFrame {
    private static final String FILE_NAME_1 = "Selected SOLL:";
    private File file;
    private void setNewButton(Container contentPane, final String fileName, String format) {
        contentPane.add(Box.createVerticalStrut(5));
        final Label label = new Label("Select " + fileName + " in ." + format +" format");
        contentPane.add(label);
        contentPane.add(Box.createVerticalStrut(10));
        Button selection = new Button("Select " + fileName);
        contentPane.add(selection);
        selection.addActionListener(new FileSelectionListener("Only " + format + " is allowed", format) {
            @Override
            protected void setSelection(File selectedFile) {
                setFile(selectedFile);  // call file setter here
                label.setText("Selected" + fileName + selectedFile.getAbsolutePath());
            }
        });
    }

    // define a setter for your File member
    private void setFile(File file) {
        this.file = file;
    }

    public void uploadFiles() {
        Container contentPane = this.getContentPane();
        setNewButton(contentPane, FILE_NAME_1, "xls");
    }
}

Answer 3

你可以自己为文件创建一个可变的包装类：

public class FileWrapper {

    /** The file. */
    private File file;

    public File getFile() {
        return file;
    }

    public void setFile(File file) {
        this.file = file;
    }
}

然后您可以使用该类的最终实例：

final private FileWrapper fileWrapper = new FileWrapper();

// ...     

selection.addActionListener(new FileSelectionListener("Only " + format + " is allowed", format) {
        @Override
        protected void setSelection(File selectedFile) {
            fileWrapper.setFile(selectedFile);  
            label.setText("Selected" + fileName + selectedFile.getAbsolutePath());
        }
    });

并通过在内部类之外调用fileWrapper.getFile() 来获取最后选择的文件。

Answer 4

只需将您的动作侦听器提取为解决方案的内部类，您就可以从内部类中分配 Solution.this.file ：

public class Solution extends JFrame {
   private class MyListener extends FileSelectionListener{
@Override
            protected void setSelection(File selectedFile) {
                Solution.this.file = selectedFile;  // NO compilation error here
            }
}
    private static final String FILE_NAME_1 = "Selected SOLL:";
    private File file;
    private void setNewButton(Container contentPane, final String fileName, String format, File file) {
        contentPane.add(Box.createVerticalStrut(5));
        final Label label = new Label("Select " + fileName + " in ." + format +" format");
        contentPane.add(label);
        contentPane.add(Box.createVerticalStrut(10));
        Button selection = new Button("Select " + fileName);
        contentPane.add(selection);
        selection.addActionListener(new MyListener() );
    }

    public uploadFiles() {
        Container contentPane = this.getContentPane();
        setNewButton(contentPane, FILE_NAME_1, "xls", file1);
    }
}