- 1
- 2
- 3
- 4
- 5
<?php
if(!@isset($_POST[submit]))
{
...
?>
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+144.8
<?php
if(!@isset($_POST[submit]))
{
...
?>
guest 09.04.2010 22:55 # −1.6
и где константа submit объявлена ?
а так не особенно говнисто
HyperGeek 09.04.2010 22:56 # +1.2
guest 10.04.2010 14:15 # 0
DmitryDick 09.04.2010 23:09 # −2.8
HyperGeek 09.04.2010 23:26 # +0.4
larrygingras2 10.04.2010 19:00 # −1
PHPшные движухи в плане "как надо" на самом деле абсолютно неуместны. Либо надо было сразу делать "как надо" либо уже молчать в тряпочку.
Особенно меня убило ВНЕЗАПНОЕ отключение magic_quotes_gpc в рекомендуемой версии php.ini для production environment... (Кто в курсе тот поймет)...
guest 09.04.2010 23:44 # +0.4
pee 10.04.2010 00:17 # +3
guest 10.04.2010 06:05 # +0.8
larrygingras2 10.04.2010 19:09 # 0
Лично я быдло-руководитель и я РЕКОМЕНДОВАЛ писать строковые ключи в кавычках, но некоторые так и пишут без кавычек, мне похуй. Главное что работает...
guest 10.04.2010 23:30 # +0.2
larrygingras2 10.04.2010 23:38 # 0
А в чем проблема-то?
Это всё ведет только к тому, чтобы вас быстренько взломали, причем не хакеры, а дети-со-скриптами.
Я не думаю что ты будешь крайним в этом случае, раз ты предупреждал...
Или ещё лучше (для тебя), раздобудь любой (похуй какой, я думаю там любой что надо покажет) сканер уязвимостей, проскань и приди сутречка к директору с отчетом...
При грамотном подходе премия обеспечена...
larrygingras2 10.04.2010 19:04 # −0.2
DmitryDick 10.04.2010 11:10 # +1.2
Death 10.04.2010 18:34 # −1.4
разве нельзя просто написать if ($_POST[submit]) { ... } ???
larrygingras2 10.04.2010 19:06 # −1
В твоей конструкции условие не сработает если, например, в $_POST[submit] будет 0 (или "0", не важно)
Death 10.04.2010 19:26 # 0
larrygingras2 10.04.2010 19:31 # 0
Но всё же я бы не советовал лишний раз делать подобные допущения. А то мало ли что... Вдруг очередному заказчику захочется кнопку сабмита подписать как "0"...
Хотя грешен, у самого такое есть в коде :(
Death 10.04.2010 20:58 # 0
Хотя лично я всегда использую hidden поля типа <input type='hidden' name='op' value='send' />
if ($op) { ... }
larrygingras2 10.04.2010 21:09 # 0
larrygingras2 10.04.2010 21:10 # 0
larrygingras2 10.04.2010 21:13 # 0
Как вы сказали?
if ($op)?
То есть вы до сих пор пользуетесь register_globals?!
Oh, my Buddha...
Death 10.04.2010 21:15 # 0
larrygingras2 10.04.2010 21:18 # 0
Dreyk 10.04.2010 23:45 # +1.2
Конечно, давайте говорите, что нотисы - это херня, продолжайте юзать строки без кавычек, а так же инкременьте необъявленные переменные и т.д.
Да и потом не забудьте поругать PHP за то, что он, сука такая, медленный очень.
PHP - не быдло-язык, просто это язык, на котором говнокод писать оочень легко.
DmitryDick 10.04.2010 23:53 # 0
DmitryDick 10.04.2010 23:49 # 0