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

    +161.5

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    $keys = array_keys($_POST);
$vals = array_values($_POST);
for($i=0; $i<=count($keys); $i++)
{
    if($keys[$i]!='act' && $keys[$i]!='mode' && $keys[$i]!='pid')
    {
            $sql->Query('UPDATE `user_services_options` SET `value`=\''.$f->QuoteSql($vals[$i]).'\' WHERE `option`=\''.$f->QuoteSql($keys[$i]).'\' AND `service_id`='.$f->QuoteSql($service_id).' AND user_id='.$f->QuoteSql($user_id));
    }
}

    кагбе про foreach($_POST as $key => $value) я тогда и не знал.

    Запостил: nolka4, 18 Декабря 2009

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

    • ava Yozheg 18.12.2009 14:12 # 0

      Ну ладно, про foreach не знал, но зачем создавать массив значений? Достаточно итерировать по массиву ключей
      $len = count($keys);
      for($i=0; $i < $len; $i++)
      {
      $key = $keys[$i];
      $val = $_POST[$key];
      ...
      }

      PS
      ООП конечно решает, но вот это $f->QuoteSql() я бы назвал как-нибудь покороче, ибо ленив
      Ответить

      • ava nolka4 18.12.2009 14:14 # 0

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

    • ava paranoid 18.12.2009 14:31 # 0

      изврат
      Ответить

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