Нормально ли в PHP определять переменную при вызове функции в списке аргументов?

Question

[Евгений Обыкновенный]

Нормально ли в PHP впервые определять переменную при вызове функции в списке аргументов? Ошибку вроде не выдает, но вдруг могут быть какие-то проблемы.

// $matches не определена
if( preg_match( '#^(abc)|(def)$#ui', $str $matches ) )
{
  ... // использование $matches
}

Answer 1

[FanatPHP]

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

Но по возможности такого подхода следует избегать.

И чисто технически это не определение переменной при вызове, а передача переменной по ссылке.

Comments

[rPman]

не понял почему это не нормально?
в php нет необходимости определять локальные переменные в принципе

[FanatPHP]

rPman, вот этот вопрос и есть причина, по которой это ненормально. Ход выполнения кода должен быть четким и понятным, легко прослеживаться, а не завязываться кренделем. аргументы функции должны быть аргументами, а не вдруг возвращаемыми значениями.
возвращаемые значения должны быть возвращаемыми значениями, а не появляться вдруг откуда не возьмись по волшебству.

а "необходимость определять локальные переменные" тут вообще не при чем.

[rPman]

вот именно если переменная локальная по контексту дальше следующих пары строчек не уйдет, а именно это и происходит с preg_match, я не представляю зачем заморачиваться и тратить на синтаксическую мишуру время и место в коде

[Adamos]

rPman, обычно "не представляю, зачем заморачиваться" === "не работает в следующей версии РНР".

[rPman]

Adamos, надеюсь ЭТО поведение не изменят еще скоро
от слишком строгой типизации уходят и к синтаксическим удобствам пришли даже в строгом c++, было бы очень обидно потерять в удобстве в таком хорошем инструменте как php

Answer 2

[Newto]

В данном случае определять переменную $matches не нужно. Если вы посмотрите на документацию к функции preg_match (https://www.php.net/manual/ru/function.preg-match.php) то увидите валидный пример кода:

<?php
preg_match('/(foo)(bar)(baz)/', 'foobarbaz', $matches, PREG_OFFSET_CAPTURE);
print_r($matches);
?>

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