将一个$ _POST值传递给带有PDO变量的函数参数

我已经建立了一个pdo连接，并将其作为变量传递给一个函数。这一切工作正常，该功能正确返回。如果我使用PDO变量和名称在条件语句中运行该函数，它将正确运行 - 如果名称位于数据库中，则它会正确回应，如果不是，则它也会正确回显。我想要做的是将表单帖子的值传递给函数，以便检查它是否存在于数据库中。这里是我的代码：

函数检查列计数是否为1。

function user_exists($pdo, $username) { 
$stmt = $pdo->prepare('SELECT COUNT(uid) FROM users WHERE username = :username'); 
$stmt->execute(['username' => $username]); 
$result = $stmt->fetchColumn(); 
return ($result == 1); 
}

如果管理员用户在数据库中存在回声“存在” - 只是用于测试。

if(user_exists($pdo,'admin') == true) { 
echo "exists"; 
} else { 
echo "doesnt exist"; 
}

检查，看看这两个领域已经进入那么我想它来检查输入的用户名在数据库中，但我做错了什么。

if(!empty($_POST) === true) { 
$username = $_POST['username']; 
$pwood = $_POST['password']; 
if(empty($username) === true || empty($pwood) === true) { 
echo "You need to enter a username and password"; 
} else if (user_exists($pdo,$_POST['username']) == false){ 
echo 'We can\'t find that username. Please try again or register'; 
} 
}

来源

2017-10-17 cozmo smallpiece

此行'如果（！空（$ _ POST）===真）{' – Akintunde007

完美非常感谢。我不知道为什么。 post的值不为空...... –

@GrumpyCrouton，使用'global'变量*不推荐。请参阅https://stackoverflow.com/questions/5166087/php-global-in-functions或https://stackoverflow.com/questions/1557787/are-global-variables-in-php-considered-bad-practice-if -so-why为什么 –

最好不要与布尔值进行比较，只是用

//...see below 
require_once('UserRepository.php'); 
$ur = new UserRepostory($pdo); 

if(!empty($_POST)) { 
    if (empty($username) || empty($pwood)) { 
     // something 
    } else if (!$ur->exists($_POST['username'])) { // your user_exists($pdo, $username) works fine, too 
     // something else 
    } 
}

尤其是初始if(!empty($_POST) === true) {是难以阅读和导致错误“的原因误解操作的优先级。基于上述意见

更新，这里是一个类的实例：

// UserRepository.php 
class UserRepository { 
    private $pdo; 

    // '\PDO' instead of 'PDO', see PHP Namespacing 
    public function __construct (\PDO $pdo) 
    { 
     $this->pdo = $pdo; 
    } 

    public function exists($username) 
    { 
     $sql = 'SELECT COUNT(uid) FROM users WHERE username = :username'); 
     $stmt = $this->pdo->prepare($sql); 
     $stmt->execute(['username' => $username]); 
     $result = $stmt->fetchColumn(); 

     return (bool) $result; 
    } 
}

来源

2017-10-17 20:39:10 clemens321

我可以看到这是如何合理的。 OO目前略高于我的技能水平。虽然我相信我会回到这个答案。 –

正如代码注释中提到的，没有类的函数可以正常工作，并且对于小型项目和初学者也很有用。从小开始没有什么不好，只是为了显示“正确”的方式，而不是使用全局变量。 – clemens321

真的很棒的编辑，我希望我能第二次upvote。 – GrumpyCrouton

将一个$ _POST值传递给带有PDO变量的函数参数

回答

相关问题