[PHP]Problem z chroniona funkcja w klasie Opcje

[uki8877]

Witam

Mam nadzieje ,ze to ostatnie pytanie na dzis (IMG:style_emoticons/default/smile.gif)

chce utworzyc funkcje ktora bedzie mi zwracac kilka linijek kodu:

[PHP] pobierz, plaintext 
protected function domDOC() {
			libxml_use_internal_errors(true);
			$dom = new DOMDocument;
			$dom->loadHTML($this->getHighscore());
			$domXPath = new DomXPath($dom);
 
		}
[PHP] pobierz, plaintext 

no i tu mam problem bo nie wychodzi mi to

probowalem tak:

[PHP] pobierz, plaintext 
protected function domDOC() {
			libxml_use_internal_errors(true);
			$dom = new DOMDocument;
			$dom->loadHTML($this->getHTML());
			$domXPath = new DomXPath($dom);
 
			return $this->domDOC();
 
		}
[PHP] pobierz, plaintext 

Jednak to nie dziala. Natomiast gdy ten kod wkleje "normalnie' to wszystko dziala

czyli np w klasie count taki kod nie zadziala :

[PHP] pobierz, plaintext 
public function count() {
	$this->domDOC();
 
	return $dom->getElementsByTagName('option')->length;
}
[PHP] pobierz, plaintext 

a taki zadziała:

[PHP] pobierz, plaintext 
public function count() {
	libxml_use_internal_errors(true);
	$dom = new DOMDocument;
	$dom->loadHTML($this->getHTML());
	$domXPath = new DomXPath($dom);
 
	return $dom->getElementsByTagName('option')->length;
}
[PHP] pobierz, plaintext 

Bede wdzieczny za porady jak to usprawnic (IMG:style_emoticons/default/smile.gif)

[IProSoft]

Cóż za stworek, chyba podstawy się kłaniają...

W konstruktorze powinieneś zadeklarować argument DOMDocument, który później możesz wielokrotnie wykorzystać bez potrzeby ładowania na nowo.

Podaję Ci tylko przykład, nie rozwiązanie.

[PHP] pobierz, plaintext 
public $dom;
 
public function __construct()
{
	$this->dom = $this->getDOMDocument();
}
 
protected function getDOMDocument()
{
	libxml_use_internal_errors( true );
	$dom = new DOMDocument;
	$dom->loadHTML( $this->getHTML() );
 
	return $dom;
}
 
protected function getDomXPath( \DOMDocument $dom )
{
	return new DomXPath( $dom );
}
 
public function count($string)
{
	return $this->dom->getElementsByTagName('option')->length;
}
[PHP] pobierz, plaintext 

[uki8877]

Cóż za stworek, chyba podstawy się kłaniają...

W konstruktorze powinieneś zadeklarować argument DOMDocument, który później możesz wielokrotnie wykorzystać bez potrzeby ładowania na nowo.

Podaję Ci tylko przykład, nie rozwiązanie.

[PHP] pobierz, plaintext 
public $dom;
 
public function __construct()
{
	$this->dom = $this->getDOMDocument();
}
 
protected function getDOMDocument()
{
	libxml_use_internal_errors( true );
	$dom = new DOMDocument;
	$dom->loadHTML( $this->getHTML() );
 
	return $dom;
}
 
protected function getDomXPath( \DOMDocument $dom )
{
	return new DomXPath( $dom );
}
 
public function count($string)
{
	return $this->dom->getElementsByTagName('option')->length;
}
[PHP] pobierz, plaintext 

No klaniaja, klaniaja (IMG:style_emoticons/default/smile.gif) ale będę się uczył dalej, dzięki :-)