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

    +169

    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
    if (($_GET['var'])==0 and ($_GET['email'])==1 and ($_GET['numbers'])==1)
$label='Вы неправильно указали логин';
elseif (($_GET['var'])==0 and ($_GET['email'])==0 and ($_GET['numbers'])==1)
$label='Вы неправильно указали логин и е-мейл';
elseif (($_GET['var'])==1 and ($_GET['email'])==0 and ($_GET['numbers'])==1)
$label='Вы неправильно указали е-мейл';
elseif (($_GET['var'])==0 and ($_GET['email'])==0 and ($_GET['numbers'])==0)
$label='Вы неправильно указали логин, е-мейл и числовой набор';
elseif (($_GET['var'])==1 and ($_GET['email'])==1 and ($_GET['numbers'])==0)
$label='Вы неправильно указали числовой набор';
elseif (($_GET['var'])==0 and ($_GET['email'])==1 and ($_GET['numbers'])==0)
$label='Вы неправильно указали логин и числовой набор';
else
$label='';

    Вывод ошибки

    Запостил: Dedan, 28 Февраля 2011

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

    • ava nepster 28.02.2011 03:48 # 0

      а это от куда?
      Ответить

    • ava eth0 28.02.2011 09:49 # 0

      И суповой набор.
      Я всегда думал, что лень - двигатель прогресса. Видимо, такие кодоавторы не понимают, что копипаста отнюдь не "ленивое действие", а только кажется таким.
      Ответить

    • ava TarasB 28.02.2011 12:36 # +4

      А в ПХП можно так?
      label = switch ($_GET['var'], $_GET['email'], $_GET['numbers']) {
  (0, 1, 1): 'Вы неправильно указали логин'; break;
  (0, 0, 1): 'Вы неправильно указали логин и е-мейл'; break;
  (1, 0, 1): 'Вы неправильно указали е-мейл'; break;
  (0, 0, 0): 'Вы неправильно указали логин, е-мейл и числовой набор'; break;
  (1, 1, 0): 'Вы неправильно указали числовой набор'; break;
  (0, 1, 1): 'Вы неправильно указали логин и числовой набор'; break;
};
      Ответить

      • ava Skelet0n 28.02.2011 16:57 # 0

        Очень, кстати, наглядно. Такой код даже выглядит красиво.
        Ответить

        • ava Sulik78 28.02.2011 17:04 # 0

          только красивее бы тогда уже было
          $label = 'Вы неправильно указали ';
          $label .= switch ($_GET['var'], $_GET['email'], $_GET['numbers']) {
          (0, 1, 1): 'логин'; break;
          (0, 0, 1): 'логин и е-мейл'; break;
          .........
          }
          Ответить

          • ava punishment 01.03.2011 22:40 # 0

            Но сработает только 
            $label = 'Вы неправильно указали ';
switch ( array($_GET['var'], $_GET['email'], $_GET['numbers']) ) 
{
	case array(0, 1, 1): $label .= 'логин'; break;
	case array(0, 0, 1): $label .= 'логин и е-мейл'; break;
	case array(1, 0, 1): $label .= 'е-мейл'; break;
	case array(0, 0, 0): $label .= 'логин, е-мейл и числовой набор'; break;
	case array(1, 1, 0): $label .= 'числовой набор'; break;
	case array(0, 1, 1): $label .= 'логин и числовой набор'; break;
};
echo $label;
            Ответить

          • ava TarasB 01.03.2011 23:00 # 0

            $label = 'Вы неправильно указали:';
            if ($_GET['var'] == 0) $label.= ' логин,';
            if ($_GET['email'] == 0) $label.= ' пароль,';
            if ($_GET['numbers'] == 0) $label.= ' суповой набор,';
            if ($label[$label'last] == ':') then $label = '' else $label[$label'last] = '.';
            Ответить

      • ava xXx_totalwar 28.02.2011 17:14 # 0

        расмусик не знает, что такое pattern-matching
        Ответить

      • ava Sulik78 28.02.2011 19:16 # 0

        это двоичная проверка:
        3
        1
        5
        0
        6
        Ответить

      • ava Мистер Хэнки 28.02.2011 22:23 # +2

        А теперь вообразим на минутку, что в форме 10 или даже 20 полей, требующих валидации...
        Ответить

        • ava ZX_Spectrum 01.03.2011 02:35 # 0

          всего-то:
          10 полей - 1023 проверки
          20 полей - 1048575 проверок
          Ответить

    • ava gegMOPO4 28.02.2011 14:18 # 0

      Почему только 7 вариантов? Должно быть 8.
      Ответить

    • ava Dedan 28.02.2011 22:26 # 0

      И никого не смутили названия переменных, и что так сравнивать строковые переменные на пустоту - нельзя в принципе?
      Ответить

      • ava istem 01.03.2011 03:58 # 0

        Нет. Потому что - это "похапе", а это мало кого ибёт (кроме тов. Расмуса, возможно).
        Ответить

      • ava ZX_Spectrum 01.03.2011 15:32 # 0

        а вот меня смущает наличие скобочек:
        ($_GET['email'])==...
        Ответить

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