局部变量范围的问题。如何解决？答案

【问题标题】：Problems with local variable scope. How to solve it?局部变量范围的问题。如何解决？
【发布时间】：2014-11-11 17:20:04
【问题描述】：

尝试在我的代码中执行 statemet.executeUpdate() 时出现以下错误：

Local variable statement defined in an enclosing scope must be final or effectively final.

这是我目前的代码：

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;.

import org.eclipse.swt.SWT;
import org.eclipse.swt.events.MouseAdapter;
import org.eclipse.swt.events.MouseEvent;
import org.eclipse.swt.widgets.Button;
import org.eclipse.swt.widgets.Display;
import org.eclipse.swt.widgets.Label;
import org.eclipse.swt.widgets.Shell;
import org.eclipse.swt.widgets.Text;

public class a1 {

    protected Shell shell;
    private Text text;
    private Text text_1;
    private Text text_2;
    private Text text_3;

    /**
     * Launch the application.
     * @param args
     */
    public static void main(String[] args) {
        try {
            a1 window = new a1();
            window.open();
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

    /**
     * Open the window.
     */
    public void open() {
        Display display = Display.getDefault();
        createContents();
        shell.open();
        shell.layout();
        while (!shell.isDisposed()) {
            if (!display.readAndDispatch()) {
                display.sleep();
            }
        }
    }

    /**
     * Create contents of the window.
     */
    protected void createContents() {

        Connection connect = null;

        ResultSet resultSet = null;

        try {
            Class.forName("com.mysql.jdbc.Driver");
        } catch (ClassNotFoundException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
        try {
            connect = DriverManager.getConnection("jdbc:mysql://localhost/railwaydb", "root", "");
        } catch (SQLException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }

        Statement statement = null;
        // statements allow to issue SQL queries to the database
        try {
            statement = connect.createStatement();
        } catch (SQLException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }

        shell = new Shell();
        shell.setSize(450, 300);
        shell.setText("SWT Application");

        Label lblName = new Label(shell, SWT.NONE);
        lblName.setBounds(10, 43, 47, 15);
        lblName.setText("Name");

        Label lblFrom = new Label(shell, SWT.NONE);
        lblFrom.setBounds(10, 74, 55, 15);
        lblFrom.setText("From");

        Label lblTo = new Label(shell, SWT.NONE);
        lblTo.setBounds(10, 105, 55, 15);
        lblTo.setText("To");

        Label lblPrice = new Label(shell, SWT.NONE);
        lblPrice.setBounds(10, 137, 55, 15);
        lblPrice.setText("Price");

        text = new Text(shell, SWT.BORDER);
        text.setBounds(64, 43, 76, 21);

        text_1 = new Text(shell, SWT.BORDER);
        text_1.setBounds(64, 74, 76, 21);

        text_2 = new Text(shell, SWT.BORDER);
        text_2.setBounds(64, 105, 76, 21);

        text_3 = new Text(shell, SWT.BORDER);
        text_3.setBounds(64, 137, 76, 21);

        Label lblRailwayDatabase = new Label(shell, SWT.NONE);
        lblRailwayDatabase.setBounds(174, 10, 97, 15);
        lblRailwayDatabase.setText("Railway Database");

        Label lblCreateView = new Label(shell, SWT.NONE);
        lblCreateView.setBounds(189, 43, 76, 15);
        lblCreateView.setText("Create View");

        Button btnName = new Button(shell, SWT.CHECK);
        btnName.setBounds(189, 73, 93, 16);
        btnName.setText("Name");

        Button btnFrom = new Button(shell, SWT.CHECK);
        btnFrom.setBounds(189, 105, 93, 16);
        btnFrom.setText("From");

        Button btnTo = new Button(shell, SWT.CHECK);
        btnTo.setBounds(189, 137, 93, 16);
        btnTo.setText("To");

        Button btnPrice = new Button(shell, SWT.CHECK);
        btnPrice.setBounds(189, 171, 93, 16);
        btnPrice.setText("Price");

        Button btnInsert = new Button(shell, SWT.NONE);
        btnInsert.addMouseListener(new MouseAdapter() {
            @Override
            public void mouseDown(MouseEvent e) {
                String name = text.getText();
                String from = text_1.getText();
                String to = text_2.getText();
                String price = text_3.getText();

                String query = "INSERT INTO booking (name, fromst, tost, price) VALUES ('"+name+"', '"+from+"', '"+to+"', '"+price+"')";
                try {
                    statement.executeUpdate(query);
                } catch (SQLException e1) {
                    // TODO Auto-generated catch block
                    e1.printStackTrace();
                }
            }
        });
        btnInsert.setBounds(10, 171, 75, 25);
        btnInsert.setText("Insert");

        Button btnView = new Button(shell, SWT.NONE);
        btnView.setBounds(307, 74, 75, 25);
        btnView.setText("View");

        Button btnIndex = new Button(shell, SWT.NONE);
        btnIndex.setBounds(307, 127, 75, 25);
        btnIndex.setText("Index");

    }
}

我也尝试将statement 设置为final，但声明给了我另一个错误。

【问题讨论】：

请告诉我们：预期结果是什么，观察到的错误行为是什么（Baz 已经要求的错误消息）。并且：这是很多代码！请尝试将您的源代码范围缩小到受影响的代码行。
这是 Java 的古怪内部类/“lambda”表示法。从 lambda 表达式内部引用的变量（未在 lambda 表达式中声明）必须是实例变量或在调用范围内不会发生变化的局部变量。
要选择其他内容，您的错误处理不合适。您捕获异常，忽略它并继续代码。考虑getConnection 没有真正连接的场景。生成的SQLException 被有效地忽略（除了将堆栈跟踪打印到System.err），并且代码继续尝试createStatement——此时，由于connect 是null，你会得到一个@987654331 @扔给你。然后，您将负责调查为什么那里有一个null 指针。如果你早点抛出SQLException，你马上就知道了。
请仅包含与您的问题相关的强制性代码。我们不需要知道您的 1000 个按钮/标签来解决您的问题。该错误很可能与一个变量有关，因此您可以删除大量代码，使其更易于所有人阅读。

标签： java scope

【解决方案1】：

确实存在范围问题，因为statement 是此处定义的本地方法变量：

protected void createContents() {
    ...
    Statement statement = null; // local variable
    ...
     btnInsert.addMouseListener(new MouseAdapter() { // anonymous inner class
        @Override
        public void mouseDown(MouseEvent e) {
            ...
            try {
                statement.executeUpdate(query); // local variable out of scope here
            } catch (SQLException e1) {
                e1.printStackTrace();
            }
            ...
    });
}

当您尝试在 mouseDown() 方法中访问此变量时，您正在尝试从匿名内部类中访问局部变量，并且范围不够。所以它肯定必须是final（给定你的代码是不可能的）或声明为类成员，以便内部类可以访问这个statement变量。

来源：

如何解决？

你可以...

使statement 成为类成员而不是局部变量：

public class A1 { // Note Java Code Convention, also class name should be meaningful   
    private Statement statement;
    ...
}

你可以...

按照@HotLicks 的建议，定义另一个最终变量并使用这个变量：

protected void createContents() {
    ...
    Statement statement = null;
    try {
        statement = connect.createStatement();
        final Statement innerStatement = statement;
    } catch (SQLException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }
    ...
}

但是你应该...

重新考虑您的方法。如果在按下btnInsert 按钮之前不会使用statement 变量，那么在实际发生之前创建连接是没有意义的。您可以像这样使用所有局部变量：

btnInsert.addMouseListener(new MouseAdapter() {
   @Override
   public void mouseDown(MouseEvent e) {
       try {
           Class.forName("com.mysql.jdbc.Driver");
           try (Connection connect = DriverManager.getConnection(...);
                Statement statement = connect.createStatement()) {

                // execute the statement here

           } catch (SQLException ex) {
               ex.printStackTrace();
           }

       } catch (ClassNotFoundException ex) {
           e.printStackTrace();
       }
});

【讨论】：

请注意，简单的解决方案是添加类似final Statement innerStatement = statement; 的语句，并在mouseDown 中引用innerStatement。
谢谢 dic19。 @hot licks，但我收到此警告：空指针访问：变量内部语句在此位置只能为空
@amitsingh - 因为您在设置 statement 之前放置了它。

【解决方案2】：

首先，我们不能将变量设为final，因为它的状态可能会在程序运行期间发生变化，而我们在内部类覆盖中的决定可能取决于它的当前状态。

其次，良好的面向对象编程实践建议仅使用对类定义至关重要的变量/常量作为类成员。这意味着如果我们在匿名内部类覆盖中引用的变量只是一个实用变量，那么它不应该列在类成员中。

但是——从 Java 8 开始——我们有第三种选择，如下所述：

https://docs.oracle.com/javase/tutorial/java/javaOO/localclasses.html

从 Java SE 8 开始，如果在方法中声明本地类，它就可以访问方法的参数。

所以现在我们可以简单地将包含新内部类及其方法覆盖的代码放入一个私有方法中，该方法的参数包括我们从覆盖内部调用的变量。然后在 btnInsert 声明语句之后调用此静态方法：-

 . . . .
 . . . .

 Statement statement = null;                                 

 . . . .
 . . . .

 Button btnInsert = new Button(shell, SWT.NONE);
 addMouseListener(Button btnInsert, Statement statement);    // Call new private method

 . . . 
 . . .
 . . . 

 private static void addMouseListener(Button btn, Statement st) // New private method giving access to statement 
 {
    btn.addMouseListener(new MouseAdapter() 
    {
      @Override
      public void mouseDown(MouseEvent e) 
      {
        String name = text.getText();
        String from = text_1.getText();
        String to = text_2.getText();
        String price = text_3.getText();
        String query = "INSERT INTO booking (name, fromst, tost,price) VALUES ('"+name+"', '"+from+"', '"+to+"', '"+price+"')";
        try 
        {
            st.executeUpdate(query);
        } 
        catch (SQLException e1) 
        {
            e1.printStackTrace();                                    // TODO Auto-generated catch block
        }
    }
  });
  return;
}

 . . . .
 . . . .
 . . . .

【讨论】：

我一定在这里遗漏了一些东西。问题是变量必须是最终的。所以把它放在一个封闭的类中？但是您提供的链接说“但是，从 Java SE 8 开始，本地类可以访问最终或有效最终的封闭块的局部变量和参数。”所以这似乎是同一件事，只是结构略有不同。仍然存在（IMO 相当不合理的）限制，它必须是最终的。
必须是最终的或有效最终的，即在当前代码块中的计算期间。
但是如果我可以使它成为最终的或有效的最终的（也就是说，如果我不需要在 lambda 或匿名类中更改它），那么它就不会成为问题变量。
链接中，从Java SE 8开始，如果在方法中声明本地类，就可以访问方法的参数了。。问题在于 access 到了 OP 在覆盖方法中需要的变量，他需要访问在封闭类中定义的变量 st。错误消息只是说 在封闭类中定义的局部变量语句必须是最终的或有效的最终的。 但就访问而言，该错误消息自 Java 8 以来（可能仍然是？）已过时，因为Java 8+ 现在允许访问封闭方法的参数。
我发现了这个，它清除了它：Java Closures Using Mutable Objects。我猜对象引用本身不能改变，但对象的任何成员都可以。我仍然觉得 Java 限制毫无意义，但至少这是一个很好的解决方法（尽管该页面的作者不喜欢它）。

【解决方案3】：

不是错误：

JSONObject json1 = getJsonX();

错误：

JSONObject json2 = null;
if(x == y)
   json2 = getJSONX();

错误：在封闭范围内定义的局部变量语句必须是最终的或有效的最终。

但是你可以写：

JSONObject json2 = (x == y) ? json2 = getJSONX() : null;

【讨论】：

【解决方案4】：

我发现这种方法很有用。这样你就不需要类也不需要final

 btnInsert.addMouseListener(new MouseAdapter() {
        private Statement _statement;

        public MouseAdapter setStatement(Statement _stmnt)
        {
            _statement = _stmnt;
            return this;
        }
        @Override
        public void mouseDown(MouseEvent e) {
            String name = text.getText();
            String from = text_1.getText();
            String to = text_2.getText();
            String price = text_3.getText();

            String query = "INSERT INTO booking (name, fromst, tost, price) VALUES ('"+name+"', '"+from+"', '"+to+"', '"+price+"')";
            try {
                _statement.executeUpdate(query);
            } catch (SQLException e1) {
                // TODO Auto-generated catch block
                e1.printStackTrace();
            }
        }
    }.setStatement(statement));

【讨论】：