Как объединить 3 массива из файлов после цикла foreach?

Question

[Andrej Kopp]

Всем привет. Подскажите пожалуйста, каким образом можно объединить несколько массивов в один через функцию array_merge() после цикла foreach?

В переменной $config массив:

<?php

return [
    'genres' => [
        'langs' => ['ru', 'en', 'de'],
    ]
];

Записи в языковых файлах выглядят так:



Пишу такую функцию:

public function getFilesGenres() {
        $files = scandir(__DIR__.'/genres');
        $config = config('songscrud.genres.langs');

        $array = [];
        foreach($config as $lang) {
            foreach ($files as $file)
            {
                if(preg_match('/\.(php)/', $file))
                {
                    $path = __DIR__.'/genres/' . $file;
                    $ext = pathinfo($file, PATHINFO_FILENAME);

                    if ($lang === $ext) {
                        $array[$lang] = require($path);
                        $array = array_merge($array[$lang]);
                    }
                }
            }

        }

        return $array;
    }

Записываю каждый массив в массив с языком типа en, ru, de. В итоге выдаёт только массив из первого файла и ничего не объединяется. Пробовал записывать так:

$array1 = ['id' => 1, 'parent_id' => NULL, 'lft' => NULL, 'rgt' => NULL, 'depth' => 1, 'name_ru' => 'Народная музыка', 'slug_ru' => 'народная-музыка', 'translit_ru' => 'narodnaja-muzyka'];
        $array2 = ['id' => 1, 'parent_id' => NULL, 'lft' => NULL, 'rgt' => NULL, 'depth' => 1, 'name_de' => 'Ethnische Musik', 'slug_de' => 'ethnische-musik'];
        $array3 = ['id' => 1, 'parent_id' => NULL, 'lft' => NULL, 'rgt' => NULL, 'depth' => 1, 'name_en' => 'Ethnic music', 'slug_en' => 'ethnic-music'];

        return array_merge($array1, $array2, $array3);

Такой код работает хорошо и объединяет всё в один массив. Как объединять массивы динамично через файлы с массивами используя цикл?

Answer 1

[ThunderCat]

1) Вопрос изначально был хреново описан, слава яйцам ТС догадался что мы не знаем что хранится в его массивах и что надо получить в итоге... Первое правило решения задачи - хорошо описать исходные данные и требуемый результат.
2)

В переменной $config массив:

Массив состоит из 1 элемента, было бы странно ожидать что итераций foreach будет больше одной. Возможно имеет смысл проверять переменные чаще, если результат не соответствует ожиданиям, var_dump/dd($lang) внутри foreach был бы очень кстати...
3) У вас есть список языков, который соответствует названиям файлов, зачем дергать каждый раз весть список файлов, когда можно просто открыть файл __DIR__/$lang.php? Ну, разве что нужна проверка на file_exists()...
4) Вам дважды указали на ошибку с отсутствием второго аргумента в array_merge, которую вы проигнорировали...

Answer 2

[O . J]

public function getFilesGenres() {
    $files = scandir(__DIR__.'/genres');
    $config = config('songscrud.genres.langs');

    $resultArray = [];
    foreach($config as $lang) {
        foreach ($files as $file) {
            if(preg_match('/\.(php)/', $file)) {
                $path = __DIR__.'/genres/' . $file;
                $ext = pathinfo($file, PATHINFO_FILENAME);

                if ($lang === $ext) {
                    $array = require($path);
                    $resultArray = array_merge($resultArray, $array);
                }
            }
        }
    }

    return $resultArray;
}

Comments

[Andrej Kopp]

тоже самаое, что и ответ выше. Выдаёт 2228 записей, вместо 743 и количество элементов в массиве должно быть 12, а не 7.

В переменной $config массив:

<?php

return [
    'genres' => [
        'langs' => ['ru', 'en', 'de'],
    ]
];

Записи в языковых файлах выглядят так:



Может что-то не так в этих массивах?

Answer 3

[iljaGolubev]

Не совсем понятно что лежит в __DIR__/genres/ и что в итоге должно быть в $array. Но если догадаться...

Измените подход: заполняйте один массив цикл. Маска файлов в вашем коде фиксированная. так что как-то так:

public function getGenres() {
    $genres=[];
    foreach( config('songscrud.genres.langs') as $lang ){
        $filename = __DIR__ . '/genres/' . $lang . '.php' ;
        $genres[$lang] = require($filename);
    }
}