function zu: Userstatus

[sipoh]

Hallo,

ich habe momentan eine einfache if-Anweisung, die je nach identifizierter UserID die Rechte vergibt.

 // Wenn reg Benutzer
 if (session_id() == $_SESSION['mysession']) {
	 $sql="SELECT UserID FROM users WHERE UserSession = '".$_SESSION['mysession']."'";
     $result = mysql_query($sql);
	 $row = mysql_fetch_assoc($result);
     $myid = $row['UserID'];

	 if ($myid=="1") {
		 // Wenn ID = 1 (ADMIN-ID) status 3
		 $status = 3;
	 } else {
		 // registrierte Benutzer
		 $status = 1;
	 }
 }

 // Wenn Gast
 if (!isset($_SESSION['mysession'])) {
     $status = 0;
 }

?>

Sehr simple. Kann man das vielleicht auch vereinfacht in eine funktion packen?

 

[Flex]

function getStatus()
{
  if(session_id() == $_SESSION['mysession'])
  {
    $sql="SELECT UserID FROM users WHERE UserSession = '".mysql_real_escape_string($_SESSION['mysession'])."'";
    $result = mysql_query($sql);
    $row = mysql_fetch_assoc($result);
	  
	  $status = ( $row['UserID'] == 1 ? 3 : 1 );
	  return $status;
  }
  return 0;
}

$status = getStatus();

 

[hasta_jesaja]

Uhm... no offense but...

$status = ( $row['UserID'] != 1) ? 1 : 3;

Die Klammer war falsch.

 

[Flex]

Öh, nein, war sie nicht. Ein ternärer Operator kann auch ganz eingeklammert werden.

 

[hasta_jesaja]

Verdammt, ich wusste doch da stimmt was nicht

 

[KD3]

bei if(session_id() == $_SESSION['mysession']) wird bestimmt immer true zurückgeliefert. Da wäre session_regenerate_id() sicherer und angebrachter.

MfG
KD3