Loginscript probleem(pje)

[Verwijderd]

PHP:

<?php
session_start(); 
header("Cache-control: private"); // IE 6 Fix for hitting the back menu
include("connect.php");

$_SESSION['naam'] = $_POST['naam']; 
$_SESSION['wachtwoord'] = $_POST['wachtwoord']; 

if(!isset($HTTP_POST_VARS ['Submit'])) {
    echo "You are not authorized to enter this page the way you tried to, so kindly <h1>leave!!</h1>";
}else{


if( (!$_SESSION['naam']) || (!$_SESSION['wachtwoord']) )
{ 
    echo "Vul de gevraagde login informatie in aub! <br />"; 
    include("login.html"); 
    exit(); 
} 
// Convert password to md5 hash 
$password = md5($password); 
// check if the user info validates the db 
$sql = mysql_query("SELECT * FROM persoon WHERE Naam='".$_SESSION['naam']."' AND Password='".$_SESSION['wachtwoord']."'"); 
$login_check = mysql_num_rows($sql); 
if($login_check > 0){ 
    
while($row = mysql_fetch_array($sql)){ 
foreach( $row AS $key => $val ){ 
        $$key = stripslashes( $val ); 
    } 
        // Register some session variables! 
        session_register('first_name'); 
        $_SESSION['first_name'] = $first_name; 
        session_register('last_name'); 
        $_SESSION['last_name'] = $last_name; 
        session_register('email_address'); 
        $_SESSION['email_address'] = $email_address; 
        session_register('special_user'); 
        $_SESSION['user_level'] = $user_level; 
         
        echo 'Login success';
    } 
} else { 
    echo "You could not be logged in! Either the username and password do not match or you have not validated your membership!<br /> 
    Please try again!<br />"; 
    include 'login.html'; 
} 
?> 

Ik snap niet precies waarom ik hierbij een "Parse error: parse error in c:\phpdev\www\rottertours\login.php on line 14" krijg

Dezelfde if statement werkt wel op mijn andere php scriptjes dus ik snap het ff niet meer

[Verwijderd]

regel 11: }else{

Waar eindigt die else?

[Dennis]

Verwijderd schreef op 02 May 2003 @ 13:54:
regel 11: }else{

Waar eindigt die else?

Als de topicstarter nou eens gaat nesten...

[Verwijderd]

ddc schreef op 02 May 2003 @ 13:56:
[...]

Als de topicstarter nou eens gaat nesten...

Wat ik bedoel is dat (als ik correct geteld heb) er een } te weinig staat in de code.

[Verwijderd]

Verwijderd schreef op 02 May 2003 @ 13:54:
regel 11: }else{

Waar eindigt die else?

Vroeg ik me ook al af, ik mis volgens mij wat }-tjes.

Tabs zouden handig zijn (niet alleen voor ons, ook voor jezelf)

[Verwijderd]

Ik denk niet dat het daaraan ligt, want als ik gewoon else{ weghaal krijg ik nog steeds dezelfde parse error. Ik heb die statement er ff bijgezet voor experimentele doeleinden.

Ben overigens nog supern00b dus ben niet al te goed in het overzichtelijk maken van php coden yet

[ Voor 23% gewijzigd door Verwijderd op 02-05-2003 14:00 ]

[djluc]

Alles valt te leren he...
Ik vind het een beetje slap om te zeggen dat je de code niet overzichtelijk kunt krijgen omdat je een noob bent. Dat is meer een kwestie van logisch nadenken.

Als je deze code nu eens gaat opmaken dan zul je waarschijnlijk al meteen de oplossing voor je probleem hebben.

[Verwijderd]

Dat is niet helemaal waar, want de definitie van overzichtelijk is niet echt objectief. Dus ik kan wel "logisch" gaan nadenken (wat ook vrij subjectief is), maar als het toch zo subjectief is heeft dat toch veel minder effect.

[MBV]

IK zie er zoieso al een fout zitten. Je zegt: IF (!$_SESSION['blaat']). Waarom je $_POST omzet naar $_SESSION omzet, volg ik zoieso niet. Maar je gaat nu een boolean testen. Hier zal (denk ik) altijd true uitkomen. Je moet het volgende doen: strcmp($_POST['blaat'],"")==0.

En over opmaken: Dat is waarschijnlijk al heel goed voor je foutherkenning. Zet elke accolade op een nieuwe regel (dus ook na if enz.). Alles tussen de accolade voeg je 2 spaties toe.

PHP:

if (true)
{
  doe iets
}
else
{
  doe iets anders
  if (test)
    while (boe)
      {
        blaat
      }
  en nog iets
}

Dan kom je er vanzelf achter dat je nog een else moet sluiten

[ Voor 9% gewijzigd door MBV op 02-05-2003 14:13 ]

[whoami]

Hoi.

Syntaxerrors zijn meestal makkelijk zelf op te lossen. Je hoeft er allereerst zeker niet een hele lap code voor posten, want je weet meestal al rond welke lijn de fout zich voordoet.
Als je dan de errormelding goed interpreteerd, en eens goed je code kritisch bekijkt, dan kan je het meestal zelf wel oplossen.