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

[Cypherq]

Może się mylę, ale jak już używasz funkcji to nie lepiej opakować je w klasę i dodać pole publiczne zamiast bawić się globalami?

[Kildyt]

- poczytaj o programowaniu obiektowym,
- global'e nie są najlepszym rozwiązaniem,
- polskie nazwy - po tym poznasz amatora

To tak na pierwszy rzut oka.

[lysiu]

ok, dzięki. Dzisiaj to poprawie (IMG:style_emoticons/default/czarodziej.gif)

[patryczakowy]

- polskie nazwy - po tym poznasz amatora

Gratuluje poczucia humoru

[darko]

Logiki, ani wydajności nie analizowałem, ale

dobrze:

-utrzymanie jednolitej konwencji nazewniczej funkcji: oddzielające podkreślniki (_) i rozpoczynanie nazw od "polodm"

żle:

- lepiej, wygodniej, estetyczniej i przejrzyściej użyć klas(y) i metod zamiast grup funkcji
- polskie nazewnictwo funkcji (standardem jest używanie języka angielskiego w kodzie, nawet jeśli odbiorcą skryptu jest Polak/Polka)
- jeśli chcesz sprawdzić czy zmienna jest NULL to robi się to tak:

if(is_null($var)) {

}

- zamiast zwracać z funkcji typ NULL lub string "(NULL)" - zwróć wartość false
- te drugie nawiasy nie są potrzebne, wynik operacji modulo i tak zostanie przypisany do zmiennej $liczba: if( $liczba > 9999 ) $liczba = ($liczba%10000);


To tak pobieżnie, już na początku napisałem, że nie analizowałem ani logiki ani pod kątem "co by można tu zrobić szybciej i krócej".

Życzę powodzenia w dalszej nauce PHP

[Cypherq]

Gratuluje poczucia humoru

A nie? Jak pracujesz w firmie i robisz jakiś system internetowy to gdy zajmie się nią zagraniczna grupa, szybko uczy się Polskiego. Co więcej, jak bierzesz system zrobiony przez Duńczyków, to uczysz się Duńskiego. No i CakePHP, on pewnie też był pisany po polsku.

[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.

[Cypherq]

Z jakichś powodów Twoja funkcja przekracza maksymalny czas który dostaje od serwera na wykonanie się. Przetestuj ją (czas jej wykonania w różnych momentach, żeby znaleźć dziurę) funkcją microtime()

[lysiu]

Z jakichś powodów Twoja funkcja przekracza maksymalny czas który dostaje od serwera na wykonanie się. Przetestuj ją (czas jej wykonania w różnych momentach, żeby znaleźć dziurę) funkcją microtime()

Przecież napisałem kiedy i gdzie się zapętla. Tylko nie wiem dlaczego, na #wordpress też nie wiedzą, a pewnie błąd jest zbyt oczywisty by go dostrzec (IMG:style_emoticons/default/smile.gif)

[Cypherq]

Oj, bo takie rzeczy się pisze w poście a nie komentując kod.

http://wordpress.org/tags/apply_filters
http://codex.wordpress.org/Function_Reference/apply_filters
http://codex.wordpress.org/Function_Reference/get_the_time

Masz i grzeb, że tak powiem. Wtedy naprawdę się czegoś nauczysz i będziesz miał z tego satysfakcję.

[lysiu]

tak doładniej to zawiesza się na:

[PHP] pobierz, plaintext 
$dateformatstring = preg_replace( "/([^\\\])a/", "\\1" . backslashit( $datemeridiem ), $dateformatstring );
[PHP] pobierz, plaintext 

;-)

przydało by się gdb, a pozatym jak to się może zawiesić, tego nie rozumiem.

[Cypherq]

A na jakim tekscie operuje ta funkcja?

[lysiu]

dodałem echo przed zapętlającą się funkcją:

[PHP] pobierz, plaintext 
		echo '<br />$datemonth_abbrev=\''.$datemonth_abbrev.'\',<br /> $dateformatstring=\''.$dateformatstring.'\';<br /><br />';
		$dateformatstring = preg_replace( "/([^\\\])a/", "\\1" . backslashit( $datemeridiem ), $dateformatstring );
[PHP] pobierz, plaintext 

i oto część outputu: http://pastie.org/648233
a tu jak chcesz masz cały, wczytuje się dokładnie 30s: http://irci.mine.nu:666/

[Cypherq]

Łe? Tego kodu nie ma w kodzie wtyczki który wkleiłeś, wklej cały kod.

[lysiu]

heh (IMG:style_emoticons/default/winksmiley.jpg) To jest funkcja wordpress; wp-include/functions.h -> http://xref.yoast.com/2.8/nav.html?wp-incl...source.html#118 <-- linia 118, przed tą linią dodałem echo. Wcześniej wkleiłem swój kody przy którym się zawiesza, i napisałem jaki jest błąd, wątpię, żeby to był problem wordpress'a, raczej moja funkcja, wywołuje go za dużo razy, przynajmniej tak mi się wydaję.

jak widać w linku http://irci.mine.nu:666/ $dateformatstring cały czas się rozrasta, i gdyby nieograniczenia czasu wykonywania skryptu, rozrastało by się w nieskończoność (IMG:style_emoticons/default/smile.gif)

Ten post edytował lysiu 9.10.2009, 16:03:01

[thomson89]

- polskie nazwy - po tym poznasz amatora

Ja siedzę w php już kilka lat i nadal używam polskich nazw, bo w końcu jestem polakiem!

[Cypherq]

Piszesz dla siebie - ok. I to nazywamy programowaniem amatorskim. Gdybyś chciał stworzyć coś większego, niekoniecznie dla firmy, dla klienta, po prostu napisać klasę, którą chciałbyś się podzielić ze światem, pewnie napiszesz ją po polsku, co? A ludzie z innych państw, chcący przeanalizować działanie Twojej klasy, lub ją zmienić, będą musieli się nauczyć polskiego, tak?