Razzeal Posted April 30, 2006 Share Posted April 30, 2006 Hey,I was trying to make a function (within a class) for logging in, this to make it easier on my websites to log in and not have to create al the code over and over again.Now my problem is here:[code]$v_user = mysql_query("SELECT * FROM $u_table WHERE $u_row = $username") or die("Error: " .mysql_error());[/code]The error I get is a mysql error, saying I should check my code considering the "[b]'WHERE = '[/b]" on line 1.... Is it possible to have variables in my sql like this or do I have to change the constants every time I use the function for another website? How else should I put the variables so that ik might still work.Thx in advance,Razzeal... Quote Link to comment Share on other sites More sharing options...
High_-_Tek Posted April 30, 2006 Share Posted April 30, 2006 I almost NEVER code in OOP, but if you are using this inside a class it should be:[code]$this->$v_user = mysql_query("SELECT * FROM $u_table WHERE $u_row = $username") or die("Error: " .mysql_error());[/code]or[code]$this::$v_user = mysql_query("SELECT * FROM $u_table WHERE $u_row = $username") or die("Error: " .mysql_error());[/code] Quote Link to comment Share on other sites More sharing options...
Razzeal Posted May 1, 2006 Author Share Posted May 1, 2006 Thx for your reply!I tried your code and it kept returning the same error this is my function:[code]function login($username, $password){ //controleren of ingevulde username bestaat: $this -> $v_user = mysql_query("SELECT * FROM $u_table WHERE $u_row = '$username'") or die("Error: " .mysql_error()); $exist = mysql_num_rows($v_user); if($exist < 1) { $error = "Username not found!"; return $error; } else { //wachtwoord controleren: $this->$get_pass = mysql_query("SELECT * FROM $u_table WHERE $u_row = '$username'") or die("Error: " .mysql_error()); $p_array = mysql_fetch_array($get_pass); $e_pass = $p_array["password"]; if($e_pass != md5($password)) { //errors weergeven: $error = "Wrong Password!"; return $error; } else { //login gelukt! return true; } }}[/code]I would like to keep it als variable is possible so that i can use this function wherever I want :DThx Quote Link to comment Share on other sites More sharing options...
rawb Posted May 1, 2006 Share Posted May 1, 2006 Make sure you are using the 'this->' keyword to reference your variables! Quote Link to comment Share on other sites More sharing options...
Razzeal Posted May 1, 2006 Author Share Posted May 1, 2006 That means before every variable? anyway I think I got it now :D thx Quote Link to comment 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.