В вашем коде не должно быть неиспользуемых переменных, что делает его менее понятным. Кроме того, конструктор - это (как следует из его названия) метод для инициализации объекта, в вашем случае это еще не сделано.
Я предлагаю вам использовать статический метод вместо:
ButtonCellEditor.niceNameHere(table, 2);
Единственный случай, в котором я могу подумать, что конструктор был бы адекватным, это когда ему требуются параметры, чтобы инициализировать себя и затем выполнить некоторые действия позже, но не для выполнения действия внутри, и это не похоже на ваш.