2

我有以下代码：

$sql = "SELECT name, address, city FROM tableA, tableB WHERE tableA.id = tableB.id";

if (isset($price) ) {
    $sql = $sql . ' AND price = :price ';
}
if (isset($sqft) ) {
    $sql = $sql . ' AND sqft >= :sqft ';
}
if (isset($bedrooms) ) {
    $sql = $sql . ' AND bedrooms >= :bedrooms ';
}


$stmt = $dbh->prepare($sql);


if (isset($price) ) {
    $stmt->bindParam(':price', $price);
}
if (isset($sqft) ) {
    $stmt->bindParam(':price', $price);
}
if (isset($bedrooms) ) {
    $stmt->bindParam(':bedrooms', $bedrooms);
}


$stmt->execute();
$result_set = $stmt->fetchAll(PDO::FETCH_ASSOC);

我注意到的是我有多余的多个 IF 语句。

问题：有没有办法清理我的代码，这样我就没有准备好的语句的多个 IF 语句？

4

2 回答 2

2

这与一个用户最近在我的书 SQL Antipatterns 的论坛上向我提出的问题非常相似。我给了他一个类似这样的答案：

$sql = "SELECT name, address, city FROM tableA JOIN tableB ON tableA.id = tableB.id";

$params = array();
$where = array();

if (isset($price) ) {
    $where[] = '(price = :price)';
    $params[':price'] = $price;
}
if (isset($sqft) ) {
    $where[] = '(sqft >= :sqft)';
    $params[':sqft'] = $sqft;
}
if (isset($bedrooms) ) {
    $where[] = '(bedrooms >= :bedrooms)';
    $params[':bedrooms'] = $bedrooms;
}

if ($where) {
  $sql .= ' WHERE ' . implode(' AND ', $where);
}

$stmt = $dbh->prepare($sql);

$stmt->execute($params);
$result_set = $stmt->fetchAll(PDO::FETCH_ASSOC);
于 2010-04-28T20:05:48.797 回答
1

而不是 if else 只是使用 PHP 三元运算符

     if (isset($_POST['statusID']))
{
  $statusID = $_POST['statusID'];
}
else
{
  $statusID = 1;

}

而不是你可以这样做：

 $statusID =  (isset($_POST['statusID'])) ? $_POST['statusID'] : 1;

三元运算符的格式为：$variable = condition ? if true : if false

它的美妙之处在于您可以将 if/else 语句缩短到一行，如果编译器给您错误，您可以随时返回该行而不是 3 行。

于 2010-04-28T21:07:38.177 回答