Проблемы с локальной областью переменных. Как его решить?

Ответ 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.

Источники:

• Анонимные классы
• Как анонимные (внутренние) классы используются в Java?

Как его решить?

Вы могли бы...

Сделайте 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();
       }
});

Ответ 2

Во-первых, мы просто НЕ МОЖЕМ сделать переменную окончательной, так как ее состояние может изменяться во время выполнения программы, и наши решения в переопределении внутреннего класса могут зависеть от ее текущего состояния.

Во-вторых, хорошая практика объектно-ориентированного программирования предлагает использовать в качестве членов класса только переменные/константы, которые имеют жизненно важное значение для определения класса. Это означает, что если переменная, на которую мы ссылаемся в пределах анонимного внутреннего переопределения класса, является просто служебной переменной, то она не должна быть указана среди членов класса.

Но - с 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;
}

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

Ответ 3

not Error:

JSONObject json1 = getJsonX();

Ошибка:

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

Ошибка: Локальная инструкция переменной, определенная в охватывающей области, должна быть окончательной или окончательной.

Но вы можете написать:

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

Ответ 4

Я нашел этот подход полезным. Таким образом, вам не нужен класс и окончательный

 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));