Почему не работает return?

Question

[Виктор Волков]

function encrypt($text, $n) {
	
	$len = strlen($text);
	
		for($i = 1; $i < $len; $i += 2){

		$str1 .= substr($text, $i, 1);
		}	
		for($i = 0; $i < $len; $i += 2){

		$str2 .= substr($text, $i, 1);
		}

	if($n > 0 ){
		$m = $n - 1;
		$text = $str1.$str2;		
		encrypt($text, $m);
		
	} else {
		//echo  $text.' '; //выводит правильный результат!
		return $text;
	}
	
}
	
echo encrypt('This is a test!', 2); // результата нет...

Задачка на шифрование, необходимо формировать строку из каждого второго символа исходной строки. В аргументе указано сколько раз нужно прокрутить строку через этот механизм (решил рекурсией). Вопрос, почему не работает return? причем если в else рядом выводить, то видно, что функция правильно сработала, и результат есть, но если выводить результат функции, то экран пуст...

Comments

[zorca]

Код оберните, анализировать этот компот невозможно. Сразу навскидку - неинициализированы переменные: $str1 и $str2. Вы "что-то" конкатенируете с "чем-то", результат предсказуем.

Answer 1

[twobomb]

У вас что вывода ошибок нет?

function encrypt($text, $n) {	
$str1 = $str2 = "";
for($i = 0; $i < strlen($text); $i++)
	if($i%2 == 0)
		$str2 .= $text[$i];
	else
		$str1 .= $text[$i];
if($n > 0 )
	$text = encrypt($str1.$str2, $n - 1);	
return $text;
}
echo encrypt('This is a test!', 2);

Comments

[Виктор Волков]

прописаны вот такие директивы:
error_reporting(E_ALL);
Error_Reporting(E_ALL & ~E_NOTICE);
но тишина... будьте добры, пришли правильную (Вашу) шапку...

Answer 2

[Андрей Николаев]

Ну так все правильно!
Вы же получаете результат для своей строки на второй итерации, а return на первой не работает.
Заменяем:
encrypt($text, $m);
На
return encrypt($text, $m);

И видим результат: s eT ashi tist!
Можно конечно еще и функцию упростить:

function encrypt($text, $n)
{
	if ( $n < 1 )
	{
		return $text;
	}
	$str1 = $str2 = "";
	for($i = 0, $len = strlen($text); $i < $len; $i++)
	{
		if ( $i%2 == 0 )
		{
			$str2 .= $text[$i];
		}
		else
		{
			$str1 .= $text[$i];
		}
	}

	return encrypt( $str1.$str2, ($n-1));
}
  
echo encrypt('This is a test!', 2);

Answer 3

[doublench21]

Вывод ошибок для кого? $str1 и $str2 вы вообще не определили. Ну судя по коду что вы кинули.

Comments

[Виктор Волков]

я объявил их уже придав значение, повторюсь, результат через закоменченное echo выводит правильный, функция работает

[doublench21]

Виктор Волков,

function encrypt($text, $n) {
  
  $len = strlen($text);
  
    for($i = 1; $i < $len; $i += 2){

    $str1 .= substr($text, $i, 1);
    }	
    for($i = 0; $i < $len; $i += 2){

    $str2 .= substr($text, $i, 1);
    }

  if($n > 0 ){
    $m = $n - 1;
    $text = $str1.$str2;		
    encrypt($text, $m);
    
  } else {
    var_dump($text);  //выводит правильный результат!
    return $text;
  }
}
  
encrypt('This is a test!', 2); // Смотреть вывод var_dump

Возможно на каком-то этапе это переменная оказывается пуста. Смотрим var_dump