Det er ret mange linjer kode, for et simpelt script. Hvis du f.eks vil have 10 farver på et tidspunkt, er det tidskrævende, og det vil komme til at fylde alt for meget, og gentagelse "background-color, skal du af med under alle omstændigheder.
Her er et eksempel, hvor der kan tilføjes et utal af farver:
- <?php
- $colours = array( "yellow", "blue", "pink", "green" );
- shuffle( $colours );
- ?>
- <style type="text/css">
- body {
- <?php
- print "background-color: {$colours[0]};";
- ?>
- }
- </style>
- ?>
Du kan også bruge:
- <?php
- $colours = array( "yellow", "blue", "pink", "green" );
- $num_colours = count($colours);
- $colour = rand( 0, $num_colours);
- ?>
- <style type="text/css">
- body {
- <?php
- print "background-color: {$colours[$colour]};";
- ?>
- }
- </style>
- ?>
Bare et par eksempler på at det kan gøres nemmere. Et lille hint, når du laver scripts; undgå at gentage dig selv. Hvis du ser gentagelser i dit script, så omskriv det så du undgår det.
I dit tilfælde kunne du gøre flg:
- <?php
- $color = rand(1,4);
- echo "<style type=\"text/css\"> body { background-color: ";
- switch($color){
- case 1:
- echo "yellow";
- break;
-
- case 2:
- echo "blue";
- break;
-
- case 3:
- echo "pink";
- break;
-
- case 4:
- echo "green";
- break;
- default:
- echo "white";
- }
- echo "; } </style>";
- ?>
Læg mærke til, at jeg har tilføjet en default til din switch. Der bør altid være en default i en switch, ligesom der f.eks. skal være en alt="" i en <img>