PHPケーススイッチ（効率）

Question

以下のコードは、ユーザーがコントロールパネルにアクセスしたときに、クイック検証プロセスを実行してエンティティの内容を検証します。たとえば、ユーザーがレベル1の場合、ビデオフィードへのアクセスのみが許可されます。これは、他に利用可能なものがないことを意味します。

私はコードを見ると、ケース1と3が呼び出されたときにビデオフィードが呼び出されるのがわかります。私はおそらくコードをより効率的にするための選択肢を楽しむだろう。

私は可能な配列が少し楽になるかもしれないと言われましたが、もう一度これは速くなります。

switch ($_SESSION['permission']) { 
     case 1: // Level 1: Video Feed 
      include ("include/panels/videofeed.index.php"); 
      break; 
     case 2: // Level 2: Announcements/Courses/Teachers 
      include ("include/panels/announcements.index.php"); 
      include ("include/panels/courses.index.php"); 
      include ("include/panels/teachers.index.php"); 
      break; 
     case 3: // Level 3: Announcements/Video Feed/Courses/Teachers/Accounts/Logs 
      include ("include/panels/announcements.index.php"); 
      include ("include/panels/videofeed.index.php"); 
      include ("include/panels/courses.index.php"); 
      include ("include/panels/teachers.index.php"); 
      include ("include/panels/accounts.index.php"); 
      include ("include/panels/log.index.php");  
      break; 
     case 4: // Level 4: Teachers 
      include ("include/panels/teachers.index.php");   
    } 

Answer 1

可能であれば、require_onceを使用する場合は、フリーリストをに実行するとよいでしょう。 2つ目は、すべてのURLが同じであると思われるURLを短縮することです。

たぶん例えば機能でそれを使用しよう：

間違いましたか？私を正す！

Answer 2

これは問題ありません。私は、あなたが "繰り返し"を参照するときにあなたが "効率"を意味しないと思う。スイッチのフォールスルーを使用してコードを圧縮することができます。

これによりコードが小さくなる可能性がありますが、効率（スクリプトの実行に要する時間）に大きな影響はなく、実際にはコードを読みにくくします。それを残す。