singleton - nie można pobrać instancji w klasach dziedziczonych Opcje

[czychacz]

problem polega na tym, że przy próbie uruchomienia skryptu dostaję błąd:

Allowed memory size of 134217728 bytes exhausted (tried to allocate 43 bytes)

w projekcie stworzyłem 4 singletony. dwa z nich podczas tworzenia ustalają sobie jakieś parametry (ale to chyba nie jest ważne). dalej - w projekcie posiadam klasę "_controller". inne klasy mogą z niej dziedziczyć. gdy próbuję odpalić kod, który zawiera tworzenie obiektu $a (klasy "lol" dziedziczącej po "_controller"), dostaję wyżej wymieniony błąd.
poniżej klasa controller:

[PHP] pobierz, plaintext 
class _controller{
 
	protected $engine = null;
	protected $view = null;
	protected $db = null;
	protected $config = null;
 
	public $defaultAction = 'index';
 
	public function __construct(){
		$this->engine = _engine::getInstance();
		$this->view = _view::getInstance();
		$this->db = _db::getInstance();
		$this->config = _config::getInstance();
	}
 
}
[PHP] pobierz, plaintext 

a tu jeden z singletonów:

[PHP] pobierz, plaintext 
<?php
 
class _view{
 
	static private $instance;
	static public function getInstance(){
		if(is_null(self::$instance))
			self::$instance = new _view();
		return self::$instance;
	}
	private function __construct(){}
	private function __clone(){}
 
}
 
?>
[PHP] pobierz, plaintext 

czy ktoś wie, w czym może być problem?
edit: dodam, że po wywaleniu kodu odpowiedzialnego za przypisanie zawartości w konstruktorze klasy "_controller" wszystko idzie normalnie. kod "getInstance" jest prawie taki sam w każdym singletonie (nie licząc nazwy klasy przy wywołaniu "new").

Ten post edytował czychacz 28.04.2012, 15:01:04

[Sinevar]

Spróbuj pokombinować z tymi ustawieniami (dodatkowo możesz wklepać zaklęcie, o ile siedzisz na unixie: ulimit -a):

[PHP] pobierz, plaintext 
ini_get('memory_limit');
memory_get_usage();
memory_get_peak_usage();
[PHP] pobierz, plaintext 

- na początku sprawdziłbym coś takiego

[PHP] pobierz, plaintext 
public function __construct()
{
  echo memory_get_usage();
  $this->engine = _engine::getInstance();
  $this->view = _view::getInstance();
  $this->db = _db::getInstance();
  $this->config = _config::getInstance();
  echo memory_get_usage();
}
[PHP] pobierz, plaintext 

- potem wywaliłbym linijkę z $this->db (może jakiś ORM wyżera Ci pamieć)
- potem przeszukałbym kod, głównie pętle foreach, możę tam jest jakis memory leak
- potem popróbowałbym zwiększyć limit pamięci z poziomu php

[PHP] pobierz, plaintext 
ini_set('memory_limit', -1);
ini_set('memory_limit', "192M");
[PHP] pobierz, plaintext 

Generalnie baw się i próbuj, możesz nawet zapiąc xdebuga, albowiem jest to odpowiednia do tego sytuacja

[czychacz]

problem jest chyba gdzieś indziej. zacząłem pisać nowy projekt i wszystkie klasy są prawie puste. np. klasy _db i _view mają tylko zmienną przechowującą instancję i metody pobierające tą instancję.
próba zwiększenia limitu pamięci nawet do 192 MB kończy się tym samym. ten kod nie pozwala także wyświetlić użycia pamięci.
w całym kodzie są tylko 2 pętle (foreach, konstruktor klasy _config, przetwarza on plik ini).

temat do zamknięcia. poradziłem sobie - okazało się, że w konstruktorze klasy _controller wywoływałem pobieranie instancji, która jeszcze nie została utworzona a sam obiekt klasy _controller był tworzony w konstruktorze klasy _engine. fakt - był to rodzaj pętli

[bastard13]

Taki offtopic.
A dlaczego chcesz tutaj koniecznie mieć singletony?
Rozumiem, że każda dziedzicząca klasa ma posiadać takie same (a dokładniej - te same) atrybuty. Ja bym zrobił coś takiego:

[PHP] pobierz, plaintext 
class _controller{
 
protected static $engine;
protected static $view;
protected static $db;
protected static $config;
 
public function __construct()
{
  if (is_null(self::engine))
    $this->init();
}
 
private function init()
{
  self::engine = new _engine();
  self::view = new _view();
  self::db = new _db();
  self::config = new _config();
}
 
}
[PHP] pobierz, plaintext 

Atrybuty są statyczne, czyli dotyczą każdej klasy rozszerzającej rodzica. init() jest wywoływany tylko raz, gdy atrybuty jeszcze nie zostały zainicjalizowane. Dodatkowo unikasz tworzenia singletonów, co jest z pewnością na plus:)


Ten post edytował bastard13 30.04.2012, 10:14:40