Что не так с областью видимости переменной?

Question

[OstapO]

public function importFileIntoDB(Request $request){
    $data = array();
    if($request->hasFile('excel_file')){
      $path = $request->file('excel_file')->getRealPath();
      Excel::load($path, function($reader) {
        $results = $reader->get();
        foreach ($results as $item) {
          if(!empty($item->firstname) && !empty($item->lastname) && !empty($item->sex) && !empty($item->year) && !empty($item->user_group) && !empty($item->email) && !empty($item->school) && !empty($item->birthday) && !empty($item->mentor)){
            array_push($data, $item);
          };
        };
      });
      print_r($data);
      exit();
}
}

если объявить переменную $data внутри функции Excel::load, то все ок
но мне нужно переменная в глобальной области видимости

Error:
ErrorException: array_push() expects parameter 1 to be array, null given

Comments

[Дмитрий]

Не загрузился файл

[2cha.headz]

...
global $data;
$results = $reader->get();
...

[OstapO]

нет,файл загружается
если объявить переменную $data внутри функции Excel::load, то все ок
но мне нужно переменная в глобальной области видимости

[Александр Ковальчук]

Дмитрий Глаголев, оч плохой совет

[Adamos]

Александр Ковальчук, просто ТС неверно сформулировал вопрос.
Ему, конечно же, не нужна глобальная переменная.
Ему всего лишь нужно пробросить локальную в замыкание.
Например, так, как посоветовали вы ;)

[Ninazu]

Судя по public function предположу что это метод класса. А если это метод класса, то зачем глобальная переменная, есть же свойства у того же класса, и не переживаешь потом что где-то кто-то перетрёт твою глобальную переменную

Answer 1

[Moses Fender]

Еще лучше function($reader) use (&$data) {…}

Comments

[OstapO]

Благодарю!

Answer 2

[Александр Ковальчук]

Надо использовать use
function($reader) use ($data) {
подробнее в доках https://secure.php.net/manual/ru/functions.anonymo...

Comments

[OstapO]

Благодарю!