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

    +164

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    $arr = array(1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1);
    
    function EtpGetEtpRoleCheckboxes($checked = array(1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1))
    {
    	$roles = EtpGetEtpRoleArray();
    	foreach($roles as $key => $role)
    	  $checkboxes .= "<input type=checkbox value='1' name='etp_roles[".$key."]' ".($checked[($key)] > 0 ? 'checked' : '')."> ".$role."<br />";
      return substr($checkboxes,0,-6);
    }

    с array_fill программист был не знаком. И так по всему скрипту. Это к первой строчке.

    В функции меня удивило задание имен <input>'ам и value="1".

    Запостил: windrun, 23 Сентября 2011

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

    • С array_fill тоже не знаком, но это ..здец
      Ответить
    • он со многим незнаком - с implode, например
      Ответить
      • а при чем здесь implode?
        Ответить
        • а то, что последний <br /> отрезали
          Ответить
          • ну эт да... хотя....
            Ответить
            • оффтоп: хотя, да. если массивы для этого дела претят, можно все же обойтись строками и без обрезания:
              1. инициализируем пустую строку
              2.1 в цикле, если строка непуста, сначала добавляем разделитель
              2.2 добавляем очередной элемент
              Ответить
              • ну это и подразумевалось под моим "хотя..." :)
                Ответить
    • А что не так с заданием имен в инпутах? Говнокодец, конечно, отменный, но использование массивов при посте формы - вполне адекватная идея)
      Ответить
    • показать все, что скрытоvanished
      Ответить

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