Verzweigung kürzen

[Forumsgast]

Hi,
kann mir bitte jemand sagen, ob man folgende Verzweigung kürzen kann?
Es dreht sich um eine Seite, die ohne id aufgerufen (www.domain.de/index.php) wird und auf der Seite kann man die Ausgabe über ein Menü ändern, da werden dann die ids übergeben (www.domain.de/index.php?id=1).

if (isset($_GET['id'])) {
 $var = 'anderer Name';
} else {
$var = 'Name';
} 

Gekürzt. So in Ordnung?

$var = 'Name';
if (isset($_GET['id'])) {
$var = 'anderer Name';
} 

[Forumsgast]

Kann man so auch machen, finde ich jedenfalls. Es funktioniert doch...

[Forumsgast]

noch kürzer:

$var = isset($_GET['id']) ? 'anderer name' : 'Name';

Ob man das verwendet ist Geschmackssache. macht den Code evtl. etwas unleserlicher, aber für das initialisieren von Variablen mach ich das gerne mal:

$id = isset($_GET['id']) ? (int) $_GET['id'] : 0;

für ids (numerische Werte) bietet sich auch max() an:

 $id = max(0, $_GET['id']);

[Forumsgast]

für ids (numerische Werte) bietet sich auch max() an:

 $id = max(0, $_GET['id']);

Gibts max() auch für Buchstaben? So das nur einer oder vielleicht drei Buchstaben zugelassen werden?

[Forumsgast]

ne die funktion max geht nur auf Zahlen

um die länge von Strings zu überprüfen kannst du

 strlen( ) 

verwenden.

um sie dann "abzuschneiden" kannst du die funktion

substr( $string, Start, [Ende] ); 

verwenden.

[Forumsgast]

Also deine Version ganz oben verwende ich am liebsten. Und mit Buchstaben könnte man sowas basteln:

<?php
$id = 'a';
if (in_array($_GET['id'], array('b', 'c', 'tufftuff')))
    $id = $_GET['id'];
?>

[Forumsgast]

Wenn ich das Array aber z.B. mit 50 Werten voll packe, geht das nicht auf kosten der Geschwindigkeit?

[Forumsgast]

ne die funktion max geht nur auf Zahlen

Wo hast den das her?

PHP:max - Manual[/url]]mixed max ( mixed $value1 , mixed $value2 [, mixed $value3... ] )

Kannst ja selber mal testen:

<?php
echo max('abcd', 'defg', 'xy', 'xzab');
?>

[Forumsgast]

Wenn ich das Array aber z.B. mit 50 Werten voll packe, geht das nicht auf kosten der Geschwindigkeit?

Wenn du 50 verschiedene Werte hast dann kannst du auch gerne 50 verschiedene IF-Abfragen machen. Aber ob damit die Performance besser wird ... also "in_array" mag nicht die schnellste Funktion sein aber sag mir wie man einfacher eine white-List von 50 verschiedenen Einträgen überprüft.

[Forumsgast]

Re: Verzweigung kürzen

if (isset($_GET['id'])) {
 $var = 'anderer Name';
} else {
$var = 'Name';
} 

Gekürzt. So in Ordnung?

$var = 'Name';
if (isset($_GET['id'])) {
$var = 'anderer Name';
} 

Um solche Kleinigkeiten solltest du dich nicht kümmern ..

Aber du kannst das ganz einfach selbst überprüfen:
ID gesetzt:
Skript 1:
Funktionsaufruf + Bedingungsprüfung + Zuweisung
Skript 2:
Zuweisung + Funktionsaufruf + Bedingungsprüfgung + Zuweisung
Gewinner: Skript 1


ID nicht gesetzt:
Skript 1:
Funktionsaufruf + Bedingungsprüfung + Zuweisung
Skript 2:
Zuweisung + Funktionsaufruf + Bedingungsprüfung
Gewinner: Unentschieden

Und hilft dir das weiter? Neuzuweisungen dauern vermutlich länger, als reines überschreiben - ist das in PHP wirklich so .. weiß mans? Keine Ahnung?! Dann lieber an anderen Kriterien festhalten: Lesbarkeit!

[Forumsgast]

Wenn du 50 verschiedene Werte hast dann kannst du auch gerne 50 verschiedene IF-Abfragen machen.

Das würde ich eigentlich nicht wollen, nein.

Aber ob damit die Performance besser wird ... also "in_array" mag nicht die schnellste Funktion sein aber sag mir wie man einfacher eine white-List von 50 verschiedenen Einträgen überprüft.

Ja, ich kenn da keine Alternative.

Um solche Kleinigkeiten solltest du dich nicht kümmern ..

Stimmt natürlich. Ich lern da aber auch immer wieder was drauß für andere Sachen.

... Und hilft dir das weiter? ...

JA! Danke!