toelPHP - przekształcanie obrazów, gotowe rozwiązanie Opcje

[toel]

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 (IMG:style_emoticons/default/winksmiley.jpg)

P.S.2. Jeśli ktoś już będzie oceniał to był bym wdzięczny od razu za prostą ocenkę layoutu strony projektu (IMG:style_emoticons/default/winksmiley.jpg)

Pozdrawiam,
Paweł

[Crozin]

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ść

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