OXPUS.de

Hallo, wäre folgender Schnipsel so:

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

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.

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)

Das ist egal.

ok, danke

OXPUS.de

Schnipsel - Was ist besser?

Schnipsel - Was ist besser?