emptyの条件分岐に関する疑問。

Question

現在、ウェブサービスの作成にあたり、
ユーザー登録の実装をしています。その関連で、

ネットで見つけたコードについて、
「自分の認識に誤りがあるのか？」「コードに誤りがあるのか？」知りたく質問があります。

該当コードは下段に記します。

質問内容は、5行目の
「if ($urltoken == '')」の部分です。

ここは必要ないのでは？と思います。
理由は、if(empty($_GET))で、「''」がfalse判定されて、
処理Aに入る、と思うからです。

以上になりますが、
どうぞよろしくお願いいたいます。

以下に掲載のあるコードの一部を転載します。※省略しています。
【PHP】メール認証を利用した会員登録機能の作成方法

if(empty($_GET)) {
//処理A
}else{
	$urltoken = isset($_GET["urltoken"]) ? $_GET["urltoken"] : NULL;
	if ($urltoken == ''){
	$errors['urltoken'] = "トークンがありません。";
　　　　　//以下、つづく

Comments

[m.ts10806]

個人記事のようですし、意図を聞きたいのであれば投稿者に確認しては。
そこだけ抜き出しても要不要は論じれません。

Answer 1

意図は投稿主に確認すべきとは思いますが、$_GETがnot emptyであることとその中の特定のキーが存在するかどうかは保証されません。

そもそも提示されたコードを見る限りあくまでurltokenの存在有無をチェックしたいのであって、GETでパラメータがなにもないときに別の処理をさせたい（デフォルトの表示など）のであれば、必要はあると言えます。
要件次第です。

Comments

[退会済みユーザー]

ご回答ありがとうございます。

ご指摘をいただき考えたのは、
「コードだけみるのか」、「全体を通して考えるのか」でした。

まだはっきりとは分かっていないと思いますが、
意図や、それに伴う各コードの役割も踏まえて、考えたほうがよいのでしょうか。

どうもありがとうございました。