Witam

Mam nadzieje ,ze to ostatnie pytanie na dzis

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

Nie przedstawiłeś nam pełni klasy, ale czy nie mylisz sobie protected z private?

W tym przypadku nie ma to znaczenia ponieważ do funkcji odwołuje się tylko wewnątrz klasy, nie poza nią

[PHP] pobierz, plaintext 
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) {
	$dom = $this->getDOMDocument();
 
	return $dom->getElementsByTagName('option')->length;
}
[PHP] pobierz, plaintext 

Tylko, że to całkowicie bezsensu za każdym razem gdy będziesz chciał coś zrobić stworzysz nową instancję DOMDocument. Przemyśl to jeszcze

@CuteOne

Skoro uwazasz ze to bez sensu to nie mam zamiaru polemizować :-)

W takim razie może podpowiesz jak byś rozwiązał ten problem ?

stwórz sobie metodę która będzie inicjować w twoim wypadku najlepiej w konstruktorze.

czyli to co podal CuteOne ? ->

[PHP] pobierz, plaintext 
public function __construct() {
 
			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);
 
			}
		}
[PHP] pobierz, plaintext 

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 ale będę się uczył dalej, dzięki :-)