どこか間違っているんでしょうか。

php
1//in model
2public function generatePath($url){
3		$path = substr(str_shuffle('1234567890abcdefghijklmnopqrstuvwxyz'),0,7);
4		$record = $this->Urlpath->findByUrlAndPath($url,$path);
5			if($record==false){
6		     $path('path');
7			}
8			return $path;
9
10	}

どこが間違っているか分からなくて、迷っています。

行動規範の内容に同意します

回答2件

ベストアンサー

エラーや、具体的に何を達成したいかが不明ですが、
ランダムな文字列を関数としている$path('path');この部分が構文違反になっていると思われます。
エラーメッセージや、どうしたいかの追記をお願いします。

投稿2015/08/19 07:18

mhashi

総合スコア408

$path('path')の部分で、自身を同じ引数で呼び出し直せばいいのです。
関数自身を呼び出しなおすことを再帰呼び出しといいます。
コメントをよみやすく整理したものを載せておきます。

PHP
1public function generatePath($url){
2    //ランダムなURLパスを生成する
3    $path = substr(str_shuffle('1234567890abcdefghijklmnopqrstuvwxyz'),0,7);
4
5    //(ドメイン＋生成されたパス)のURLが存在すればそのレコードを探す
6    $record = $this->Urlpath->findByUrlAndPath($url,$path);
7
8    //一致するデータがあるならば自身を再実行（再帰処理）
9    if($record===false){
10        //TODO
11    }
12
13    //一致するデータがないならばパスを返す
14    return $path;
15}

ちなみに、再帰呼び出しはforやwhileで置き換えられます。

PHP
1public function generatePath($url){
2    do {
3        //ランダムなURLパスを生成する
4        $path = substr(str_shuffle('1234567890abcdefghijklmnopqrstuvwxyz'),0,7);
5
6        //(ドメイン＋生成されたパス)のURLが存在すればそのレコードを探す
7        $record = $this->Urlpath->findByUrlAndPath($url,$path);
8
9        //一致するデータがあれば繰り返す
10    } while ($record === false);
11
12    //一致するデータがないならばパスを返す
13    return $path;
14}