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

    +155

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

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

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

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

    • гм... а ежели ему надо будет этот самый $bonusюзать во многих местах кода? тады может и оправдано...
      Ответить
      • Да, спасибо, я не уточнил.
        Больше ни где не используется. Более того, по идее это значение должно было браться из формы регистрации.. То есть бонусы не светят ни кому.. вот так печально.. =)
        Ответить
        • Еще как предположение, раньше этот бонус рассчитывался, а потом сказали, что больше бонусов нет. Лично я бы тогда поменял значение переменной. Правда, еще закомментировал бы старый код, чтобы можно было вернуться обратно.
          Ответить
    • Если уж всё равно двойные кавычки, зачем явная конкатенация?(
      Ответить
      • Многие так привыкли
        Ответить
      • А что плохого? Ещё с php4 или даже с 3 учили что нужно делать явную конкатенацию для облегчения работы компилятора и соответственно увеличения скорости.
        Ответить
        • Имелось ввиду что плохого в ответ на фразу
          Если уж всё равно двойные кавычки, зачем явная конкатенация?(
          Ответить
        • Дык там в любом случае пустая как понятно по куску кода переменная, зачем нужно делать конкатенацию, если можно просто присвоить ей значение?
          Ответить
    • а типа мы так нотайсы обходим :)
      Ответить
    • А зачем двойная установка переменной?
      Ответить
      • Чтобы нотиса не было, полагаю у него там ещё данные добавляются в $message.
        Ответить
        • Да, там

          $message = "";

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

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

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

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