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

    +134.9

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    // Объявлен класс GOSTDocument extends TechDocument ( ввожу в курс дела:) )

public class TechDocument
{

  protected TechDocument(){} //Ибо нефиг

  public TechDocument getDocumentA ( Type DocType )
{

        TechDocument tmpl = null;
  switch (DocType)
  {
      case DOCTYPE_GOST : tmpl = (GOSTDocument) new GOSTDocument();
      break;
// ... Тут еще CASE всякие
      default: tmpl = new TechDocument();
      break;
  }
  return tmpl;

}

}

    Как бы кастинг.:)

    Запостил: guest, 21 Июля 2009

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

    • ava guest 21.07.2009 12:44 # +1

      Tinhol:
      Видимо это такой factory method...

      Вобщем то особо говенного кроме

      ненужного преобразования типов

      здесь вроде нет. Или я ошибаюсь?
      Ответить

    • ava guest 21.07.2009 14:27 # +1

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

      Да ещё и в силу protected конструктора.
      Ответить

    • ava guest 21.07.2009 15:36 # 0

      public TechDocument getDocumentA ( Type DocType ) по идее быть вообще не должно. М.б. это наследство какого-нибудь доисторического языка?
      Ответить

    • ava пъянь 16.08.2009 16:41 # 0

      имхо не самая важная часть похоже древняя заглушка 100 раз зарефактореная
      Ответить

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