1. C# / Говнокод #11594

    +134

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    private string GetNextDishCode()
    {
        string[] vesCode = { "A", "B", "C", "D", "E", "F", "G", "H", "I", "J", "K", "L", "M", "N", "O", "P", "Q", "R", "S", "T", "U", "V", "W", "X", "Y", "Z" };
        for (int i = 0; i < vesCode.Length; i++)
            if (!CheckDishes(vesCode[i]))
                return vesCode[i];
        return "AA";
    }

    В одном, довольно крупном и дорогом проекте.

    Запостил: minuzZ, 15 Августа 2012

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

    • return "-A";

      А в точке вызова if ((GetNextDishCode()).Length >1) /*...*/
      Ответить
      • Вообще, чтобы пробежаться по алфавиту, не обязательно выделять массив, хардкодить его алфавитом, а потом бежать по нему) А то что вы говорите, да, пожалуй так и используется. Но гавнокод явно в том, о чем написал я :) Не стал добавлять в описание - так как думал, что это и так очевидно.
        Ответить
        • >Вообще, чтобы пробежаться по алфавиту, не обязательно выделять массив...

          КЭЭЭЭП!?
          Ответить
          • Ну, если вам это было очевидно, к чему первый комментарий?
            Ответить
            • Тег сарказм включён по умолчанию(с)
              Ответить
              • Надо выводить это как предупреждение в хедере или при первом заходе на сайт.
                Ответить
                • sayidandrtfm - user/3428
                  minuzZ - user/3186

                  ГК надо посещать регулярно.
                  Ответить
                  • Правильно. Он наверное ещё даже не в курсе, что тут появилась модерация, а циферка более-меннее олдфажная. Кругом обман.
                    Ответить
                    • Увы и ах, появляюсь здесь только с целью поделиться найденным. Учитывая размеры текущего проекта и количество человек над ним работающих, думаю еще много чего откопаю, обещаю поделиться :)
                      Ответить
                    • Да что там он? Вот я(user/29), а про модерацию ничего не знаю:( Хоти захожу сюда регулярно: утром, в обед, и вечером.
                      Ответить
                      • > утром, в обед, и вечером.
                        До приёма пищи, или после?
                        > про модерацию ничего не знаю
                        Видимо качество прочтения тредов низкое.
                        Ответить
        • if(!CheckDishes("A") return "A";
          else if (!CheckDishes("B") return "B";
          else if (!CheckDishes("C") return "C";
          ...
          Ответить

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