phpで複数のカンマ区切りの文字から直積を作って配列に入れたいのですが、入れ子のforeachのエラーを直したい

Question

はじめまして、ご覧になっていただきありがとうございます。

質問なんですが、仕事のことでカンマ区切りの言葉それぞれの
全組み合わせをphpでもとめたいのですが、

基本はできたのですがどうしても最後の方のforeachの入れ子で
エラーが出てしまいます。

foreachにわたす変数が配列じゃないと出てくるのですが
配列を入れてるつもりなのでお手上げになってしまいました。

どなたかわかる方がいらっしゃればよろしくお願いします。

結果としてこんなカンジに出力できたらと思ってます。

http://qiita.com/mpyw/items/e0b6e0842a460b3f901f#1-2

	$errorm = 's-non-used';


	//キーワードの直積の一覧を作成
	//全変数を宣言

	$k['a'] = array(

		'product' => 'pc,phone,heater',

		'event' => '',

		'star' => '',

		'number' => '',

		'trouble' => '',

		'real' => '',

		'desire' => '',

		'price' => '5800,4200,3400,980,1980,580',

		'labor' => '今売れてます,安い,配送無料,無料保証,サービス第一,安全第一,他社よりすごい,今すぐお値引き,係員にお申し付けください',

	);


	//各変数にエラーフラグを格納

	foreach ($k['a'] as $key => $value){

		if($value === ''){

			$value = $errorm;

//本来はelseでGETから変数を取得しています。

		}
	}


	//各変数をカンマで切り分けて配列にする
	foreach ($k['a'] as $value){

		if($value === $errorm){

			unset($value);

		} else {


			$value = explode( ',', preg_replace ("/,+/",',',$value));

		}

	}

//指摘により追加しました
$words = $k;
	$k = '';

	//全nameに対しての繰り返し処理ex)product,price,star...
	foreach ($words['a'] as $key => $value){

		//受け入れ配列の要素数を入力配列の要素数分増やす

		if($k === ''){

			$k = $value;

		} else {

			$t = 0;


//ここから下でエラーが出てます。

			//新しく入れる分重複要素を増やす
			foreach($k as $value2){

				foreach($value as $i){

					$depot[] = $value2;

				}

			}

			//増やした重複要素に新しく入れる分を全パターン入れる
			foreach($k as $i){

				foreach ($value as $value2){

					$depot["$t"]["$key"] = $value2;

					++$t;

				}

			}

			$k = $depot;

			unset($depot);

		}

	}

Answer 1

お二方のソースを実行しましたが直積にはなりませんでしたが、
とても参考になる意見をありがとうございました。

自己解決できましたので、ソースをそのまま貼っつけときます。
直積を作りたい複数の配列を一つの配列に2次元配列としてまとめて渡すと
直積の全パターンを2次元配列に格納して出力します。

ミスをしていたのはイテレータのvalueに値を入れても元の配列に反映されないということでした。

	$errorm = 's-non-used';

	$i = 'a';



	//検索ボックスから値取得

	//全変数を宣言

	$k['a'] = array(

		'product' => 'pc,phone,,heater',

		'event' => '',

		'star' => '',

		'number' => '',

		'trouble' => '',

		'real' => '',

		'desire' => '',

		'price' => '5800,,,,,4200,3400,980,1980,580',

		'labor' => '今売れてます,安い,配送無料,無料保証,サービス第一,安全第一,他社よりすごい,今すぐお値引き,係員にお申し付けください',

	);


	//各変数に値を格納

	foreach ($k['a'] as $key => $value){

		if($value === ''){

			$k['a']["$key"] = $errorm;

		}
	}

	//priceとlaborをもう１つの変数にまとめる

	if($k['a']['price'] === $errorm and $k['a']['labor'] === $errorm){

		$k['a']['cost'] = $errorm;

	} else {

		$k['a']['cost'] = str_replace ($errorm , '' ,$k['a']['price'] . ',' .$k['a']['labor']);

	}




	//各変数をカンマで切り分けて配列にする
	foreach ($k['a'] as $key => $value){

		if($value === $errorm){

			unset($k['a']["$key"]);

		} else {

			$k['a']["$key"] = explode( ',' , preg_replace ("/,+/",',',$value));

		}

	}



	//キーワードの直積の一覧を作成
	$words = $k;

	$k = array();

	//全nameに対しての繰り返し処理ex)product,price,star...
	foreach ($words['a'] as $key => $value){

		//受け入れ配列の要素数を入力配列の要素数分増やす

		$t = 0;

		if(empty($k) === TRUE){

			foreach ($value as $value2){

				$k["$t"]["$key"] = $value2;

				++$t;

			}

		} else {

			//新しく入れる分重複要素を増やす
			foreach($k as $value2){

				foreach($value as $i){

					$depot[] = $value2;

				}

			}

			//増やした重複要素に新しく入れる分を全パターン入れる
			foreach($k as $i){

				foreach ($value as $value2){

					$depot["$t"]["$key"] = $value2;

					++$t;

				}

			}

			$k = $depot;

			unset($depot);

		}

	}


print_r($k);

Answer 2

お手数ですが、
$words の設定値を教えていただけないでしょうか？
foreachの参照元に $words['a']が設定されていますが、掲載されているソースには、$wordsが宣言されていないようです。

ソース確認いたしました。
3か所に間違いと思われる個所がございました。（コメントをご参照ください）
されたいことがあっているかどうか判断しかねますが、文法エラーはなくなりました。

<?php
$errorm = 's-non-used';


    //キーワードの直積の一覧を作成
    //全変数を宣言

 $k['a'] = array
 (

        'product' => 'pc,phone,heater',
        'event' => '',
        'star' => '',
        'number' => '',
        'trouble' => '',
        'real' => '',
        'desire' => '',
        'price' => '5800,4200,3400,980,1980,580',
        'labor' => '今売れてます,安い,配送無料,無料保証,サービス第一,安全第一,他社よりすごい,今すぐお値引き,係員にお申し付けください',

    );

    //各変数にエラーフラグを格納
    foreach ($k['a'] as $key => $value)
    {
        if($value === '')
        {
            $value = $errorm;

//本来はelseでGETから変数を取得しています。
        }
    }

    //各変数をカンマで切り分けて配列にする
    foreach ($k['a'] as $key => $value)
    {
        if($value === $errorm)
        {
            unset($value);
        } 
        else 
        {
//修正点①
//ここでカンマ区切りのデータを配列化していますが、$kに反映されていませんでした。
            $k['a'][$key] = explode( ',', preg_replace ("/,+/",',',$value));
        }
    }

	$words = $k;
    $k = '';

    //全nameに対しての繰り返し処理ex)product,price,star...
    foreach ($words['a'] as $key => $value)
    {

        //受け入れ配列の要素数を入力配列の要素数分増やす
        if($k === '')
        {
            $k = $value;
        } 

//修正点２
// 上記if 文のelse時のみ実行されますが、$words配列１個目が必ず処理されていません　1件目の"product"が処理されていませんでした。

		$t = 0;
//ここから下でエラーが出てます。

		//新しく入れる分重複要素を増やす
		foreach($k as $value2)
		{
			foreach($value as $i)
			{
				$depot[] = $value2;
			}
		}

		//増やした重複要素に新しく入れる分を全パターン入れる
		foreach($k as $i)
		{
			foreach ($value as $value2)
			{
// 修正点３
// $depot への代入文法に間違いがございました。（2次配列の宣言後に代入）

				$depot[$t] = array($key=>$value2);
				++$t;
			}
		}

		$k = $depot;
		unset($depot);
    }
?>

Comments

[kooo]

訂正ありがとうございます！
$wordsの設定値ですが、$words = $k;
で$kの一時置き場としてそのままの値を使ってます。

修正点2についてなのですが、最初$kが空なので
//新しく入れる分重複要素を増やす
のイテレータがエラーになるかと思って分岐させてます。

"product"を空の$kに格納できればと思うのですが、
もしわかればよろしくお願いします。

Answer 3

こんにちは。

$k = ''
なので、配列じゃないというエラーが出るんじゃないですか？

$k = array();
にしてみたらどうでしょう？
あと記載されたソースを見る限り、$wordsも宣言されていないので同じエラーが出そう。別のところで宣言されているなら問題ありませんが。

Comments

[kooo]

$wordsが抜けてました、ありがとうございます。

foreach内のif文の中で配列を代入してるから大丈夫かと思ったのですができなかったのですね。。
いま出先なので家に帰ったら修正してみます！
ありがとうございます！