[php] if commando werkt niet?

[Elect]

PHP:

<? $HTTP_COOKIE_VARS["lidnaam, passgecodeerd"]; include('config/config.php');
           
mysql_connect("$hostname","$username","$password");
mysql_select_db("$database");
           
$select = "SELECT * FROM leden WHERE lidnaam='$lidnaam'";
$query = mysql_query($select);

while ($list = mysql_fetch_object($query)) {
if ( ($list->status == 'Administrator') || ($list->status == 'Moderator') ) {
    echo "";
    }
else {
    exit;
    }
}
               
?>

Ik ben al een half uur bezig aan dit stomme stukje, maar ik zie niet waarom hij niet werkt.

Als ik niet ben ingelogd heb ik geen cookie, en nog laat hij me toe tot de pagina. Oftwel hij moet me nog herkennen als Administrator of Moderator?
Zie ik nu iets cruciaals over het hoofd of?

[ Voor 12% gewijzigd door Elect op 11-10-2003 15:54 ]

[Helmet]

Zoek eens op superglobals
hint: ($_COOKIE['lidnaam'])

[dArtagnan]

Iets anders:
Je hoeft geen " en " te zetten om $databse (dit geldt ook voor de regel er boven

PHP:

mysql_connect($hostname, $username, $password); 
mysql_select_db($database);

PHP:

$select = "SELECT * FROM leden WHERE lidnaam=".$_COOKIE['lidnaam'];

[ Voor 30% gewijzigd door dArtagnan op 11-10-2003 16:02 ]

[Verwijderd]

Koraalduivel schreef op 11 October 2003 @ 16:00:

PHP:

$select = "SELECT * FROM leden WHERE lidnaam=".$_COOKIE['lidnaam'];

Ik mag aannemen dat je $_COOKIE['lidnaam'] als String wil behandelen en dat je niet zomaar aanneemt dat de waarde van het cookie veilig is, dus:

PHP:

$select = "SELECT * FROM leden WHERE lidnaam = '" . mysql_escape_string($_COOKIE['lidnaam']) . "'";

Daarnaast is het gebruik van * niet erg netjes. Selecteer alleen de velden die je nodig hebt, en als je ze allemaal nodig hebt kun je ze het beste ook gewoon volledig specificeren.

[Elect]

Bedankt voor de reacties alvast, hij is wel beter beveiligd
alleen krijg ik nogsteeds dat ik gewoon kan inloggen op de pagina. Oftewel hij exit nogsteeds niet als ik niet ingelogd ben (en dus geen cookie heb).

[Olaf van der Spek]

Dat komt doordat de while lus niet wordt uitgevoerd.

[ Voor 4% gewijzigd door Olaf van der Spek op 11-10-2003 18:38 ]

[dArtagnan]

Je krijgt toch maar één resultaat (lijkt me wel de bedoeling) dus je kunt net zo goed het volgende doen:

PHP:

$list = mysql_fetch_object($query);
if ( ($list->status == 'Administrator') || ($list->status == 'Moderator') ) { 
    echo ""; 
    } 
else { 
    exit; 
    } 
}

[Suepahfly]

PHP:

<? 
include('config/config.php');

/* de volgende 2 regels zou ik in config.php zetten */
mysql_connect("$hostname","$username","$password");
mysql_select_db("$database");



$lidnaam = $HTTP_COOKIE_VARS["lidnaam"];

           
/* $select is een beetje overbodig */
$query = mysql_query("SELECT [benodigde velden] FROM leden WHERE lidnaam='$lidnaam'") or die("[regel vd query]: ". $mysql_error());
/* 
regel vd query is wel makkelijk als een script hebt dat veel queries uit voert.
Verder moet je alleen de benodige velden in je query zetten aangezien dat een kan schelen in de tijd dat je quey word uitgevoerd

passgecodeerd wordt helemaal niet gebruikt, kun je beter wel doen

mysql_query("SELECT [benodigde velden] FROM leden WHERE lidnaam =' $lidnaam AND pass = 'passgecodeerd'
*/


while ($list = mysql_fetch_object($query)) 
{
    if ( ($list->status == 'Administrator') || ($list->status == 'Moderator') ) 
    {
        echo "";
    }
    else 
    {
        exit;
    }
}
               
?>

Verder mag je wel iets aan je coding style doen.

PHP:

if ()
{
    doe_iets();
     
    if(iets_anders()
    {
         doe_nog_iets();
    }
else
{
    doe_iets_anders();
}

ipv _{extreem voorbeeld}

PHP:

if (){
doe_iets();    
if(iets_anders() {
 doe_nog_iets();
}else{
doe_iets_anders();
}

Het eerste wat ik heb geleerd toen ik ging programeren voor een werkgever.
Het maakt je code een stuk leesbaarder.

Kijk verder eens naar de cooies die je zet.

Kleine tip:
Zet slechts een cookie met alle waarden die nodig hebt.

PHP:

$value = $naam.":".$pass.":".$nog_iets;
setcookie($naam, $value, $expire, $domain, $sucure);



/* om je cookie terug te halen: */

$cookie = $_COOKIE[$naam];
$cookie = expolode(":", $cookie);

/* Nu heb je al alle waarden in een array,
Persoonlijk vind ik dit mooier als een aparte cookie zetten voor elke waarde die nodig hebt */

[PrisonerOfPain]

Suepahfly schreef op 11 October 2003 @ 22:50:

PHP:

if (){
doe_iets();    
if(iets_anders() {
 doe_nog_iets();
}else{
doe_iets_anders();
}

die gaat een parse error geven. je mist namelijk een haakje