Code optimieren

[Scott]

Hallo zusammen,

ich habe mir etwas gebastelt, um aus einer Reihe von Werten immer nur 5 per Zufall auszugeben.

Grundlage war ein Script von www.php.net bei der Funktion shuffle:

<?php
$inhalt = array(
           "1111",
           "2222",
           "3333",
           "4444",
           "5555",
           "6666",
           "7777",
           "8888",
           "9999"
           );

srand ((float)microtime()*1000000);
shuffle ($inhalt);

$i = 1;
while ((list (, $inhalt_array) = each ($inhalt)) && $i <= 5)
{
   echo $inhalt_array.'<br />';
   $i++;
}
?>

Es funktioniert!
Ich wollte aber mal Eure Expertenmeinung dazu haben, ob der Code so ok ist, oder ob man ihn noch optimieren kann.

Danke im Voraus!!!
### Tips schon eingearbeitet ###

[derHund]

$inhalt = array(1111, 2222, 3333, ...
echo $inhalt.'<br />'; 

[Scott]

ok. Danke.

Variablen ausserhalb "" ausgeben.
Array ohne "" geht leider nicht, weil ich dort Sätze mit Sonderzeichen anstelle der Zahlen nehmen will.

Sonst ok?

[wahsaga]

Re: Code optimieren

wenn man eine feste anzahl von schleifendurchläufen haben will - warum nimmt man dann keine for-schleife?

und wozu list() bei einem array mit nur einer ebene gut sein soll ...?

[marc75]

ohne da jetzt genau hingeschaut zu haben, php hat für die Ausgabe die foreach schleife angedacht.

http://de.php.net/manual/de/control-...es.foreach.php

[schlimmerfinger]

Re: Re: Code optimieren

Original geschrieben von wahsaga
wenn man eine feste anzahl von schleifendurchläufen haben will - warum nimmt man dann keine for-schleife?

Ich glaube auch das eine For schleife da geeigneter wäre.

cheers

[S.Oliver]

Re: Code optimieren

Original geschrieben von Scott
Es funktioniert!

Schön

Original geschrieben von Scott
Ich wollte aber mal Eure Expertenmeinung dazu haben, ob der Code so ok ist, oder ob man ihn noch optimieren kann.

Wie viele Millionen mal am Tag wird denn das bißchen Code aufgerufen, dass es sich lohnt darüber nachzudenken?

Ich will ja Deinen Wissensdurst nicht bremsen, aber es gibt für nahezu jedes Problem dutzende Ansätze und Schreibweisen - und keine davon ist gut oder schlecht. Wenn es doch funktioniert und die Funktion 5 mal am Tag genutzt wird, ist es doch prima. Selbst bei perfekter Optimierung wirst Du dabei vielleicht 2 ms rausholen im Vergleich zu "schlechtesten" Version

[Scott]

Vielen Dank für die ganzen Tips!

Habe es jetzt so gelöst:

$inhalt = array(..........);

srand ((float)microtime()*1000000);
shuffle ($inhalt);

for ($i = 1; $i <= 5; $i++)
{
  echo $inhalt[$i].'<br />';
} 

@S.Oliver: Nenne mich ruhig Code-optimier-Freak
Ich brauche das eben für mein Ego...