Да ну бросьте. Хороший тон это понятие растяжимое.
АПД:
Ладно, раз никто толком ничего не объяснил
Объясню я.
В примере с вложенной функцией есть такие недостатки:
1) при каждом клике создается новая функция
2) отписаться нельзя т.к. ссылка на функцию нигде не сохраняется.
Но при всем при этом вложеннаая функция - не зло само по себе, но надо понимать, что она будет создаваться заново при каждом вызове функции контейнера - это утечка памяти ну и просто некрасиво. Если же используется метод вместо функции то он не будет пересоздаваться много раз. Методы вобще не пересоздаются даже для каждого экземпляра, а создаются единожды для класса. Скажем, у вас есть 100 экземпляров одного класса у которых есть метод method, данный метод не будет дублироваться в памяти 100 раз.