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

[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