Archief - PHP: Fout in script, maar waar?

r

raintjah`

Legacy Member

• 3 jul 2004

• #1

Ik ben dus PHP aan het leren, en ik heb net een form leren maken enzo, en nu wou ik eens een initiatief nemen:

login.html :

<form action="loginverwerken.php" method="post">
Login:<br><br>
Username: <input type="text" name="username"><br><br>
Password: <input type="text" name="password"><br><br>
<input type="Submit" value="Login">
</form>


loginverwerken.php
<?php

if (isset($_POST["password"] && $_POST["password"]=="rofl") && isset($_POST["username"] && $_POST["username"]=="raintjah") {
echo "U bent nu ingelogt";
}

else {
echo "Wrong username/password";
}



?>


Er zit een fout in line 3 van loginverwerken.php. Ik denk dat het iets is met de haakjes omdat ik nie echt snap waar ik die precies moet zetten.

---> Ik weet dat dit niet de beste manier is om een login te maken, het was ook maar een probeersel... Weidt dus aub niet uit naar een beter login script

TY

m

maT'

Legacy Member

• 3 jul 2004

• #2

PHP:

loginverwerken.php
<?php

if (isset($_POST["password"]) && $_POST["password"]=="rofl" && isset($_POST["username"]) && $_POST["username"]=="raintjah" {
echo "U bent nu ingelogt";
}

else {
echo "Wrong username/password";
}
?>

de IsSet() functie vereist een variabele tussen die haakjes, bv: if (isset($_POST['pwd'])) { ...}

r

raintjah`

Legacy Member

• 3 jul 2004

• #3

thw, het werkt!
maaruhm, kan ik dat gebruiken als login script?
Grtz.



BTW: Hoe kan ik daar verschillende waardes aan geven?

Bv: dat de username stijn ook kan inloggen? grtz.

********************** EDIT **************

http://users.skynet.be/fa008497/raintjah2/layout.html

Als ik daar zo een tekstvlak insteek, dan flipt mn layout omdat dat ding te breed is, hoe kan ik de breedte aanpassen van zo een tekstvlak..?
Grtz.

S

SpoBo

Legacy Member

• 3 jul 2004

• #4

die issets moeten er eigenlijk ni bij aangezien ge toch vergelijkt met een waarde.
tzou zoiet worden dan:

Code:

if (($_POST['password']=="rofl" && $_POST['username']=="raintjah") OR (($_POST['password']=="stijnpass" && $_POST['username']=="stijn"))) {
      echo "U bent nu ingelogt";
}

ma als ik u was zou ik eerst is een deftig loginscriptje met session bezien enzo want dees is echt wel gelimiteerd :/

|

|t3st

Legacy Member

• 3 jul 2004

• #5

neen users.skynet.be ondersteund geen php en dus oko nie dit loginscript

beetje zoeken op logische fouten is niet al teveel gevraagd hoop ik
en probeer u scripts eerst es lokaal te testen (www.google.be + easyphp voor host)

greetz Sluttie

C

Col.Kurtz

Legacy Member

• 3 jul 2004

• #6

die isset is eig. overbodig want in je validatie zijn die vars altijd "ge-set", dus enkel checken of user & pass juist zijn

k

killgore

Legacy Member

• 3 jul 2004

• #7

Col.Kurtz zei:

die isset is eig. overbodig want in je validatie zijn die vars altijd "ge-set", dus enkel checken of user & pass juist zijn

Klik om te vergroten...

gpc vars moet je ALTIJD checken of zo bestaan.

C

Col.Kurtz

Legacy Member

• 3 jul 2004

• #8

killgore zei:

gpc vars moet je ALTIJD checken of zo bestaan.

Klik om te vergroten...

deze 2 scriptjes hebben toch hetzelfde resultaat ?

Code:

if(isset($_POST["name"]) && $_POST["name"]!="") {
   //ok
} else {
   exit;
}

if($_POST["name"]!="") {
   //ok
} else {
   exit;
}

of niet ?

R

Radiance

Legacy Member

• 4 jul 2004

• #9

Col.Kurtz zei:

deze 2 scriptjes hebben toch hetzelfde resultaat ?
of niet ?

Klik om te vergroten...

niet, die tweede geeft je dit als die waarde niet bestaat:

Notice: Undefined index: name in e:\easyphp\www\test.php on line 2

waar uw eerste gaat afbreken omda de variable ni geset is

m

maT'

Legacy Member

• 4 jul 2004

• #10

Dan kan je't natuurlijk ook iets minder proper oplossen:

PHP:

if(@$_POST["name"]!="") {
   //ok
} else {
   exit;

maar soit, 't is idd beter om te checken of de variabele bestaat of niet *muggezifting*
IsSet() zou altijd voorrang moeten krijgen.

k

killgore

Legacy Member

• 4 jul 2004

• #11

maT' zei:

Dan kan je't natuurlijk ook iets minder proper oplossen:

PHP:

if(@$_POST["name"]!="") {
   //ok
} else {
   exit;

maar soit, 't is idd beter om te checken of de variabele bestaat of niet *muggezifting*
IsSet() zou altijd voorrang moeten krijgen.

Klik om te vergroten...

Niks te muggezifting, correct is correct .
zeggen dat je hier beter !empty($_POST['name']) gebruikt, DAT is muggeziften .

s

servi

Legacy Member

• 4 jul 2004

• #12

ge gebruikt hier beter if ( !empty($_POST['name']) )

k

killgore

Legacy Member

• 4 jul 2004

• #13

servi zei:

ge gebruikt hier beter if ( !empty($_POST['name']) )

Klik om te vergroten...

Twas te denken da gij da hier weer ging komen zeggen .