Tom18 Posted September 13, 2018 Share Posted September 13, 2018 Hello i am trying to create a prepared statement with mysqli and php to see if the username entered in the HTML form already exists the in table. I am getting this error Fatal error: Uncaught Error: Call to a member function bind_param() on string My code: $sql_u = "SELECT user FROM users WHERE user=?"; $sql_u->bind_param('s', $user); $sql_u->execute(); if($result = $sql_u->fetch_array()) { if($result >= 1) { print "That username is taken!\n"; } else { //Do nothing } } The variable $user is equal to $_POST['user']; The code below is working it inserts records into the table with no problems but i would just like to check if the user is already existing. $stmt = $conn->prepare("INSERT INTO users (user, password, email) VALUES (?, ?, ?)"); $stmt->bind_param('sss', $user, $pass, $email); $stmt->execute(); if(!$stmt) { die("Something went wrong."); } Quote Link to comment https://forums.phpfreaks.com/topic/307683-check-if-username-already-exists/ Share on other sites More sharing options...
Barand Posted September 13, 2018 Share Posted September 13, 2018 (edited) If you look at your second block of code you will see you are using prepare() method. That bit is missing from the first block - you are defining a string instead of a prepared statement. bind_param() needs a statement object. EDIT: BTW fetch_assoc() returns an array, not a numeric value. Edited September 13, 2018 by Barand Quote Link to comment https://forums.phpfreaks.com/topic/307683-check-if-username-already-exists/#findComment-1560795 Share on other sites More sharing options...
benanamen Posted September 13, 2018 Share Posted September 13, 2018 Your your logic is incorrect. You do not check to see if the user already exists. You set a unique constraint on the username column and then attempt the insert capturing the duplicate error if any. Your method will create a race condition. Quote Link to comment https://forums.phpfreaks.com/topic/307683-check-if-username-already-exists/#findComment-1560798 Share on other sites More sharing options...
Zane Posted September 14, 2018 Share Posted September 14, 2018 For starters, a string is not an object $sql_u = "SELECT user FROM users WHERE user=?"; $sql_u->bind_param('s', $user); $sql_u->execute() Therefore, the bind_param function does not exist; and nor does execute Quote Link to comment https://forums.phpfreaks.com/topic/307683-check-if-username-already-exists/#findComment-1560827 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.