Как в цикле php переопределить (перезаписать) исходный массив который передается в функцию в виде параметра?

Question

[tripcollor]

Я хочу создать многомерный массив меню у меня есть одномерный массив меню, который получается из БД.
В нем есть такие поля: "id ", "parent_id", "title".
На время разработки я упростил немного массив и убрал поле "url".

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

Вот отрывок кода который не отрабатывает как я хочу. Нужно чтобы $this->menu массив изменился (чтобы туда для каждого элемента добавился ключ "lavel" с определенным значением) в результате отработки функции function buildMenu() в которую он передается в качестве параметра, как это сделать?

Сейчас в принте выводится старая не измененная версия массива.

<?php
class Menu{

  protected $menu = [
    [ 'id' => 1,  'parent_id' => 0, 'title' => 'Главная' ],
    [ 'id' => 2,  'parent_id' => 0, 'title' => 'Категории' ],
    [ 'id' => 3,  'parent_id' => 2, 'title' => 'Спорт' ],
    [ 'id' => 4,  'parent_id' => 3, 'title' => 'Тенис' ],
    [ 'id' => 5,  'parent_id' => 3, 'title' => 'Баскетбол' ],
    [ 'id' => 6,  'parent_id' => 2, 'title' => 'IT' ] ,
    [ 'id' => 7,  'parent_id' => 6, 'title' => 'Пограммирование' ],
    [ 'id' => 8,  'parent_id' => 5, 'title' => 'Турниры' ],
    [ 'id' => 9,  'parent_id' => 0, 'title' => 'Товары' ],
    [ 'id' => 10, 'parent_id' => 11, 'title' => 'Компьютеры' ],
    [ 'id' => 11, 'parent_id' => 9, 'title' => 'Техника' ],
    [ 'id' => 12, 'parent_id' => 9, 'title' => 'Мячи' ],
  ];

  public function __construct(){
    $this->getMenu();
  }

  protected function getMenu(){

    $this->buildMenu($this->menu);

    echo('<pre>');
      print_r($this->menu);
    echo('</pre>');

  }

  protected function buildMenu($menu, $pid = 0, $level = 0) {
    foreach ($menu as $item) {
      if($item['parent_id'] == $pid){
        $item['level'] = $level;
        $this->buildMenu($menu, $item['id'], $level + 1);
      }
    }
  }
}

new Menu();

Напомню что мне нужно сделать именно тем методом которым я описал , я знаю что можно создать новый массив и скопировать туда все значения или

даже так

protected function buildMenu($menu, $pid = 0, $level = 0) {
    foreach ($menu as $i=>$item) {
      if($item['parent_id'] == $pid){
        $this->menu[$i]['level'] = $level;
        $this->buildMenu($menu, $item['id'], $level + 1);
      }
    }
  }

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

Comments

[Сергей Герасимов]

А зачем внутри объекта передавать ваше меню в качестве параметра? Оно и так доступно из метода...

Проще сделать foreach вашего $this->menu, а значение передавать по ссылке:

protected function buildMenu($pid = 0, $level = 0) {
foreach ($this->menu as &$item) {
if($item['parent_id'] == $pid){
$item['level'] = $level;
$this->buildMenu($menu, $item['id'], $level + 1);
}
}
}

[tripcollor]

Сергей Герасимов, я знаю и об этом методе да жаль не упомянул его. Но мне нужно именно передавать как параметр, потому что реальная задача решается рекурсивной функцией которая обязательно для работы принимает параметр который нужно изменить.
В конце я напиал же:

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

[Сергей Герасимов]

tripcollor, так если рекурсивно вызывать эту же функцию, с этим же меню (которое было дописано во время цикла), то нет никакой разницы - будет это передача в качестве пераметра по ссылке или использование через $this...

[tripcollor]

Сергей Герасимов, возможно... но я проверил ваш код и он не работает как надо
prntscr.com/jt1yis
массив остается прежним и не изменяется

[tripcollor]

Сергей Герасимов,

а вот это

<?php
class changeArray{
  protected $array = [
    [ 'title' => 'Заголовок1' ],
    [ 'title' => 'Заголовок2' ],
    [ 'title' => 'Заголовок3' ],
  ];

  public function __construct(){
    $this->start();
  }

  protected function start(){

    $this->newArray($this->array);

    echo('<pre>');
      print_r($this->array);
    echo('</pre>');

  }

  protected function newArray(&$array) {
    foreach ($array as $i=>&$item) {
      $item['new_key'] = $i;
    }
  }
}

new changeArray();

работает как надо

[Сергей Герасимов]

tripcollor, странно. Точно такой же результат: phpio.net/s/8bxe

[tripcollor]

Сергей Герасимов, Вы подшаманили мой код перед его проверкой)?
protected function newArray(&$array) {
не вижу у вас в скобках после названия функции параметр &$array
Вы меня тролите да) prntscr.com/jtscq9?

[Сергей Герасимов]

tripcollor, если внимательно посмотреть мою первую запись, то там видно что я убрал передачу меню в качестве параметра, о чем и написал.

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

[tripcollor]

Сергей Герасимов, ну так я попробовал запустить ваш код и он не работает, если передача параметра работает так как мне нужно, зачем мне использовать ваш нерабочий метод? не вижу в этом смысла

[Сергей Герасимов]

tripcollor, а почему у меня работает? Пожалуйста, используйте как вам нравится.

[tripcollor]

Сергей Герасимов, ваш код не добавляет новое значение под ключом "level" в исходный массив, я же вам отсылал скрин отработки вашего скрипта выше

[Сергей Герасимов]

tripcollor, как скажите. Спор бессмысленен. По ссылке выше я вам привел код скрипта, который добавляет, и сделан он на основании совета, предложенного в 1ом комментарии. По ссылке добавляет?

[tripcollor]

Сергей Герасимов, да по ссылке добавляет

Answer 1

[Александр Аксентьев]

Видимо так
php.net/manual/en/language.references.pass.php

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

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

Comments

[tripcollor]

Да я понимаю, я уже почти решил всю задачу и проблема именно в этом моменте уже в самом конце когда я делаю рекурсию и отрисовываю все вложенности именно эти два метода плохо годятся, а в этом фрагменте который я привел использую копирование в новый массив.
Просто код который у меня есть еще на стадии разработки и не хотелось вас утруждать разбираться в нем и для этого дал простой наглядный пример.
Спасибо у меня получилось, пришлось еще поставить "&" в цикле foreach перед перебираемым элементом

вот так

<?php
class changeArray{
  protected $array = [
    [ 'title' => 'Заголовок1' ],
    [ 'title' => 'Заголовок2' ],
    [ 'title' => 'Заголовок3' ],
  ];

  public function __construct(){
    $this->start();
  }

  protected function start(){

    $this->newArray($this->array);

    echo('<pre>');
      print_r($this->array);
    echo('</pre>');

  }

  protected function newArray(&$array) {
    foreach ($array as $i=>&$item) {
      $item['new_key'] = $i;
    }
  }
}

new changeArray();

Спасибо!