Ocena prostego skryptu, ;-)))) Opcje

[lysiu]

Zależy mi na ocenie mojego skyptu. Uczę się i chcę wiedzieć co robię źle. Skrypt już ściągneło 14 osób ale nikt nawet nie raczył mi powiedzieć czy działa, czy może jest całkiem do kitu. Skrypt jest bardzo mały i jego przeanalizowanie nie zajmie wam dużo czasu. To mój drugi skrypt po totalnie beznadziejnej księdze gości, którą napisałem jakiś czas temu ;-) [źródło]. Planuję już następny bardziej skomplikowany skrypt (so beware!!!) ;-))))

Dzięki!!

Edit [przykład użycia]:

[PHP] pobierz, plaintext 
$l = polodm_rozloz_liczbe( 3456 );
$s = polodm_zamien_na_slowa( $l );
$k = polodm_odmieniaj( $l );
 
echo $s, ' ', $k; // wynik: Trzy tysiące czterysta pięćdziesiąt sześć komentarzy
[PHP] pobierz, plaintext 

Ten post edytował lysiu 2.10.2009, 19:39:29

[lysiu]

Dzięki za wszystkie poprawki, niektóre wprowadziłem. Miałem trochę problemu z przejściem na język obiektowy, ale sobie poradziłem. poprawione

Ale mam inny problem. Ponieważ nie byłem pewien czy moje zmienianie daty dobrze działa, dodałem opcję by ten ficzer wyłączyć.

Metoda (kod, przy którym się zapętla zaznaczyłem wykrzyknikami):

[PHP] pobierz, plaintext 
    // Niewiele zmienia
    public function data( $d = '' )
    {
        if( get_option( 'polodm_uzyj_daty' ) == 'no' )
        {
            // !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!
            // proszę mi wytłumaczyć, czemu to się zapętla w nieskończność 
            echo apply_filters('the_time', get_the_time( $d ), $d);
            // !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!
            echo "To się nigdy nie wyświetli!";
 
           // Narazie jest tak w rep:
           // echo $d;
 
            return;
        }
 
        $id = $GLOBALS['id'];
 
        // narazie, więc godzina jest po prostu wyświetlana normalnie 
        if( preg_match( "/\d*:\d*.?.?.?/", $d ) ) 
        {
            echo $d;
            return;
        }
 
        $pat_daty = get_option( 'polodm_pat_daty' );
        $uzyj_custom = trim(get_option( 'polodm_uzyj_custom'));
        $custom = stripslashes(get_option( 'polodm_custom' ));        
 
 
        $rok = get_post_time('Y', false, $id, true);
        $miesiac = get_post_time('n', false, $id, true);
        $dzien = get_post_time('j', false, $id, true);
        $go = '-go';
 
        $sprawdz = str_replace(' ', '', $custom );
        $sprawdz = str_replace("\t", '', $sprawdz);
 
 
        $pat_daty = str_replace( '[dzien]', $dzien, $pat_daty );
        $pat_daty = str_replace( '[rok]', $rok, $pat_daty );
        $pat_daty = str_replace( '[miesiac]', $this->miesiace[$miesiac], $pat_daty );
        $pat_daty = str_replace( '[]', $go, $pat_daty );
 
        if( $uzyj_custom == 'yes' && !empty($sprawdz) )
        {
            $W_POKAZ = FALSE;
            $W_DATA = TRUE;
 
            if( @eval( $custom ) !== FALSE )
            {
                if(!empty($wynik)) $pat_daty = $wynik;    
            }
        }
 
        echo $pat_daty;
    }
 
[PHP] pobierz, plaintext 

Funkcję rejestruje w wordpress:

[PHP] pobierz, plaintext 
             add_action ( 'the_time', array( &$this, 'data' ), 1, 5 );
[PHP] pobierz, plaintext 

A błąd dostaje taki:

Kod

Fatal error: Maximum execution time of 30 seconds exceeded in E:\Programy\WebServ\httpd\vhosts\wp\wp-includes\functions.php on line 118

Nie wiem, czy potrzebna jest znajomość wordpressa by to zdiagnozować (IMG:style_emoticons/default/smile.gif)
Z góry dziękuję, i pozdrawiam.