[php] Als gebruiker ingelogd verstop en verplaats <LI>

[Zakkenwasser]

Opgelost:
Had mezelf wat meer tijd moeten gunnen om te debuggen, ik raakte te snel in paniek.

[code=php]
if (logged_in()) {
echo "<li>Welkom ";
echo ucfirst($_SESSION['username']);
echo "</li>";
echo "<li id=\"nav-nieuws\"><a href=\"http://www.overdatum.nl/testt/index.php\">Nieuws</a></li>";
echo "<li id=\"nav-plaatjes\"><a href=\"http://www.overdatum.nl/testt/eigen_productie.php\">Eigen productie</a></li>";
echo "<li id=\"nav-filmpjes\"><a href=\"http://www.overdatum.nl/testt/filmpjes.php\">Filmpjes</a></li>";
echo "<li id=\"nav-overons\"><a href=\"http://www.overdatum.nl/testt/over_ons.php\">Over ons</a></li>";
echo "<li id=\"nav-webshop\"><a href=\"http://www.overdatum.nl/testt/webshop/webshop.php\">Webshop</a></li>";
echo "<li id=\"nav-login\"><a href=\"http://www.overdatum.nl/testt/logout.php\">Logout</a></li>";
}
else {
echo "<li id=\"nav-nieuws\"><a href=\"http://www.overdatum.nl/testt/index.php\">Nieuws</a></li>";
echo "<li id=\"nav-plaatjes\"><a href=\"http://www.overdatum.nl/testt/eigen_productie.php\">Eigen productie</a></li>";
echo "<li id=\"nav-filmpjes\"><a href=\"http://www.overdatum.nl/testt/filmpjes.php\">Filmpjes</a></li>";
echo "<li id=\"nav-overons\"><a href=\"http://www.overdatum.nl/testt/over_ons.php\">Over ons</a></li>";
echo "<li id=\"nav-webshop\"><a href=\"http://www.overdatum.nl/testt/webshop/webshop.php\">Webshop</a></li>";
echo "<li id=\"nav-login\"><a href=\"http://www.overdatum.nl/testt/login.php\">Login</a></li>";
echo "<li id=\"nav-aanmelden\"><a href=\"http://www.overdatum.nl/testt/aanmelden.php\">Aanmelden</a></li>";

}
[/code=php]


Het probleem was:
Kort door de bocht:

Als een gebruiker nog niet ingelogd is wil ik dat hij de menu items <li>login</li> en <li>aanmelden</li> ziet

Als de gebruiker al in gelogd is, moet de <li>aanmelden</li> verdwijnen en moet <li>login</li> veranderen in <li>logout</li>


Dit heb ik ervan weten te maken:
[code=php]
$true = true;
$false = false;
if ($_SESSION['username'] == $true) {
echo "<li>Welkom ";
echo ($_SESSION['username']);
echo "<li id=\"nav-login\"><a href=\"http://www.overdatum.nl/testt/logout.php\">Logout</a></li>";
} elseif (!$_SESSION['username'] == $false) {
echo "<li id=\"nav-login\"><a href=\"http://www.overdatum.nl/testt/login.php\">Login</a></li>";
echo "<li id=\"nav-aanmelden\"><a href=\"http://www.overdatum.nl/testt/aanmelden.php\">Aanmelden</a></li>";
} else {
echo "<li id=\"nav-login\"><a href=\"http://www.overdatum.nl/testt/login.php\">Login</a></li>";
echo "<li id=\"nav-aanmelden\"><a href=\"http://www.overdatum.nl/testt/aanmelden.php\">Aanmelden</a></li>";
}
[/code=php]


Dit geeft echter een foutcode aan dat Username nog niet bestaat als in een "undentified index"

en de foutmelding is hetzelfde of er nou is ingelogd of niet.

[ Voor 47% gewijzigd door Zakkenwasser op 22-01-2009 00:10 ]

[MsG]

moet het niet true zijn ipv $true? Lees niet goed .

[ Voor 27% gewijzigd door MsG op 21-01-2009 23:01 ]

[Megamind]

code:

$true = true; 
        $false = false; 
        if ($_SESSION['username'] == $true) {

Dit lijkt me toch wel erg standaard debug werk. Je zegt dat username niet bestaat, maar hoe kunnen wij nou weten hoe deze geset wordt?

Zowiezo zie ik een missende </li> en zit er volgens mij een " te veel.

[ Voor 12% gewijzigd door Megamind op 21-01-2009 23:02 ]

[sanzut]

session_start()?
verder denk ik dat je niet op tweakers.net moet kijken maar op php.net en sites als phpfreakz.nl of wmcity.nl

PHP:

<?php
session_start();
if($_SESSION['username']) echo("");
else echo("");
?>

redelijk triviaal? De rest moet je maar op bovengenoemde sites leren

[ Voor 41% gewijzigd door sanzut op 21-01-2009 23:07 ]

[--MeAngry--]

Eh, kijk alleen al eens naar de syntax highlighting hier binnen GoT en zie dat je gewoon een syntax fout maakt.
Neem daarnaast een fatsoenlijke PHP-editor, (liefst IDE) dan had je dit meteen gezien.

[edit]
En toen was de syntax-fout ineens weg...

En WTH idd

PHP:

<?php
        $true = true;
        $false = false; 
?>

Ik mag hopen dat dat een grapje is?

[ Voor 32% gewijzigd door --MeAngry-- op 21-01-2009 23:04 ]

[ofniedan]

Je moet zowiezo niet

PHP:

if ($_SESSION['username'] == $true)

gebruiken, wat trouwens precies t zelfde is als

PHP:

if ($_SESSION['username'])

maar je wilt kijken of er een waarde in $_SESSION['username'] staat (en eigenlijk ook nog checken of deze waarde valid is, door in de db te kijken of deze username ook bestaat als minimale controle, maar dat moet uitgebreider hoor!).

probeer het eens met

PHP:

if (!empty($_SESSION['username']))

trouwens, als je wilt checken op een true waarde, dan heb je 3x een = nodig ipv 2!

[BBrunekreeft]

MrJey schreef op woensdag 21 januari 2009 @ 22:59:

Dit geeft echter een foutcode aan dat Username nog niet bestaat als in een "undentified index"
en de foutmelding is hetzelfde of er nou is ingelogd of niet.

Je eerste controle zou moeten zijn of $_SESSION['username'] wel bestaat:
[code=php]
if (isset($_SESSION['username'])) {
[/code=php]

Overigens begrijp ik de bedoeling van regel 1 en 2 niet zo.
Bovendien zal de logout link alleen verschijnen bij de persoon die de username "1" (denk ik) heeft

[sanzut]

BBrunekreeft schreef op woensdag 21 januari 2009 @ 23:06:
[...]


Je eerste controle zou moeten zijn of $_SESSION['username'] wel bestaat:
[code=php]
if (isset($_SESSION['username'])) {
[/code=php]

Overigens begrijp ik de bedoeling van regel 1 en 2 niet zo.
Bovendien zal de logout link alleen verschijnen bij de persoon die de username "1" (denk ik) heeft

Nou.. dat hangt er vanaf hoe die wordt beschreven, maar gezien de kwaliteit van bovenstaande code denk ik dat het met iedere username wel werkt:P

[Zakkenwasser]

sanzut schreef op woensdag 21 januari 2009 @ 23:08:
[...]

Nou.. dat hangt er vanaf hoe die wordt beschreven, maar gezien de kwaliteit van bovenstaande code denk ik dat het met iedere username wel werkt:P

Is dat positief ?

[MueR]

Je hebt nog veel te leren. Volgens mij is je dat al vaker verteld, maar weiger je te luisteren.

[Verwijderd]

MueR schreef op woensdag 21 januari 2009 @ 23:20:
Je hebt nog veel te leren.

Ik vind het flauw om zoiets te zeggen tegen iemand die er geen talent voor heeft.

[ Voor 13% gewijzigd door Verwijderd op 21-01-2009 23:24 ]

[Zakkenwasser]

Verwijderd schreef op woensdag 21 januari 2009 @ 23:24:
[...]

Ik vind het flauw om zoiets te zeggen tegen iemand die er geen talent voor heeft.

vergat de sessie.php te includen op elke pagina, vandaar dus.
toch bedankt voor het reageren!

Weinig tijd en wil veel leren.

Ik ben wat meer gewend op tweakers dan op andere sites, kwestie van gewenning waarschijnlijk.

[ Voor 35% gewijzigd door Zakkenwasser op 21-01-2009 23:39 ]

[TeeDee]

Volgens mij is

PHP:

!$_SESSION['username'] == false

hetzelfde als

PHP:

$_SESSION['username'] == true

en hetzelfde als

PHP:

if($_SESSION['username'])

Ik denk dat je in ieder geval moet kijken of je session niet null is.

O en MueR, MrJey kan misschien wel luisteren, en dat doet hij misschien ook wel, maar dan moet je verder op de bijdrage van Cheatah. Het zou zomaar kunnen zijn dat dat niks met weigering te maken heeft. Leren mag, weet je nog. Of wist jij meteen alles?

Of bovenstaande in PHP opgaat laat ik even buiten beschouwing

[ Voor 7% gewijzigd door TeeDee op 21-01-2009 23:40 ]

[Patriot]

Tsja, jokes aside kom je hier met problemen die van wel héél laag niveau zijn. Dat maakt niet uit, maar het lijkt er niet veel beter op te worden.

Om op je probleem terug te komen: Kijk nou eens per regel naar je code en vraag jezelf dan af wat je wilt doen. Stel nu dat je dit stukje:

PHP:

if ($_SESSION['username'] == $true) {

neemt. Zeg nou eens hardop wat daar precies gaat gebeuren. Dan moet vrij snel duidelijk worden wat er misgaat. Als dat niet zo is, dan moet je toch echt terug de boeken in (of een goede tutorial zoeken).

TeeDee schreef op woensdag 21 januari 2009 @ 23:35:
O en MueR, MrJey kan misschien wel luisteren, en dat doet hij misschien ook wel, maar dan moet je verder op de bijdrage van Cheatah. Het zou zomaar kunnen zijn dat dat niks met weigering te maken heeft. Leren mag, weet je nog. Of wist jij meteen alles?

MueR's punt is juist dat de problemen van MrJey in al zijn topics eigenlijk vergelijkbaar zijn. Hij klopt wat code maar lijkt zelf niet te begrijpen wat het doet. Dat is enorm lastig, en zolang MrJey niet verder leert (misschien probeert hij dat wel maar lukt dat niet) komen er keer op keer dit soort topics. Dat hoeft niet erg te zijn, maar persoonlijk vraag ik me af of GoT daar een geschikte plek voor is. Ik denk dat een gemiddelde webmastercommunity daar geschikter voor is.

NOFI overigens, ik ben ook zo begonnen. Ik ben alleen niet zo op GoT begonnen.

[ Voor 45% gewijzigd door Patriot op 21-01-2009 23:46 ]

[Zakkenwasser]

Patriot schreef op woensdag 21 januari 2009 @ 23:38:
Tsja, jokes aside kom je hier met problemen die van wel héél laag niveau zijn. Dat maakt niet uit, maar het lijkt er niet veel beter op te worden.

Om op je probleem terug te komen: Kijk nou eens per regel naar je code en vraag jezelf dan af wat je wilt doen. Stel nu dat je dit stukje:

PHP:

if ($_SESSION['username'] == $true) {

neemt. Zeg nou eens hardop wat daar precies gaat gebeuren. Dan moet vrij snel duidelijk worden wat er misgaat. Als dat niet zo is, dan moet je toch echt terug de boeken in (of een goede tutorial zoeken).

een naam kan geen 1 of 2 zijn

dan zou ["id"] beter wezen

by the way:

MrJey schreef op woensdag 21 januari 2009 @ 23:25:
[...]

vergat de sessie.php te require_once op elke pagina, vandaar dus.
toch bedankt voor het reageren!

[ Voor 13% gewijzigd door Zakkenwasser op 21-01-2009 23:43 ]

[TeeDee]

MrJey schreef op woensdag 21 januari 2009 @ 23:41:
[...]
een naam kan geen 1 of 2 zijn

dan zou ["id"] beter wezen

Wat heeft dat met wat operator geneuzel te maken? Waar haal je ["id"] nou weer vandaan. Als je 1 of 2 als true of false wil definieren is dat niet logisch en gebruikelijk.

by the way:

Dat stond er eerst niet he, vandaar de extra reacties! Trouwens, als dat de oplossing is voor je operator probleem kan ik helaas haast niet wachten op je volgende topic.

Patriot schreef op woensdag 21 januari 2009 @ 23:38:
MueR's punt is juist dat de problemen van MrJey in al zijn topics eigenlijk vergelijkbaar zijn. Hij klopt wat code maar lijkt zelf niet te begrijpen wat het doet. Dat is enorm lastig, en zolang MrJey niet verder leert (misschien probeert hij dat wel maar lukt dat niet) komen er keer op keer dit soort topics. Dat hoeft niet erg te zijn, maar persoonlijk vraag ik me af of GoT daar een geschikte plek voor is. Ik denk dat een gemiddelde webmastercommunity daar geschikter voor is.

Daar ben ik het ook mee eens, alleen de aanname van MueR over de weigering gaat bij mij een beetje scheef. En de history van TS ben ik ook mee bekend, maar dat laat ik in deze context stug buiten beschouwing.

[ Voor 52% gewijzigd door TeeDee op 21-01-2009 23:50 ]

[Patriot]

MrJey schreef op woensdag 21 januari 2009 @ 23:41:
[...]

een naam kan geen 1 of 2 zijn

dan zou ["id"] beter wezen

Zeg nu eens precies wat jij denkt dat die code doet. Want ook uit deze reactie blijkt weer dat je niet goed lijkt te begrijpen wat er precies gebeurt.

[Zakkenwasser]

Weetje, het werkt, ik verdiep me erin met alle vrije tijd die ik op dit moment heb.

Mijn welgemeende excuses voor de mensen waarvan ik het bloed heb laten koken, het was niet mijn bedoeling
Ik vertrouw de mensen hier toch meer dan op enig ander forum.

Ik ben nog lang niet perfect, maar iedereen moet het een keer leren niet?
Toch blijf ik gemotiveerd om beetje bij beetje met de kleine tijd die ik op een dag heb mijn php kennis te verbreden
en soms kom ik er niet uit.

Again, sorry dat ik wat mensen hier geirriteerd heb.

[ Voor 15% gewijzigd door Zakkenwasser op 21-01-2009 23:56 ]

[CodeCaster]

Ach, ik wil niks roepen naar de TS hoor, maar topics van deze kwaliteit komen erg vaak langs 'hier'.

Wat wil je nou, TS, als je deze code aan PHP voert:

code:

$true = true;
        $false = false;
            if ($_SESSION['username'] == $true) {

Lees je code eens door, kijk eens goed wat je aan het doen bent... ben je code aan het misbruiken om het aan jouw ideeën te laten voldoen, of ben je aan het programmeren om jouw ideeën om te zetten in code?

"Als de naam van de ingelogde gebruiker gelijk is aan 'true'"... dit kan je toch niet serieus in je hoofd hebben gehad toen je deze code schreef?

---

In ieder geval hier een gratis lesje logisch nadenken. Jij wil: "Als een gebruiker nog niet ingelogd is" <-- stop hier. Hoe hou jij bij dat een gebruiker wel ingelogd is? Is dan $_SESSION['username'] geset? Mooi, dan heb je een basis in psuedocode:

code:

if (session_username != null)
{
  print "Je bent ingelogd! Klik hier om uit te loggen";
}
else
{
  print "Klik hier om in te loggen";
}

Hoe je dat in php verder regelt is aan jou, of dat een isset($_SESSION['username']) is moet je zelf weten, maar met constructies als $true = true ga je veel, veel te ver.

[ Voor 32% gewijzigd door CodeCaster op 22-01-2009 00:07 ]

[MsG]

Sowieso snap ik niet waarom je weer met een omweg werkt via $true en $false. Ik ben ook nog nieuw in PHP, maar die manier ben ik nog nooit tegengekomen. Wat is het voordeel hier aan TS?

[Zakkenwasser]

CodeCaster schreef op donderdag 22 januari 2009 @ 00:03:
Ach, ik wil niks roepen naar de TS hoor, maar topics van deze kwaliteit komen erg vaak langs 'hier'.

Wat wil je nou, TS, als je deze code aan PHP voert:

code:

$true = true;
        $false = false;
            if ($_SESSION['username'] == $true) {

Lees je code eens door, kijk eens goed wat je aan het doen bent... ben je code aan het misbruiken om het aan jouw ideeën te laten voldoen, of ben je aan het programmeren om jouw ideeën om te zetten in code?

"Als de naam van de ingelogde gebruiker gelijk is aan 'true'"... dit kan je toch niet serieus in je hoofd hebben gehad toen je deze code schreef?

Dat had ik niet in mijn hoofd.

Ik had mezelf wat meer tijd moeten gunnen voor debuggen en niet zo snel in paniek raken waardoor ik dit soort laag drempelige topics plaats.

[code=php]
<ul id="nav">
<?php
if (logged_in()) {
echo "<li>Welkom ";
echo ucfirst($_SESSION['username']);
echo "</li>";
}
?>
<li id="nav-nieuws"><a href="http://www.overdatum.nl/testt/index.php">Nieuws</a></li>
<li id="nav-plaatjes"><a href="http://www.overdatum.nl/testt/eigen_productie.php">Eigen productie</a></li>
<li id="nav-filmpjes"><a href="http://www.overdatum.nl/testt/filmpjes.php">Filmpjes</a></li>
<li id="nav-overons"><a href="http://www.overdatum.nl/testt/over_ons.php">Over ons</a></li>
<li id="nav-webshop"><a href="http://www.overdatum.nl/testt/webshop/webshop.php">Webshop</a></li>
<?php
if (logged_in()) {
echo "<li id=\"nav-login\"><a href=\"http://www.overdatum.nl/testt/logout.php\">Logout</a></li>";
}
else {
//
echo "<li id=\"nav-login\"><a href=\"http://www.overdatum.nl/testt/login.php\">Login</a></li>";
echo "<li id=\"nav-aanmelden\"><a href=\"http://www.overdatum.nl/testt/aanmelden.php\">Aanmelden</a></li>";

}

?>
</ul>
[/code=php]
op deze manier werkt het perfect

nogmaals excuses, ik weet dat het irritant is.

[ Voor 14% gewijzigd door Zakkenwasser op 22-01-2009 00:18 ]

[MueR]

Waaarom print je dat hele menu 2 keer? Is enkel een if ($loggedIn) op Welkom en Aanmelden niet voldoende?

[Patriot]

MsG schreef op donderdag 22 januari 2009 @ 00:06:
Sowieso snap ik niet waarom je weer met een omweg werkt via $true en $false. Ik ben ook nog nieuw in PHP, maar die manier ben ik nog nooit tegengekomen. Wat is het voordeel hier aan TS?

Dat heeft geen voordeel. Dat is juist totaal onnodig.

EDIT:

Ach, ik wil niks roepen naar de TS hoor, maar topics van deze kwaliteit komen erg vaak langs 'hier'.

True, maar bij iemand met 5k posts mag het wel gezegd worden, toch?

EDIT2:
@hieronder: lol

Ik vroeg me al af waarom mijn systemen gisteren andersom werkten.. als ik toch wist dat de oplossing zo simpel was!

[ Voor 33% gewijzigd door Patriot op 22-01-2009 01:24 ]

[Verwijderd]

Patriot schreef op donderdag 22 januari 2009 @ 00:39:
[...]
Dat heeft geen voordeel. Dat is juist totaal onnodig.

Dat is juist super handig als je server in the flipside staat, even de waarde van true en false omkeren en alles is weer normaal.

Ik programmeer altijd al mijn code zo, vooral omdat ik zelf ook regelmatig naar de flipside ga.

[MueR]

MrJey schreef op donderdag 22 januari 2009 @ 00:08:
Ik had mezelf wat meer tijd moeten gunnen voor debuggen en niet zo snel in paniek raken waardoor ik dit soort laag drempelige topics plaats.

Wat nou debuggen. Je moet eerst een code gaan schrijven, dan pas valt er iets te debuggen. Als je bij elke scheet hier een topic opent terwijl je de basisbeginselen van het programmeren niet beheerst. Zowel klok als klepel zijn nog erg ver weg. Ga deze tutorial eens lezen, daarna hier eens goed kijken. Daarna een keer een boek gaan lenen (heb ik je ook pas drie keer gezegd) en dat doorlezen. Daarna lief beginnen met Hello World niveau spul. Maar ach, waarom neem ik nog de moeite voor zo'n HK meubel. Misschien heeft Cheatah gelijk.

[ Voor 0% gewijzigd door MueR op 22-01-2009 08:57 . Reden: Niet replyen voor de koffie, dan maak je tikfouten ]

[Creepy]

Ik had mezelf wat meer tijd moeten gunnen voor debuggen en niet zo snel in paniek raken waardoor ik dit soort laag drempelige topics plaats.

Leer hier aub van.........