我有一个学校项目,我正在制作一个登录系统,你可以用电子邮件或用户名登录。但我有一个问题,显示正确的回声,电子邮件或用户名是不正确的。
这是我的密码:
if(strpos($emailOrUsername, "@")){
$query = $dbh->prepare('SELECT * FROM register WHERE email = ?');
$query->execute(array($emailOrUsername));
}
if($query->rowCount() == 0){
echo('Email is incorrect');
}
if($emailOrUsername){
$query2 = $dbh->prepare('SELECT * FROM register WHERE username = ?');
$query2->execute(array($emailOrUsername));
}
if($query2->rowCount() == 0){
echo('Gebruikersnaam bestaat niet');
}现在我只想显示一个回声。因为如果你填写错误的电子邮件和用户名,它将显示两个回显,我希望它只显示一个。
发布于 2017-11-02 20:18:47
你不需要这么做。在一个查询中,如果没有返回任何行,那么您将回答“没有这样的用户”,否则您将帮助黑客进一步推断哪些用户名或电子邮件是正确的或不正确的。
增添:
不要透露用户名/电子邮件或密码是否不正确。给出一个一般的错误信息。
发布于 2017-11-02 20:14:03
我认为你必须改变
if(strpos($emailOrUsername, "@"))
至
if(strpos($emailOrUsername, "@") !==false)
和
if($emailOrUsername)至
elseif($emailOrUsername)或elseif(!empty($emailOrUsername))
发布于 2017-11-02 22:14:07
虽然有很大的空间即兴您的查询,但我已经完善了它,以满足您的需要,从现在起。
if(strpos($emailOrUsername, "@")){
$query = $dbh->prepare('SELECT * FROM register WHERE email = ?');
$query->execute(array($emailOrUsername));
if($query->rowCount() == 0){
echo('Email is incorrect');
}
}
else {
$query2 = $dbh->prepare('SELECT * FROM register WHERE username = ?');
$query2->execute(array($emailOrUsername));
if($query2->rowCount() == 0){
echo('Gebruikersnaam bestaat niet');
}
}如果可以在一个if块中实现这一点,则不需要有多个if条件。
https://stackoverflow.com/questions/47083819
复制相似问题