php probleem

Hallo,

Middels de volgende code probeer ik te controleren of de ingevoerde gegevens juist zijn, zoja: url naar controlepaneel weergeven. Echter geeft hij deze fout:

Quote:
Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in /home/onehosted/domains/***********.com/public_html/serial/admin.php on line 35

Script

[COLOR= #0000BB]<?php[/COLOR]

[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_SERVER[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]REQUEST_METHOD[/COLOR][COLOR= #007700]] ==[/COLOR] [COLOR= #DD0000]POST[/COLOR][COLOR= #007700]){[/COLOR]

[COLOR= #0000BB]$naam[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/mysql_real_escape_string"]mysql_real_escape_string[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]gebr_naam[/COLOR][COLOR= #007700]]);[/COLOR]
[COLOR= #0000BB]$wachtwoord[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/md5"]md5[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]wachtwoord[/COLOR][COLOR= #007700]]);[/COLOR]

    [COLOR= #0000BB]$adminQuery[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/mysql_query"]mysql_query[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]"SELECT * FROM admin WHERE gebruikersnaam="[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$naam[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]" AND wachtwoord="[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$wachtwoord[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]""[/COLOR][COLOR= #007700])[/COLOR] or [URL="http://www.php.net/die"]die[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/mysql_error"]mysql_error[/URL][COLOR= #007700]);[/COLOR]
    
    [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/mysql_num_rows"]mysql_num_rows[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$adminQuery[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]1[/COLOR][COLOR= #007700])){[/COLOR]
        
        [URL="http://www.php.net/echo"]echo[/URL][COLOR= #DD0000]U bent succesvol ingelogd, klik <a href="controlepaneel.php">hier</a> om verder te gaan![/COLOR][COLOR= #007700];
        
    }[/COLOR][URL="http://www.php.net/else"]else[/URL][COLOR= #007700]{[/COLOR]
        
        [URL="http://www.php.net/echo"]echo[/URL][COLOR= #DD0000]Uw inloggegevens zijn onjuist, probeer het nog eens.[/COLOR][COLOR= #007700];
        
    }
    
    
}[/COLOR]

?>

Alvast bedankt voor jullie hulp!

[COLOR= #0000BB]<?php[/COLOR]

[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_SERVER[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]REQUEST_METHOD[/COLOR][COLOR= #007700]] ==[/COLOR] [COLOR= #DD0000]POST[/COLOR][COLOR= #007700]){[/COLOR]

[COLOR= #0000BB]$naam[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/mysql_real_escape_string"]mysql_real_escape_string[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]gebr_naam[/COLOR][COLOR= #007700]]);[/COLOR]
[COLOR= #0000BB]$wachtwoord[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/md5"]md5[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]wachtwoord[/COLOR][COLOR= #007700]]);[/COLOR]

    [COLOR= #0000BB]$adminQuery[/COLOR] [COLOR= #007700]=[/COLOR] [URL="http://www.php.net/mysql_query"]mysql_query[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]"SELECT * FROM admin WHERE gebruikersnaam="[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$naam[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]" AND wachtwoord="[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$wachtwoord[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]""[/COLOR][COLOR= #007700])[/COLOR] or [URL="http://www.php.net/die"]die[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/mysql_error"]mysql_error[/URL][COLOR= #007700]);[/COLOR]
    
    [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/mysql_num_rows"]mysql_num_rows[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$adminQuery[/COLOR][COLOR= #007700]) ==[/COLOR] [COLOR= #0000BB]1[/COLOR][COLOR= #007700]){[/COLOR]
        
        [URL="http://www.php.net/echo"]echo[/URL][COLOR= #DD0000]U bent succesvol ingelogd, klik <a href="controlepaneel.php">hier</a> om verder te gaan![/COLOR][COLOR= #007700];
        
    }[/COLOR][URL="http://www.php.net/else"]else[/URL][COLOR= #007700]{[/COLOR]
        
        [URL="http://www.php.net/echo"]echo[/URL][COLOR= #DD0000]Uw inloggegevens zijn onjuist, probeer het nog eens.[/COLOR][COLOR= #007700];
        
    }
    
    
}[/COLOR]

?>

Haakje verkeerd neergezet... Verder niet gecontroleert.

Als ik jou was zou ik de gebruikersnaam en wachtwoord apart controleren.

Shahiem schreef:

Als ik jou was zou ik de gebruikersnaam en wachtwoord apart controleren.

want

Omdat dit in mijn inzien niet overzichtelijk is maar ook niet netjes. En als hij die melding geeft van onjuist dan weet je niet wat onjuist is.

Shahiem schreef:

Omdat dit in mijn inzien niet overzichtelijk is maar ook niet netjes. En als hij die melding geeft van onjuist dan weet je niet wat onjuist is.

Dit is gewoon overzichtelijk hoor dus ik zie in waarom je het apart moet checken? Doe ik ook niet en ik denk nog wel veeeeel meer niet. Als jij op een login site gaat zetten je ww is verkeerd dan weet ik dat het gebruikersnaam goed is. Makkelijk voor hackers...

Als je weet dat de gegevens onjuist zijn, dan weet je toch niet wat er uberhaupt fout is. Dan kan je net zo goed de manier wat ik doe proberen.

Het is gewoon te rommelig vind ik.

Wat Shahiem zegt vind ik ook; als ik een tijd niet op een website ben geweest, moet ik eerst uitvogelen van mijn gebruikersnaam was. Als ik wat invoer en er staat dat het fout is, weet ik niet of mijn wachtwoord of mijn gebruikersnaam nou fout is. Als mijn gebruikersnaam goed is en hij meldt dat, kan ik mijn wachtwoord opvragen en ben ik klaar.

R

Shahiem schreef:

Als je weet dat de gegevens onjuist zijn, dan weet je toch niet wat er uberhaupt fout is. Dan kan je net zo goed de manier wat ik doe proberen.

Het is gewoon te rommelig vind ik.

Rommelig = slordig en dit is niet slordig en het slaat nergens op. Goed hoor Jos

Slaat wel degelijk ergens op. Het is gewoon niet netjes om het zo te controleren. Je kan dan wel zeggen dat het niet zo is, maar dit is mijn mening. Misschien geeft Jos me wel gelijk.

Het is alleen voor een admin en zal dus niet gebruikt worden door mensen van buitenaf de site. Ik vind de manier van Shahiem ook goed hoor, maar in dit geval is dat denk ik alleen maar meer werk. Verder hoop ik dat iemand de fout ziet, want ik krijg nog steeds die error.
In ieder geval bedankt allemaal!

EDIT:
Oplossing was al gegeven door Robbert, had er even overheen gelezen, bedankt!