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

    • ava KirAmp 23.09.2011 15:45 # 0

      С array_fill тоже не знаком, но это ..здец
      Ответить

    • ava Lure Of Chaos 23.09.2011 15:59 # +1

      он со многим незнаком - с implode, например
      Ответить

      • ava jokz 23.09.2011 16:15 # 0

        а при чем здесь implode?
        Ответить

        • ava Lure Of Chaos 23.09.2011 16:32 # +1

          а то, что последний <br /> отрезали
          Ответить

          • ava jokz 23.09.2011 16:41 # −1

            ну эт да... хотя....
            Ответить

            • ava Lure Of Chaos 23.09.2011 16:45 # +3

              оффтоп: хотя, да. если массивы для этого дела претят, можно все же обойтись строками и без обрезания:
              1. инициализируем пустую строку
              2.1 в цикле, если строка непуста, сначала добавляем разделитель
              2.2 добавляем очередной элемент
              Ответить

              • ava jokz 24.09.2011 01:06 # −1

                ну это и подразумевалось под моим "хотя..." :)
                Ответить

    • ava greevex 25.09.2011 11:42 # 0

      А что не так с заданием имен в инпутах? Говнокодец, конечно, отменный, но использование массивов при посте формы - вполне адекватная идея)
      Ответить

    • ava guest8 09.04.2019 11:34 # −999

      показать все, что скрытоvanished
      Ответить

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