[Singleton] Wydajność Opcje

[Astarot]

Witam, postanowiłem przetestować czas wykonywania skryptu dla klasy skonstruowanej wg. wzorca Singleton, wywoływanej poprzez metode statyczną która decyduje czy należy tworzyć nowy obiekt klasy. Wywoływałem tą klasę również w tradycyjny sposób. Czas wykonywania skryptu wyglądał
następująco :

Kod

Skrypt wykonany w czasie: 1.485986 sekundy (tradycyjnie wywołany obiekt)
Skrypt wykonany w czasie: 2.203427 sekundy(wg. singleton)

Byc moze zle implementuje ten wzorzec w swojej aplikacji wiec podaje poniżej kod :

[PHP] pobierz, plaintext 
<?php
 
class Test2
{
public $licznik;
 
  private static $oInstance = false;
 
 
  public static function getinstance ( )
  {
	if ( self::$oInstance == false )
	{
	  self::$oInstance = new Test2( );
	}
	return self::$oInstance;
  }
  public function mnozenie()
	{
		return $this->licznik*10;
	}
} 
 
?>
[PHP] pobierz, plaintext 

Klasę wywołuję wg singleton tym kodem:

[PHP] pobierz, plaintext 
<?php
 
for($i=0; $i<80000; $i++)
{
$test =& Test2::getinstance ( ); 
$test->licznik=$test->licznik++;
echo $test->mnozenie();
}
 
?>
[PHP] pobierz, plaintext 

A tak tworze obiekt klasy w tradycyjny sposób(wg. moich testów ten sposób jest szybszy)

[PHP] pobierz, plaintext 
<?php
 
for($i=0; $i<80000; $i++)
{
$test = New Test2( ); 
$test->licznik=$test->licznik++;
echo $test->mnozenie();
}
 
?>
[PHP] pobierz, plaintext 

Czy może mi ktoś wyjaśnić dlaczego kod wg singleton jest wykonywany wolniej i kiedy warto
w swoich aplikacjach implementować ten wzorzec ?

Z góry dzieki za pomoc.

[hwao]

Roznica jest nie wielka.

Singleton jak sama nazwa wskazuje jest uzywany tam gdzie w calej aplikacji musi byc tylko jedna wersja obiektu i tego trzeba sie trzymac.

[LBO]

Dodam tylko, że funkcjonalność tego wzorca jest na tyle duża, że pomija się raczej takie małe wady. O ile tę małą różnicę można nazwać wadą, bo wyobraź sobie jakie narzuty na czas wykonywania, miałoby przekazywanie obiektów przez parametr/referencję - bo właśnie temu singleton zapobiega.

[mariuszn3]

Hmm.. a spróbuj taką wersję (przynajmniej to co napisałeś trochę mi nie pasuje):
Singleton:

[PHP] pobierz, plaintext 
<?php
 
class TestSingleton {
	private $licznik = 0;
	private static $oInstance = false;
 
	public static function set_instance()  {
		if (self::$oInstance === false) {
			self::$oInstance = new Test2();
			++self::$oInstance->licznik;
		}
	}
 
	public static function mnozenie() {
		return self::$oInstance->licznik*10;
	}
}
for($i=0; $i<80000; ++$i) {
	TestSingleton::set_instance();
	echo TestSingleton::mnozenie();
}
 
?>
[PHP] pobierz, plaintext 

Tradycyjny sposób:

[PHP] pobierz, plaintext 
<?php
 
class Test {
	private $licznik = 0 ;
 
	public function __construct() {
		++$this->licznik;
	}
 
	public function mnozenie() {
		return $this->licznik*10;
	}
}
 
for($i=0; $i<80000; ++$i) {
	$test = new Test;
	echo $test->mnozenie();
}
 
?>
[PHP] pobierz, plaintext 

Nie mówię, że będzie dużo lepiej, ale kod jakby właściwszy mi się wydaje.

Ten post edytował mariuszn3 3.05.2006, 15:18:20

[Hacker]

Klasę wywołuję wg singleton tym kodem:

[PHP] pobierz, plaintext 
<?php
 
for($i=0; $i<80000; $i++)
{
$test =& Test2::getinstance ( ); 
$test->licznik=$test->licznik++;
echo $test->mnozenie();
}
 
?>
[PHP] pobierz, plaintext 

A tak tworze obiekt klasy w tradycyjny sposób(wg. moich testów ten sposób jest szybszy)

[PHP] pobierz, plaintext 
<?php
 
for($i=0; $i<80000; $i++)
{
$test = New Test2( ); 
$test->licznik=$test->licznik++;
echo $test->mnozenie();
}
 
?>
[PHP] pobierz, plaintext 

zły test, chociaż wyniki poprawne singleton jest trochę wolniejszy.

[PHP] pobierz, plaintext 
<?php
$test->licznik=$test->licznik++;
?>
[PHP] pobierz, plaintext 

nigdy nie zwiększy $test->licznik
$zmienna++ zwiększa wartość zmiennej PO użyciu, a nie PRZED
mój kod testujący

[PHP] pobierz, plaintext 
<?php
class Test2
{
 
public $licznik = 1;
 
  public static function getinstance()
  {
	static $instance;
	if (!isset($instance))
	  $instance = new Test2();
	return $instance;
  }
  public function mnozenie()
	{
		return $this->licznik*10;
	}
} 
 
class Test
{
public $licznik = 1;
 
  public function mnozenie()
	{
		return $this->licznik*10;
	}
}
function getmicrotime()
{
	list($usec, $sec) = explode(" ", microtime());
	return ((float)$usec + (float)$sec);
}
$temp = 0;
$time_start = getmicrotime();
for($i=0; $i<1000000; $i++)
{
$test = Test2::getinstance();
$test->licznik=++$temp;
$temp = $test->licznik;
$test->mnozenie();
}
$time_end = getmicrotime();
echo 'Singleton: '.($time_end-$time_start).'s <br>';
 
$temp = 0;
$time_start1 = getmicrotime();
for($i=0; $i<1000000; $i++)
{
$test = New Test( ); 
$test->licznik=++$temp;
$temp = $test->licznik;
$test->mnozenie();
}
$time_end1 = getmicrotime();
echo 'Normal: '.($time_end1-$time_start1).'s';
?>
[PHP] pobierz, plaintext 

a wyniki

Kod

Singleton: 8.5278367996216s
Normal: 6.6726980209351s

Takie wyniki uzystałem na moim AMD Athlon 2500+ (Barton)

-edit-
2 min później lol

Ten post edytował Hacker 3.05.2006, 12:55:22

[em1X]

Przeciez w singletonie nie chodzi o wydajnosc, ale o to, zeby klasa nie mogla byc powielana wieloma obiektami!

Przypominam, ze powinien byc jeszcze zdefiniowany prywatny konstruktor (w twoim przypadku go nie ma i mozna tworzyc obiekty tej klasy)

Ten post edytował em1X 3.05.2006, 18:33:26