Einfaches script - Seite 3

Benjamin · 15.11.2006, 22:39

Zitat:

Zitat von Der_Gerhard

Weil Du zwei unterschiedliche Dinge damit machst. Oder eher machen sollst.

öhm das versteh ich immo nicht ich mein ich kann doch einfach schreiben if(isset($_GET['action']=="setup_profile")) das reicht doch ich mein das eine $_GET['action'] ist da ja völlig unnötig oder was meinst du genau?

// Edit: Achso jetzt weiß ich was du meinst.. meintest du etwa das man einmal $_GET['action'] für setup_profile brauche und einmal $_GET['action'] für den rest des scriptes?^^

Der_Gerhard · 15.11.2006, 22:44

Zitat:

Zitat von iNTENSiON

Zitat:

Zitat von Der_Gerhard

Weil Du zwei unterschiedliche Dinge damit machst. Oder eher machen sollst.

öhm das versteh ich immo nicht ich mein ich kann doch einfach schreiben if(isset($_GET['action']=="setup_profile")) das reicht doch ich mein das eine $_GET['action'] ist da ja völlig unnötig oder was meinst du genau?

Du solltest mal kurz überlegen, was da eigentlich gemacht wird.
Dein Konstrukt ist Schrott. Sorry.

isset prüft, ob eine Variable definiert ist.
Und ein Vergleich ist nunmal keine Variable. Da dürfte Dir auch der Parser vor Aufregung umkippen.

Du sollst also zuerst prüfen, ob die Variable definiert ist und nur dann den Inhalt vergleichen.
Andernfalls vergleichst Du eine undefinierte Variable mit einem Wert. Das wirft Dir eine entsprechende Notice raus.

Generell gilt: Was nicht existiert, darf nicht verwendet werden. Ganz wie im richtigen Leben.

GelaMu · 15.11.2006, 23:06

Net, eben nicht, iNTENSiON

Hast Du if(isset($_GET['action']=="setup_profile")) überhaupt ausprobiert?
So spuckt mir mein PHP diese Fehlermeldung aus: PHP Parse error: syntax error, unexpected T_IS_EQUAL, expecting ',' or ')'

Wenn Du if(isset($_GET['action']) AND $_GET['action'] == "setup_profile") tut Dein PHP zuerst überprüfen ob action in $_GET überhaupt vorhanden ist, und erst dann wird Dein $_GET['action'] auf "setup_profile" verglichen.

Wenn Du $_GET['action'] nur auf EINE Übereinstimmung überprüfen möchtest, ist es glaube ich kein Vorteil zuerst die Funktion isset() zu benutzen.

Benjamin · 15.11.2006, 23:14

soo alles geht danke

also ich hab bei $action nun $action=$_GET['action'];
und bei "if" hab ich: if($action=="setup_profile") { ... } und alles funzt :P

hab jetzt noch

PHP-Code:

  $query=mysql_query("SELECT * FROM `users` WHERE id='".$_SESSION['id']."'");

$data=mysql_fetch_array($query);

 
if($data) {

$username=$data['username'];

$firstname=$data['firstname'];

$lastname=$data['lastname'];

$msn=$data['msn'];

$icq=$data['icq'];

$birthday=$data['birthday'];

$email=$data['email'];

$id=$data['id'];

}

so das was in der datenbank steht vorher ausgegeben wird das man auch sieht was da im moment steht

danke euch und der_gerhard ich werds mir merken

nikosch · 15.11.2006, 23:23

du solltest es dir nicht nur merken, sondern auch gleich anwenden. dein konstrukt verschiebt nur das problem:
$action=$_GET['action'];
ist $_GET['action'] nicht definiert bekommst du dann doch ne notice.

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Ganz einfaches Script nur wo ist der Fehler?	aerow90	PHP Tipps 2008	10	20.07.2008 23:18
Script beendet sich ohne Fehlermeldung	Zahl	PHP Tipps 2006	6	04.06.2006 20:41
[Erledigt] Verzögerung durch included Script? Umgehen?		PHP Tipps 2005-2	4	10.07.2005 19:26
[Erledigt] Relay Script		PHP-Fortgeschrittene	11	01.06.2005 16:02
[Erledigt] Hilfe bei Download Verwaltungs Script		PHP Tipps 2005	8	04.05.2005 10:29
Einfaches Script für Datenbankfeld-Manipulierung gesucht	robo47	PHP Tipps 2004-2	8	27.12.2004 14:56
"das" Script oder "der" Script??	Simbo	Off-Topic Diskussionen	31	16.09.2004 17:04
[Erledigt] Brauch ein einfaches Formmail script		PHP Tipps 2004	2	18.08.2004 20:40
[Erledigt] suche gutes einfaches script zum email mit attachment versen		PHP Tipps 2004	2	05.08.2004 19:05


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

15.11.2006, 23:06
GelaMu Erfahrener Benutzer Registriert seit: 05.03.2006 Beiträge: 183	Net, eben nicht, iNTENSiON Hast Du if(isset($_GET['action']=="setup_profile")) überhaupt ausprobiert? So spuckt mir mein PHP diese Fehlermeldung aus: PHP Parse error: syntax error, unexpected T_IS_EQUAL, expecting ',' or ')' Wenn Du if(isset($_GET['action']) AND $_GET['action'] == "setup_profile") tut Dein PHP zuerst überprüfen ob action in $_GET überhaupt vorhanden ist, und erst dann wird Dein $_GET['action'] auf "setup_profile" verglichen. Wenn Du $_GET['action'] nur auf EINE Übereinstimmung überprüfen möchtest, ist es glaube ich kein Vorteil zuerst die Funktion isset() zu benutzen.

15.11.2006, 23:14
Benjamin Benutzer Registriert seit: 08.10.2007 Beiträge: 33	soo alles geht danke also ich hab bei $action nun $action=$_GET['action']; und bei "if" hab ich: if($action=="setup_profile") { ... } und alles funzt :P hab jetzt noch PHP-Code: $query=mysql_query("SELECT * FROM `users` WHERE id='".$_SESSION['id']."'"); $data=mysql_fetch_array($query); if($data) { $username=$data['username']; $firstname=$data['firstname']; $lastname=$data['lastname']; $msn=$data['msn']; $icq=$data['icq']; $birthday=$data['birthday']; $email=$data['email']; $id=$data['id']; } so das was in der datenbank steht vorher ausgegeben wird das man auch sieht was da im moment steht danke euch und der_gerhard ich werds mir merken

15.11.2006, 23:23
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.987 PHP-Kenntnisse: Fortgeschritten	du solltest es dir nicht nur merken, sondern auch gleich anwenden. dein konstrukt verschiebt nur das problem: $action=$_GET['action']; ist $_GET['action'] nicht definiert bekommst du dann doch ne notice.