Parser szablonów Opcje

[seba_bwo]

Proszę o ocenę oraz o to jak mogę ulepszyć moje "dziecko", komentarze powinny wystarczyć.

[PHP] pobierz, plaintext 
<?php
#sprawdzane warunków:
 # {if zalogowany}Lol{/if} => if($zalogowany){return "Lol";}
 preg_match_all("@{if (!|)(.*)}(.*){/if}@Usi",$strona,  $out);
 foreach($out[2] as $in=>$val)
 {
     if($out[1][$in]=='!')
     {
         #zamiana spełnionej negacji na wartość pola
         if(!$val){$strona=str_replace($out[0][$in], $out[3][$in], $strona);}
     }
     else
     {
         #zamiana warunku na wartość pola
         if($val){$strona=str_replace($out[0][$in], $out[3][$in], $strona);}
     }
 }
 
 #usuwanie warunków nie spełnionych
 $strona=preg_replace('@{if (!|)(.*)}(.*){/if}@Usi', '', $strona);
 
 
 # zamiana {var zmienna} na $zmienna
 preg_match_all("@{var (.*)}@Usi",$strona,  $out);
 foreach($out[0] as $in=>$val)
 {
     $name=$out[1][$in];
     $strona=str_replace($val,$name, $strona);
 }
 #
 
 # wstawienie pętli dla tablicy XYZ w miejscuy {loop XYZ} >tu akcja< {/loop}
 # {1} w html odpowiada elementowi $XYZ[1] 
 # indexy puki co tylko numeryczne
 preg_match_all("@{loop (.*)}(.*){/loop}@Usi",$strona,  $out);
 foreach($out[1] as $in=>$val)
 {
     #tymczasowa zmienna
     $redesign="af_$val";
     
     #tablica z mozliwosciami podmiany (nie udawało mi się bez tej tablicy :( )
     $a=array('{0}','{1}','{2}','{3}','{4}','{5}','{6}','{7}','{8}','{9}','{10}','{11}');
         foreach($val as $value)
         {
             #zamiana elementów z tablicy $a na elementy z zapętlanej tablicy
             $redesign.=str_replace($a, $value, $out[2][$in]);    
         }
         
     #zamiana pętli html na wygenerowany kod
     $strona= str_replace($out[0][$in], $redesign, $strona);
 }
?>
[PHP] pobierz, plaintext 

Ten post edytował seba_bwo 4.04.2009, 10:04:39

[Zyx]

Jeśli piszesz hobbystycznie, aby się czegoś nauczyć, to jak najbardziej ma to sens - w końcu najlepiej uczyć się przez praktykę. Jednak do praktycznego wykorzystania takie kilkunastolinijkowe parsery (kilkudziesięcio- i kilkuset- często też) są bez sensu z powodów, które zostały już przytoczone. Po prostu aby to faktycznie pomagało, trzeba nad tym wybitnie długo posiedzieć i stworzyć niesamowicie dobry projekt; inaczej taki parser będzie utrudnieniem, a nie pomocą.

Od strony czysto technicznej, zabierasz się za problem od niewłaściwej strony. Manualne wykonywanie kodu to ślepy zaułek, który nigdzie Cię nie zaprowadzi. Masz PHP, parser PHP ma wszystkie kwestie związane z interpretacją kodu już rozwiązanie i w dodatku zrobi to dużo szybciej, niż skrypt, więc najlepiej aby parser kompilował szablony do kodu PHP. Przy okazji miałbyś też załatwiony problem z tą Twoją tablicą. Druga rzecz to brak jakiejkolwiek logiki w kodzie: masz trzy elementy, które z technicznego punktu widzenia są tym samym, a od A do Z obsługujesz każdy z nich w inny sposób.

Ten post edytował Zyx 5.04.2009, 13:47:25