CakePHPの複雑なconditionsの書き方について

Question

CakePHPのfindメソッドのconditionsについての質問です。

viewファイルにcalculation.ctpを作っておき、formからデータを送信するようにしておきます。
（現在は練習、テスト段階なのでそこは詳しく触れません）
コントローラーで目標とする動きは、データを何らかの値で受け取ったら、ランダム変数$randamに1、もしくは2が入ります。そして、配列$arrayから$randamの数だけランダムにキーを取ってきます。
もし、$randamが1なら$randkeyは変数、2なら配列なのでなのでif文で分けます。
それぞれ、findメソッドのconditionsを作る作業をしています。

ここで$randamが1なら問題ないのですが、2の場合が上手くいきません。
area_idがランダムに選んだ二つのうちのどちらかを含んでいるものを取り出したいのですが、Warning (2): Illegal offset type と出てしまい動作してくれないのです。
連想配列のキーに「配列」を指定したら発生するというものらしいのですが、$array[$randkey[0]]はintegerなので関係ないはずです。実際に変数でなく直接１など数値を入力してみましたが、同じエラーが出ました。
その後、OR,ANDの書き方も調べて試して下のように書いてみたのですが、同様の結果となってしまいます。
Queryが WHERE MySampleData.area_id IS NULLとなってしまっているようです。
(これが１と２、とか、２と４になってほしいのですが)

複雑なconditionsの書き方はどうすればいいのでしょうか？
cookbookも見て色々試したのですが、解決しません。よろしくお願いします。

<?PHP
  function calculation(){
    if (isset($this->data['id'])){
		$randam = rand(1, 2);
		$array = [1, 2, 3, 4];
		$randkey = array_rand($array, $randam);
	  if($randam = 1){
		  $status = array(
			'conditions' => 
            array('MySampleData.area_id' => $array[$randkey]),
		  );
	  }else{
		  $status = array(
			"conditions" => 
            array('OR' => array(
                    'MySampleData.area_id' => $array[$randkey[0]],
                    'MySampleData.area_id' => $array[$randkey[1]],
                ))
		  );
	  };
      $data = $this->MySampleData->find('all'/*'first'*/, $status);
    } else {
      $data = null;
    }
    $this->set('data', $data);
  }

Answer 1

提示いただいコードだと、$randamの値に関わらずすべて1の方を通ってしまいます

 if($randam = 1){ // これだとすべてthenと判断されてしまう

 if($randam == 1){ // これで$randamが1またはそれ以外の判断となる

 if(1 == $randam){ // ちなみにこのように書く癖をつけておくと、==を誤って=としたときにエラーとなり見つけやすい

また、miyabi-sunさんが記述しているように、ORで同じキー値を使用すると上書きされてしまので、IN句で検索するように修正する必要があります

Comments

[miyabi-sun]

これが回答ですわ…
$randkeyが配列のまま1に入ってエラーになってるわけですね。

[tacchan]

ありがとうございました。確かに＝一つではだめですよね。ふと頭からそのことが消えていました。ワンポイントアドバイスもしていただき、ありがとうございました。

Answer 2

Warningの原因は突き止めてないのですが、
CakePHPのconditionsの書き方に関してです。

CakePHPのコンディションとしてSQL文に展開されるまではただの配列なので、
上の例では1つの配列に同じキーを2つ宣言してしまっています。

          $status = array(
            "conditions" => 
            array('OR' => array(
                    'MySampleData.area_id' => $array[$randkey[0]],
                    'MySampleData.area_id' => $array[$randkey[1]], // ←キーが重複
                ))
          );

下記のように記述すると、
CakePHPはIN演算子を使ってよしなにやってくれるようになります

         $status = array(
            "conditions" => array(
              'MySampleData.area_id' => [$array[$randkey[0]], $array[$randkey[1]]],
            ),
          );

// SELECT * FROM MySampleData WHERE MySampleData.area_id IN ($array[$randkey[0]], $array[$randkey[1]) となるイメージ

Comments

[tacchan]

ありがとうごさいました。期待通りの動きになりました。Cakeのエラーで慌てていて、キーの重複を見落とすなんて恥ずかしいです。最初は[]でくくっているところをarray()でくくっていましたが、駄目でした。[]とarray()でも違いがあるんですね。

[miyabi-sun]

JavaScriptでは厳密には違いがあったかと思いますが、
PHPでは[]とarray()には違いは全くありませんよ。
PHP5.4以降、めんどくさいarray()を[]でも書けるようになっただけの話です。

もうPHP5.3のサポートも終了していますし、
今後は基本的に[]で統一することをオススメします。