Schnipsel - Was ist besser?

[JaneDoe]

Hallo, wäre folgender Schnipsel so:

Code: Alles auswählen

if ( $poster_id != ANONYMOUS )
	{
.
.
.
		if ( !empty($postrow[$i]['user_viewemail']) || $is_auth['auth_mod'] )
		{
			if ( $userdata['session_logged_in'] ) 
			{
			$email_uri = ...; 
			$email_img = '<a href="....</a>';
			$email = '<a href="...</a>';
			}
		}
		else
		{
			$email_img = '';
			$email = '';
		}
	}	
else
	{
		$email_img = '';
		$email = '';
	}

oder so

Code: Alles auswählen

if ( $poster_id != ANONYMOUS )
	{
.
.
.
		if ( (!empty($postrow[$i]['user_viewemail']) || $is_auth['auth_mod']) && $userdata['session_logged_in']  )
		{
			$email_uri = ...; 
			$email_img = '<a href="....</a>';
			$email = '<a href="...</a>';
		}
		else
		{
			$email_img = '';
			$email = '';
		}			
	}	
else
	{
		$email_img = '';
		$email = '';
	}			

besser, oder sagen wir der ordenlicherere Code?

grüsse

[oxpus]

Code: Alles auswählen

$poster_id != ANONYMOUS 

und

Code: Alles auswählen

$userdata['session_logged_in']

sind das gleiche, bzw. bringen die gleichen Ergebnisse.
Auf eines von beiden kannst Du also verzichten.
Ansonsten ist diese Abfrage okay.

[JaneDoe]

Sorry. Der Teil ist aus der viewtopic.php, hätt ich dazuschreiben sollen.
So war das missverständlich.
Mit dem

Code: Alles auswählen

$userdata['session_logged_in']

Teil will ich verhindern, dass gäste den
Emailbutton sehen.

Das klappt ja mit der Abfrage auch.

Wollte nur wissen, ob es vom Codestandpunkt besser ist

Code: Alles auswählen

if ((a || b) && c) 

zu schreiben oder

Code: Alles auswählen

if (a || b )
 if (c)

[oxpus]

Das ist egal.

[JaneDoe]

ok, danke