PHP if-then-else 语句不起作用

Question

我的网址是这样的：“inventory.php?sorting=1”等等。页面加载正常，但无法正确显示信息。

mysql_connect("localhost","user","pass"); 
mysql_select_db("database"); 

if ($sorting == 1){
$result = mysql_query("select * from vehicles ORDER BY year DSC");
}
elseif ($sorting == 2){
$result = mysql_query("select * from vehicles ORDER BY make DSC");
}
elseif ($sorting == 3){
$result = mysql_query("select * from vehicles ORDER BY miles DSC");
}
elseif ($sorting == 4){
$result = mysql_query("select * from vehicles ORDER BY downpay DSC");
}
elseif ($sorting == 5){
$result = mysql_query("select * from vehicles ORDER BY pricepay DSC");
}
elseif ($sorting == 6){
$result = mysql_query("select * from vehicles ORDER BY pricecash DSC");
}
else {
$result = mysql_query("select * from vehicles");
}

while($r=mysql_fetch_array($result))

Answer 1

为什么不直接使用字段名称作为 GET 变量呢？

$sortField = $_GET['sorting'];
// Ensure we don't get any SQL injection:
$validFields = array('year', 'make', 'miles' ... 'pricecash');


$sql = "select * from vehicles";

if(in_array($sortField, $validFields)){
    $sql .= ' ORDER BY ' . $sortField .' DESC';
}

mysql_query($sql);

然后使用 inventory.php?sorting=year 等访问页面。

这使得 URL 更具可读性、可预测性，并且意味着您只需将新字段添加到数组中即可支持新字段，而无需编写新的 switch case。

Answer 2

您需要将 $sorting 替换为 $_GET["sorting"]

但同时：

使用 switch 不是一个更好的主意吗陈述？

switch($_GET["sorting"]{
    case 1:
    $result = mysql_query("select * from vehicles ORDER BY year DSC");
    break;
case 2:

ETC。

Answer 3

简短回答：将 $sorting 替换为 $_GET["sorting"]，或添加 $sorting = $_GET['sorting' ]; 到代码顶部。

长答案：很久以前，register_globals用于自动使 URL 参数显示为变量。这会导致很多安全问题（上面的链接包含一个示例），因此最终被默认关闭（PHP 4.2.0）。在 PHP 6 中，此选项不再存在。因此，您需要通过 $_GET 或 $_REQUEST 显式访问 URL GET 参数。

作为替代方案，您可以使用 import_request_variables 命令。

Answer 4

为了让它变得更好，你可以这样做：

$sortBy = '';
switch($_GET["sorting"]{
  case 1:
    $sortBy = 'year';
    break;
  case 2:
    $sortBy = 'make';
    break;
  //...
}  

if(!empty($sortBy)) {
  $result = mysql_query('select * from vehicles ORDER BY ' . $sortBy . ' DSC');
}
else {
  $result = mysql_query('select * from vehicles');
}

这样，如果有一天你必须改变你的查询，你只需要在某一时刻改变它。

Answer 5

有一些地方吗

$sorting = $_GET['sorting'];

你的代码中 ？它不会自动获取它的值。

Answer 6

在代码开头添加此行。

$sorting = $_REQUEST['sorting'];

Answer 7

您需要从 $_GET 数组中获取 $sorting 变量。我还将其重写为 switch 语句，如下所示：

switch($_GET['sorting'])
{
  case 1:
    $result = mysql_query("select * from vehicles ORDER BY year DSC");
  brek;

  case 2:
    $result = mysql_query("select * from vehicles ORDER BY make DSC");
  break;

  ...

  default:
    $result = mysql_query("select * from vehicles");
  break;
}

Answer 8

为什么不使用 switch：

switch ($sorting) {
    case 1:
        $result = mysql_query("select * from vehicles ORDER BY year DSC");
        break;
    case 2:
        $result = mysql_query("select * from vehicles ORDER BY make DSC");
        break;
    // ...
    default:
        $result = mysql_query("select * from vehicles");
        break;
}

另外，请确保分配了 $sorting：

$sorting = $_GET['sorting']; // Place somewhere before the switch

Answer 9

如果可以通过 get 或 传入，则可以使用 $_GET['sorting'] 或 $_REQUEST['sorting'] >post，但为什么不这样做呢？

$query = "SELECT * FROM `vehicles`";

$sort_values = array( 1 => 'year', 'make', 'miles', 'downpay', 'pricepay', 'pricecash' );
$sort_number = $_GET['sorting'];
if( $sort_number <= count($sort_values) ) {
    $query .= " ORDER BY `{$sort_values[ $sort_number ]}` DESC";
}

$result = mysql_query($query);

请注意，数组的 1 => 部分是因为您对查询列表进行了 1 索引。
if 语句的 <= 部分也是出于这个原因——如果您对其进行 0 索引，则只需使用 < 即可。

看起来可能还不是这样，但您很快就会发现尝试找到编写更少代码的方法是值得的。使用数组意味着您不必复制/粘贴任何代码（重复编写 $result = mysql_query(...); 等），并且几乎可以毫不费力地将新列添加到表中，如果您需要显示更多信息。

人们甚至可以直接从数据库中获取列名，并避免再次接触此代码。