Jump to content

Archived

This topic is now archived and is closed to further replies.

Razzeal

Problem making a login function within a class...

Recommended Posts

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...

Share this post


Link to post
Share on other sites
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]

Share this post


Link to post
Share on other sites
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 :D

Thx

Share this post


Link to post
Share on other sites
Make sure you are using the 'this->' keyword to reference your variables!

Share this post


Link to post
Share on other sites
That means before every variable? anyway I think I got it now :D thx

Share this post


Link to post
Share on other sites

×

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.