[php] Hoe het niet moet :p

[Verwijderd]

Soms ben je in kladblok lekker html, php en javascript door elkaar aan het mixen, maar is het eindresultaat niet om aan te zien...
Heeft iemand misschien tips om zoiets overzichtelijk te houden?

PHP:

<a href="#" class="s12"<?php if(!isset($gobanner)){ ?> onclick="open('<?php if(isset($admin)) {echo ".";} ?>./pop_vote.php?ID=<?=$line["ID"]?>','_blank','<?=$vparam?>'); return(false);"<?php } ?>><b><?=round($line["rating"],1)?></b></a>

[MisterData]

Gebruik een template parser.

[Verwijderd]

40 KebaB overhead om een <a>'tje te formuleren?

[Erkens]

Verwijderd schreef op 30 June 2003 @ 18:34:
Soms ben je in kladblok lekker html, php en javascript door elkaar aan het mixen, maar is het eindresultaat niet om aan te zien...
Heeft iemand misschien tips om zoiets overzichtelijk te houden?

je script en je output scheiden van elkaar, dus van te voren alle "dynamisch" stukjes html in variablen maken zodat je die makkelijk kan uitpoepen met <?=$var?>

trouwens,

PHP:

<?php if(isset($admin)) {echo ".";} ?>

schrijf ik (zeker als het in de html moet) verkort op:

PHP:

<?=(isset($admin)?".";"")?>

[damouzer]

Je zou er commentaar bij kunnen zetten //Dit stukkie code doet dit en dit

De php regels op een nieuwe regel beginnen + inspringen.

Handig lijkt het me ook om even de FAQ van Programming & webscripting door te lezen. Daar staat het nodige wel in.

[Verwijderd]

Erkens schreef op 30 June 2003 @ 18:42:
je script en je output scheiden van elkaar, dus van te voren alle "dynamisch" stukjes html in variablen maken zodat je die makkelijk kan uitpoepen met <?=$var?>

trouwens,

PHP:

<?php if(isset($admin)) {echo ".";} ?>

schrijf ik (zeker als het in de html moet) verkort op:

PHP:

<?=(isset($admin)?".";"")?>

<?=$var?> gebruik ik al een hoop zoals je ziet, $vparam bevat namelijk:

PHP:

  $vparam = "top=' + (screen.height-88) / 2 + ',left=' + (screen.width-410) / 2 + ',";
  $vparam .= "location=0,menubar=0,resizable=0,scrollbars=0,status=0,toolbar=0,height=88,width=410";

Dat heb ik er slim genoeg al tussenuit gehaald

<?=(isset($admin)?".";"")?> ben ik al wel erg blij mee

[beetle71]

Ik vind zelf in dit soort gevallen ook de functies printf en sprintf erg handig om het overzichtelijk te houden.

Voorbeeld:

PHP:

$extrapoint=(isset($admin))?".":"";
$onclick=sprintf("onclick=\"open('%s./pop_vote.php?ID=%s',
           '_blank','%s');return(false);\"",$extrapoint,$line["ID"],$vparam);
printf("<a href='#' class='s12' %s><b></b>%s</a>",$onclick,round($line["rating"],1));

Regel 2 & 3 in het voorbeeld zijn moeten natuurlijk 1 regel zijn, maar ik wil de layout hier niet helemaal ver***ken