[php] return и рекурсивная функция

Не понял, зачем передавать во внутреннюю рекурсию $tree,, её тебе нужно только встречать. Попробуй в рекурсивном вызове вместо

ShowTreeWithCat($ID1, $lvl,$mod,$tree);

написать

$tree .= ShowTreeWithCat($ID1, $lvl,$mod);

а в начале функции поставь на всякий случай $tree = "";

Если не разберешься с переменными, то есть ещё один способ (будет работать 100%, но метод тормозной и кривой :)). Можно перед вызовом функции сделать кэширование вывода (для поиска ключевое слово ob_start), после функции забрать весь буфер в переменную и очистить кэш.

Kpd, попадает в вечный цикл если сделать замену

почему все данные сохраняются, кроме тех которые во втором запросе в функции?

Это не может влиять на зацикливание, что-то ещё изменил.

Потому что криво работаешь в переменными.

Я не понимаю какую роль у тебя играют параметры, передаваемые в функцию. Там достаточно только одной переменной - ID родительского раздела.

попробуйте сделать переменную ссылочной - &$tree, а от return совсем оказаться.

это хороший способ - будет работать.

Это хороший способ если делать по типу "лишь бы работало", однако желательно именно разобраться как оно работало бы, что не работает, почему не работает...

(не окуратный код одна из главных причин дыр в безопасности)

Аналогично будет работать объявление $tree глобальной переменной (и по сути это аналогично передаче по ссылке), но тоже некрасивый способ.

не то пишите -

1. у вас рекурсивный вызов не присваивает ничего ни какой переменной, а сама функия возврашает значение.

2. я вам дал совет, а использовать его или нет - дело ваше.

3. неаккуратный код и мой совет никак не соприкасаются, так что и безопасность тут не причем.

4. если эта функция совсем кривая - лучше напишите свою.

я не разбирался в тонкостях этой проги, я дал лишь совет как иногда полезно обойти трудности с возвратом функции при рекурсии. Или вы хотите, что-бы вам ее написали?

Написал, что я кривой - так напиши плиз ещё и где. А про лишние переменные, так они работают, а в рамках этого форума я их исключил из кода.

Всё вышепредложенное не работает (((((((((((((((

Ещё раз

Ты возвращаешь из рекурсии какое-то накопленное значение (вернее хочешь вернуть), но не делаешь его сохранение в точке вызова

Поставь здесь присваивание и всё.

function  ShowTreeWithCat($ParentID, $lvl,$mod,$tree) {

global $my_link;

global $lvl;



$lvl++;		

	

//echo $tree."<br>";



$sSQL = "SELECT * FROM cat_category WHERE pid = " . $ParentID . " and status='yes' ORDER BY sort";

	

$result = mysql_query($sSQL, $my_link);

	

if (mysql_num_rows($result) > 0) 

   {

	$tree.=("<UL>\n");

			while ( $row = mysql_fetch_array($result) ) 

			{

				$ID1 = $row["id"];

				$tree.=("<LI>\n");

				$tree.=("<A title=\"$row[title]\" HREF=\"" . "index.php?mod=cat&lmod=category&id=" . $ID1 . "\"><b>" . $row["title"] . "</b></A>" . "&nbsp;&nbsp;\n");

				

				$resultA = mysql_query("SELECT id,title,status,sort,pid FROM  cat_content WHERE pid=$row[id] and status='yes' order by sort",$my_link);

			  	if (mysql_num_rows($resultA) > 0)

			  	{

			  	$tree.= "<ul>\n";

						if ($myrowA = mysql_fetch_array($resultA))

						 do

						  {

							$tree.= "<li><a title=\"Редактировать товар\" href=\"index.php?mod=cat&lmod=content&id=$myrowA[id]\">$myrowA[title]</a></li>";

						  }

						while ($myrowA = mysql_fetch_array($resultA));

				$tree.= "</ul>\n";

			  	}

				ShowTreeWithCat($ID1, $lvl,$mod,$tree); 

				$lvl--;

			}

	$tree.=("</UL>\n");

	}

	return $tree;

}

щас выглядит вот так и не работает