1. Java / Говнокод #5633

    +71

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    class CellEditor {

   protected CellEditor(Composite parent, int style) {
	this.style = style;
	create(parent);
   }

   public void create(Composite parent) { ... }

}

    А вот это уже JFace...
    5 строка подарил много положительных эмоций, при попытке сконструировать кастомный CellEditor

    Запостил: tir, 11 Февраля 2011

    Комментарии (17) RSS

    • ava tir 11.02.2011 17:49 # −2

      ммм, те кто минусуют аргументировали бы

      P. S. Для большей очевидности можно слово abstract добавить в 1-й строке
      Ответить

      • ava StriderMan 11.02.2011 18:01 # −1

        минусы символизируют, что код - говно. Ты разве не знал?
        Ответить

        • ava absolut 11.02.2011 18:20 # 0

          то же самое для комментариев справедливо ;)
          Ответить

    • ava Lure Of Chaos 11.02.2011 18:59 # 0

      комментарии минусуют когда код говно?
      кэп, а в чем говно-то?
      Ответить

    • ava gegMOPO4 11.02.2011 21:48 # 0

      Где здесь Капитан?
      Ответить

    • ava istem 11.02.2011 23:18 # −1

      Пацаны, шухер! ЭТО ПОСТ ЗЛА!!!111
      Ответить

    • ava tir 12.02.2011 00:30 # −1

      Ок. Объясняю. Перегрузка метода, вызываемого из конструктора - зло, причем великое. Почему - читайте Джошуа Блоха "Java. Эффективное программирование" (Joshua Bloch "Effective Java"). Если кто-то из java кодеров ее еще не читал - срочно качайте и читайте. Так сказать книга must read. Кому лень читать, попробую . объяснить. Допустим у меня есть потомок, например MyCellEditor. Чтобы создать мой собственный редактор, мне надо перегрузить метод create. Допустим, что я хочу, чтобы это был комбобокс , а количесто элементов в списке зависело от передаваемого параметра. Я бы мог определить конструктор
      MyCellEditor(Composite parent, int style, int numberOfLines) {
      super(parent, style);
      this.numberOfLines = numberOfLines;
      }, ну и в методе create() соответственно написать код в стиле
      ...
      combo.setVisibleLinesCount(this.numberOf Lines)
      ...

      Да как говорится, не тут-то было. В момент когда я вызываю super(parent, style) вызывается перегруженный мною метод create(), и в данный момент переменная this.numberOfLines не определена (точнее равна нулю, т. к. это поле). И красивых способов исправить данную ситуацию нет.

      Вывод: НИКОГДА НЕ ПЕРЕГРУЖАЙТЕ МЕТОД ВЫЗЫВАЕМЫЙ ИЗ КОНСТРУКТОРА!!!
      Ответить

      • ava gegMOPO4 12.02.2011 14:01 # 0

        Всё так. Методы, вызываемые из конструктора должны быть final или private. Нормальные IDE недовольно ворчат по этому поводу.
        Ответить

        • ava tir 12.02.2011 14:20 # 0

          угу. а в JFace'е приведенный пример - Public API и другого способа для создания cell editor'ов нет.
          Ответить

      • ava StriderMan 14.02.2011 09:55 # −1 

        super(parent, style);
this.numberOfLines = numberOfLines;

        а поменять местами не судьба?

        не раскрыта суть метода create(). Нахер его перегружать? Заведите свой .MyCreate() в наследнике.
        Ответить

        • ava tir 14.02.2011 12:07 # 0

          в методе create надо реализовывать создание редактора, собсно в этом методе мне уже как бы и надо знать numberOfLines. create() - метод из public API, также как и сам класс CellEditor.
          Ответить

          • ava StriderMan 14.02.2011 14:19 # −2

            так чем не нравится в конструкторе поменять местами строчки?
            Ответить

            • ava tir 14.02.2011 15:28 # +2

              в java нельзя ничего вставить до вызова super()
              Ответить

              • ava StriderMan 14.02.2011 15:41 # +1

                гыг, ацтой. не знал. Я Вообще-то на Delphi пишу обычно ))))
                Ответить

                • ava Lure Of Chaos 15.02.2011 16:48 # 0

                  а еще, если явно super() не указан, ява считает, что он все равно там есть
                  Ответить

              • ava PascalGovno 14.02.2011 16:24 # +3

                вы всё еще считаете, что нет ничего сранее, чем дельфи? тогда мы несем яву к вам!
                Ответить

    Добавить комментарий