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

Question

[MaksPaverov]

Мой код:

index.php

case '': { 
            include("moduls/main.php");
            $txt = menu();
            break;
}

main.php

<?
function menu()
{
$sm_read = file("templates/menu.html");
$sm_read = implode("",$sm_read);
return $sm_read;
}
?>

Так вот, menu.html не выводится
Если return заменить на echo, то всё работает

Comments

[Александр Шаповал]

return на экран ничего не выводит, потому и не выводится
на экран выводят echo и всякие print*

[MaksPaverov]

Александр Шаповал, у меня был другой скрипт, там немного другой код, решил вставить в свой:

index.php

if($page == "index"){
include("moduls/menu.php");
$txt = menu();
}

menu.php такой же
И с таким всё хорошо выводилось

[Александр Шаповал]

MaksPaverov, а что в moduls/menu.php?

[MaksPaverov]

Александр Шаповал,

main.php

<?
function menu()
{
$sm_read = file("templates/menu.html");
$sm_read = implode("",$sm_read);
return $sm_read;
}
?>

Answer 1

[Lander]

$txt = menu();
А что дальше? Ну засунули вы меню в переменную. Потом что происходит?

Comments

[MaksPaverov]

return $sm_read;
Возвращается файл, насколько я понимаю, и выводится на экран

[Lander]

и выводится на экран

Где выводится? Какой командой? Выводить можно либо через echo, либо через var_dump / print_r и так далее. Код у вас отрабатывает правильно. return работает. Просто вы не выводите переменную $txt.

[MaksPaverov]

Lander, у меня был другой скрипт, там немного другой код, решил вставить в свой:

index.php

if($page == "index"){
include("moduls/menu.php");
$txt = menu();
}

menu.php такой же
И с таким всё хорошо выводилось

[Lander]

MaksPaverov, уверены что case срабатывает? Вообще, что за идиотская привычка, кидать три строчки кода и всё. Давайте весь!

[MaksPaverov]

Lander,

<?php

    $url = explode('/',strtolower(substr($_SERVER['REQUEST_URI'], 1)));

    switch($url[0]) {
        case '': { 
            include("moduls/main.php");//?????????? ??????
            $txt = menu();
            break;
        }
?>

[Lander]

MaksPaverov, А вас не смущает что у вас количество открытых фигурных скобок не совпадает с количеством закрытых?

[MaksPaverov]

Lander, их 2, криво вставил

[Lander]

var_dump($url); что выводит?

[MaksPaverov]

Lander, array(2) { [0]=> string(0) "" [1]=> string(0) "" }

[Lander]

Это всё содержимое файла index.php

[MaksPaverov]

Lander, не всё, есть ещё другие case, но не вижу смысла их показывать

[DanKud]

О чем вообще все эти комментарии? Вам же сразу написали, что вы ничего не выводите на экран. Вы присваиваете переменной значение выполнения функции, но никуда ее не выводите:

.....
$txt = menu();
echo $txt;
break;
.....

Если вы все-таки где-то потом ее выводите и ничего не выводиться, значит проблема в конструкции switch

Answer 2

[Antony Tkachenko]

Вы просто не понимаете как работает return

php.net/manual/ru/function.return.php

function getMenu() {
   return 'getMenu';
}

function printMenu() {
  echo 'printMenu';
}

$a = getMenu();
echo $a; // можно сразу вывести: echo getMenu();
printMenu();

Answer 3

[sa1ch]

Всё просто:
return возвращает.
echo выводит.

Вам нужно вывести через echo то, что вернул return.