NO
@predvoditelev вот кейс. В твоём варианте, будет сюрприз...
Size: a a a
2020 September 06
СП
Пока не будет, я это не правил)
NO
Вот тут было обсуждение https://github.com/yiisoft/arrays/issues/1 я не вникал, но такое поведение убрали.
В общем неоднозначность поведения ведёт к скрытым и неожиданным проблемам
В общем неоднозначность поведения ведёт к скрытым и неожиданным проблемам
Поэтому поведение должно быть однозначным. Самое простое всегда в приоритет ставится.
СП
Поэтому поведение должно быть однозначным. Самое простое всегда в приоритет ставится.
у тебя одно другому противоречит. "Самое просто" - это значит несколько вариантов. "Однозначно" - вариант один
NO
Есть базовое поведение. Запись и чтение по ключу одномерного массива. И расширенное поведение - магия с вложенностью через точки. Первое как основа должно быть всегда приоритетней.
СП
Есть базовое поведение. Запись и чтение по ключу одномерного массива. И расширенное поведение - магия с вложенностью через точки. Первое как основа должно быть всегда приоритетней.
Согласен. но вот здесь должно быть 2:
$array = [
'a' => [
'b.c' => 1,
'b' => [
'c' => 2,
],
],
];
ArrayHelper::getValue($array, 'a.b.c');NO
Тупо потому что чаще используется и ожидается именно такое.
NO
Нет, не должно. Я выше расписал кейс
СП
Нет, не должно. Я выше расписал кейс
У тебя вот такой кейс:
$array = [Что должен вернуть
'a.b' => 1,
'a' => [
'b' => 2,
],
];
ArrayHelper::getValue($array, 'a.b');СП
И вот тут 1
NO
1 конечно
NO
При рекурсии те же правила должны работать что на первом уровне. Само собой. Иначе вообще всех запутаешь )
СП
Тогда смотри вот этот тест https://github.com/yiisoft/arrays/blob/master/tests/ ArrayHelperTest.php#L536 по вот этому багу: https://github.com/yiisoft/arrays/issues/1
СП
Я не вникал в то описания, но в рекурсии такое поведение убирали целенаправлено
СП
То есть логика в итоге такая:
Если в массиве есть ключ с таким именем - вернуть его, если нет, то сделать explode по точками и попытаться получить из вложенностей.
Если в массиве есть ключ с таким именем - вернуть его, если нет, то сделать explode по точками и попытаться получить из вложенностей.
T
Qiang Xue it got wrong creating ArrayHelper.
NO
Я не вникал в то описания, но в рекурсии такое поведение убирали целенаправлено
Потому и накосячили.
АГ
Вот здесь простое и ожидаемое 2 получить
думается что получить в этом случае 1 - просто труднее в реализации, больше кода, и работать будет медленнее, т.к. нужно больше проверок. вообще пример надуман если честно. использовать точки в ключах в таком случае будет только идиот
NO
Тогда смотри вот этот тест https://github.com/yiisoft/arrays/blob/master/tests/ ArrayHelperTest.php#L536 по вот этому багу: https://github.com/yiisoft/arrays/issues/1
Если ты сделаешь как я описал, то всё будет исправлено в том кейсе.
АГ
можно просто кидать эксепшн если точки в ключе и все)