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

    • ava sayidandrtfm 15.08.2012 14:57 # +1

      return "-A";

      А в точке вызова if ((GetNextDishCode()).Length >1) /*...*/
      Ответить

      • ava minuzZ 15.08.2012 15:16 # −3

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

        • ava sayidandrtfm 15.08.2012 15:21 # +3

          >Вообще, чтобы пробежаться по алфавиту, не обязательно выделять массив...

          КЭЭЭЭП!?
          Ответить

          • ava minuzZ 15.08.2012 15:24 # +2

            Ну, если вам это было очевидно, к чему первый комментарий?
            Ответить

            • ava sayidandrtfm 15.08.2012 15:31 # +3

              Тег сарказм включён по умолчанию(с)
              Ответить

              • ava 3.14159265 15.08.2012 15:34 # +2

                Надо выводить это как предупреждение в хедере или при первом заходе на сайт.
                Ответить

                • ava sayidandrtfm 15.08.2012 15:41 # +3

                  sayidandrtfm - user/3428
                  minuzZ - user/3186

                  ГК надо посещать регулярно.
                  Ответить

                  • ava vistefan 15.08.2012 15:48 # +2

                    Правильно. Он наверное ещё даже не в курсе, что тут появилась модерация, а циферка более-меннее олдфажная. Кругом обман.
                    Ответить

                    • ava minuzZ 15.08.2012 15:53 # 0

                      Увы и ах, появляюсь здесь только с целью поделиться найденным. Учитывая размеры текущего проекта и количество человек над ним работающих, думаю еще много чего откопаю, обещаю поделиться :)
                      Ответить

                    • ava psina-from-ua 17.08.2012 01:35 # +1

                      Да что там он? Вот я(user/29), а про модерацию ничего не знаю:( Хоти захожу сюда регулярно: утром, в обед, и вечером.
                      Ответить

                      • ava vistefan 17.08.2012 12:53 # 0

                        > утром, в обед, и вечером.
                        До приёма пищи, или после?
                        > про модерацию ничего не знаю
                        Видимо качество прочтения тредов низкое.
                        Ответить

        • ava Lennis 15.08.2012 15:36 # +3

          if(!CheckDishes("A") return "A";
          else if (!CheckDishes("B") return "B";
          else if (!CheckDishes("C") return "C";
          ...
          Ответить

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