- 1
- 2
- 3
- 4
<?php
if ("$payment_method" == "1") {
?>
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+27.5
<?php
if ("$payment_method" == "1") {
?>
еще один шедевр, зачем так делать?
какой красивше?
Говногость, причем тут это
красивее будет не первый не второй а третий
так как он понятней не программистам..
но я думаю автор имел ввиду почему переменная в скобки запихнута
Если, например, объект, то код эквивалентен $payment_method->__toString() == "1"
Если число, то можно и не приводить.
(string)1 == "1"
1 == "1" // то же самое
Если boolean - тоже можно не менять тип.
(string)true == "1"
true == "1" // то же самое
А вот если массив, то другое дело.
(string)array(1,2) != "1"
array(1,2) != "1"
array(1,2) == true
Вопрос в том, нахрена в PHP приводить к стрингу переменную, которая потом еще и сравнивается с интом?
единственный шанс этого кода быть не говном такой:
но и он безнадёжно провален.
Диагноз: говнокод! :)
это же очевидно что там int ($payment_method=1;)
нахрена и то и это приводить к string?
Автор кода видимо незнает про типы данных вообще т.к далее по его коду
наблюдается такое
Литеральные константы -- это зло.
Что обозначает "1"? Каков его сакральный смысл?