Good bad error message

Ja maar wqarom doe je dan niet zo..

<?php
if($var == '1')
{
?>
<!--HTML CODE HIER-->
<div class="error">Shittymessage</div>
<?php
}
else
{
?>
<!--HTML CODE HIER-->
<div class="good">Goodiemessage</div>
<?php
}
?>

En HTML en PHP gescheiden?

<?php
$var = 2;
if($var = 1){
	echo "<style> #error {	visibility:visible !important;}</style>"; //echo error message
}else{
	echo "<style>#good { visibility:visible !important; } </style>"; //echo error message
}
?>

Wat is dit dan?

Ik zou voor de manier van ferhat gaan.

Ik zou van het id sowieso een class maken omdat je dingen als "good" en "error" waarschijnlijk vaker en ook meerdere keren op 1 pagina zal gebruiken.

En er dan iets in de richting van;

<?php
if($inloggenSuccessvol)
{
    $loginStatusClass = "good";
    $loginStatusText = "Inloggen gelukt";
}
else
{
    $loginStatusClass = "error";
    $loginStatusText  = "Inloggen mislukt";
}
?>


<div class="<?= $loginStatusClass; ?>"><?=$loginStatusText; ?></div>

van maken.

Waarom dan geen function..?

Citaat van Ferhat

Waarom dan geen function..?

Kan moeilijk een function in een view gaan definiereen, en om zijn hele structuur te gaan veranderen op basis van een paar regels html lijkt me ook niet echt handig.

<?php
function error($melding) {
    echo'<div class="error"><p>' . $melding . '</p></div>';
}
?>

<?php
function goed($melding) {
echo'<div class="good"><p>' . $melding . '</p></div>';
}
?>

Oproepen met

<?php echo goed('Goed zo')?>