Кресты / Говнокод #4546 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 guest666, (Updated )

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

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

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

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

Из-за тебя ушел bormand, guest!

    А не использовать ли нам bbcode?


    8