Witam,
Napisałem funkcje rekurencyjną:

[PHP] pobierz, plaintext 
function child_counter($child_id)
{
	 global $level;
 
	if($child_id!=0)
	{
	$level++;		
$query6=mysql_query("SELECT categories_id,parent_id from categories where categories_id='$child_id'",$GLOBALS[db]);	
	$d=mysql_fetch_array($query6);
	child_counter($d[1]);
 
	}
	else
	return $level;	
 
 
 
}
[PHP] pobierz, plaintext 

Do której podaje jako argument id rodzica z tabeli z kategoriami. Tabela jest wzorowana na OSCommerce:

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `categories` (
  `categories_id` int(11) NOT NULL AUTO_INCREMENT,
  `categories_image` varchar(64) DEFAULT NULL,
  `parent_id` int(11) NOT NULL DEFAULT '0',
  `sort_order` int(3) DEFAULT NULL,
  `date_added` datetime DEFAULT NULL,
  `last_modified` datetime DEFAULT NULL,
  `pos` int(3) NOT NULL,
  PRIMARY KEY  (`categories_id`),
  KEY `idx_categories_parent_id` (`parent_id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin2 AUTO_INCREMENT=130 ;
 
[SQL] pobierz, plaintext 

Problem w tym, że funkcja po kilku iteracjach gdy dojdzie już do rodzica równego 0 nic nie zwraca. Gdzie może być błąd?

[PHP] pobierz, plaintext 
if ($child_id != 0) {
	$level++;
	$query6 = mysql_query("SELECT categories_id,parent_id from categories where categories_id='$child_id'", $GLOBALS[db]);
	$d = mysql_fetch_array($query6);
	child_counter($d[1]);
        //Tutaj można by coś zwrócić ;]
 
}else{
	return $level;
}
[PHP] pobierz, plaintext 

Teraz popatrz ; )

Dzięki, ale już sobie poradziłem do...while zamiast rekurencją.

[PHP] pobierz, plaintext 
function child_counter($child_id)
{
	if($child_id!=0)
	{
 
	do
	{
	$level++;		
$query6=mysql_query("SELECT categories_id,parent_id from categories where categories_id='$child_id'",$GLOBALS[db]);	
	$d=mysql_fetch_array($query6);
	$child_id=$d[1];
 
	}while($child_id>0);
	}
 
	return $level;	
 
 
 
}
[PHP] pobierz, plaintext 

A tak z ciekawości jeszcze dwie kwestie: w tej rekurencji jest problem z zerowaniem tego levelu - w momencie gdy wywołuje się funkcje wiele raz - unset nie zdaje egzaminu.

A po drugie dlaczego: return musi być po pierwszym if a nie w else? Na logikę przecież: funkcja jest wywoływana jeszcze raz - z parametrem 0 (dochodzi do 0) - wtedy niespełniony jest if i else powinno zwracać? Coś przeoczyłem w moim rozumowaniu?