PHPがおかしい

質問

$GET['type']が"login"でも"tutorial_hsp"と同じ挙動をしてしまいます。
部分のスクリプトはこんな感じです。

if(!isset($_GET['type'])){
echo "<h2>14ちゃんねるデベロッパーメニュー</h2>";
echo "<p><a href=\"menu.php?type=tutorial_hsp\">14ちゃんねるの扱い方<a href=\"https://hsp.tv\">(HSP)</a></a></p>";
echo "<p><a href=\"https://ux.getuploader.com/14channel/\">作ったツールのアップロード(別サイトに飛びます)</a></p>";
echo "<p><a href=\"../program\">作ったツールの宣伝</a></p>";
}else{
if($_GET['type'] = "tutorial_hsp"){
echo "<h2>14ちゃんねるの扱い方(HSP)</h2>";
echo "<p>";
echo "まずは、<a href=\"hsp.tv\">HSP</a>をインストールしてください。<br>";
echo "そうしたら、「HSPスクリプトエディタ」を開いて、以下のスクリプトを入力してください。";
echo "</p>";
echo "続きは制作中です...";
}else{
if($_GET['type'] = "login"){
echo "<h2>404 Not Found</h2>";
}
}
}

URL:
http://arkmonster.r.ribbon.to/developer/menu.php?type=login

回答No.3

AsarKingChang · ベストアンサー

なので、整理すると、
$type=isset($_GET['type'])? $_GET['type']:false;
で一度代入するか？
中に丸ごと入れてしまうか？
switch(isset($_GET['type'])? $_GET['type']:false) {
case false:
パラメタがなかった時の処理
break;
case 'tutorial_hsp':
case 'login':
各処理
break;
}
あと、
特に変数が関係していないのにダブルクオートを使ってるので、
if($_GET['type'] = "tutorial_hsp"){
こういう場合が適切な時でも、シングルクオートで
if($_GET['type'] = 'tutorial_hsp'){
の方が、速度が速いですよ。

連続３連投稿しちゃったので、これにて自粛～！

お礼

ありがとうございます！解決しました！

回答No.2

AsarKingChang

おまけ
if(!isset($_GET['type'])){
これ、あちこちに書くと面倒の種にしかならないので。
if(isset($_GET['type'])){
$type=$_GET['type'];
}else{
$type=false;
}
３連書式なら、
$type=isset($_GET['type'])? $_GET['type']:false;
でまとめたほうがいいですよ。

PHPでデバッガーでトレースする時、
あちこちにあると、トラブルの元にしかならないので。
または最低でも
define("TYPE","type");
などと、宣言化するのもありでしょう。

水商売のじじいからのコメントです。

お礼

ありがとうございます。

回答No.1

AsarKingChang

> if($_GET['type'] = "tutorial_hsp"){

”＝”が一つなので、代入してます。
比較なら”＝＝”です。
型チェックあり比較は”＝＝＝”
です。

一発解決で、ごめんね。

お礼

ありがとうございます。