Size: a a a

2020 April 08

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
Параметры без индекса тоже с номерами...
Ну так с нуля если считать, то как раз норм. Но я за то, чтобы оставить только именованные
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
Параметры без индекса тоже с номерами...
По номеру полюбэ надо, у Reference ты тип не определишь
источник

DS

Dmitriy S in Yii Framework 3
То, что номер скрытый будет, то ничего не меняет
источник

Д

Дмитрий in Yii Framework 3
Dmitriy S
По номеру полюбэ надо, у Reference ты тип не определишь
$reference->getId()?
источник

Д

Дмитрий in Yii Framework 3
Dmitriy S
Ну так с нуля если считать, то как раз норм. Но я за то, чтобы оставить только именованные
т.е. ты предлагаешь запретить такое:
__construct() => [
$arg1,
$arg2,
$arg3
]

?
источник

AM

Alexander Makarov in Yii Framework 3
Dmitriy S
То, что номер скрытый будет, то ничего не меняет
Меняет. Можно ошибиться.
источник

AM

Alexander Makarov in Yii Framework 3
Ну и не ясно, что будет если

[
 1 => ...,
 ...,
 ...
]
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
$reference->getId()?
И там алиас. Поможет?
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
Ну и не ясно, что будет если

[
 1 => ...,
 ...,
 ...
]
Ну тогда только именованные параметры
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
т.е. ты предлагаешь запретить такое:
__construct() => [
$arg1,
$arg2,
$arg3
]

?
В общем - да, оставить только именованные
источник

Д

Дмитрий in Yii Framework 3
Dmitriy S
В общем - да, оставить только именованные
тогда решай следующий кейс:
class RulesService {
constructor(RuleInterface ...$rules)
}

как это конфижить?
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
тогда решай следующий кейс:
class RulesService {
constructor(RuleInterface ...$rules)
}

как это конфижить?
А как ты сейчас кофижить такое будешь?
источник

Д

Дмитрий in Yii Framework 3
construct=>[
$rule1,
$rule2,
$rule3
...
]
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
construct=>[
$rule1,
$rule2,
$rule3
...
]
Ну тогда либо оставлять по индексам, либо не юзать такое в конструкторах. Я бы лично такое в конструкторах точно не делал.
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
Ну и не ясно, что будет если

[
 1 => ...,
 ...,
 ...
]
А так, норм?
[
    'name1' => 'value1',
    'value2',
    'name2' => 'value3',
    'value4',
    Reference::to('service1'),    
]
источник

AM

Alexander Makarov in Yii Framework 3
Dmitriy S
В общем - да, оставить только именованные
Или же сделать как в injector: или только именованные или только позиционные без индексов.
источник

AM

Alexander Makarov in Yii Framework 3
Dmitriy S
А так, норм?
[
    'name1' => 'value1',
    'value2',
    'name2' => 'value3',
    'value4',
    Reference::to('service1'),    
]
Нет. Я не представляю как оно будет работать.
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
Нет. Я не представляю как оно будет работать.
Ну вот и я об этом
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
Или же сделать как в injector: или только именованные или только позиционные без индексов.
в инжекторе нет референса без типа
источник

DS

Dmitriy S in Yii Framework 3
Штука в том, что если мы сейчас оставим только именованные, а будут ишью по таким кейсам, как у Дмитрия выше с запаковкой параметров, то мы сможем допилить, обратная совместимость сохранится.
источник