[php] Sessie is weg na veranderen pagina

[Anoniem: 281519]

Hallo,

Ik een tijdje geleden een inlogscript geschreven. Van de week ben ik weer eens begonnen met wat te spelen met php. Nu heb ik dit inlogscript bijna letterlijk gekopieerd, het enige verschil is dat ik nu gebruik (wil) maken van classes en functies. Nu is echter het probleem, ik kan inloggen, maar wanneer ik naar een andere pagina ga blijkt de sessie weer leeg te zijn. Ik heb het nu al een paar keer na gelopen en ik snap eigenlijk niet wat ik fout doe. Vergeleken met het vorige script en weinig verschil te vinden. Ziet iemand wat ik fout doe?

Hier de relevante code:

Functies:

PHP:

            function checkLogin(){
                session_start();
                if(isset($_SESSION['id'])){
                    return true;
                }else {
                    return false;
                }
            }


    function login($user, $password){
            
        $this->connect();
        if(isset($_SESSION['id'])){
            return false;
        }else {

            $user = mysql_real_escape_string($user);
            $pass = mysql_real_escape_string(md5($password));

            $sql = ("SELECT * FROM users WHERE username='" . $user . "' AND pass='" . $pass . "'");
            $uitvoer = mysql_query($sql);
            $controle = mysql_num_rows($uitvoer);

            if ($controle != 0) { //kijk of er een match is gevonden


                while ($record = mysql_fetch_object($uitvoer)) { //haal gegevens uit de DB en stop ze in vars
                    $id = $record->id;
                    $naam = $record->username;
                }

            session_start(); //check. gegevens klopppen. maak sessie aan met de gegevens
                $_SESSION['id'] = $id;
                $_SESSION['naam'] = $naam;
                return true;

            } else {
                return false;
                    
            }}}

Code waarmee ik inlog:

PHP:

if(isset($_POST['Submit']) && $func->checkLogin()==false) {
    if(empty($_POST['username']) || empty($_POST['password'])) {
        $error = "niet alle velden zijn ingevuld!";
    } else {
        $func->login($_POST['username'],$_POST['password']);
        if( $func->checkLogin() == false) {
            $error = "Foutieve loggingegevens!";
        }
    }

}

Het laatste stukje php staat in een andere file, genaamd index.php, en bovenaan heb ik de class file geinclude en de variable $func aan toegekend.

Alvast bedankt!

[disjfa]

Zet je

PHP:

session_start();

eens en maar 1 keer bovenaan in je bestanden. Dus als eerste na de <?php

[Anoniem: 26306]

disjfa schreef op dinsdag 13 september 2011 @ 21:51:
Zet je

PHP:

session_start();

eens en maar 1 keer bovenaan in je bestanden. Dus als eerste na de <?php

Gewoon ergens in je code maar voordat je $_SESSION gebruikt en voordat je output genereert is voldoende.

Voor de rest is dit natuurlijk zo klaar als een klontje. TS roept nergens session_start aan als het géén POST request met het veld 'Submit' betreft, en dan heb je dus inderdaad geen beschikking over de $_SESSION variabelen.

[Anoniem: 281519]

Met andere woorden, ik moet de session_start() aanroepen voordat ik ook maar iets met variabelen ga doen. Dat is logisch, maar alsnog krijg ik het niet werkend, terwijl ik nu in de index en in de class bovenaan begin met session_start()

wel krijg ik nu de melding:
Notice: A session had already been started - ignoring session_start() in
Dit is vrij logisch, aangezien ik twee keer de sessie start, 1 keer in de index.php en daarna include ik meteen de class waarin ook een session_start wordt aangeroepen.

Ik heb in de tussentijd nog eens nagelopen wat er fout kan gaan en wanneer ik eenmaal "inlog" en de $_SESSION['naam'] (die tijdens het inloggen wordt toegekend!) probeer te lezen krijg ik de volgende melding:

Notice: Undefined index: naam in ...

Wat ik hier zelf uit op maak: De sessie is wel gestart, maar de variabelen zijn niet gevuld. Ik weet niet zeker of dat het is maar als dat zo is lijkt me dat vreemd aangezien ik tijdens het inloggen dit doe:

PHP:

$_SESSION['id'] = $id;
$_SESSION['naam'] = $naam;

[Beatboxx]

Verlooptijd?

[Ventieldopje]

Tip: om je fout te vinden probeer je situatie zo klein mogelijk te maken. Probeer dus te constateren of a. de sessie wel gestart wordt, b. je wat naar de sessie kan schrijven (geen login, gewoon heel simpel) en of het er dan ook in blijft staan.

Hier meer info over php sessies, wellicht nuttig http://nl.php.net/manual/en/session.examples.basic.php

[ Voor 19% gewijzigd door Ventieldopje op 13-09-2011 22:14 ]

[Anoniem: 281519]

Verlooptijd lijkt mij niet.

Bedankt Ventieldopje, ga ik proberen!

[ameesters]

PHP:

<?php
session_start();

if(!isset($_SESSION['count']))
    $_SESSION['count'] = 0;
else
    $_SESSION['count'] + 1;

echo $_SESSION['count'];

Als hij optelt weet je dat session werken, een klein test scriptje als het waren.

daarnaast kan je ook kijken of een sessie al gestart is, en zo niet starten:

PHP:

<?php
if(!isset($_SESSION)) session_start();
?>

Meschien dat deze voorbeeldjes je een gooi in de goede richting geeft

[Ventieldopje]

ameesters schreef op dinsdag 13 september 2011 @ 22:57:

PHP:

<?php
session_start();

if(!isset($_SESSION['count']))
    $_SESSION['count'] = 0;
else
    $_SESSION['count'] + 1;

echo $_SESSION['count'];

Als hij optelt weet je dat session werken, een klein test scriptje als het waren.

daarnaast kan je ook kijken of een sessie al gestart is, en zo niet starten:

PHP:

<?php
if(!isset($_SESSION)) session_start();
?>

Meschien dat deze voorbeeldjes je een gooi in de goede richting geeft

Wat betreft je eerste code is het altijd nog

PHP:

<?php
$_SESSION['count'] += 1;

Wat betreft je 2e code, dit lijkt me niet eens nodig en kan alleen maar bugs opleveren (uitroepteken vergeten bijv.). Beter is om gewoon altijd session_start(); aan te roepen bovenaan de pagina aangezien een sessie altijd maar één keer gestart kan worden zal de 2e session_start() (bijv. in een include) genegeert worden en een notice genereren Tenzij je dus session_name gebruikt

@Fulairyn
Zoals ameesters helaas al heeft voor gedaan kun je mooi testen of sessies werken maar van zelf doen leer je veel meer imo

[Anoniem: 281519]

Dank jullie allen,

Ik heb besloten dat ik m'n code helemaal ga nalopen en waar nodig aanpassen.
Ook de functies herschrijven en dergelijke. Dit omdat het op sommige plekken gewoon een rommel is en omdat ik nu ook meer weet wat ik doe. Dit aangezien het om niet al te veel code gaat en ik de fout niet kan vinden (misschien niet hard genoeg gezocht, misschien niet goed gezocht).

Ook snap ik nu beter waar ik mee bezig ben en heb ik me meer in sessie's verdiept, dus zoveel werk zal het niet zijn en het resultaat zal alleen maar beter zijn.

[Ventieldopje]

Heb je code niet gezien verder maar het lijkt me een goed plan, ik leer ook het meeste terwijl ik bezig ben met een project en pak ik het meestal bij een nieuw project beter aan dan de vorige keer