Optymalizacja kodu, ereg_replace + mysql Opcje

[Esc]

Schemat jest taki dany tekst naznaczam znacznikiem [OBRAZEK=nazwa.jpg][/OBRAZEK] Nastepnie sprawdzam ile jest takich obrazkow w tekscie - zapamietuje nazwe. Ta nazwe porownuje w bazie danych - jezeli istnieje to wyciagam reszte informacji o obrazku typu numer katalogowy, opis czy inne tam po czym dopiero w petli zamieszczam ereg_replace do podmiany znacznika na wszystkie opisy - czy to nie jest troche za duzo dla serverka ? Nie dalo by sie tego jakos zoptymalizowac ?

[PHP] pobierz, plaintext 
<?php
 
$tekst = explode(&#092;"[OBRAZEK=\", $opis);
for($x=1; $x < count($tekst); $x++){
$slowo= explode(&#092;"][/OBRAZEK]\", $tekst[$x]);
$szukane_slowo[$x]= $slowo[0];
}
 
$table_name3 = &#092;"01_zdjecia\";
$sql3 = &#092;"SELECT * FROM $table_name3\";
$pyt3 = mysql_query($sql3,$connection);
while ($rekord3 = mysql_fetch_array ($pyt3)) {
$y++;
if (($szukane_slowo[$y] == $rekord3[Nazwa]) or ($x > $y)) {
$z_bazy[$y] = $rekord3[Nazwa];
$z_bazy_opis[$y] = $rekord3[Opis];
}}
 
for ($z=1; $z < $x; $z++) {
$opis = ereg_replace(&#092;"[OBRAZEK=$z_bazy[$z]][/OBRAZEK]\", \"<BR><CENTER><a href =\"img/$z_bazy[$z]\">$z_bazy[$z]</A><BR>$z_bazy_opis[$z]</CENTER><BR>\", $opis);
}
 
?>
[PHP] pobierz, plaintext 

[bendi]

Ja bym to zrobil tak:

[PHP] pobierz, plaintext 
<?
//szukamy obrazkow
preg_match_all( '%[OBRAZEK=\"(.*?)\"][/OBRAZEK]%s', $opis, $aSzukaneSlowo );
 
//tablica $aSzukaneSlowo[1] zawiera interesujace nas informacje
//nastepnie pobieramy informacje o plikach z bazy
$hRes = mysql_query( 'SELECT * FROM 01_zdjecia WHERE Nazwa IN (\"'. join( '\",\"', $aSzukaneSlowo[1] ) . '\")' );
 
//no i zamieniamy co trzeba korzystajac z str_replace zamiast ereg_replace, bo jes
 szybsze
while( ( $aRow = mysql_fetch_array( $hRes ) ) !== false ) {
    $opis = str_replace('[OBRAZEK=\"' . $aRow['Nazwa'] . '][/OBRAZEK]', '<div style=\"text-align:center;\"><a href =\"img/' . $aRow['Nazwa'] . '\">' . $aRow['Nazwa'] . '</a><br >' . $aRow['Opis'] . '</div>', $opis);
}
?>
[PHP] pobierz, plaintext 

Zastapilem tez znacznik <center> div'em, gdyz ten pierwszy zostal wycofany nawet ze specyfikacji HTML 4.0.

Ten post edytował bendi 22.03.2005, 15:09:52

[Esc]

Dzieki serdeczne bendi po Twojemu znacznie sie przyspieszylo wykonywanie a dziala dokladnie tak samo jak powinno.