✔ $_GET['show'] Fehlermeldung

[unrealzero_php]

hallo zusammen

habe eine kleine website geschrieben über die ich mit $_GET['show'] die variable show übergebe.
beim ersten aufruf der datei bekomme ich jedoch eine fehlermeldung, dass die variable $show nicht definiert ist...das kommt daher dass, beim ersten aufruf $_GET['show'] noch nicht definiert ist.

wie kann ich diesen fehler abfangen...gibt es irgend etwas wie if exist ...?

danke für die mithilfe

 

[Dennis Wronka]

Es gibt 2 Varianten um zu pruefen ob eine Variable gesetzt ist.
Entweder

if ($variable)
 {
   //machwas
 }

oder

if (isset($variable))
 {
   //machwas
 }

Beide funktionieren, jedoch ist die 2. programmiertechnisch sauberer.

 

[unrealzero_php]

danke, dass habe ich gesucht!

 

[Sebastian Wramba]

unrealzero_php: Bitte halte dich bei deinen nächsten Beiträgen doch an die Netiquette, insbesondere Punkt 12, danke.

 

[SilentWarrior]

unrealzero: Wenn du mit einer Fallunterscheidung (switch, case) arbeitest (was du eigentlich aus Sicherheitsaspekten immer tun solltest), dann kannst du, wenn das Error Reporting von PHP nicht auf E_ALL (d. h. alle Fehler/Warnungen werden angezeigt) gestellt ist, auch mit einem default arbeiten.

Falls du noch keine Fallunterscheidung verwendest, möchte ich dir dies ans Herz legen, da sonst ein User mit bösen Absichten möglicherweise ziemliches Unheil auf deiner Page anrichten kann.

-> http://www.php.net/switch/

 

[Sebastian Wramba]

Er kann doch genausogut eine if-Abfrage machen? Bzw. in der show-Variable nur den Dateinamen übertragen und in der Datei die Endung hinten dran setzen, sprich:

if(isset($_GET['show']) && $_GET['show'] != "") {
    include($_GET['show'].".php");
}

 

[Dennis Wronka]

Du kannst auch direkt eine Variable includen vorausgesetzt Du ueberpruefst, dass kein http: oder ftp: drin vorkommt.

$subsite=$_GET["subsite"];
if ($subsite)
	{
		if ((strpos($subsite,"index.php")>-1) || (strpos($subsite,"http:")>-1) || (strpos($subsite,"ftp:")>-1))
			{
				header("Location:index.php");
			}
		else
			{
				include($subsite);
			}
	}
else
	{
		include("welcome.php");
	}

Das ist der Code aus meiner index.php

 

[SilentWarrior]

Sebastian: Das wäre eben sehr unsicher, weil man so evtl. auch Scripte von fremden Seiten includen kann, die dann ganz schön viel Unheil anstellen können.

So wie reptiler das gemacht hat, finde ich das eine sehr gute Lösung. Wenn hier keiner irgendwas sicherheitstechnisches daran auszusetzen hat, werde ich das wohl in Zukunft auch so machen.

 

[Sebastian Wramba]

Ah.. sozusagen index.php?show=http://www.url.de/boeses_script. Hm clever, hab ich gar nicht dran gedacht, sicher geht das so.

 

[unrealzero_php]

unrealzero: Wenn du mit einer Fallunterscheidung (switch, case) arbeitest (was du eigentlich aus Sicherheitsaspekten immer tun solltest), dann kannst du, wenn das Error Reporting von PHP nicht auf E_ALL (d. h. alle Fehler/Warnungen werden angezeigt) gestellt ist, auch mit einem default arbeiten.

Falls du noch keine Fallunterscheidung verwendest, möchte ich dir dies ans Herz legen, da sonst ein User mit bösen Absichten möglicherweise ziemliches Unheil auf deiner Page anrichten kann.

-> http://www.php.net/switch/

Danke für den Input.

Werde es mal genauer anschauen.