Можно ли не прописывать переменные в функцию если их нет?

Question

[PageUp]

Здравствуйте, уважаемые кодеры. Подскажите, пожалуйста, можно ли игнорировать при вызове функции Function указание отсутствующих переменных? Раньше помню в лог ошибок постоянно это сыпалось, что функция в себя включает список скажем из 5 входящих переменных, а в нее передается только 3 или 2. Когда стал прописывать пустыми кавычками "", "", "", до нужного количества, ошибки эти пропали. Сейчас по логам такое не встречается. даже не знаю, то ли я устранил все это, то ли это больше не считается за ошибку. Так вот. К примеру есть функция

function delitem($iditem, $login, $kolvo, $linefile, $catalog, $effect34, $effect33)

В тех местах где пара эффектов передается, я прописываю значения effect34 и effect33
в тех местах где не нужно, я пишу "", "") и идут пустые данные.

Сейчас мне потребовалось для передачи добавить аж 5 новых переменных и они точно так же будут местами заполнены, местами нет. Могу ли я добавить нужные значения только в тех местах где эти переменные приобретают свойства, не добавляя кучу пустых кавычек в тех местах, где это не требуется?

Как это может сказаться на производительности и вообще. Не хочется просто теперь переписывать около 50 файлов, везде где встречается данная функция. что бы туда просто добавить после $effect34, $effect33 пустые кавычки. Надеюсь подробно объяснил и суть ясна, что я имею ввиду.

Answer 1

[Владислав Лысков]

ну так задай параметры по дефолту

function delitem($iditem, $login, $kolvo, $linefile, $catalog, $effect34 = null, $effect33 = null)

Comments

[PageUp]

А если переменные будут передаваться, то эти NULL приобретут значения?

[Владислав Лысков]

PageUp, да
https://www.php.net/manual/ru/functions.arguments.php

[PageUp]

Владислав Лысков, спасибо огромное. Как же я пропустил такой казалось бы элементарный момент.

Answer 2

[Page-Audit.ru]

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

function myFunc( $a, array $b = [ ] ) {
...
  ...$y = $b[ 'varY' ] ?? 'defaultValue';
...
}

myFunc( 1, [ 'var2' => 2, 'var3' => 3, ... 'varX' => 'X' ] );

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

Comments

[Mylistryx]

Я бы таким пальцы ломал!

[Page-Audit.ru]

Хм, почему? CodeStyle?

[Дмитрий]

Page-Audit.ru, а потом приходит другой программист, вызывает метод и не понимает, почему у него эксепшены сыпятся из-за отсутствующих индексов в это массиве. Например.

[Page-Audit.ru]

Дмитрий, неудачный пример, если переменная (как описано в этом вопросе) может отсутствовать, значит факт отсутствия её инициализации уже прописан в коде и не является эксепшеном + доку по входным параметрам никто не отменял.

[Mylistryx]

Переменные надо передавать явно, причем крайне желательно с указанием типов или через DTO, но никак не массивом. А то есть любители... потом extract(...) и сиди, разгадывай! В общем ненадо так!

[Page-Audit.ru]

Mylistryx, vvv@Mylistryx, хе-хе, вот это вы расскажите разработчикам PHP, которые вообще позволяют передавать в функцию неопределённое количество параметров. О каком указании типов и явной передаче идёт речь! 8)
Тут уже все переходят к вопросу Code Style, но он вторичен. Первичный же вопрос был "как".

[Mylistryx]

public function test (string $name, int $count = null) {
   doSomeWith($name);
   if ($count) {...}
}
class DtoConfig {
public $name;
public $count;

public function __construct($name, $count = null) {
    $this->name = $name;
    $this->count = $count;
  }
}

$Dto = new ConfigDto('Vasya', 100);

public function test (DtoConfig $Dto) {
   doSomeWith($Dto->name);
   if ($Dto->count) {...}
}
$this->test($Dto);
public function fuckedMagic(array $args)
{
 $args = extract($args);
 /** сношайтесь как хотите */
}

Что приятней? Мне 2й вариант.

[Page-Audit.ru]

Mylistryx, да, DTO более кошерно. 8) Но в нём не выполняется постановка задачи - "количество инициализируемых параметров не определено".
Вся кайфовость DTO ломается в его конструкторе - он принимает всего навсего два параметра... а автору вопроса вдруг понадобилось иметь 5 новых параметров. Перед каждым запуском кода переписывать класс под новое количество переменных?..