From 924f395a39eeccbcd54d540613e440cbedd6ba3d Mon Sep 17 00:00:00 2001
From: Arun Persaud <arun@nubati.net>
Date: Sat, 26 Apr 2008 13:01:41 -0700
Subject: CLEANUP: cleaned up index.php a bit more by introducing an 'action'
 variable

no more if, but one nice switch statement in index.php
Signed-off-by: Arun Persaud <arun@nubati.net>
---
 index.php | 66 +++++++++++++++++++++++++++------------------------------------
 1 file changed, 28 insertions(+), 38 deletions(-)

(limited to 'index.php')

diff --git a/index.php b/index.php
index 54e2cb7..31c497a 100644
--- a/index.php
+++ b/index.php
@@ -31,51 +31,41 @@ output_header();
 
 /* The rest of the file consists of handling user input.
  * The user sends information via html GET and POST variables,
- * the script checks if these are set via "myisset"
- * which can check a list of variables.
+ * and the action variable tells the prog what the user wants to do
  */
+if(myisset("action"))
+  $action=$_REQUEST['action'];
+else
+  $action=""; /* so that we can use a default option below */
 
-/* does the user want to log out? */
-if(myisset("logout"))
-  {
-    require './include/logout.php';
-  }
-/* check if we want to start a new game */
-else if(myisset("new"))
+switch($action)
   {
+  case 'new':
     require './include/newgame.php';
-  }
-/*check if everything is ready to set up a new game */
-else if( myisset("PlayerA", "PlayerB","PlayerC","PlayerD","dullen","schweinchen","callrule" ))
-  {
-    require './include/newgameready.php';
-  }    
-/* cancel a game, if nothing has happend in the last N minutes */
-else if(myisset("cancel","me"))
-  {
+    break;
+  case 'cancel':
     require './include/cancelgame.php';
-  }
-/* send out a reminder */
-else if(myisset("remind","me"))
-  {
+    break;
+  case 'reminder':
     require './include/reminder.php';
-  }
-/* handle request from one specific player for one game,
- * (the hash is set on a per game base) */
-else if(myisset("me"))
-  {
+    break;
+  case 'logout':
+    require './include/logout.php'; 
+    require './include/welcome.php';
+    break;
+  case 'login':
+    require './include/login.php'; 
+    require './include/user.php';
+    break;
+  case 'game':
     require './include/game.php';
- }
-/* user status page */
-else if( myisset("email","password") || isset($_SESSION["name"]) )
-   {
-     require './include/user.php';
-   }
-/* default login page */
- else
-   {
-     require './include/welcome.php';
-   }
+    break;
+  default:
+    if(isset($_SESSION["name"]))
+      require './include/user.php';
+    else
+      require './include/welcome.php';
+  }
 
 output_footer();
 
-- 
cgit v1.2.3-18-g5258