Удаление элементов из массива в пределах foreach

Question

В настоящее время я работаю над классом создания общей формы и вчера имел проблему. Я сделал фрагмент, чтобы воспроизвести проблему.

По существу, я хочу удалить элементы, которые сгруппированы из массива исходных элементов после того, как вся группа была нарисована, и я делаю это, перебирая массив элементов.

Фрагмент кода должен охватывать проблему, я что-то пропустил здесь? По моим сведениям, удаление элемента в то время как foreach полностью безопасно и законно, поскольку foreach внутренне использует только копию, которая может быть изменена во время цикла.

$ids = array('a' => array(), 'b' => array(), 'c' => array()); 
$groups['g1'] = array('a', 'c'); 
foreach($ids as $id => $element) { 

    //var_dump($ids); 
    $g_id = ''; 

    // search the id in all groups 
    foreach($groups as $group_id => $group) { 
     if(in_array($id, $group)) { 
      $g_id = $group_id; 
      break; 
     } 
    } 

    // element is part of a group 
    if($g_id !== '') { 

     //echo $g_id; 

     // element a and c gets unset within loop and should not be in $ids anymore 
     foreach($groups[$g_id] as $field_id) { 
      unset($ids[$field_id]); 

      echo $field_id; 
     } 
     unset($groups[$g_id]); 
    } else { 
     if($id === 'a' || $id === 'c') 
      echo $id; 
    } 
} 

элемент «с» получает сбросить в пределах Foreach (группы ..) цикл, но потом снова выводятся в другом отделении. Также, когда я сначала var_dump ($ fields), я всегда получаю 'a', 'b' и 'c' внутри. Я использую PHP 5.4.7.

Заранее спасибо

EDIT: я сделал ошибку в примере кода, его в настоящее время обновляется. Разумеется, все комментарии об использовании неправильного индекса (это было бы 0,1 и т. Д.). Значения при использовании var_dump не установлены сейчас, но я все равно попадаю в else с 'c' один раз.

EDIT2: Im не сделано с исходным кодом, но после прочтения комментариев, которые я в настоящее время придумал следующее решение отправленного фрагмента кода выше:

$ids=array("a"=>array(),"b"=>array(),"c"=>array(),"d"=>array(),"e"=>array()); 
$groups=array(array("a"),array("c", "e")); 
array_walk($groups,function($v,$i)use(&$ids){ 

    $in_both = array_intersect(array_keys($ids),$v); 
    //var_dump($in_both); 
    foreach($in_both as $b) { 
     unset($ids[$b]); 
    } 
}); 
print_r($ids); 

или

$ids=array("a"=>array(),"b"=>array(),"c"=>array(),"d"=>array(),"e"=>array()); 
$groups=array(array("a"),array("c")); 
array_walk($ids,function($v,$i)use(&$ids, $groups){ 
    $in_both = array(); 
    foreach($groups as $g) { 
     if(in_array($i,$g)) { 
      $in_both = array_intersect(array_keys($ids),$g); 
     } 
    } 

    foreach($in_both as $b) { 
     unset($ids[$b]); 
    } 
}); 
print_r($ids); 

Использование foreach не работает для меня в этом случае, потому что мне нужно изменить массив $ ids, в то время как цикл выполняет итерацию по нему.

В самом самом базовом положении код что-то вроде этого:

$ids = array('a', 'b'); 

while(count($ids)) { 
    array_pop($ids); 
    echo 'pop'; 
} 

echo 'empty'; 

Allthough Еогеасп может изменить исходные значения из массива не изменит копию массива, используемого для итерации как NL-х уже заявлено. Благодаря Passerby за идею использования array_walk для этого.

EDIT3: Обновлен код, отрезанный еще раз. Второй отрезанный allthough ведет себя также неопределенным. Удаление элементов из массива при итерации по нему кажется плохой идеей.

Answer 1

Крис, если я правильно понимаю, вы не ожидаете, 'C', которая должна быть выведена в ELSE филиал?

Но он должен быть выдан. Ваша логика такова:

• вы делаете идентификаторы и начинаете с id 'a'.
• затем вы очищаете идентификаторы a и c от идентификаторов и удаляете группу g1, содержащую 'a'. Во время этого шага будут удалены удаленные идентификаторы, являющиеся a и c. (Очищающий и с из идентификаторов не будет иметь никакого влияния на foreach($ids as $id) как Еогеасп будет продолжаться с нетронутой копией даже после того, как массив идентификаторов был очищен.)
• тогда вы ID «Ь»: не найден ни в одном группа. (фактически, пока нет какой-либо группы)
• поэтому для 'b' вы вводите ветку else. Но if() внутри ветви else предотвращает вывод
• , тогда вы делаете id 'c', который также не встречается ни в одной группе, потому что у вас есть уже удаленная группа g1! Помните, нет групп?
• поэтому для 'c' вы также вводите ветку else. И на этот раз if() внутри ветви else дает выход! Выход был только c

Таким образом, общий выход действительно соответствует.

Хорошо знать, что foreach(), который продолжается с нетронутой копией даже после того, как ее элементы были очищены, представляет собой специфическую вещь PHP. Другой язык не обязательно делает то же самое.

Answer 2

Если вы хотите удалить элемент из массива, разве вы не должны «сращивать» его с array_splice?

Из PHP инструкции: http://php.net/manual/en/function.array-splice.php

Answer 3

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

while(list($key,$value) = each($array)){ 
    if(your reason to unset) 
     unset($array[$key]); 
} 

это удалит элемент из массива.

Answer 4

$ids[$field_id] не существует, вы используете значение вместо ключа.

Вы должны просто снята с охраны с помощью правой клавиши:

if (in_array($field_id, $ids)) 
    unset($ids[array_search($field_id, $ids)]); 

Answer 5

Я перепроверки в настоящее время. Но я думаю, что отключение массива с foreach - не действительно безопасно.

То, что я обычно делал, это взять забег и начать с самых высоких индексов и уменьшить индекс по пути. for($i = count($arr)-1; $i >= 0; $i--) { unset($array[$i]); }

Я отредактировал это сообщение через несколько минут.

Редактировать: Я был в замешательстве. Претендент с $ i ++ действительно является виновником. Еогеасп безопасен (в PHP не на всех языках!)

<?php 

$arr = Array(1,2,3,4,5,6,7,8,9,10); 
foreach ($arr as $key=>$val) 
    unset($arr[$key]); 
echo implode(',',$arr); // returns nothing 


$arr = Array(1,2,3,4,5,6,7,8,9,10); 
for ($i=0; $i<count($arr); $i++) 
    unset($arr[$i]); 
echo implode(',',$arr); // returns 6,7,8,9,10 


$arr = Array(1,2,3,4,5,6,7,8,9,10); 
for ($i=count($arr)-1; $i>=0; $i--) 
    unset($arr[$i]); 
echo implode(',',$arr); // returns nothing 

?> 

Answer 6

провел некоторое время на чтение кода, и я предполагаю ваша процедура:

• Для каждого элемента в $ids, проверка если он существует в некоторой подматрице в $groups;
• Если он существует, удалите все в $ids, которое также существует в этом подматрице.

Следуя приведенной выше логике, я придумал это:

$ids=array("a","b","c","d","e"); 
$groups=array(array("a","c"),array("c","e")); 
array_walk($groups,function($v,$i)use(&$ids){ 
    $ids=array_diff($ids,$v); 
}); 
print_r($ids);//debug 

Live demo