使用 WHERE 选择所有内容

Question

我目前正在开发一个存在多个访问权限的 PHP 项目。逻辑很简单：
- 当您的访问级别为2时，您只能看到您自己的项目列表
- 当您的访问级别为 3 时，您可以看到所有项目，

因此在 SQL 中：
第一个：

SELECT project_name FROM projects

第二个：

SELECT project_name FROM projects WHERE user_id = user_id

问题是我将 PDO 与准备好的语句一起使用，并且此类查询在脚本中执行多次。它是这样的：

if ($_SESSION['access_level'] == 3) {
    $sql = "SELECT project_name FROM projects";
  } else {
    $sql = "SELECT project_name FROM projects WHERE user_id = ?";
}

$res = $db->prepare($sql);

// Some more PHP 

if ($_SESSION['access_level'] == 3)
      $res->execute();
    else
      $res->execute(array($_SESSION['user_id']));

当我在脚本的多个部分中执行此操作时，它会变得一团糟。有更好的方法吗？就我个人而言，我正在考虑一个 WHERE 子句，其中选择每条记录。这样，这在脚本开始时就可以实现：

if ($_SESSION['access_level'] == 3)
      $id = *;
    else
      $id = $_SESSION['user_id'];

现在查询更容易了：（

$res = $db->prepare("SELECT project_name FROM projects WHERE user_id = ?");
$res->execute(array($id));

现在，当您的访问级别为 3 时，它将获取所有记录，但当您只有级别 2 时，它只会获取您自己的记录）

这看起来很漂亮在我看来，转储解决方案是因为我并没有真正使用 WHERE 子句应该如何使用它。另外，使用 * 是不可能的。

对此最好的选择是什么？

谢谢你！

Answer 1

您正在以完全错误的方式寻找解决方案。

每当您遇到在脚本的多个部分执行某些操作并且变得一团糟的情况时，您都必须创建一个函数。

事实上，您仍然在为页面上的每个查询编写所有这些丑陋的重复代码，包括准备、执行、获取、准备、执行、获取、准备、执行、获取。你不觉得这看起来很混乱吗？

因此，您必须创建两个函数。

通用的一个，只是为了从查询中获取一些值而不重复无用的代码，像这样使用它：

$proj_names_arr = $db->getColumn("SELECT project_name FROM projects");

Raisen 提到的一个，基于第一个，像这样使用

$proj_names_arr = getProjects();

它将是唯一的真实< /strong> 改进你的代码

至于功能，这并不难
一个粗略的例子：

function getColumn() {
  $args  = func_get_args();
  $query = array_shift($args);
  $res = $db->prepare($query);
  $res->execute($args);
  $data = array();
  while ($row = $res->fetch(PDO::FETCH_NUM)) {
    $data[] = $row[0];
  }
  return $data;
}

所以，它可以被称为

$proj_names = $db->getColumn("SELECT project_name FROM projects WHERE user_id = ?",
                             $_SESSION['user_id']);

Answer 2

我认为将 where 子句以及绑定参数保留在变量中更容易，然后在运行时编写查询：

<?php

$sql = 'SELECT project_name FROM projects';
$where = array();
$parameters = array();


if ($_SESSION['access_level'] !== 3) {
    $where[] = 'user_id = ?';
    $parameters = array($_SESSION['user_id']);
}



if( !empty($where) ){
    $sql .= ' WHERE ' . implode(', ', $where);
}
$res = $db->prepare($sql);

$res->execute($parameters);

Answer 3

那么像这样的 SQL 呢：

$sql = 'SELECT project_name FROM projects WHERE user_id = ? OR 3 = ?';
$res = $db->prepare($sql);
$res->execute(array($id, $access_level);

Answer 4

创建一个函数，例如 getProjects()，它将根据用户级别返回项目。

例如在函数内部使用 Alvaro 的代码并返回结果集。

Answer 5

我是否明白：您想要一个 WHERE 语句，这总是正确的？

WHERE 1 = 1

Answer 6

由于我在脚本的多个部分中执行此操作，因此变得一团糟。

不要重复。如果您发现自己重复代码块，则表明您应该将该通用代码重构为函数或方法。

您还可以通过向查询附加额外信息而不是重写查询来消除一些较小的重复，并使用诸如 bindParam() 之类的内容来添加附加参数，而不是修改您的 execute() 语句：就

$userOnly = ($_SESSION['access_level'] == 2);

$sql = "SELECT project_name FROM projects";

if ($userOnly) {
    $sql .= " WHERE user_id = ?";
}

$res = $db->prepare($sql);

if ($userOnly) {
 $res=>bindParam(bindParam(1, $_SESSION['user_id'], PDO::PARAM_INT);
}

$res->execute(array($_SESSION['user_id']));

我个人而言，我认为这比原来的更简洁，因为仅适用于较低访问级别的部分被明确标记并且是独立的，而不是重复已经呈现的代码。

此外，使用具有访问级别描述性名称的新变量可以防止重复幻数，并为您提供单点更改。

希望有帮助。