wiso ist das sooo langsam

GreenRover

Wiso ist dieser code Soooo CPU lastig??

Code

$smiles=array( '*wink*' => 'wink.gif',
               '*bla*   => 'bla.gif',
               '*LOL1*' => 'lol.gif',
               '*LOL2*' => 'LOL.gif );




$eintrag=strtr( $eintrag, $smiles );

Wobei zu beachten ist, das die array $smilesi n wirklichkeit 10+ einträge enthält und der string $eintrag ca 1A4 seite lang ist.

das legt meine 3Ghz CPU für 2min lahm.

Gibt es einen schnelleren code ???

oder wie würdet ihr das Problem lösen ??

Torty

Was genau soll der Code machen? und
Müssten da nicht 3 Parameter angegeben werden? -> Befehlsreferenz: strtr

GreenRover

es ist ein teil eines gästebuches und er soll z.B. *lol* im Test gegen den Smile austauschen.

Empon

Wieso nimmste da nicht das gängige str_replace?

Torty

Schau mal: Hinter '*bla* fehlt ein '.
Vielleicht hilft das schon...

GreenRover

weil das noch langsamer war!!!!

Torty

Wie wärs damit:

Code

$smiles=array( '*wink*' => 'wink.gif',
               '*bla*'  => 'bla.gif',
               '*LOL1*' => 'lol.gif',
               '*LOL2*' => 'LOL.gif );


foreach ($smiles as $change)
  str_replace(array_keys($change), array_values($change), $eintrag);

Torty

Ok, also der letzte Codesnippsel war nur so schnell dahin geschmissen. Sorry, denn er weißt Fehler auf.
Nun hab ich aber ein kleines Script verfasst, welches deine Aufgabe erledigt:

HTML

<?
if (isset($eingabe)) {
  $smiles = array('*wink*' => 'wink.gif',
                  '*bla*'  => 'bla.gif',
                  '*LOL1*' => 'lol.gif',
                  '*LOL2*' => 'LOL.gif' );


  $keys = array_keys($smiles);  
  for($i=0; $i < count($keys); ++$i) {
    $eingabe = str_replace($keys[$i], $smiles[$keys[$i]], $eingabe);
  }
  echo "Guten Tag ".$eingabe;

}
else {
  // Formularansicht
?>
<html>
<body>
  <form action="usereingabe.php" method=post">
  Bitte Name eingeben:
  <input name="eingabe" type="text" size="60">
  <input type="submit" value="abschicken">
  </form>
</body>
</html>
<?
}
?>

Alles anzeigen

Und das funktioniert diesmal auch.

GreenRover

Ne leute der Fehler war ein ganz anderer:

Code

$ergebniss=mysql_query('SELECT * FROM `smiles`;');
  while ($Wert=mysql_fetch_array($ergebniss)) 
  { $smiles[$Wert['kuerzel']]='[img]images/smiles/'.$Wert['datei'].'[/img]'; }


  for ( $i=0; $i<=count($eintraege)-1 ; $i++ )
  {
   foreach ($smiles AS $key => $value)
   {
   $eintraege[$i]['text']=str_replace($key , $value, $eintraege[$i]['text']);
   }
  }

Alles anzeigen

und in der Forschleife hat die -1 gefehlet so das das ganze auf eine nicht existenten teil einer arrayangewendet werden sollte.

was aufeinmal zu 100% CPU auslastung mit 3GB ram verbrauch führt.

Torty

Hmmm...ist aber schon ziemlich weit weg von deiner 1.Version...
Nunja, zudem war in deinem 1.Post ein Doppelfehler. Es fehlten 2 ' - so dass sich der Fehler augenscheinlich aufhob und nicht bemerkt wurde.
1.) hinter bla*
2.) hinter LOL.gif

Nunja, scheint sich nun aber erledigt zu haben...

GreenRover

ja das erste war einvereinfachte version, zum reinen Verständniss des Problems.

habe die array bloß für das forum geschriben, bei mir kommt die shcon imemr direkt aus der DB.

Torty

Ok, wenn du dein Problem nun im Griff hast, ist es ja gut.

GreenRover

jup das war bloß ein kleiner denkfehler den ich aber immer wieder übersehen habe.

ODer ich muss meienr CPU beibringen im nichts zu suchen, das währe auch was.

Tags