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?

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

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.

[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

Już rozumiem. Dzięki!

Co ciekawe działa z oboma konstruktorami a mam PHP 5

Dziala chyba dlatego ze w php5 mozna uruchamiac kody z php4

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 

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

Ź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?

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

Skoro prosicie

[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.