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

    +162

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if(!$RISK && in_array($USER->GetID(), array("1", "14"))){
	$RECCURING = true;
}else{
	$RECCURING = false;
}

    Подписка на оплату. Магические числа 1 и 14.

    Запостил: hdkeeper, 28 Января 2011

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

    • ava Lure Of Chaos 28.01.2011 15:31 # +1

      и не только в числах беда.
      как и.о. К.О., заявляю:
      1. числа как стринги и в двойных кавычках
      2. in_array вместо двух примитивных сравнений
      3. именование переменных в упперкейсе - так должны именоваться только константы. Иначе кажется, что кодер или слепой (что бы лучше видеть) или глухой(привык орать)
      4. ифная конструкция лучше себя чувствовала в инкарнации тернарного оператора

      все, кажется, больше придраться не к чему
      Ответить

      • ava Анонимус 28.01.2011 15:58 # 0

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

        • ava Lure Of Chaos 28.01.2011 16:39 # 0

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

      • ava Uchkuma 28.01.2011 16:33 # +2

        5. и RECCURING вместо RECURRING, судя по всему.
        Ответить

      • ava gegMOPO4 28.01.2011 17:55 # 0

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

        • ava Under 28.01.2011 17:59 # +1 

          $RECCURING = (!$RISK && in_array($USER->GetID(), array("1", "14"))

          тернарный оператор там тоже кагбе не очень нужен
          Ответить

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