foreachの中で生成した配列を結合させたい。

Question

foreachの中で生成した配列を結合させたいのですが、
以下のようにすると上書きされてしまい、最後の配列のみが残ります。

###試したソースコード

$r=0;
$lines_array = array();
	foreach ($access as $access_val) {
		$line_list_array[$r] = line_list($access_val); //line_listはユーザー定義関数で配列を作っています。
		$lines_array += $line_list_array[$r][0];
		$r++;
}

print_r($lines_array);// Array ( [0] => line99303 [1] => 9930221 );

下記のような配列が入っているとした場合、
（実際には$access_valの値や数によって変動します。）
$line_list_array[0][0] = Array ( [0] => line99301 [1] => 9930118 );
$line_list_array[1][0] = Array ( [0] => line99302 [1] => 9930220 );
$line_list_array[2][0] = Array ( [0] => line99303 [1] => 9930221 );

下記のような結果を期待しているのですが思ったようになりませんでした。
$line_list_array = Array ( [0] => line99301 [1] => 9930118 [2] => line99302 [3] => 9930220 [4] => line99303 [5] => 9930221 );

恐れ入りますが、アドバイス頂ければ幸いです。
よろしくお願いいたします。

Answer 1

素直に書くならこうですかね。

$r=0;
$lines_array = array();
foreach ($access as $access_val) {
    $line_list_array[$r] = line_list($access_val); 
    foreach ($line_list_array[$r][0] as $line) {
        $lines_array[] = $line;
    }
    $r++;
}

Comments

[igaiga]

ありがとうございます！

Answer 2

// $r=0;
$lines_array = array();
foreach ($access as $access_val) {
    // $line_list_array[$r] = line_list($access_val); //line_listはユーザー定義関数で配列を作っています。
    // $lines_array += $line_list_array[$r][0];
    // $r++;

    $lines_array = array_merge($lines_array, line_list($access_val)[0]);
}

実行結果

$access = array(0, 1, 2);

function line_list($access_val) {
    $data = array(
        array('line99301', '9930118'),
        array('line99302', '9930220'),
        array('line99303', '9930221'),
    );
    return array($data[$access_val]);
}

/* ------------------------------------------------------- */

// $r=0;
$lines_array = array();
foreach ($access as $access_val) {
    // $line_list_array[$r] = line_list($access_val); //line_listはユーザー定義関数で配列を作っています。
    // $lines_array += $line_list_array[$r][0];
    // $r++;

    $lines_array = array_merge($lines_array, line_list($access_val)[0]);
}

print_r($lines_array);

Array
(
    [0] => line99301
    [1] => 9930118
    [2] => line99302
    [3] => 9930220
    [4] => line99303
    [5] => 9930221
)

Comments

[igaiga]

ありがとうございます！この方法でもイメージ通りできました。

[igaiga]

1行で済むのでベストアンサーにさせていただきました。

Answer 3

    foreach ($access as $access_val) {
        $line_list_array[$r] = line_list($access_val); //line_listはユーザー定義関数で配列を作っています。
        $lines_array[] = $line_list_array[$r][0];
        $r++;

じゃないでしょうか。

---

改修

    foreach ($access as $access_val) {
        $line_list_array[$r] = line_list($access_val); //line_listはユーザー定義関数で配列を作っています。
        foreach($line_list_array[$r][0] as $data) {
            $lines_array[] = $data;
        }
        $r++;

ですかね、もとの意図に沿うならば。

Comments

[igaiga]

ありがとうございます。試してみたところ、こんな感じになりました；；
Array ( [0] => Array ( [0] => line99301 [1] => 9930118 ) [1] => Array ( [0] => line28006 [1] => 2800621 ) [2] => Array ( [0] => line28003 [1] => 2800312 ) )

[igaiga]

ありがとうございます！
理解が追いつきませんが、とりあえずイメージ通りにできました！

Answer 4

こんな感じでしょうか。

$r=0;
$lines_array = array();
foreach ($access as $access_val) {
    $line_list_array[$r] = line_list($access_val);
    foreach(array_values($line_list_array[$r]) as $v){
        $lines_array[] = $v;
    }
    $r++;
}

print_r($lines_array);

Comments

[ttyp03]

マイナス評価を付けた方、どこがマイナス点なのか教えてください。

[Y.H.]

array_values($line_list_array[$r][0])かと。
$line_list_array[$r] は大外にもう一段arrayがあるようです。
あと
$line_list_array[$r][0] は$lines_listにmargeしたいとの事ですが
$line_list_array[$r][1]以降を考慮すると全てforeachで回るのはどうかと。

[ttyp03]

$line_list_array[$r] = line_list($access_val);
これで代入しているので、一次元配列なんじゃないでしょうか？
二次元であるなら、こう書かれているべきですし。
$line_list_array[$r][0] = line_list($access_val);

[Y.H.]

line_list(){
return array(array('foo','bar'),array('hoge,'fuga'));
}
な感じで。
tacsheavenさんの回答のコメントで
$lines_array[] = $line_list_array[$r][0];
とすると
print_r($lines_array)でこうなるとのことなので
Array ( [0] => Array ( [0] => line99301 [1] => 9930118 ) [1] => Array ( [0] => line28006 [1] => 2800621 ) [2] => Array ( [0] => line28003 [1] => 2800312 ) )

[ttyp03]

line_list()の戻りがArray ( [0] => line99301 [1] => 9930118 )なのだから、全体を考慮して回す必要があると思うのですが。

これで代入したら、
$line_list_array[$r] = line_list($access_val);
こうなりますよね。
$line_list_array[$r][0] = line99301
$line_list_array[$r][1] = 9930118

なので$line_list_array[$r][0] だけを対象にしてはダメだと思うのですが…。

[Y.H.]

こうなりませんか？

$ cat -n ddd.php
1 <?php
5 function line_list($foo){
6 if ($foo == "dummy1") {
7 return array(array('line99301', '9930118'), array('foo', 'bar'));
8 }
9 return array(array($foo.'line99301', $foo.'9930118'), array($foo.'foo', $foo.'bar'));
10 }
11
12 $line_list_array = array();
13
14 $r=0;
15 $lines_array = array();
16 $access=array('dummy1', 'dummy2');
17 foreach ($access as $access_val) {
18 $line_list_array[$r] = line_list($access_val);
19 foreach(array_values($line_list_array[$r]) as $v){
20 $lines_array[] = $v;
21 }
22 $r++;
23 }
24 print_r($lines_array);// Array ( [0] => line99303 [1] => 9930221 );
25

$ php ./ddd.php
Array
(
[0] => Array
(
[0] => line99301
[1] => 9930118
)

[1] => Array
(
[0] => foo
[1] => bar
)

[2] => Array
(
[0] => dummy2line99301
[1] => dummy29930118
)

[3] => Array
(
[0] => dummy2foo
[1] => dummy2bar
)
)

[Y.H.]

$line_list_array[$r][0] だけを対象にすると
こうなります。

$ php ./ddd.php
Array
(
[0] => line99301
[1] => 9930118
[2] => dummy2line99301
[3] => dummy29930118
)

[ttyp03]

line_listの戻りが違くないですか？
こうですよね？
return array('line99301', '9930118');
ここの考え方が違っているので、お互いずれているんですよ。
tacsheavenさんのコードおよびコメント欄から推測すると、line_listの一回の戻り値はこれですよね。
Array ( [0] => line99301 [1] => 9930118 )
であるなら、
return array(array('line99301', '9930118'), array('foo', 'bar'));
これは違いますよね。
元々曖昧なコードを基に回答しているので、もし私の勘違いでしたらマイナス評価は受け入れるしかありません。仕方なし。

[Y.H.]

質問では
$r=0;
foreach ($access as $access_val) {
$line_list_array[$r] = line_list($access_val);
$r++;
}
の結果以下になるとの事なので
$line_list_array[0][0] = Array ( [0] => line99301 [1] => 9930118 );
$line_list_array[1][0] = Array ( [0] => line99302 [1] => 9930220 );
$line_list_array[2][0] = Array ( [0] => line99303 [1] => 9930221 );

以下になるならおっしゃるとおりだと思います。
$line_list_array[0] = Array ( [0] => line99301 [1] => 9930118 );
$line_list_array[1] = Array ( [0] => line99302 [1] => 9930220 );
$line_list_array[2] = Array ( [0] => line99303 [1] => 9930221 );

[igaiga]

ttyp03さん、Y.H.さんありがとうございます。理解できました。