Не работает return в конструкции if?

Question

[php_raper]

Пример кода (кусок функции)

public function moveFile($file, $path)
    {
        if (move_uploaded_file( $file, $path))
        {
            //$path = "https://" . $_SERVER["SERVER_NAME"] . $path;
            return 1; //return $path
        }
 
    }

Возвращает NULL, хотя файл успешно загружается.

move_uploaded_file( $file, $path) - возвращает TRUE

то есть конструкция выполняется в ветку IF заходим.

Если написать

public function moveFile($file, $path)
    {
        if (move_uploaded_file( $file, $path))
        {
            //$path = "https://" . $_SERVER["SERVER_NAME"] . $path;
           echo 1;//echo $path
        }
 
    }

На экран выведет
1

соответственно если заменить 1 на $path изменений нет, нужно получить $path

public function moveFile($file, $path)
    {

        return (move_uploaded_file( $file, $path)) ? "https://" . $_SERVER["SERVER_NAME"] . $path : 'Ошибка загрузки';
 
    }

так ТОЖЕ возвращает NULL

и через IF / ELSE тоже NULL

ИТОГО что имеем:

return возращает всегда NULL,

if (move_uploaded_file( $file, $path))
        {
            //$path = "https://" . $_SERVER["SERVER_NAME"] . $path;
           echo <b>ЛЮБОЕ ВЫРАЖЕНИЕ</b>;
        }

работает нормально

Comments

[Игорь]

напишите правильно тот вариант который не работает, а то у вас два раза 1

[Lander]

Кусок кода как вызываете moveFile в студию!

[php_raper]

Lander,

$form = new ClassName();

var_dump($form->moveFile($file, $path)) // в сокращенном варианте так

Answer 1

[Lander]

Если всё действительно так как вы говорите, то есть мнение что move_uploaded_file не всегда возвращает true. Просто вы смотрите в разные моменты.

Comments

[php_raper]

if (move_uploaded_file($file, $path))
        {
            echo 1;

            if (true)
                return "https://" . $_SERVER["SERVER_NAME"] . $path;
        }

даже так будет

1NULL

[php_raper]

Lander

даже так NULL

[FanatPHP]

Очевидно, функция работает корректно, а проблема в коде проверки. Какая нибудь тупая опечатка.

Ну неужели не хватает здравого смысла понять, что если программа делает невозможные вещи, то ошибка в коде, и самая тупая?

[php_raper]

FanatPHP,

error_reporting(E_ALL);
ini_set('display_errors', 1);

нет ошибок, варнингов, ноутисов

[FanatPHP]

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

Answer 2

[FanatPHP]

Идеальный пример реализации анекдота у таракана уши в ногах в реальной жизни.

Автор уже сам понял, что, разумеется, дело не в условном операторе (и не может быть). Но все равно упорно стучит по столу, и проверяет что угодно, но не возврат конкретной функции

Comments

[php_raper]

FanatPHP, спасибо, проблема была из-за повторного вызова