1. C++ / Говнокод #5603

    +162

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    SeismicDriver* SeismicSelectionDG::seismicDriver()
{
  Exf2dMW* mw = Exf2dMW::getMW(this);
  _seismicDriver = mw->seismicDriver();
  if ( !mw || !_seismicDriver)
    return 0;

  return _seismicDriver;
}

    Все тот же коллега китаец. Exf2dMW - класс главного окна приложения. Во-первых бесмыссленно проверять его на 0, потому что без него вообще бы ничего не работало, но и как видно проверяет его он все равно не там. Ну и проверка на 0 _seismicDrvier тоже лишняя. Итог никакие проверки не нужны да и вообще первую строку в методе можно перенести в конструктор а возвращать член класса _seismicDriver.

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

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

    • ava ningun 09.02.2011 15:46 # 0

      Рассказывал коллега, прислали на допилку индокод. Срок допилки — 5 суток.
      На 3 сутки послали процесс рефака и тех кто писал код в Калькутту и дальше.
      За оставшееся время перепилили с нуля. Код работает и читается.
      Ответить

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