[Jake]

Hi Leute!

Erstmal vielen Dank an die vielen Freaks, die sich die Mühe machen uns Absolute Beginners weiterzuhelfen!

Auch ich hab mich auf Bens Login-Tut gestürzt und die ersten Hürden (das übliche "Header already sent"-Problem) sind übersprungen.
Jetzt hab ich aber ein Prob, das (laut Forensuche) vor mir offensichtlich noch keiner hatte: Wenn die "validateLogin.inc.php" geladen wird, bekomme ich einen Parse Error!
Ich hab die Datei genau wie im Tut aufgebaut (nur mit meinen Variablen für Namen und Passwort

PHP-Code:

session_start();
if (isset($_POST['name']), ($_POST['pwd']) 


"Vielen Freaks" *g? Nun gut

Also wenn du eigene Variabeln benutzt, kann es unter Umständen sein,
dass du in einer Datei nicht konsequent über die POST-Variabeln auf
deine umgeschrieben hast, sodass da Unstimmigkeiten vorfallen.

[Niedi]

du hast eine klammer zu wenig beim if

mfg niedi

[MrNiceGuy]

Der Fehler ist nicht die fehlende Klammer, sondern die falsche Syntax-Logik.

Du versuchst durch ein Komma zu sagen, dass das isset () auf beide Variablen anzuwenden ist, das ist aber so nicht möglich. Richtig wäre:

PHP-Code:

session_start ();
if (isset ($_POST['name']) === TRUE
&& isset ($_POST['pwd']) === TRUE)
{
  /* Ja, beide sind gesetzt */
} 


Du musst isset für jede Variable einzeln angeben und die beiden Bedingungen mit einem logischen AND verknüpfen (das ist das '&&').

Eine ander Alternative soll es zwar noch geben, die finde ich aber eher unpraktisch und von der Syntax her unlogisch, deswegen werde ich hier auch nicht näher darauf eingehen.

http://www.php.net/isset

[Jake]

Sorry Leute! War noch nicht mit Tippen fertig, als mir der Fehler aufgefallen ist, hab dann einfach das Fenster mit dem Post geschlossen. Warum das trotzdem hier eingetragen wurde... k.a. Trotzdem danke für die Mühe!
Der Fehler war, wie MrNiceGuy geschrieben hat, ein Klammernproblem. So ists auch richtig:

PHP-Code:

if (isset($_POST['name'], $_POST['pwd']) {
...
} 


Wenn man nicht richtig abtippen kann...

[Niedi]

aehm

PHP-Code:

if (isset($_POST['name'], $_POST['pwd'])) 


die findest du unlogisch? oder was meintest du?

[MrNiceGuy]

Zitat von Niedi

aehm

PHP-Code:

if (isset($_POST['name'], $_POST['pwd'])) 


die findest du unlogisch? oder was meintest du?

Japp, ich finde, dass diese Syntax nicht sauber oder in irgendeiner Weise logisch ist. Ich setze isset für jede Variable einzeln mit && bzw. || dann ist gleich klar, wa sich meine und ich brauche nicht einmal isset mit 3 werten aufrufen und einmal mit einem, weil ich ein || einbauen muss oder dergleichen, es ist in meinen Augen einfach eindeutiger, als dieser eine Befehl.

EDIT: Außerdem auch wesentlich einfacher zu erweitern, sollte man eine IF-Anweisung mal etwas ausdehnen müssen.

[Niedi]

Außerdem auch wesentlich einfacher zu erweitern

du findest deine gepostet lösung einfacher zu erweitern als meine? hrm muss ich das verstehen?

bei meiner muss ich nur ne variable reinhängen. bei deiner musst du das ganze stück mit isset(...) dranhängen + nen vergleichsoperator.. ich finde die kurze form wesentlich übersichtlicher, einfacher und auch platzsparender

najo jeder hat seine richtung

mfg niedi

[MrNiceGuy]

Mit einfacher zu erweitern gehe ich von komplexen Strukturen durch mehrere && und || aus, die es einem erleichtern die ZUsammenhänge der einzelnen Parts zu verstehen, als wenn man die && durch deine Schreibweise zusammenfassen würde. Ich mache einfach C&P und änder dann die Variable, geht bestimmt genauso fix wie wenn du einfach nur ne Variable anhängst, ist alles ne Sache der Übung.

Letztlich ist die Schreibweise mit den mehreren Parametern für isset () eher irreführend, da nicht auf Anhieb erkannt werden kann, was die Funktion eigentlich macht, sofern die Aufgabe der Funktion bei mehreren Parametern unbekannt ist. Man müsste extra wieder nen Blick in das Manual machen. Wenn sowas vermeidbar ist, ist es für mich saubere Programmierung. Aber jeder hat da sicher auch seine eigene kleine Welt. Ich setze auch Backticks in SQL-Commands.