1. PHP / Говнокод #4119

    +156

    1. 1
    if($urate['time'] < $time and count($urate) > 0) { // Если в течении суток голосовали и есть записи - фиг

    Запостил: Мартин, 28 Августа 2010

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

    • ava Lure Of Chaos 28.08.2010 23:42 # −1

      я правильно понял что второе условие после первого теряет смысл?
      Ответить

    • ava vaseninm 29.08.2010 12:55 # 0

      echo (NULL < 1231231) ? "тру" : "фолс"; //тру
      Только я бы isset() заюзал.
      Ответить

      • ava Lure Of Chaos 29.08.2010 13:35 # −1

        define("ONE_TWO_THREE_ONE_TWO_THREE_ONE" ,123123);

        echo (NULL < ONE_TWO_THREE_ONE_TWO_THREE_ONE) ? "труc" : "фол"; //труc!
        Ответить

      • ava GhOsTMZ 29.08.2010 14:30 # 0

        Не всегда имеется смысл... Тут возможно значение берется из базы, а значит элемент time в массиве будет в любом случае, пусть даже с пустым значением. А зачем проверять наличие того, что при любом раскладе должно быть?
        Ответить

        • ava vaseninm 30.08.2010 10:54 # 0

          А первый голос?
          Ответить

        • ava vaseninm 30.08.2010 11:35 # 0

          1) первый раз проголосвать не получится ибо поначалу выборка пустая.
          2) наверняка для каждго юзера выбрается. То есть если человек еще не голосовал - у него будет пустая выборка и первый раз он проголосовать не сможет.
          Ну а вообще такие вещи нужно проверять в SQL.
          Ответить

    • ava Holden 30.08.2010 14:27 # 0

      "and" - это совсем другое. Хотя в данном случае будет работать.
      В любом случае правильно писать "&&".
      Ответить

      • ava mindfreakthemon 30.08.2010 18:53 # +1

        and - это то же самое, что и && с отличием в том, что приоритет у него ниже плинтуса.
        Ответить

        • ava 7ion 31.08.2010 03:43 # −1

          Разве?
          Разве это не аналог для легкой адаптации к языку с других ЯП?
          Ответить

          • ava mindfreakthemon 31.08.2010 04:34 # +2 

            $a = 1 and (print 'o');
var_dump($a);

print "<br />\n";

$a = 1 && (print 'o');
var_dump($a);


            на выходе:
            oint(1) 
obool(true)
            Ответить

            • ava 7ion 31.08.2010 14:37 # 0

              Хм, не знал.
              Благодарю за просвещение.
              Ответить

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