Witam!

Ostatnio z przypływu weny stworzyłem sobie własną klasę do przetwarzania obrazów (przede wszystkim chciałem móc szybko tworzyć miniatury) i postanowiłem ją upublicznić.

Dlatego też podaję link

http://pawelmichalski.pl/toelphp/

i proszę o konstruktywną krytykę oraz jakieś uwagi co powinno i mogło się znaleźć w kolejnych wersjach (jeśli takowe powstaną).


P.S. Wiem, że istnieją inne gotowe rozwiązania, ale tak jak mówiłem...przypływ weny

P.S.2. Jeśli ktoś już będzie oceniał to był bym wdzięczny od razu za prostą ocenkę layoutu strony projektu

Pozdrawiam,
Paweł

Pod IE6 strona się pruje i flash nie zawiera żadnej treści.

Powinieneś do klasy dodać możliwość generowania efektu cienia i zrobić dokumentacje po polsku.

To daj jeszcze otwartą licencję i
Zastrzeżenia/wątpliwości:
1. pomieszane nazewnictwo polski i angielskie ($rozszerzenie)
2. brak komentarzy-dokumentacji phpDoc
3. czy Ty kochasz static ? - po co sobie tak utrudniać życie ?
4. nie piszesz nic o typach rozszerzeń obsługiwanych obrazków (jpg, png, gif)
5. komunikat o treści: No - server doesn\'t support technology used in this library <--- mi nic nie mówi, lepiej byłoby wypluć, że serwer nie obsługuje rozszerzenia GD
6. brakuje mi możliwości zmiany czcionki dla znaku wodnego (domyślnie jest Verdana)
7. nie znalazłem także możliwości zapisywania utworzonych miniatur na serwerze (ale pewnie gdzieś jest)
8. zamiast tego - swoją drogą fajnego ascii arta dałbyś komentarz phpDoc z: opisem klasy, autorstwem, wersją, wymaganiami, licencją etc.

Podsumowując:
ogólnie kawał dobrej roboty, kontynuuj prace i aktualizuj swój projekt, myślę, że wielu ludziom może przydać się taka podręczna niewielkich rozmiarów klasa do tworzenia miniatur.

ps. layoutu strony nie ocenię, bo na grafice się nie znam i niech tak zostanie

Och...czy na prawdę dalej musimy wspierać IE6 - już nawet Google się go ma wyprzeć ze swoimi aplikacjami

Nad rozwiązaniem cienia pomyślę, bo faktycznie może się przydać.

Dokumentacji PL w swoim czasie na pewno powstanie, tylko najpierw chciałem wystartować do szerszego grona (w razie gdyby ktoś spoza PL się na moją klasę naciął




Fakt - licencja nie jest wspomniana, ale w założeniach jest "bierzcie i róbta co chceta" ;]

2. Były, ale się zmyły - ograniczyłem się do dokumentacji poza klasą (chociaż fakt, teraz widzę, że nie zawiera wszystkich informacji)
3. static jest tylko na użytek singletona
6. Dowolny Truetype font. Wrzucasz plik, w parametrach funkcji podajesz nazwę ze ścieżką i proszę bardzo
7. funkcja EndCreate jako drugi parametr przyjmuje nazwę pliku (z rozszerzeniem i może być ze ścieżką) - oczywiście jeśli zapisze na serwer to już nie wyświetla AdHoc
8. goto 2; Ale ok... przy następnej wersji postaram się z powrotem dodać

1. Kod
   private static $image,$typ,$rozszerzenie,$img,$quality,$transparent,$mini,$r,$g,$b;
   Tam jakieś polskie wstawki pominę, ale cały skrypt mimo iż nie sam w sobie jakoś specjalnie rozbudowany przez powyższe stracił w ogóle jakikolwiek sens użytkowania. Nie mogę tworzyć/modyfikować dwóch obrazów równolegle...
2. Nie za bardzo wiem po co zaimplementowałeś tam wadliwego singletona? - wadliwego bo nie zablokowałeś możliwości sklonowania obiektu.
   toelPHP::CanIUseThisLib() jest kompletnie bezużyteczna. Zamiast wyświetlać jakiś napis niech zwróci proste true/false (ew. wyrzuci wyjątek)
3. Bardzo dziwne nazewnictwo: startCreate, endCreate?
4. Dokumentacji nie czytałem, ale w kodzie widzę, że kolor podaje się jako... tekst: "215.4.154"? Przecież to będzie straszne w użyciu. Wymuś tam podanie tablicy (array(215, 4, 154)), obiektu (new Color(215, 4, 154)) czy chociażby trzech oddzielnych parametrów. Nie będziesz się wtedy też musiał bawić w jakąś "dekodowanie" koloru z postaci heksadecymalnej itp. - zawsze można podać: array(52, 0xFF, 075).
5. Mogłeś oddzielić Twojego toelPHP-a (nie wiem czy tak to powinienem odmieniać) od GD. Dzięki dodatkowej warstwie abstrakcji mógłbyś łatwo utworzyć sterownik dla GD, ImageMagick itp.
6. Do pobrania rozszerzenia z nazwy pliku służy pathinfo - nie ma sensu bawić się w jakieś end(explode...))
7. Włącz sobie raportowanie błędów na E_ALL
8. Na kod samych "efektów" nie patrzę, ale IMO lepiej jakbyś wydzielił je od samego "głównego obiektu". Przedrostki "Effect" to średni pomysł. Lepiej zrobić coś w stylu:

   [PHP] pobierz, plaintext 
   $toelObj->effect('rotate', array(90))->effect('colorize', array(255, 220, 50))
   [PHP] pobierz, plaintext 

Co do 1 to faktycznie - nie wiem jak to przegapiłem i oczywiście jest przez static pozbawione sensu...
2. hmm...czemu wadliwy ? Tworzy tylko jeden obiekt i nawet jeśli ponownie wywołasz to nie stworzy obiektu tylko zwróci już istniejący. A konstruktor jest prywatny więc przez niego obiektu nie utworzysz. Tak więc uświadom mnie, bo chyba przegapiam jego wadliwość

Pozostałe informacje "przyjmuję na klatę" i postaram się je w jakiś sposób uwzględnić przy poprawkach

Tak ale właśnie przez singletona uniemożliwiasz ŁATWĄ prace na wielu zdjęciach. Dodatkowo poszedłeś w taką niezbyt dobra praktykę jak większość klas, a mianowicie umieszczania wszystkich metod w klasie. Jest to bez celowe z tego między innymi względu, że jeżeli chcesz dodać nowy filtr musisz ingerować w kod klasy (a nie powinno się). Lepiej umieścić opcje transformowania zdjęcia w oddzielnym katalogu.

Tzn. chciałbyś przetwarzać wiele zdjęć równolegle, czy w jednym wywołaniu ? Bo jeśli chodzi o to drugie, to własnie temu służy startCreate i EndCreate, które pozwalają wykonać operacje dla wielu obrazków na tym jednym obiekcie...

No chyba, że chodzi o to pierwsze - bo fakt...jeśli byś chciał "otworzyć" nowe przekształcanie i przed jego zakończeniem dla danego obrazu, uruchomić nowe, to się nie da...


----------------



{a w ogóle to własnie o to chodziło - dzięki za wszelkie wskazówki i krytykę }

Taka, że nadal mam możliwość utworzenia drugiego obiektu tej klasy - popatrz na poniższy przykład:

[PHP] pobierz, plaintext 
<?php
 
class MyAbcClass {
	private static $instance;
 
	private $property;
 
	private function __construct() {}
 
	public function setProperty($property) {
		$this->property = $property;
	}
 
	public function getProperty() {
		return $this->property;
	}
 
	public function getInstance() {
		if (null == self::$instance) {
			self::$instance = new self();
		}
 
		return self::$instance;
	}
}
 
// $obj = new MyAbcClass(); # Fatal Error
 
$obj = MyAbcClass::getInstance();
$obj->setProperty('test value');
$obj->getProperty(); // Returns: test value
 
$clone = clone $obj;
$clone->setProperty('another value');
$clone->getProperty(); // Returns: another value
$obj->getProperty();   // Returns: test value 
[PHP] pobierz, plaintext 

Musiałbyś dodać jeszcze:

[PHP] pobierz, plaintext 
private function __clone() {}
[PHP] pobierz, plaintext 

By poprawnie zaimplementować ten wzorzec.

Co do równlogegłości... chodzi o coś takiego:

[PHP] pobierz, plaintext 
$imgA = new Toel('filepath.png');
$imgB = new Toel('another/filepath.png');
 
if (true) {
    $imgA->doSth();
} else {
    $imgB->doSthElse();
}
 
$imgA->save();
$imgB->rotate(-90)->save();
[PHP] pobierz, plaintext