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

    +168

    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
    /*++
      прямое ли?
    --*/
    bool ModelCurve3D::IsStraight() const {
      return m_mathCurve ? m_mathCurve->IsStraight() : false;
    }
    
    /*++
      замкнутость
    --*/
    bool ModelCurve3D::IsClosed()  const
    {
      return m_mathCurve ? m_mathCurve->IsStraight() : false;
    }

    Замкнутый... ну знаешь, это типа как прямолинейный. И такое лежит годами. А как поправишь, еще может и регрессия случится. Ну как так можно?

    Запостил: guest666, 06 Ноября 2010

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

    • замкнуто-прямолинейный
      Ответить
    • Похоже потомок Лобачевского писал
      Ответить
      • ни фига... сам Риман! ( на что намекает: "И такое лежит годами" )
        Ответить
    • Да ладно, если брать прямые, например, на расширенной комплексной плоскости, то они замкнуты :)

      Хотя безрасличие между данными терминами в данном куске кода забавляет.
      Ответить
      • безразличие*
        Ответить
      • Ну это уже из области эзотерики...
        Ответить
        • Лично я почти каждый день использую различные выводы на комплексной плоскости, для вполне практических нужд
          Ответить
      • Хмм... Смотря что покурить...
        Ответить
    • Если функции делают одно и то же - зачем платить больше ?
      Ответить
      • К.О. приходит на помощь и дает подсказку: у m_mathCurve есть метод IsClosed
        Ответить
        • Как-то это совсем не О. из кода. А какой по-вашему правильный вариант ?
          Ответить
          • Не поверишь, но правильный вариант это когда ModelCurve3D::IsStraight вызывает IsStraight, а ModelCurve3D::IsClosed вызывает IsClosed :)
            Ответить
    • это так специально написано или докопипастились?
      Ответить
      • Я думал четко видно что копипаст, который на выходе привел к крайне идиотскому результату. Но вообще немного не прав был, коду где то пол года, может еще и случится чего хорошего.
        Ответить
    • Copy-paste programming detected!
      Ответить
    • Замкнул твою прямую. Проверь.
      Ответить

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