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

    +24

    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
    int MyForm::modelId(int button, bool rarefied, bool grouped)
{
    if (button == 4)
        return 9;
    else
    {
        Q_ASSERT(button == 1 || button == 2 || button == 3);
        if (!rarefied)
            return (button - 1);
        else
        {
            if (!grouped)
                return 3 + (button - 1);
            else
                return 6 + (button - 1);
        }
    }
}

    Энумы? Не, не слышал.

    Запостил: ABBAPOH, 14 Февраля 2013

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

    • ava santa_microbe 15.02.2013 04:28 # +2

      magic numbers, voodoo bumbers
      Ответить

    • ava dwarf_with_beer 15.02.2013 18:47 # +3

      надо было так:
      if(button == 4)
      return button+5;
      Ответить

      • ava bormand 15.02.2013 18:52 # 0 

        int MyForm::modelId(int button, bool rarefied, bool grouped) {
    return button + (button < 4 ? rarefied ? grouped ? 5 : 2 : -1 : 5);
}
        Ответить

        • ava scriptin 15.02.2013 19:01 # 0

          Это более читаемо, чем оригинал.
          Ответить

          • ava bormand 15.02.2013 19:03 # +2

            О_о

            Ну в оригинале автор, конечно, перестарался с отрицаниями, но кроме волшебных цифр ничего непонятного там нет.
            Ответить

          • ava bormand 15.02.2013 19:08 # 0

            Кнопки 1-3 выбирают:
            Если не стоит rarified - модели 0-2.
            Если стоит rarified но не grouped - модели 3-5.
            Если стоят и rarified и grouped - модели 6-8.
            Кнопка 4 всегда выбирает модель 9.

            Вполне читаемо и понимаемо. Знать бы еще что-за пронумерованные кнопки и модели ;)
            Ответить

            • ava ABBAPOH 15.02.2013 21:53 # 0

              Бинго:) А вот для понимания номеров как раз подойдут энумы. Ничего дико плохого в коде, на самом деле, нет, просто показалось забавным.
              Ответить

        • ava 3.14159265 15.02.2013 22:23 # 0

          ВЫ СОВЕРЕШЕННО НЕ ПОНИМАЕТЕ СМЫСЛ ЭТОГО КОДА. СУТЬ В ЦИФРЕ 5!
          return button + (button+1 <5 ? rarefied ? grouped ? 5 : 5*5/10 : -5/5 : 5);

          @dwarf_with_beer во многом прав.
          Ответить

          • ava 3.14159265 15.02.2013 22:26 # 0

            Ехал button через 5.
            Видит button: 5,5,5,
            Кликнул button цифру 5
            5,5,5, 5,5, 5,5!
            Мой голос стал ↑+5. Это подтверждает правоту моих слов: http://www.peeep.us/14b7eaa0
            Ответить

          • ava bormand 16.02.2013 07:34 # +2 

            return button + (button+5/5 < 5 ? rarefied ? grouped ? 5 : 5*5/(5+5) : -5/5 : 5);
            fxd
            Ответить

            • ava absolut 16.02.2013 11:46 # 0

              кстати, возможны ли ситуации когда в выражении A*B/C сначала выполнится деление, а потом умножение?
              Ответить

              • ava TarasB 16.02.2013 12:51 # +1

                Нет, очевидно, это нарушит результат
                Ответить

                • ava Lure Of Chaos 17.02.2013 17:27 # +1

                  погрешность изменится и не будет переполнения (в случае больших цифр)?
                  Ответить

    • ava TarasB 15.02.2013 19:51 # 0

      Это с каких пор слово button стало крестоблядским зарезервированным словом?
      Не, если из кода смотреть, то всё нормально, а если из стока, то оно синее
      Ответить

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