Hallo, ich habe ein kleines PHP Script für eine simple Web Gallery geschrieben.

Sourcecode einsehbar unter: http://echizen.ec.funpic.de/


Da ich zuvor noch nicht unter PHP programmiert habe, habe ich ein paar Fragen zum Variablenzugriff. Beim Script werden einige Variablen beim Aufruf mitübergeben und dann weiterverwendet:

gallery.php?show=index&event=Example Gallery


Was mir ein wenig Bauchschmerzen bereitet sind Stellen wie diese hier:

if(($show == 'index') && (file_exists($event)))

funktioniert genau so wie auch

if(($show == 'index') && (file_exists("$event")))

Welche Variante wäre die sauberste? $event wäre in diesem Fall Example Gallery. Müsste die Variable demnach nicht wenigstens in "" stehen?


Außerdem wäre da noch die Sache mit register_globals on/off. Um das Problem zu umgehen habe ich derzeit extract($_REQUEST); drin. Aber das soll wohl auch nicht die ordentlichste Methode sein.

Gerne nehme ich auch andere Verbesserungsvorschläge entgegen.

Mister Ad

Buhmann

if(($_GET['show'] == 'index') && (file_exists($_GET['event'])))

ist immernoch die sauberste methode

robo47

krass ein mensch der es freiwillig ohne register globals = on amchen will :P

http://www.phpfriend.de/ftopic26569.html

if(($show == 'index') && (file_exists("$event")))

wozu die " sie sind einfach nur unnötig, da du ja direkt auf eine variable zugreifst, nicht auf einen string mit ner variable drin.

und nach möglichkeit variablen die via POST oder GET übergeben werden immer erst mit isset() prüfen, nicht so sachen machen if ($var = 'asdf') oder if ($var).

am besten am anfang deines scriptes error_reporting(E_ALL); schreiben dann siehst du schon was unsauber programmiert ist.

mfg
robo47

__________________
robo47.net - Blog, Codeschnipsel und mehr
| Caching-Klassen und Opcode Caches in php | Robo47 Components - PHP Library extending Zend Framework

Danke für die schnellen Antworten.

Eine Frage habe ich noch. Warum sollte man unbedingt mit isset vorher prüfen?


reicht nicht:

if(

($_GET['show'] == 'index')

&&

(file_exists($_GET['event']))

)


anstatt:

if(

(isset($_GET['show']) && $_GET['show'] == 'index')

&&

(isset($_GET['event']) && file_exists($_GET['event']))

)

RudiS

Nein, da PHP meckern wird, wenn $_GET['show'] nicht vorhanden ist. Immer erst prüfen ob eine Variable vorhanden ist und dann damit weiterarbeiten.
mfg RudiS

__________________
Kunst kommt von Können und nicht von wollen, denn sonst würde es ja Wunst heißen.

Waq

Der Beitrag wurde verschoben, wegen...
... Postings im falschen Forum.

Bemerkung:
Die gestellte Frage entspricht nicht dem Wissensstand eines/einer Fortgeschrittenen.

moved to PHP - Anfänger

__________________
mod = master of disaster

Bedeutet das nicht, dass man auch alle anderen normalen Variablen prüfen muss? Nicht nur die, die übergeben werden, sondern auch die die nicht beim Aufruf mitübergeben werden sollen.

derHund

variablen, die du im script selber anlegst - die also 100%ig existieren - brauchst du nicht auf existenz prüfen. alle anderen ($_POST, $_COOKIE, $_SESSION, ...) schon.

__________________

Aber dass die Variablen 100% existieren, schließt doch nicht aus, dass man sie beim Script-Aufruf mitangeben bzw. falsch angeben kann?


Die if-Abfragen sind natürlich um weiten länger als zuvor. Würde das Script so wie es ist, sowohl mit register_globals on als auch off funktionieren?


Ich habe mir gerade PHP 5.0.3 installiert. "Undefined variable: event" Auf dem Webspace funktioniert's. Und wenn ich in das Scipt extract($_REQUEST); reinpacke, dann geht es in beiden Fällen. Also irgendwas ist die anscheinend noch nicht korrekt.

if((isset($_GET['show']) && $_GET['show'] == 'index') && (isset($_GET['event']) && file_exists($_GET['event'])))

Die Prüfung für event ist doch die selbe wie für show. Warum aber liefert mir PHP für event eine Fehlermeldung?

Klar, die Meldung kommt, wenn register_globals = Off ist.

Falsch:
Richtig:
Ob es noch mehr Fehler gibt, habe ich nicht kontrolliert.

EDIT:
Das ist auch falsch:
Richtig:
file_exists & Co. möchten nicht nur den Filenamen sondern auch den Pfad dazu. ./ kennzeichnet das aktuelle Verzeichnis.