预期

Question

这是按钮

<a href="index.php?p=contact">contact<a> 

这是php脚本我的脚本不工作：

<?php 
$p = isset($_GET['p']); 

if($p == "artist") 
{ 
    include 'artist.php'; 
} 

if($p == "contact") 
{ 
    include 'contact.php'; 
} 

if($p == "releases") 
{ 
    include 'releases.php'; 
} 

if($p == "downloads") 
{ 
    include 'downloads.php';  
} 

else 
{ 
    include 'home.php'; 
} 
?> 

所以我的脚本应包括contact.php当我打的按钮，但而不是仅包含contact.php它包含所有的php文件。 （这也发生在其他按钮上）。

Answer 1

现在您$p变量等于true（这是isset回报）。

变化$p = isset($_GET['p']);到$p = $_GET['p'];，你会好

更妙的是：在这种情况下你固定以防止p被空

编辑

$p = isset($_GET['p']) ? $_GET['p'] : false;

：

Th ere也是你的代码的另一个问题 - 最后的else声明。当$p与downloads不同时总是如此。所以，要么你改变每if到else if这样的：

if($p == 'artist') 
{ 
    include 'artist.php'; 
} 

else if($p == 'contact') 
{ 
    include 'contact.php'; 
} 
(...) 
else 
{ 
    include 'home.php'; 
} 

或将其更改为switch声明：

switch($p) 
{ 
case 'artist': 
    include 'artist.php'; 
    break; 
(...) 
default: 
    include 'home.php'; 
} 

Answer 2

isset（$ _ GET [ 'P']）返回true或false，所以代码将$ p与某些字符串进行比较将始终返回true并在if块内运行代码。

变化$ P = isset（$ _ GET [ 'P']）只是$ P = $ _GET [ 'P']

Answer 3

为了使这一（更多）可重复使用的，你应该改变你的脚本。创建一个简单的白名单，检查它的参数是允许的，如果是，则包括$ _GET [ 'P']的直接值：

$allowed = false; 

if (isset($_GET['p'])) 

{ 

switch $_GET['p'] { 

    case 'contact': 
    $allowed = true; 
    break; 

    case 'artist': 
    $allowed = true; 
    break; 

    // and so on for all your IFs 

} 

    if ($allowed === true) { 
    include $_GET['p'].'.php' 
    } 
    else 
    { 
    die('illegal parameter found') 
    } 
}