Skrypt uploadu iimg, sourceforge Opcje

[nanetorg]

Witam, mam do oceny skrypt uploadu obrazków.
@demo: http://iimg.pl

Skrypt do pobrania i testowania:
http://sourceforge.net/projects/iimg

[rzymek01]

wygląd oceniony to teraz kod (IMG:style_emoticons/default/smile.gif)
nie jest za dobrze:
- kod jest chaotyczny, mało czytelny
- używasz ciągle @ (ukrywanie błędów) i to nawet w miejscach gdzie to niepotrzebne, moja rada: nie używaj w ogóle @ tylko obsługuj błędy
- do dołączania plików typu engine.php użyj require zamiast include, bo w przypadku jakiegoś błędu zatrzyma się wykonywanie skryptu
- stałe pisz z wielkich liter oraz używaj funkcji defined
- nie filtrujesz danych od użytkownika, pozwalasz na ładowanie obrazków spoza katalogu `udir`
- mieszasz często ze sobą kilka różnych aspektów, podziel to
- k'gwoli optymalizacji, zamiast $size /= 1024 , możesz użyć przesunięcia bitowego (IMG:style_emoticons/default/smile.gif)
- trochę nie pomyślany ten system obsługi języków, masz swój format i za każdym razem parsujesz to wszystko,
jeśli nie zrzucasz tego do kogoś od frontendu, to zrób po prostu tablicę asoscjacyjną, wychodzi dokładnie to samo, co masz po parsowaniu
- przykład:

[PHP] pobierz, plaintext 
$data = file(idir . '/' . $_type . '.' . $_value . '.php')) {
      @array_shift($data);
      $data = implode('', $data);
[PHP] pobierz, plaintext 

zamiast użyć funkcji file_get_contents

Podsumowując przepisz cały kod pisząc obiektowo, bo teraz jakość tego kodu jest naprawdę mizerna, i już sobie wyobrażam jak będzie wyglądać dodanie jakiejś funkcjonalności czy pielęgnacja takiego kodu.
Nie chce Cię zniechęcać tylko uzmysłowić, że czeka Cię wiele pracy i żebyś nie osiadał na laurach (IMG:style_emoticons/default/wink.gif)