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

    +152.3

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    if($_POST[tel]=='')
	{
		print "<script language=\"javascript\">\n";
		print "showmsg('tel');\n";
		print "</script>\n";
	}
	if($_POST[name]=='')
	{
		print "<script language=\"javascript\">\n";
		print "showmsg('name');\n";
		print "</script>\n";
	}
	if(($_POST[name]!='')&&($_POST[tel]!=''))
	{
		print "<meta http-equiv=\"Refresh\" content=\"0;URL=send.php?name=$_POST[name]&tel=$_POST[tel]\" />\n";
		die();
	}

    Разбираю один движок инет-магазина. Первое что я понял - автор идиот. Второе - он совершенно ничего не знает об elseif'ах

    Запостил: guest, 18 Июля 2009

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

    • ava guest 18.07.2009 04:24 # +1

      lolwho:
      Феерический бред :D
      Ответить

    • ava guest 18.07.2009 07:46 # +3

      Иван:
      > Второе - он совершенно ничего не знает об elseif'ах

      Объясни тогда как с помощью elseif переписать этот код, чтобы одновременно выводилось showmsg('tel') и showmsg('name').

      Код с логической точки зрения правильный
      Ответить

    • ava guest 18.07.2009 11:21 # +2

      TEHEK:
      я даж не знаю, что поставить =)

      Логика такова:
      если отсутствует телефон или имя, вывести соответствующие сообщения. если все на месте - перенаправить...

      Ельзифы тут действительно ни при чем, но сама по себе логика остается отстоем.

      Что может помешать сделать
      send.php?name=Вася+пупкин&tel=лапша
      Ответить

    • ava guest 18.07.2009 19:42 # −2

      [quote]send.php?name=Вася+пупкин&tel= лапша[/quote]
      Может, то что это POST? =)
      Ответить

    • ava guest 18.07.2009 19:42 # 0

      Хотя, конечно, это не даёт никаких гарантий.
      Ответить

    • ava guest 20.07.2009 13:57 # +2

      O_O:
      [quote=Говногость]Может, то что это POST? =[/quote]
      с каких пор [quote]send.php?name=Вася+пупкин&tel=лапша[/quote] это пост? :)
      Ответить

    • ava guest 20.07.2009 22:40 # +1

      Во-первых, запостивший идиот и сам ничего не знает о elseif'ах, они здесь не подойдут к логике. Во-вторых, код не столько говно, сколько просто некрасив, но при этом легко читается.
      Ответить

    • ava guest 21.07.2009 12:37 # 0

      тут должна быть стандартная валидация.. заполняем массив errors, в конце смотрим,если он не пустой - то есть ошибки, которые и выводим.
      Ответить

    • ava guest 22.07.2009 09:57 # +1 

      <?php
$e = false;
foreach (array('name', 'tel') as $v)
  if ($_POST[$v] == '')
  {
    echo "<script type='text/javascript'>\nshowmsg('$v');\n</script>\n";
    $e = true;
  }
if ($e)
{
  echo '<meta http-equiv="Refresh" content="0;URL=send.php?name='.htmlspecialchars($_POST['name']).'&tel='.htmlspecialchars($_POST['tel'])."\" />\n";
  die();
}
?>


      З.Ы. За вещи вроде $_POST[name], когда имя ключа массива не заключено в кавычки, надо бить по ебалу. При уровне ошибок E_NOTICE такие конструкции устраивают на страницах настоящую веселуху.
      Ответить

    • ava guest 27.07.2009 23:24 # 0

      по ебалу бить не стоит - достаточно просто обьяснить.... а вот стоит бить по ебалу за <b>if ($_POST[$v] == '')<b/> (ибо эксепшн на пустом месте от долбоебизма), и за использование в данном случае <b> foreach<b/> - заебали "академики"((((
      Ответить

    • ava guest 02.08.2009 07:13 # 0

      да тут все академики куда ни плюнь, свой код выложите а мы всей толпой посмотрим и разберем по косточкам, а потом и в каментах выебывайтесь... от собственной злобы дристос еще не пробрал? По ебалу они тут друг дружку херачат, хе хе
      Ответить

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