cakephp ソースコード

Question

		if ( $u_out == "history" ){
			$conditions=array();
			if ( $u_userid != "" ){
				$conditions=array('username' => $u_userid);
			}
			$ips=CommonComponent::GetAllowIPsByAccessIp();
			if(empty($ips)){
				$querys = $this->Runhists->find('all', array(
					'conditions' => array(
				 		'Runhists.username' => $conditions,
						'Runhists.private_flg' => 0),
					'order' => 'created desc', 'limit'=>100));
			} else {
				$querys=$this->Runhists->find('all', array( 'conditions' => $conditions, 'order' => 'created desc','limit'=>100));
			}

これを綺麗にまとめたく思い
find()を一度にだけにしたく
条件を$conditionsで条件分岐しようと思うのですが一度使用してしまっています。
どうしたらうまく綺麗にまとめられますでしょうか？

Answer 1

単に連想配列に入れていくだけで大丈夫ですよ

<?php
$conditions=array();
if ( $u_userid != "" ){
    $conditions['Runhists.username'] = $u_userid;
}
$ips=CommonComponent::GetAllowIPsByAccessIp();
if(empty($ips)){
    $conditions['Runhists.private_flg'] = 0;
}
$querys=$this->Runhists->find('all', array(
	'conditions' => $conditions, 'order' => 'created desc','limit'=>100
));

Answer 2

usernameでif文でconditionを変えているので、それをそのまま$ipsに適用して、それぞれの条件で先に$conditionsを作ってしまえばいいのでは。

$conditions = array();
if ( $u_userid != "" ){
   $conditions = array_merge($conditions, array('Runhists.username' => array('username' => $u_userid)));
}
if ( empty($ips) ){
   $conditions = array_merge($conditions, array('Runhists.private_flg' => 0));
}
$querys=$this->Runhists->find('all', array( 'conditions' => $conditions, 'order' => 'created desc','limit'=>100));

こんな感じで。