anpera.net

Hey!

Habe in der SuFu nichts gefunden.. Oder habe einfach nichts richtiges eingegeben..

Weiß nicht mehr in welchem script, aber da habe ich sowas auf die Art gesehen:
$this->bbcode_second_pass_code('', 'switch ($_GET['op']);

case ""

output("Text1");

break;

case "1"

output("Text2");
break;')

So.. Wollte jetzt wissen, worin der Unterschied zwischen

$this->bbcode_second_pass_code('', 'if ($_GET['op']=""){
output("Text1");
}
elseif ($_GET['op']="1"){
output("Text2");
}
')

Ist das erstere jetzt "richtiger" (wie z.b. anstatt $session[user][gold] eben $session['user']['gold'] ) oder hat das auch Performance Gründe? dass z.b. die ewig vielen If-Abfragen das Skript langsam machen?

Danke schonmal für Antworten

_________________
Wenn ihr mir sagt wie ich es als Datei hochlade, dann kann ich die Source hochladen die ihr wollt .D

Aus meiner persönlichen Erfahrung würde ich folgendes sagen: Ein switch-Konstrukt läuft grundsätzlich etwas schneller als ein if-else-Konstrukt. Ausserdem finde ich, dass ein switch-Konstrukt oft auch etwas übersichtlicher aussieht, und eine gute Struktur bietet.

Allerdings muss man auch sagen, dass der Geschwindigkeitsunterschied nicht gross ist, und es - gerade bei LoGD - keine spürbaren Unterschiede gäbe.

Im übrigen kannst du es nicht mit "$session['user']['gold']" und "$session[user][gold]" vergleichen. Die Single-Quotes in einem Array sollten dastehen, damit PHP den Wert nicht als Konstante interpretiert ( Wirft sonst einen Notice-Fehler ).

Im Übrigen ist ein switch-Konstrukt eher so aufgebaut ;-)
$this->bbcode_second_pass_code('', 'switch( $_GET['op'] ) {
case 'bla':
output( 'Bla' );
break;

default:
output( 'Blabla' );
break;
}')

Da war wohl jemand schneller, allerdings stimme ich dem allen zu.

Und um weitere NOTICE zu vermeiden, sollte

switch(isset($_GET['op'])) {

verwendet werden, falls 'op' nicht gesetzt ist ^^


Ach ja. Wer noch Zeilen sparen will, kann sich im switch das letzte "break;" sparen.

Und noch etwas.
$this->bbcode_second_pass_code('', '
if($_GET['op']=="bla" || $_GET['op']=="blub"){}
')
ist gleich
$this->bbcode_second_pass_code('', '
switch(isset($_GET['op'])) {
case"bla":
case"blub":
output();
break;
default:
output();
break;
}
')

Danke für die schnelle Antwort ihr beiden




Ok danke



Ok.. dann muss ich mal eine .php durchschaun.. Wirft mir nämlich 100 Notice-Fehler aus (sehr grob gerundet )

_________________
Wenn ihr mir sagt wie ich es als Datei hochlade, dann kann ich die Source hochladen die ihr wollt .D

Ja, du kannst da natürlich isset verwenden. Oder empty.

LoGD Notice-frei zu kriegen ist übrigens eine ziemliche Anstrengung. Untenstehender Link könnte dir allerdings bei der Auswertung der Fehlermeldungen helfen.
http://www.php-fehlermeldungen.de/topicmenu1/index0.html

Neu schreiben ist leichter ^^

Aber ich glaub dann nimmt jeder die Notice Fehler hin, oder?

_________________
Wenn ihr mir sagt wie ich es als Datei hochlade, dann kann ich die Source hochladen die ihr wollt .D

Nicht jeder, ich schreib grad neu Aber im Grunde kann man sie hinnehmen. Ich würde nur keine Addons veröffentlichen die Solche Fehler haben, da es auch viele php-Anfänger bei LoGD gibt, die diese Fehler dann als richtig erlernen.

Uii

Joa.. Bis ich ein Addon veröffentliche.. Ich glaube, dass das noch ne Weile dauern wird

_________________
Wenn ihr mir sagt wie ich es als Datei hochlade, dann kann ich die Source hochladen die ihr wollt .D

Darf ich mich da kurz einklinken? ^^

ist es denn nun egal ob ich: if (isset($PATH_INFO) != '') { oder if (empty($PATH_INFO) != '') {

schreib und würde das isset irgendwas an der funktionalität von $PATH_INFO ändern oder einfach nur den NOTICE unterdrücken?

Hi

Erstmal eine wichtige Sache vorweg: empty und isset liefern als Ergebnis nur Bool'sche Werte, also true oder false. Damit macht $this->bbcode_second_pass_code('', 'switch(isset($_GET['irgendwas'])) {
case 'a':
//tu was
case 'b':
// tu was anderes
}') keinen Sinn, da 'a' oder 'b' nie erreicht werden. Für empty gilt selbes.

Meine Vorgehensweise zur Vermeidung von NOTICE-Fehlern ist:
$this->bbcode_second_pass_code('', '
$op = isset($_GET['op']) ? $_GET['op'] : ''; // Wobei man '' durch den Default-Wert seiner Wahl ersetzen kann.

if($op == 'blah') { /* Inhalt */ }
// bzw.
switch($op) {
case 'a':
// ...
}')

Bei einer einzelnen If-Verzweigung kann man das natürlich auch in der If-Kondition checken und braucht keine extra Variable, aber das Prinzip sollte verständlich sein. Das ganze lässt sich natürlich auch auf jede andere Variable, egal ob $_GET['blah'], $_POST['blah'], $irgendwas oder wie auch immer übertragen.

Auric

_________________
Mehr oder minder inaktiv

deswegen nimmt man auch ein

switch(){
default:
output();
break;
}

ins switch =)

Bei einem "switch(isset($_GET['irgendwas'])) {" wird meines Wissens nach einfach der erste switch-Block angewählt (Wenn isset true zurück gibt ).

switch prüft nur auf Gleichheit - also ja, es wird der Erstbeste Block angewählt, der nicht 0, false oder NULL ist. Deshalb dürfte das ganze besser so sein (So, wie ichs mache):

$this->bbcode_second_pass_code('', 'switch(isset($_GET['op']) ? $_GET['op'] : '') {
case '':
default: {
break;
}

case 'one': {
break;
}
}')

Hey!

Habe noch ne frage dazu. Wie mach ich denn dann den addnav dazu?

aslo um z.b. zum case b zu kommen? würde dann der addnav dennoch

addnav("Zu case b","file.php?op=b");

oder?

_________________
Wenn ihr mir sagt wie ich es als Datei hochlade, dann kann ich die Source hochladen die ihr wollt .D

jop

k thx

_________________
Wenn ihr mir sagt wie ich es als Datei hochlade, dann kann ich die Source hochladen die ihr wollt .D