Skulle behöva hjälp med mitt php script

Permalänk
Avstängd

Skulle behöva hjälp med mitt php script

OM JAG SKULLE LAGT TRÅDEN FEL VAR SNÄLL OCH FLYTTA DEN!

Hejsan nu är det såhär jag har jobbat lite på ett script. Jag kan inte så mkt om .php men jag försöker mitt bästa. Iallafall jag undrar om någon skulle vilja vara snäll och hjälpa mig med detta?

Det jag vill med scriptet är att den kolla om allt stämmer med username, code osv. Om inte så kommer det upp ett meddelande "Something is wrong!" Om det stämmer så visar den passet för användaren.
http://apelsinen.com/?p=lostpassword

Om ni går till http://apelsinen.com/?p=register
Så förstår ni hela grejen med Secret Code hoppas jag!

lostpassword.php

<b>Forgot Password</b> <?php if (isset($_POST['submit'])){ $user = $_POST['user']; $mail = $_POST['mail']; $name = $_POST['name']; $code = $_POST['code']; $pass = $_POST['pass']; $sql = "SELECT * FROM users WHERE mail='$mail', name='$name', code='$code', user='$user', pass='$pass'"; if(!mysql_query($sql)) { echo "Något är fel! ".mysql_error(); } else { echo "Ditt nya lösenord är ".$pass."!"; }} ?> <html> <body> <tr> <form action="?p=lostpassword" method="post"> <td><br>Username:<br></td> <td><input type="text" input type="text" name="user" value=""></td> </tr> <tr> <td><br>Mail:<br></td> <td><input type="text" name="mail" value=""></td> </tr> <tr> <td><br>Name:<br></td> <td><input type="text" name="name" value=""></td> </tr> <tr> <td><br>Secret Code:<br></td> <td><input type="text" name="code" value=""></td> </tr> <tr> <td> <tr> <br> <input type="submit" name="submit" value="Get Password"> </td> </tr> </table> </form> <br> <br> </html> </body>

Permalänk
Avstängd

Det är så här som jag sa jag kan inte PHP så bra. och nej det är inte så som du sa när username, code och det stämmer så får man ut sitt lösen. Därför heter detta lostpassword.php och jag vet inte alls om scriptet är rätt nu för jag kan inte MYSQL heller!

Permalänk
Avstängd

redan fixat det nu!

Permalänk

Några synpunkter, hämta inte allt när du bara behöver en sak:
"SELECT * " ska endast användas om du verkligen behöver allt. I ditt fall vill du ju bara veta om en användare med det namnet och lösenordet finns, så det räcker med "SELECT (1) FROM..."

echo "Ditt nya lösenord är ".$pass."!";
Du behöver inte konkatenera vanliga variabler. Detta funkar lika bra:
echo "Ditt nya lösenord är $pass!";