Да, есть потенциальные ошибки в сборке мусора с этим.Однако другие ответы верны в том смысле, что это стандартная идиома.
Когда вы создаете такой анонимный внутренний класс, у него есть скрытая ссылка на внешний класс.Это означает, что до тех пор, пока эта кнопка существует с этим действием, на внешний класс будут ссылаться, как и на все, на что ссылается этот внешний класс.
Однако на практике это, как правило, не является проблемой, поскольку внешнийКласс обычно будет окном или панелью, где находится эта кнопка, и кнопка будет находиться рядом с ней, пока панель находится.Вот почему это стандартная идиома - это нормально.