下記をもっとスマートにコーディングするとしたら、どんな風になるのでしょうか？

function verify($urlparam = null) {

$data = $this->Regurl->findAllByRandomUrl($urlparam);
if (!$data) {
$this->flash('Invalid URL!!','index');
}else{
if ($data[0]['User']['activated'] == 0) {
//print_r($this->data);
$this->User->id = $data[0]['Regurl']['user_id'];
$this->User->saveField('activated', 1);

$this->flash('Now activated!!','index');

}elseif($data[0]['User']['activated'] == 1){
$this->flash('Already activated!!','index');
}
}

}



コントローラUserから、外部モデルRegurlの中のrandom_urlフィールドと照合して、
一致するものがあればアソシエーション先のUserテーブルのactivatedフィールド（フラグ）を
1へ書きかえる処理です。

UserモデルではRegurlのuser_idに対しhasOneで、RegurlモデルではUserのidに対してbelongsToで
アソシエーションを設定してあります。