自作関数の戻り値について - PHPプロ！Q&A掲示板

2828

0P

自作関数の戻り値について

質問日時 / 2010年7月30日 15:43 (最終編集：7月30日 15:43) 回答数 / 4件

Questioner: jun0223xp Tweet

フラグを判断して、例えばフラグが『1』の場合は処理を続行、『0』の場合はトップページにリダイレクト、のような自作関数を作成するとします。

その場合、以下の[例]1と[例2]の書き方ではどちらが正しいのか教えてください。
よろしくお願いします。

[例1]では、関数の中で直接リダイレクトしています。
[例2]では、関数の戻り値としてはtrueかfalseしか返さず、その後にif文の分岐でリダイレクトさせています。
自分の考えでは[例2]の方が正しいような気がするのですがどうでしょうか？

[例1]

<?php
// 関数1の定義
function example1( $int ) {
if( $int == 1 ) {
return true;
} else {
header( 'Location: http://www.example.com/' );
exit;
}
}
// フラグの設定
$flg = 1;
// 関数1の呼び出し
example1( $flg );
?>

[例2]

<?php
// 関数2の定義
function example2( $int ) {
if( $int == 1 ) {
return true;
} else {
return false;
}
}
// フラグの設定
$flg = 1;
// 関数2の呼び出し
if( example2( $flg ) === false ) {
header( 'Location: http://www.example.com/' );
exit;
}
?>

この質問への意見の募集は締め切られ、ポイントは既に配分されました。
意見を投稿することはできますが、ポイントを受け取ることはできません。

この質問に返信

ツリー一覧

自作関数の戻り値についてjun0223xp

A01magicflute2この意見は投稿者によって削除されました。

A02pannna作り手のポリシーによる部分が大きいものをどちらが…

A03shimix自分で書くのであれば、どちらでもお好きな方でとは思

A03-1pannna今回みたいな判定してあるURLにリダイレクトするので

A03-1-1jun0223xp返信が遅くなり申し訳ありません。結局は作り手次

回答一覧

A01
answerermagicflute2 [7月30日 19:42]

この意見は投稿者によって削除されました。

A02
answererpannna [8月3日 03:03]

作り手のポリシーによる部分が大きいものをどちらが…と聞くのはちょっと？
まあ戻り値を必ず返すのが関数だ、という考えであれば、
判定のみを関数で行って判定結果によって呼び出し元でやるでしょうしね

A03
answerershimix [8月3日 13:25]

自分で書くのであれば、どちらでもお好きな方でとは思いますが・・・

私は、他人のソースを読んでいて(1)のような記述を見つけたら「要注意関数」としてリストアップします。呼び出し元からは推測できない動作ですから。なのでそんな要注意関数がたくさんあると匙を投げたくなります。

＃もちろん「関数名から容易に想像できる動作」であれば別ですが、往々にして違うんですよね・・

今自分が書いているソースも数年（いや数ヶ月）後の自分が読めば他人のソースと同じ（いや、もっと読みにくいかも）になることは念頭に置いといてください。

A03-1
replyerpannna [8月4日 01:33]

今回みたいな判定してあるURLにリダイレクトするのであれば、
リダイレクト先のURLを引数に指定して汎用性のある関数にするのが一番いいかと思いますね
(もちろんURLは定数としてiniファイルみたいなものに書いておきますが)。

>今自分が書いているソースも数年（いや数ヶ月）後の自分が読めば他人のソースと同じ（いや、もっと読みにくいかも）になることは念頭に置いといてください。

これはプログラムを書く上で永遠の課題なんですよね…

A03-1-1
replyerjun0223xp [8月11日 00:44]