array_push() expects parameter 1 to be array, null givenというエラーが出てしまう

Question

PHPでは問題なく動くコードなのですが、CakePHPで実行しようとするとarray_rand() expects parameter 1 to be array, null givenとarray_push() expects parameter 1 to be array, null givenの似たような２つのエラーが出てしまいます。
単純ですが、ランダムで$spotsから１つ要素を取り出し、$choiced_spotに追加するというものです。
何が原因でどこをどう直せばいいのでしょうか？
また、calculation以外でも使うために、外に出して関数として扱いたいとも考えていますが、どうすればいいのでしょうか？
ただ出すだけだと「viewがない」とCakePHPに怒られます。関数名の最初にアンダーバーを付ければよいというものをみて試しましたが、上手くいきませんでした。
よろしくお願いします。

<?php
App::uses('AppController', 'Controller');
 
class MySampleDatasController extends AppController {
   function calculation(){
       $choiced_spot = [];
       $spots =[1, 2, 3, 4, 5, 6, 7];
       function choiced_spot($spots, $choiced_spot){
           global $spots, $choiced_spot;
           $randkey = array_rand($spots, 1);
           array_push($choiced_spot, $spots[$randkey]);
       };
	
       choiced_spot($spots, $choiced_spot);
   }
}

Answer 1

原因は、CakePHPでの関数やグローバル変数の使い方がおかしいところです。
CakePHPの設定や必要なファイルについての説明は省きますが、以下のようにできると思います。

public function index() {
	$k = 5; // 処理の回数
	$choiced_spot = $this->_calculation($k); // 関数呼び出し
	Debugger::dump($choiced_spot); // $choiced_spotの中身確認
}

public function _calculation($k) {
	$choiced_spot = [];
	$spots = range(1,7);
	for ($i=0; $i<$k; $i++) {
		$choiced_spot[] = $spots[array_rand($spots, 1)];
	}
	return $choiced_spot;
}

// 実行結果の一例
array(
	(int) 0 => (int) 5,
	(int) 1 => (int) 1,
	(int) 2 => (int) 4,
	(int) 3 => (int) 6,
	(int) 4 => (int) 3
)

ちなみにコントローラーの関数名の先頭にアンダースコアを付けると、URLで指定してのアクセスができなくなります。

Comments

[tacchan]

本当にありがとうございます。書いていただいたコードを参考にやり直したら、きちんと動くようになりました。丁寧に書いてくださり、とても勉強になりました。

Answer 2

array_rand() expects parameter 1 to be array, null given

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

書いてある通りです。
1番目の引数は配列であることが期待されていますが、nullが与えられました。

$randkey = array_rand($spots, 1);

$spots は配列じゃないといけないのに、null になっとるでー！！！という意味。

array_push($choiced_spot, $spots[$randkey]);

$choiced_spot は配列じゃないといけないのに、null になっとるでー！！！という意味。

---

array_push

リファレンスの読み方覚えましょう。

Comments

[tacchan]

そうですね。意味はわかります。リファレンスの読み方、array、それくらいは読めますね。
質問に書いた通り、PHPにおいて同じfunctionを書いて試したら、問題なく動いたんです。
$spotsは配列ですよね？
$spots = [1, 2, 3, 4, 5, 6, 7]なので。
$choiced_spotも空の配列です。$choiced_spotに関してはarray_pushですから空でも問題ないですよね？
なぜ、CakePHPではエラーが発生してしまうのかを知りたいのです。
ただのPHPとして書いたコードはfunction calculation()の内部です。var_dumpで$choiced_spotを表示したら問題なく出ましたよ。
エラーの意味を訳してくれるだけでは問題の実質の解決にはなりません。
なぜ、PHPでは作動してCakeだとエラーが出てしまうのかをお願いします。

[退会済みユーザー]

> PHPでは作動してCakeだとエラーが出てしまうのかをお願いします。
なぜCakeの時にnullになるのか、地道にデバッグしましょう。

それに、オブジジェクト指向を目指すフレームワークを利用していながら変数をグローバルで宣言するというのはオブジェクト指向の思想そのものを反故にしてしまうもの。
global にしているがために、予期しない値が入ってきていると考える方が自然では？