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

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

      PS
      ООП конечно решает, но вот это $f->QuoteSql() я бы назвал как-нибудь покороче, ибо ленив
      Ответить
      • нужно было получить и имена ключей, и их значения. мозг кипел судя по всему
        Ответить
    • изврат
      Ответить

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