Beste Möglichkeit, einen PHP-Switch mit mehreren Werten pro Fall durchzuführen?

Wie würden Sie diese PHP-Switch-Anweisung ausführen?

Beachten Sie auch, dass dies viel kleinere Versionen sind, der 1, die ich erstellen muss, werden viel mehr Werte hinzugefügt.

Version 1:

switch ($p) { 
    case 'home': 
    case '': 
        $current_home="current";
    break; 

    case 'users.online': 
    case 'users.location': 
    case 'users.featured': 
    case 'users.new': 
    case 'users.browse': 
    case 'users.search': 
    case 'users.staff': 
        $current_users="current";
    break;

    case 'forum': 
        $current_forum = 'current';
    break; 
} 

Version 2:

switch ($p) { 
    case 'home': 
        $current_home="current";
    break; 

    case 'users.online' || 'users.location' || 'users.featured' || 'users.browse' || 'users.search' || 'users.staff': 
        $current_users="current";
    break;

    case 'forum': 
        $current_forum = 'current';
    break; 
} 

UPDATE – Testergebnisse

Ich habe einen Geschwindigkeitstest mit 10.000 Iterationen durchgeführt,

Time1: 0.0199389457703 // If-Anweisungen
Time2: 0.0389049446106 // Anweisungen wechseln
Zeit3: 0.106977939606 // Arrays

Für jede Situation, in der Sie eine unbekannte Zeichenfolge haben und herausfinden müssen, welche von vielen Sonstiges Strings, mit denen es übereinstimmt, ist die einzige Lösung, die nicht langsamer wird, wenn Sie mehr Elemente hinzufügen, ein Array zu verwenden, aber alle möglichen Strings als Schlüssel zu haben. Ihr Schalter kann also durch Folgendes ersetzt werden:

// used for $current_home="current";
$group1 = array(
        'home'  => True,
        );

// used for $current_users="current";
$group2 = array(
        'users.online'      => True,
        'users.location'    => True,
        'users.featured'    => True,
        'users.new'         => True,
        'users.browse'      => True,
        'users.search'      => True,
        'users.staff'       => True,
        );

// used for $current_forum = 'current';
$group3 = array(
        'forum'     => True,
        );

if(isset($group1[$p]))
    $current_home="current";
else if(isset($group2[$p]))
    $current_users="current";
else if(isset($group3[$p]))
    $current_forum = 'current';
else
    user_error("\$p is invalid", E_USER_ERROR);

Das sieht nicht so sauber aus wie ein switch()aber es ist das einzige schnell Lösung, die nicht das Schreiben einer kleinen Bibliothek von Funktionen und Klassen beinhaltet, um sie sauber zu halten. Es ist immer noch sehr einfach, Elemente zu den Arrays hinzuzufügen.

Version 2 funktioniert nicht!!

case 'users.online' || 'users.location' || ...

ist genau das gleiche wie:

case True:

und das case wird für jeden Wert von ausgewählt $pwenn nicht $p ist die leere Zeichenfolge.

|| Hat innerhalb von a keine besondere Bedeutung case Aussage, du vergleichst nicht $p zu jeder dieser Zeichenfolgen überprüfen Sie nur, ob dies nicht der Fall ist False.

Setzen Sie diese vielen Werte in ein Array und fragen Sie das Array ab, da der Schalterfall die zugrunde liegende Semantik dessen zu verbergen scheint, was Sie erreichen möchten, wenn eine Zeichenfolgenvariable als Bedingung verwendet wird, was das Lesen und Verstehen erschwert, z :

$current_home = null;
$current_users = null;
$current_forum = null;

$lotsOfStrings = array('users.online', 'users.location', 'users.featured', 'users.new');

if(empty($p)) {
    $current_home="current";
}

if(in_array($p,$lotsOfStrings)) {
    $current_users="current";
}

if(0 === strcmp('forum',$p)) {
    $current_forum = 'current';
}

Der Vollständigkeit halber weise ich darauf hin, dass die kaputte “Version 2”-Logik durch eine funktionierende switch-Anweisung ersetzt werden kann, und Auch Verwenden Sie Arrays für Geschwindigkeit und Klarheit, wie folgt:

// used for $current_home="current";
$home_group = array(
    'home'  => True,
);

// used for $current_users="current";
$user_group = array(
    'users.online'      => True,
    'users.location'    => True,
    'users.featured'    => True,
    'users.new'         => True,
    'users.browse'      => True,
    'users.search'      => True,
    'users.staff'       => True,
);

// used for $current_forum = 'current';
$forum_group = array(
    'forum'     => True,
);

switch (true) {
    case isset($home_group[$p]):
        $current_home="current";
        break;
    case isset($user_group[$p]):
        $current_users="current";
        break;
    case isset($forum_group[$p]):
        $current_forum = 'current';
        break;
    default:
        user_error("\$p is invalid", E_USER_ERROR);
}    

Heutzutage kann man…

switch ([$group1, $group2]){
    case ["users", "location"]:
    case ["users", "online"]:
        Ju_le_do_the_thing();
        break;
    case ["forum", $group2]:
        Foo_the_bar();
        break;
}

Hier ist ein fertiger Codesatz, der das Konzept demonstriert:

<?php

function show_off_array_switch(){
  $first_array = ["users", "forum", "StubbornSoda"];
  $second_array = ["location", "online", "DownWithPepsiAndCoke"];

  $rand1 = array_rand($first_array);
  $rand2 = array_rand($second_array);

  $group1 = $first_array[$rand1];
  $group2 = $second_array[$rand2];

  switch ([$group1, $group2]){
      case ["users", "location"]:
      case ["users", "online"]:
          echo "Users and Online";
          break;
      case ["forum", $group2]:
          echo "Forum and variable";
          break;
      default:
          echo "default";
  }
  echo "\nThe above result was generated using the array: \n" . print_r([$group1, $group2], true);
}

for ($i = 0; $i < 10; $i++){
  show_off_array_switch();
}

Folgendes hat (für einen zufälligen Lauf) Folgendes ausgegeben:

 Users and Online
The above result was generated using the array: 
Array
(
    [0] => users
    [1] => online
)
Users and Online
The above result was generated using the array: 
Array
(
    [0] => users
    [1] => online
)
default
The above result was generated using the array: 
Array
(
    [0] => users
    [1] => DownWithPepsiAndCoke
)
Users and Online
The above result was generated using the array: 
Array
(
    [0] => users
    [1] => location
)
Forum and variable
The above result was generated using the array: 
Array
(
    [0] => forum
    [1] => DownWithPepsiAndCoke
)
Forum and variable
The above result was generated using the array: 
Array
(
    [0] => forum
    [1] => DownWithPepsiAndCoke
)
Forum and variable
The above result was generated using the array: 
Array
(
    [0] => forum
    [1] => online
)
default
The above result was generated using the array: 
Array
(
    [0] => StubbornSoda
    [1] => location
)
Users and Online
The above result was generated using the array: 
Array
(
    [0] => users
    [1] => location
)
Users and Online
The above result was generated using the array: 
Array
(
    [0] => users
    [1] => location
)

Wenn jeder Wenn Sie Ihren Code sonst nie pflegen müssten, würden sie Version 2 mit ziemlicher Sicherheit doppelt angehen – das ist extrem nicht standardisiert.

Ich würde bei Version 1 bleiben. Ich bin jedoch der Meinung, dass Fallaussagen ohne eigenen Anweisungsblock explizit sein sollten // fall through kommentieren Sie daneben, um anzuzeigen, dass es tatsächlich Ihre Absicht ist, durchzufallen, wodurch jegliche Unklarheit beseitigt wird, ob Sie die Fälle anders handhaben und vergessen oder so etwas.

Einige andere Ideen, die noch nicht erwähnt wurden:

switch(true){ 
  case in_array($p, array('home', '')): 
    $current_home="current"; break;

  case preg_match('/^users\.(online|location|featured|new|browse|search|staff)$/', $p):
    $current_users="current"; break;

  case 'forum' == $p:
    $current_forum = 'current'; break; 
}

Jemand wird sich wahrscheinlich über Lesbarkeitsprobleme mit #2 beschweren, aber ich hätte kein Problem damit, solchen Code zu erben.