logo标签列表

本地变量作用域的问题。如何解决？

javascope
68
我在代码中执行 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已经要求的那样，请提供错误信息）。还有：这是很多代码！请尝试将您的源代码缩小到受影响的代码行。 - Gottlieb Notschnabel
这是Java的古怪内部类/“lambda”符号。从lambda表达式内部引用的变量（未在lambda表达式中声明）必须是实例变量或在调用范围内看不到变化的局部变量。 - Hot Licks
另外需要指出的是，你的错误处理不太合适。你捕获了异常，却忽略了它并且继续执行代码。考虑这样一种情况，getConnection 没有得到一个真正的连接。生成的 SQLException 被有效地忽略了（除了在 System.err 中打印堆栈跟踪信息），代码继续尝试 createStatement -- 然而此时 connect 是 null，你会遇到一个抛出的 NullPointerException。然后你就要去调查为什么会有一个 null 指针。如果你之前抛出了 SQLException，那么你就会立即知道发生了什么。 - ADTC
1
请仅包含与您的问题相关的强制性代码。我们不需要知道您的1000个按钮/标签来解决您的问题。错误很可能只涉及一个变量，因此您可以删除大量代码，使其更容易阅读。 - Maude
4个回答
96
您确实遇到了作用域问题，因为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();
    }
    ...
}

但你应该...

重新考虑你的方法。如果statement变量直到按下btnInsert按钮才被使用，那么在此之前创建连接是没有意义的。你可以像这样使用所有本地变量：

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();
       }
});
6
请注意，简单的解决方案是添加一个语句，如 final Statement innerStatement = statement; 并在 mouseDown 中引用 innerStatement。 - Hot Licks
谢谢dic19。@hot licks但是我收到了这个警告：空指针访问：变量innerstatement在此位置只能为null。 - Amit Chahar
1
@amitsingh - 因为您将其放置在 statement 设置之前。 - Hot Licks
5
首先，我们不能将变量设置为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;
}

 . . . .
 . . . .
 . . . .
我一定是漏掉了什么。问题在于变量必须是final的。那么把它放在封闭类里？但您提供的链接说：“但从Java SE 8开始，局部类可以访问封闭块的局部变量和参数，这些变量和参数是final的或有效地是final的。”所以这似乎是同样的事情，只是略微不同的结构。仍然有（我个人认为相当不合理的）限制，即它必须是final的。 - Ryan Lundy
必须是最终的或有效地最终的，即在当前代码块的计算期间。 - Trunk
但是，如果我可以将其声明为final或有效地final（也就是说，在lambda表达式或匿名类中不需要更改它），那么它作为变量也不会成为问题。 - Ryan Lundy
从链接中开始，从Java SE 8开始，如果您在方法中声明本地类，则可以访问该方法的参数。问题出在需要在重写的方法中访问OP所需的变量，他需要访问在封闭类中定义的变量st。错误消息只是说在封闭类中定义的局部变量语句必须是final或有效final。但是，这个错误消息（也许仍然是？）已经过时了，因为Java 8+现在允许访问封闭方法的参数。 - Trunk
我找到了这篇文章，解决了这个问题：Java Closures Using Mutable Objects。我猜对象引用本身不能改变，但是对象的任何成员都可以改变。我仍然觉得Java的限制毫无意义，但至少这是一个好的解决方法（尽管该页面的作者不喜欢它）。 - Ryan Lundy
这个“final”关键字 - 特别是与线程安全相关的 - 似乎具有更普遍的相关性。 https://softwareengineering.stackexchange.com/questions/115690/why-declare-final-variables-inside-methods 还有编译优化的说法。 - Trunk
0

我发现这种方法很有用。这样你就不需要一个类或者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));
0

非错误：

JSONObject json1 = getJsonX();

错误：

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

错误：在封闭作用域中定义的局部变量语句必须是 final 或 effectively final。

但您可以编写： 

JSONObject json2 = (x == y) ? json2 = getJSONX() : null;
网页内容由stack overflow 提供, 点击上面的
可以查看英文原文，
原文链接