1. Си / Говнокод #3386

    +132

    1. 1
    2. 2
    3. 3
    if ((strlen == 0) && (key ==BNS_CLR_K || key ==WISE_K_BSPACE))
				;
else

    Запостил: aljosha, 03 Июня 2010

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

    • ava guest 03.06.2010 10:08 # 0

      Изыди!
      Ответить

    • ava guest 03.06.2010 12:43 # +5

      с булевой алгеброй и логикой у нас плохо. PHPшником будет
      Ответить

    • ava absolut 03.06.2010 17:10 # +4

      strlen. Неудачное название переменной.
      Ответить

      • ava nil 03.06.2010 23:49 # +5

        «Намучается пацан с таким именем»
        Ответить

    • ava guest 04.06.2010 04:47 # 0

      Туплю наверное.
      если длина строки == 0, и нажата одна из клавиш, то...
      А где говнокод ?)
      Ответить

      • ava aljosha 04.06.2010 08:30 # 0

        Сразу же после..
        Ответить

        • ava nil 04.06.2010 08:36 # 0

          С другой стороны, это еще вопрос, как читаемее:

          (strlen != 0) || (key !=BNS_CLR_K && key !=WISE_K_BSPACE)

          Фиг знает, на любителя. Имхо, списки с логическим И немного проще воспринимаются.
          Ответить

          • ava Soul_re@ver 04.06.2010 09:15 # +1

            if !((strlen == 0) && (key ==BNS_CLR_K || key ==WISE_K_BSPACE))

            Любители разные бывают
            Ответить

            • ava absolut 04.06.2010 09:46 # +1

              А тем временем в ветке else такая же точка с запятой может быть.
              Ответить

            • ava nil 04.06.2010 10:07 # 0

              Вот это я больше всего не люблю, негативные условия в if.
              Знак "!" очень тонкий и визуально воспринимается с трудом.
              Читаешь какое-то условие, понимаешь его, смотришь код, охуеваешь, и только потом замечаешь, что условие обернуто.
              Но, может, это я один такой.
              Ответить

              • ava guest 04.06.2010 10:53 # +3

                Пустое тело тоже не сахар. Увидел точку с запятой и думай, что это - ошибка или шрифты глючат или вообще вот так:
                if (blabla && yohoho)                                                             do smth()
        ;
else


                Вообще в обсуждаемом случае сильно помог бы коммент ;//do nothing
                Ответить

                • ava nil 04.06.2010 11:23 # 0

                  Да, к вопросу о длине строки...
                  Да, за коммент всеми руками за!
                  Ответить

              • ava guest 04.06.2010 17:27 # 0

                #define not !
                Ответить

              • ava TarasB 05.06.2010 12:03 # +2

                Ставь три (или пять) восклицательных знака, это намного лучше видно.
                if !!!!!!!111
                Ответить

                • ava nil 05.06.2010 12:11 # 0

                  if !!!!!!!111oneone ?:)

                  Прелестно!
                  Ответить

                  • ava guest 05.06.2010 12:59 # 0

                    bool oneoneone=0;
                    if(!!!!!!!!oneoneone)
                    {
                    printf("Пук!!!!!1111/0x111");
                    };
                    Что выведется на экран?
                    Ответить

                • ava Stalker 05.06.2010 22:02 # 0

                  :))) Зашибись хороший совет!
                  Ответить

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