Return boolean ( if value exist in column ) in QSqlQuery
-
@Trav
Hi.bool UserLoginDlg::User_Searched(QString user, QString pass){ QSqlQuery query; query.prepare("select * from users where USERNAME ='"+user+"' and PASSWORD = '"+pass+"'"); query.exec(); while(query.next()){ return true; } return false; } void UserLoginDlg::on_Log_In_clicked() { if(User_Searched(ui->USERNAME->text(),ui->PASSWORD->text())){ User Searched :)) } else QMessageBox::information(this,"WARNING","Username Or Password Is Wrong"); }ui->USERNAME->text(),ui->PASSWORD->text(), USERNAME AND PASSWORD Is Linedit Text
See for an easy reference on what's wrong with your code:
https://www.w3schools.com/sql/sql_injection.asp
https://www.youtube.com/watch?v=8ZtInClXe1Q -
QSqlQuery query; query.prepare("INSERT INTO employee (id, name, salary) " "VALUES (:id, :name, :salary)"); query.bindValue(":id", 1001); query.bindValue(":name", "Thad Beaumont"); query.bindValue(":salary", 65000); query.exec();And Read This:
http://doc.qt.io/qt-5/qsqlquery.html -
See for an easy reference on what's wrong with your code:
https://www.w3schools.com/sql/sql_injection.asp
https://www.youtube.com/watch?v=8ZtInClXe1Q -
@Taz742 said in Return boolean ( if value exist in column ) in QSqlQuery:
This was just an example
Sorry mate but an "example" that can get you fired and/or sued it's a hell of a bad example
"La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
~Napoleon BonaparteOn a crusade to banish setIndexWidget() from the holy land of Qt
-
@Taz742 said in Return boolean ( if value exist in column ) in QSqlQuery:
This was just an example
Sorry mate but an "example" that can get you fired and/or sued it's a hell of a bad example
-
The query should be:
qry.prepare("SELECT * FROM known_users WHERE email_address = :email ");the part in the
TODOisuserNameField =false; if(qry.exec()) userNameField =qry.next();This does not check the password though, only the email
@VRonin said in Return boolean ( if value exist in column ) in QSqlQuery:
qry.prepare("SELECT * FROM known_users WHERE email_address = :email ");
Thanks, I found your reply the most helpful just from the clear response. The if statement can you please explain this a little more? the issue I guess I'm having is the if statement logic the components..
So we are declaring the userNameField = false and then entering a IF statement which is executing what exactly? and the final line "userNameField =qry.next();" we are moving to next bool? Im just a bit confused by this and want to understand this correctly so I can understand the logic thoroughly and have my hah moment , thank you. -
if(qry.exec())just checks that the query was executed i.e. we don't have syntax errors in our query and the SQL server is respondinguserNameField =qry.next();so if the database contains the email passed as a parameter it will return at least one row, if it does not then it will return 0 records.qry.next()moves to the next record (the first in this case) and returns true if the record exists. What i'm doing here is asking "does the query returns at least 1 record?" to do this I try accessing the first record if it's unsuccessful it means that the query returned no rows and hence the user does not exist
P.S.
Can I ask what type is usernameField?"La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
~Napoleon BonaparteOn a crusade to banish setIndexWidget() from the holy land of Qt
-
if(qry.exec())just checks that the query was executed i.e. we don't have syntax errors in our query and the SQL server is respondinguserNameField =qry.next();so if the database contains the email passed as a parameter it will return at least one row, if it does not then it will return 0 records.qry.next()moves to the next record (the first in this case) and returns true if the record exists. What i'm doing here is asking "does the query returns at least 1 record?" to do this I try accessing the first record if it's unsuccessful it means that the query returned no rows and hence the user does not exist
P.S.
Can I ask what type is usernameField?@VRonin Thanks for clarifying that ! I understand and can apply this to future applications and sure... Just a QString at the moment which is entered into a UI field then passed to the function. I will take more secure practices in the future in context of the application though will defiantly want the password hashed etc. If you have any suggestions in context of something like this ( I think you understand what I'm doing here ) I'd be glad to hear them as this is my first time doing a project of this sort with communication to a Database
-
@VRonin Thanks for clarifying that ! I understand and can apply this to future applications and sure... Just a QString at the moment which is entered into a UI field then passed to the function. I will take more secure practices in the future in context of the application though will defiantly want the password hashed etc. If you have any suggestions in context of something like this ( I think you understand what I'm doing here ) I'd be glad to hear them as this is my first time doing a project of this sort with communication to a Database
@Trav said in Return boolean ( if value exist in column ) in QSqlQuery:
Just a QString at the moment which is entered into a UI field then passed to the function.
then you need to declare another boolean instead of re-using that string
bool userNameFieldFound =false; if(qry.exec()) userNameFieldFound =qry.next();I will take more secure practices in the future in context of the application though will defiantly want the password hashed etc. If you have any suggestions in context of something like this ( I think you understand what I'm doing here ) I'd be glad to hear them
I suggest Argon2 but QCryptographicHash with
QCryptographicHash::Sha3_512algorithm should be ok as long as you always remember to salt it"La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
~Napoleon BonaparteOn a crusade to banish setIndexWidget() from the holy land of Qt
-
@Trav said in Return boolean ( if value exist in column ) in QSqlQuery:
Just a QString at the moment which is entered into a UI field then passed to the function.
then you need to declare another boolean instead of re-using that string
bool userNameFieldFound =false; if(qry.exec()) userNameFieldFound =qry.next();I will take more secure practices in the future in context of the application though will defiantly want the password hashed etc. If you have any suggestions in context of something like this ( I think you understand what I'm doing here ) I'd be glad to hear them
I suggest Argon2 but QCryptographicHash with
QCryptographicHash::Sha3_512algorithm should be ok as long as you always remember to salt it
