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

    +155

    1. 1
    2. 2
    3. 3
    $bonus = "нет";
$message = "";
$message .= "<b>Бонус-код:</b> ".$bonus."<br><br>";

    Завидная стабильность.

    Запостил: matveev, 18 Ноября 2009

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

    • ava guest 18.11.2009 17:02 # +1

      гм... а ежели ему надо будет этот самый $bonusюзать во многих местах кода? тады может и оправдано...
      Ответить

      • ava matveev 18.11.2009 17:06 # 0

        Да, спасибо, я не уточнил.
        Больше ни где не используется. Более того, по идее это значение должно было браться из формы регистрации.. То есть бонусы не светят ни кому.. вот так печально.. =)
        Ответить

        • ava ivmaroz 19.11.2009 11:45 # 0

          Еще как предположение, раньше этот бонус рассчитывался, а потом сказали, что больше бонусов нет. Лично я бы тогда поменял значение переменной. Правда, еще закомментировал бы старый код, чтобы можно было вернуться обратно.
          Ответить

    • ava guest 18.11.2009 20:14 # +1

      Если уж всё равно двойные кавычки, зачем явная конкатенация?(
      Ответить

      • ava guest 19.11.2009 08:12 # +1

        Многие так привыкли
        Ответить

      • ava guest 19.11.2009 11:55 # 0

        А что плохого? Ещё с php4 или даже с 3 учили что нужно делать явную конкатенацию для облегчения работы компилятора и соответственно увеличения скорости.
        Ответить

        • ava guest 19.11.2009 11:56 # 0

          Имелось ввиду что плохого в ответ на фразу
          Если уж всё равно двойные кавычки, зачем явная конкатенация?(
          Ответить

        • ava DanxilLs 05.01.2010 09:03 # 0

          Дык там в любом случае пустая как понятно по куску кода переменная, зачем нужно делать конкатенацию, если можно просто присвоить ей значение?
          Ответить

    • ava foxweb 20.11.2009 13:21 # +1

      а типа мы так нотайсы обходим :)
      Ответить

    • ava guest 26.11.2009 22:24 # 0

      А зачем двойная установка переменной?
      Ответить

      • ava guest 27.11.2009 11:24 # 0

        Чтобы нотиса не было, полагаю у него там ещё данные добавляются в $message.
        Ответить

        • ava matveev 27.11.2009 13:42 # 0

          Да, там

          $message = "";

          и потом сток пятьдесят типа

          $message .= "<b>Ла-ла:</b> ".$la_la."<br><br>";

          =)))
          Ответить

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