PHP strings einfügen

[toms]

Hallo

Ich bin neu bei PHP und möchte in einen string z.B
$mystring = "12345"; Kommas einfügen.

Das Resultat ($myoutput) soll so aussehen:

1, 2, 3, 4, 5

Bis jetzt sieht der code so aus:

Code:

$myoutput = "";
for ($i = 0; $i < strlen($mystring); $i++){
 if ($i < strlen($mystring) -1) {
 $myoutput .= $myoutput.$mystring[$i].", ";
 } else
 {
 $myoutput .= $myoutput.$mystring[$i];
 }
}

Er funktioniert schon halbwegs. Bug:

Wenn z.B $mystring = "34"; ist, dann gibt es 3, 3, 4 aus...

Kann man die Funktion evtl noch schöner schreiben?

Danke für Hinweise - Antworten

[mkinzler]

Code:

$myoutput = "";
for ($i = 0; $i < strlen($mystring); $i++){
    if ($myoutput != "") {
        $myoutput .= ", ";
    }
    $myoutput .= $myoutput.$mystring[$i];
}

[toms]

Danke fuer die schnelle Antwort

Bei deinem Code passiert jedoch das gleiche:

Wenn z.B $mystring = "34"; ist, dann gibt es 3, 3, 4 aus...

[mkinzler]

Hier war es doppeltgemoppelt:

Code:

$myoutput .= $mystring[$i];

[toms]

Danke, funktioniert nun bestens!

[yankee]

Benchmarke das doch mal gegen diese Lösung:
$myoutput =substr(preg_replace('/(.)/','\\1,',$mystring),0,-1);

Code:

for ($i = 0; $i < strlen($mystring); $i++)
{
  if ($myoutput != "") $myoutput .= ",";
  $myoutput .=$mystring[$i];
}

--> ~0.118 Sekunden
mit ++$i statt $i++
--> ~0.114 Sekunden

Code:

$myoutput =substr(preg_replace('/(.)/','\\1,',$mystring),0,-1);

--> ~0.073 Sekunden

Alle Werte verstehen sich über 10K iterationen und dann der Durschniktt mehrerer Aufrufe.
Ich habe viel damit rumgespielt, aber ich habe die Performance der ersten Lösung nicht nennswert anheben können. Die Regexlösung liegt weit vorne und ist auch noch die kürzeste :-).

[S2B]

Wenn ihr schon am Benchmarken seid, wie wäre es mit dieser Lösung:

Code:

$myoutput = substr(chunk_split($mystring, 1, ', '), 0, -2);

Ach ja: Die for-Lösung sollte so schneller werden:

Code:

for ($i = 0, $strlen = strlen($mystring); $i < $strlen; $i++)

[yankee]

Zitat:

Zitat von S2B

Wenn ihr schon am Benchmarken seid, wie wäre es mit dieser Lösung:

Code:

$myoutput = substr(chunk_split($mystring, 1, ', '), 0, -2);

Winner!
0,03 Sekunden!

[S2B]

Zitat:

Zitat von yankee

Winner!
0,03 Sekunden!

:firejump: :mrgreen: