您的result 变量(除了被覆盖)不是您认为的那样。这是一个 PDO 声明。
试试这个:
$stmt = $this->con->prepare("select * from ".$table_name." where username = :user_email");
$stmt->execute([
':user_email' => $user_email,
]);
if (false !== ($row = $stmt->fetchColumn()))
{
return $row;
}
但是,这只会返回第一行的第一列。相反,您可能想要:
return $stmt->fetchAll(PDO::FETCH_ASSOC);
我将$result 更改为$stmt,因为这不是结果。它是一个语句对象。
原始问题
在您的 原始 代码(见下文)中,您将使用来自 execute 的返回值覆盖它,这是一个布尔值。
// Old code (don't use this)
$result = $result->execute([
':user_email' => $user_email,
]);
//$result = TRUE|FALSE
if ($result->fetchColumn() !== false)
{
return $result;
}
然后您尝试调用 Boolean 的方法,这将不起作用。但问题远不止于此。假设您没有覆盖它。
// Old code (don't use this)
$result->execute([
':user_email' => $user_email,
]);
//$result = PDOStatment object.
if ($result->fetchColumn() !== false)
{
return $result;
}
现在结果仍然是您的 PDOStatement,这很好,但正如我所说,您没有保存获取的数据。这次您返回 PDOStatement 对象。这不是你想要的。
然后,正如我之前所说,如果您确实保存并返回它,它可能仍然不是您想要的。因为fetchColumn() 一次只能访问一行和一列。
但我无法知道你想要什么。也许这就是你想要的?在这种情况下,您的查询不是理想。也许您只是想查看给定电子邮件的用户是否存在?在那种情况下,我会使用这个查询。
$result = $this->con->prepare("SELECT id FROM ".$table_name." WHERE username = :user_email");
$result->execute([
':user_email' => $user_email,
]);
// There isn't any need to check it (see below)
return $result->fetchColumn();
PDOStatement::fetchColumn() 从结果集的下一行返回单个列,如果没有更多行,则返回 FALSE。
我也可以从你的资料中看出,你的数据库设置可能是错误的。也就是说,如果你真的需要动态表$table。我可以这么说的原因是您不应该复制任何用户数据(或者实际上任何数据,这称为规范化),并且动态表意味着电子邮件可能单独存在于两个(或更多)表中。
如果不是这样,那就不要让它动态化。为什么这是一个问题?好吧,想想如果用户现在更改他们的“电子邮件”会发生什么,因为它存在于两个表中(可能)。你必须在这两个地方更新它。但它比这更糟糕,因为它使您对电子邮件所做的任何事情都过于复杂。
没有看到您的表的架构,我只能推测它,以及如何修复它。但通常你会使用外键并将用户记录与它相关联。然后使用 JOIN 即可访问电子邮件而无需重复。
也就是说,在少数情况下这可能是可以接受的,但我无法知道您的情况是否属实。一个简单的例子是用户和管理员的单独表(基本上是两个用户系统)。
安全
最后要非常小心:
"select * from ".$table_name." where username = :user_email"
这里的问题是它对SQL injection 开放。任何时候将变量连接到 SQL 中,都会为 injection 攻击打开大门。好吧,你可能会说我正在传递一个罐装字符串account。这没问题,但在失败点没有验证。因此,也许在五个月内您重用此代码并忘记您从未验证过表名。也许不是,但事实仍然是,如果用户数据可以进入该参数,您将无法防止表名上的 injection。它的可能性是存在的。
就这么简单:
public function verify_user($table_name,$user_email){
$allowed = ['account','users'];
if(!in_array($table_name, $allowed )) throw new Exception('Invalid table name');
}
现在看,在表名中注入 东西几乎是不可能的。此外,因为它采用相同的方法(在故障点),您将永远不会失去这种保护。后者很容易赶时间并复制一段代码更改一些内容,然后......你知道的。
只要我的两分钱。
更新
因此,即使用户输入进入$table 的可能性很小,您也不能保证 100%,因为在verify_user 中,您无法知道数据来自哪里,但您相信相信这不是用户输入。说到SQL注入,你不能说这没问题,因为我只会以某种方式调用这个方法。它必须是 100% 防注入或尽可能接近人类。
你问为什么这很重要?想象一下。
$userprofileobj->verify_user('account --',$_SESSION['user_email']);
这两个小--s 类似于 PHP 中的//,但对于 SQL,它们注释掉 SQL 中的其余行,因此您的查询变成了这样。
"select * from account -- where username = :user_email"
或者(基本上)
"select * from account"
所以我们只是修改了您的查询的内容。现在幸运的是,在 PDO 中一次运行两个查询实际上是不可能的。您可以在 MySqli 中完成(通过一些工作)。但出于安全原因,他们大多取消了这种能力。原因就是这个(或者更糟的是创建数据库用户)。
$userprofileobj->verify_user('account; DROP TABLE account --',$_SESSION['user_email']);
如果你可以做两个查询,你会这样做:
SELECT * FROM account
DROP TABLE account
无论如何,这是危险的事情,必须不惜一切代价避免。太懒(我是一个 lazy 程序员,所以不要误会)把表名放进去并不是你想要在你的数据库被破坏并且你暴露之后给出的答案用户数据给第三方。这不是一个选择。
所有这些:
if(!in_array($table_name, ['table1', 'table2', ...])) throw new Exception('Invalid table name');
如果 "needle" $table_name 不在 "haystack" 中,则抛出错误 - 表名的固定列表。所以如果我这样做(使用我们上面的例子):
if(!in_array('account --', ['table1', 'table2', ...])) throw new Exception('Invalid table name');
它不会在我们的table1 和table2 列表中找到account -- 并且会爆炸,从而防止注入攻击。