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

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

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