[X] Помощник

[fedornabilkin Topic Starter]

Что-то ничего подходящего не нашел в сети. Везде какие-то страшные куски кода прелдлагают.
Конкретно надо вот что.
Из массива генерирую пары (name = '$name', login = '$login',) для запроса в бд
и надо убрать последнюю запятую. В случае WHERE, надо убрать последний AND
Или как-то иначе можно сделать?

[matroskin8]

А покажите исходный массив и запрос(ы), который должен получиться в итоге.

[fedornabilkin Topic Starter]

На данный момент метод выглядит так:

function field_val($par, $where=false){
  $sep = ($where) ? "AND" : ",";
  foreach( $par as $field => $val){
   $set .= " $field = '$val' $sep";
  }


  if($where)
   return "WHERE" . substr($set, 0, -3);
  else
   return "SET" . substr($set, 0, -1);
}

Массив where array('id'=>$id, 'login'=>$login);
Запрос: UPDATE users {то, что возвратит метод ( WHERE id = '1' AND login = 'fedornabilkin')}

[matroskin8]

Да, в принципе, так можно и оставить - обрезать 3 или 1 символ в конце.
Ну еще, как вариант, можно при формировании предиката сделать проверку и в зависимости от ее выполнения добавлять AND или запятую, т.е. проверить остались ли пары в массиве и, если таковых нет больше, то и добавляться в конец ничего не будет:

foreach( $par as $field => $val){
if($set AND $field) $set .= " $sep"; // если предикат не пуст (не первая итерация) и есть очередная пара, тогда добавим AND или ,
$set .= " $field = '$val'";
}

Тогда и обрезать, вроде как, ничего не нужно будет.

[fedornabilkin Topic Starter]

А как же отследить первую итерацию? Счетчик не охота вставлять туда.

[fedornabilkin Topic Starter]

Немного подумал и решил просто добавлять сначала сепаратор, а потом удалять первое вхождение.
Так намного проще. В итоге получился довольно таки компактный и полезный метод.

function field_val($par, $sep){
  foreach( $par as $field => $val){
   $set .= " $sep $field = '$val' ";
  }
  return preg_replace('#'.$sep.'#', ' ', $set, 1);
}

Можно еще добавить третий необязательный параметр, который будет подставляться вместо =
Получится что-то типа explode, для работы с ключами и значениями.
[member=matroskin8], спасибо, направил на нужную мысль.

[matroskin8]

А как же отследить первую итерацию? Счетчик не охота вставлять туда.

А зачем нужно отслеживать первую итерацию? Она ведь и до этого, вроде как не отслеживалась. Но если все же нужна, то что мешает добавить еще одно условие - if(empty($set)){ // это и будет первой итерацией}
Кстати, в моем коде выше условие немного избыточно. Проверять есть ли очередная пара - совсем не обязательно, поскольку если ее нет, то в тело цикла мы больше и не попадем. Итого, можно так:

foreach( $par as $field => $val){
   if(empty($set)){
      // первая итерация, поскольку предикат пока пуст
   }else{
      $set .= " $sep"; // если предикат не пуст (не первая итерация), тогда добавим AND или ,
   }
   $set .= " $field = '$val'";
}

[fedornabilkin Topic Starter]

Если на то пошло, тогда двумя строками ограничиваемся

foreach( $par as $field => $val){
   if(empty($set))  $set .= " $field = '$val'";
   else $set .= " $sep  $field = '$val";
}

Вообще красота

[matroskin8]

Ну да)

Замечание модератора: Эта тема была закрыта автоматически ввиду отсутствия активности в ней на протяжении 100+ дней. Если Вы считаете ее актуальной и хотите оставить сообщение, то воспользуйтесь кнопкой или обратитесь к любому из модераторов.