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

    +950.3

    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
    //...
for (int i = 0; i < arr.Length; i++)
{
      if (i == 5)
      {
             if (arr[i] == -1)
             {
                    break;
             }
             else
             {
                    return -1;
             }
      }
      else continue;
}
//...

    Запостил: FMB, 31 Декабря 2009

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

    • ava TarasB 31.12.2009 23:46 # 0

      Классика. Хорошо хоть, догадался break поставить.
      Ответить

    • ava guest 01.01.2010 01:11 # 0

      Проголосовал за говнокод, но пожалуйста... а не могли бы вы весь метод запостить?...
      Ответить

      • ava FMB 02.01.2010 00:34 # +2 

        public static int is5big(string str)
{
      char[] strArr = str.ToCharArray();
      int[] arr = new int[strArr.Length];
      for (int i = 0; i < strArr.Length; i++)
      {
            if ((int)strArr[i] >= (int)'a' && (int)strArr[i] <= (int)'z') arr[i] = -1;
            else if((int)strArr[i] >= (int)'A' && (int)strArr[i] <= (int)'Z') arr[i] = 1;
      }
      for (int i = 0; i < arr.Length; i++)
      {
            if (i == 5)
            {
                   if (arr[i] == -1)
                   {
                          break;
                   }
                   else
                   {
                          return -1;
                   }
            }
            else continue;
      }
      return 1;
}

        Когда - то надо было проверить является ли 5 буква строчной или нет. Тогда ничего другого придумать не смог, видимо спать очень хотел =)
        Ответить

    • ava guest 01.01.2010 18:03 # −7

      показать все, что скрытоА что не так? Вроде обычный код?
      Ответить

      • ava guest 02.01.2010 01:35 # 0

        Для Вас этот код является обычным? То есть char.IsLower(arr[4]) ? -1 : 1 и все это - одинаково красиво? IMHO, незнание возможностей платформы является признаком некомпетентности.
        Ни в коем случае не хочу обидеть автора. Если это было для него в период изучения платформы, то все ок. Если же он работает с ней не один год - увы.
        Ответить

        • ava KiTE 12.01.2010 20:05 # +1

          В первую очередь говнокод в том, что цикл делает бесполезные первые 4 итерации. И абсолютно ненужная строка else continue;
          Ответить

    • ava guest 01.01.2010 18:55 # −2

      if (arr.Length>4)
      {
      if (arr[5] != -1)
      return -1;
      }
      Ответить

      • ava TarasB 01.01.2010 19:03 # +3

        Массивы тут с нуля нумеруются, вообще-то.
        Ответить

      • ava guest 02.01.2010 01:34 # +1

        if (arr.Length > 5 && arr[5] != -1) return -1;
        Ответить

        • ava guest 02.01.2010 13:24 # −1

          Если первое условие не прокатило, второе не станет оцениваться?
          Ответить

          • ava guest 03.01.2010 10:47 # +2

            Смысла нету, ведь результат уже известен.
            Ответить

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