Как изменить массив на лету внутри цикла foreach?

Question

[alex0176]

Есть массив объектов термов, который перебираться с помощью foreach:

foreach( $terms as $term ) :
    // code...
    $hierarchy_terms = hierarchyTerms( $term->term_id, $tax, $terms );
    // code...
endforeach;

Внутри тела foreach есть функция hierarchyTerms(), которая обрабатывает элементы массива и затем изменяет сам массив (удаляет элемент массива):

function  hierarchyTerms( $termID, $taxonomy, &$terms_array )
{
     $output = '';
     $term_children = get_term_children( $termID, $taxonomy );
     if( ! is_wp_error( $term_children ) && ! empty( $term_children ) ) {
         $output .= '<div class="hierarchy-terms">';
         $output .= '<ul>';

         foreach ( $term_children as $child_id ) {
            $output .= '<li>'. get_term( $child_id )->name.'</li>';

            foreach ( $terms_array as $key => $val ) {
                if( $val->term_id === $child_id ) {						
			unset( $terms_array[$key] );						
		}
            }
         }
          $output .= '</ul>';
          $output .= '</div>';
     }

      return $output;
}

Здесь при вызове функции hierarchyTerms( $term->term_id, $tax, $terms ) в цикле foreach третий параметр $tax, $terms передаться по ссылке, что обеспечит изменение массива на лету. Но, по каким то причинам, этот массив $terms не изменяется и цикл foreach пробегается по всем ранее существующим элементам и не учитывает того, что были удалены в процессе перебора определенные элементы массива.

Comments

[Dmitry Bay]

У вас в основном массиве - termID - уникальный?

[maksam07]

alex0176, если помогло, напишите как ответ, чтобы вопрос закрылся

[alex0176]

Решение было в добавлении амперсанда (&) к параметрам foreach

foreach( $terms as &$term ) :
    // code...
    $hierarchy_terms = hierarchyTerms( $term->term_id, $tax, $terms );
    // code...
endforeach;

foreach ( $terms_array as $key => &$val ) {
   // Code...
}

[Михаил Ливач]

alex0176, это очень странно, потому что в PHP уже довольно давно объекты передаются по ссылке внутренними механиками. Какая у Вас версия языка?
Заодно хочу отметить, что Ваш код нарушает SRP: он что-то собирает для вывода, и попутно модифицирует $terms_array, это нехорошо.

[alex0176]

К сожалению данная реализация не решила моих задач и я отказался от передачи по ссылках... сейчас застрял еще с одной задачей - вывод иерархической структуры термов таксономии. вот ссылка на пост

[Ипатьев]

Михаил Ливач, объекты же, а не массивы.
Ну и говорить про SRP разработчику под вордпресс - ну, такое. Есть старый анекдот, "это наша родина, сынок". И еще "Ты когда в страну въезжал, большой красный флаг видел"?

[Михаил Ливач]

Ипатьев, так Вы и посмотрите: везде, где автор добавил & - объекты . У него единственная ссылка на массив есть в постановке проблемы..

[Михаил Ливач]

alex0176,

К сожалению данная реализация не решила моих задач

Как-то странно у Вас получилось: сначала Вы вынесли это в "решение", а потом пишете, что это не решение. Картинка не сходится

[alex0176]

Все заработало, спасибо всем за помощь.

Answer 1

[samyside]

foreach ( $terms_array as $key => $val ) {
                if( $val->term_id === $child_id ) {						
			unset( $terms_array[$key] );						
		}
            }

Надо еще здесь обращаться по ссылке к элементам массива, иначе они не изменяются:

foreach ( $terms_array as $key => &$val ) {