Что такое предупреждение «this-escape» и как с ним справиться?

Вот запись в системе ошибок JDK, в которой представлено это новое предупреждение — https://bugs.openjdk.org/browse/JDK-8299995

Короче говоря, предупреждение this-escape предназначено для предупреждения вас, когда подкласс может @Override использовать метод, который также вызывается в конструкторе суперкласса.

Это опасно, поскольку переопределение метода, используемого в конструкторе, позволяет подклассу непреднамеренно внести ошибку во время инициализации подкласса. Что, если этот метод зависит от состояния, которое еще не создано, поскольку мы все еще находимся в суперконструкторе? В конце концов, вы не можете ничего сделать в конструкторе подкласса до вызова суперконструктора (пока ).

Есть несколько способов исправить это.

1. Используйте в конструкторе только те методы, которые нельзя переопределить.

   • static методы.

   • final методы.

   • private методы.

2. Сделайте сам урок final.

3. Не передавайте/используйте this с самого начала — вместо этого передайте тот конкретный компонент this, который вам нужен.

   • По сути, перестаньте лениться и четко сформулируйте, что вам нужно. Не передавайте просто свой объект God — передавайте только те атрибуты, которые вам нужны.

Обратите внимание: эти правила применяются рекурсивно. Это означает, что когда вы вызываете метод в конструкторе, этот метод не только должен быть «непереопределяемым», но и методы, в которые этот метод передает this, ТАКЖЕ должны соответствовать одному из приведенных выше правил. Если ваш метод верхнего уровня не является переопределяемым, но один из методов внутри него переопределяем, и этот метод имеет this в своей области действия, то при компиляции вы получите ошибку this-escape. Вот пример.

import javax.swing.*;

public class GUI
{

   private final JFrame frame;

   public GUI()
   {
   
      this.frame = new JFrame();
   
      this.frame.add(this.createBottomPanel());
   
   }

   //final! Does that mean we are safe?
   final JPanel createBottomPanel()
   {
   
      final JButton save = new JButton();
   
      save
         .addActionListener
         (
            /*
             * No. We get the warning here at the start of this lambda.
             * The reason is because we have given this lambda the
             * ability to call this.toString(), and we don't know when
             * it will do that. Maybe now, maybe later. But if it does
             * it now, then we could end up doing things before the
             * object is fully created. And if that were to happen, then
             * that would be a this-escape. So, the warning occurs here,
             * to let you know that it is possible.
             */
            actionEvent ->
            {
           
               this.toString();
           
            }
           
         )
         ;
   
      return null;
   
   }

}