[php]Call to a member function on a non-object, jak prawidłowo odwołać się do metody obiektu w obiekcie Opcje

[onlyX]

Witam!

Przerabiam kod strukturalny na klasę i napotkałem błąd.
Najpierw kod:

[PHP] pobierz, plaintext 
<?php
 
require_once 'bbcode/stringparser_bbcode.class.php';
 
 
//kontener 
class BBCode{
 
	var $bbcode;
 
	//konstruktor
	function BBCode() {
		$bbcode = new StringParser_BBCode ();
 
		$bbcode->setParagraphHandlingParameters ("\n\n", "<p>", "</p>");
 
		//PHP
		$bbcode->addCode ('php', 'callback_replace', 'callbackPHP', array ('start_tag' => '<code>', 'end_tag' => '</code>'),
		'code', array ('block', 'inline'), array ('code'));
	}
	 
	//callbacki
	//PHP
	function callbackPHP($action, $attributes, $content, $params, &$node_object) {
		return codeHighlight($content, 'php');
	}
 
	//funkcja kolorujaca kod za pomoca Geshi'
	function codeHighlight($str, $lang) {
		//tymczasowo
		return 'jezyk: '.$lang.' kod: '.$str;
	}
}
 
 
//----- MAIN -------
if (!empty($_POST['text'])) {
 
	$bbcode = new BBCode();
	echo $bbcode->bbcode->parse ($_POST['text']);
}
 
?>
[PHP] pobierz, plaintext 

Po wysłaniu formularza otrzymałem błąd:

[HTML] pobierz, plaintext 
Fatal error: Call to a member function parse() on a non-object in E:\WORKS\_usr\highlight\withbbcode2.php on line 39
[HTML] pobierz, plaintext 

W przed przepisaniem do klasy nie miałem tego problemu, wszystko działało poprawnie. Gdzie zrobiłem błąd?

Ten post edytował onlyX 7.05.2008, 15:26:36

[sniezny_wilk]

$bbcode->bbcode - jest zmienną, a nie obiektem stąd nie możesz wywołać funkcji parse.

Ten post edytował sniezny_wilk 7.05.2008, 15:11:02

[nevt]

Po pierwsze - prosze dodać odpowiedni tag do tematu - zgodnie z zasadami publikacji w przedszkolu - inaczej zamknę lub usunę temat.

a co do problemu.
kod który przerabiasz pochodzi z PHP4 a prawdopodobnie odpalasz go na serwerze PHP5
a różnica jest taka, że w 4 konstruktor klasy miał taką samą nazwę jak klasa, a w 5 to po prostu __construct()

więc:

[PHP] pobierz, plaintext 
<?php
// zamień
function BBCode() {
 
// na
public function __construct() {
?>
[PHP] pobierz, plaintext 

i będzie ok.

[dr_bonzo]

[PHP] pobierz, plaintext 
<?php
private $bbcode;
function BBCode() {
		$bbcode = new StringParser_BBCode (); // tutaj uzywaz zmiennej lokalnej, zamiast $this->bbcode
?>
[PHP] pobierz, plaintext 

przez co potem

$bbcode->bbcode->parse() nie zadziala, bo ->bbcode jest nullem

[onlyX]

Już rozumiem. Dzięki!

Co ciekawe działa z oboma konstruktorami (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) a mam PHP 5

[marcio]

Co ciekawe działa z oboma konstruktorami (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) a mam PHP 5

Dziala chyba dlatego ze w php5 mozna uruchamiac kody z php4 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[onlyX]

Mniejsza o to.
Problemów ciąg dalszy: Funkcja addCode() powinna wywołać callbackPHP(), teraz nie chce. Jak się do niej odwołać. Przed przepisaniem do klasy działała.

[PHP] pobierz, plaintext 
<?php
//kontener 
class BBCode{
 
	var $bbcode;
 
	//konstruktor
	public function __construct() {
		$this->bbcode = new StringParser_BBCode ();
 
		$this->bbcode->setParagraphHandlingParameters ("\n\n", "<p>", "</p>");
 
		//dodaj tag - funkcja powinna wywlywac funkcje callbackPHP
		$this->bbcode->addCode ('php', 'callback_replace', 'callbackPHP', array ('start_tag' => '<code>', 'end_tag' => '</code>'),
		'code', array ('block', 'inline'), array ());
	}
	 
	//callbacki
	//PHP
	public function callbackPHP($action, $attributes, $content, $params, &$node_object) {
		echo '#DEBUG: wywolano callback'; //brak reakcji
		return codeHighlight($content, 'php');
	}	
}
?>
[PHP] pobierz, plaintext 

[mike]

A skąd mamy wiedzieć dlaczego addCode() nie wywołuje callbackPHP() skoro nie znamy jej implementacji?
Pokaż kod to pogadamy.

[onlyX]

Źle się wyraziłem. Funkcja addCode() dodaje BBkod, przy użyciu którego powinna się wywołać funkcja callbackPHP().

Chyba nie muszę podawać implementacji addCode. Myślałem, że wystarczy informacja, że kiedy wywoływałem je w taki sposób, przed przepisaniem do klasy, to callbackPHP się wykonywał. Teraz nie chce, więc podejrzewam, że to wina złego odwołania się do niej, źle podany parametr w addCode, czy coś podobnego.

Macie jakieś pomysły?

[sniezny_wilk]

Skoro mike poprosił to znaczy, że jest to potrzebne. Prawdopodobnie teraz masz ją (to funkcję) w klasie do której źle się odwołujesz.

[onlyX]

Skoro prosicie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php
function addCode ($name, $callback_type, $callback_func, $callback_params, $content_type, $allowed_within, $not_allowed_within) {
		if (isset ($this->_codes[$name])) {
			return false; // already exists
		}
		if (!preg_match ('/^[a-zA-Z0-9*_!+-]+$/', $name)) {
			return false; // invalid
		}
		$this->_codes[$name] = array (
			'name' => $name,
			'callback_type' => $callback_type,
			'callback_func' => $callback_func,
			'callback_params' => $callback_params,
			'content_type' => $content_type,
			'allowed_within' => $allowed_within,
			'not_allowed_within' => $not_allowed_within,
			'flags' => array ()
		);
		return true;
	}
?>
[PHP] pobierz, plaintext 

I jeszcze jedna funkcja, która może pomoże:

[PHP] pobierz, plaintext 
<?php
function validate ($action = 'validate') {
		if ($action != 'validate' && $action != 'validate_again') {
			return false;
		}
		if ($this->_codeInfo['callback_type'] != 'simple_replace' && $this->_codeInfo['callback_type'] != 'simple_replace_single') {
			if (!is_callable ($this->_codeInfo['callback_func'])) {
				return false;
			}
 
			if (($this->_codeInfo['callback_type'] == 'usecontent' || $this->_codeInfo['callback_type'] == 'usecontent?' || $this->_codeInfo['callback_type'] == 'callback_replace?') && count ($this->_children) == 1 && $this->_children[0]->_type == STRINGPARSER_NODE_TEXT) {
				// we have to make sure the object gets passed on as a reference
				// if we do call_user_func(..., &$this) this will clash with PHP5
				$callArray = array ($action, $this->_attributes, $this->_children[0]->content, $this->_codeInfo['callback_params']);
				$callArray[] =& $this;
				$res = call_user_func_array ($this->_codeInfo['callback_func'], $callArray);
				if ($res) {
					// ok, now, if we've got a usecontent type, set a flag that
					// this may not be broken up by paragraph handling!
					// but PLEASE do NOT change if already set to any other setting
					// than BBCODE_PARAGRAPH_ALLOW_BREAKUP because we could
					// override e.g. BBCODE_PARAGRAPH_BLOCK_ELEMENT!
					$val = $this->getFlag ('paragraph_type', 'integer', BBCODE_PARAGRAPH_ALLOW_BREAKUP);
					if ($val == BBCODE_PARAGRAPH_ALLOW_BREAKUP) {
						$this->_flags['paragraph_type'] = BBCODE_PARAGRAPH_ALLOW_INSIDE;
					}
				}
				return $res;
			}
 
			// we have to make sure the object gets passed on as a reference
			// if we do call_user_func(..., &$this) this will clash with PHP5
			$callArray = array ($action, $this->_attributes, null, $this->_codeInfo['callback_params']);
			$callArray[] =& $this;
			return call_user_func_array ($this->_codeInfo['callback_func'], $callArray);
		}
		return (bool)(!count ($this->_attributes));
	}
?>
[PHP] pobierz, plaintext 

Uporałem się jakoś z tamtymi problemami i doszedłem do czegoś takiego:

[PHP] pobierz, plaintext 
<?php
require_once 'bbcode/stringparser_bbcode.class.php';
 
//kontener 
class BBCode{
 
	var $bbcode;
 
	//konstruktor
	public function __construct() {
		$this->bbcode = new StringParser_BBCode ();
 
		$this->bbcode->setParagraphHandlingParameters ("\n\n", "<p>", "</p>");
 
		$this->bbcode->addCode ('b', 'simple_replace', null, array ('start_tag' => '<b>', 'end_tag' => '</b>'),
		'code', array ('block', 'inline'), array ());
 
		//PHP
		$this->bbcode->addCode ('php', 'callback_replace', array($this, 'callbackPHP'), array ('start_tag' => '<code>', 'end_tag' => '</code>'),
		'code', array ('block', 'inline'), array ());
	}
	//funkcja kolorujaca kod za pomoca Geshi'
	function codeHighlight($str, $lang) {
		require_once 'highlight/geshi.php';
		$geshi =& new GeSHi($str, $lang);
		return $geshi->parse_code();
	}
	//callbacki
	//PHP
	public function callbackPHP($action, $attributes, $content, $params, &$node_object) {
		echo '#DEBUG: wywolano callback';
		return $this->codeHighlight($content, 'cpp');
	}
 
}
 
 
//----- MAIN -------
if (!empty($_POST['text'])) {
 
	$bbcode = new BBCode();
	echo $bbcode->bbcode->parse ($_POST['text']);
}
 
?>
[PHP] pobierz, plaintext 

Jedyny problem jest taki, że callbackPHP() wywołuje się teraz, dwa razy. Skutkom ubocznym częściowo zaradziłem zamieniając rquire... na require_once 'highlight/geshi.php'; ale mimo wszystko funkcja dalej wywołuje się raz niepotrzebnie. Myślę, że powodem jest ten zapis:

[PHP] pobierz, plaintext 
<?php
array($this, 'callbackPHP')
?>
[PHP] pobierz, plaintext 

ale nie mam pojęcia jak to zapisać prawidłowo.