Телеграмм чат группы yii3ru страница 1888

На текущий момент логика в случае, когда ключ - строка, такая:

• Если есть элемент с таким ключом - вернуть его.
• Если нет элемента с таким ключом - сделать exclude('.', $key) и попытаться вернуть значение из глубин.
• Если не вышло - вернуть значение по умолчанию.

—-

Неявность такой логики появляется вот в таких ситуациях:

$array = [
  'a.b' => ['c' => 1],
  'a' => ['b' => ['c' => 2]],
  'a' => ['b.c' => 3],
];

Разве очевидно, что getVaue($array, 'a.b.c') должен вернуть?
Мне кажется, что нет.

источник

09:27пожаловаться #9

СП

Сергей Предводителев... in Yii Framework 3

PR#50 https://github.com/yiisoft/arrays/pull/50

Позволяет уйти от неявности и меняет логику так:

getValue():

• Если есть элемент с таким ключом - вернуть его.
• Если не вышло - вернуть значение по умолчанию.

getValueByPath() строки с точками разворачивает однозначно в массив и ищет сразу в глубине.

---

И вот такие случаи уже легко отвеить что вернёт getValue(), а что getValueByPath().
Всё становится прозрачно и понятно на мой взгляд - как работают функции и когда какую применять.

$array = [
  'a.b' => ['c' => 1],
  'a' => ['b' => ['c' => 2]],
  'a' => ['b.c' => 3],
];

getValue($array, 'a.b.c'); вернёт null
getValueByPath($array, 'a.b.c'); вернёт 2

GitHub

Add getValueByPath() and setValueByPath() by vjik · Pull Request #50 · yiisoft/arrays

Q
A

Is bugfix?
❌

New feature?
✔️

Breaks BC?
✔️

Fixed issues
#48

This is break BC. In another packages that use dots-way need replace getValue() and setValue() to getValueByPath() and s...

источник

09:30пожаловаться #10

Nex Otaku in Yii Framework 3

Сергей Предводителев

тест нельзя написать, чтобы потом пофиксить :)
Там функционал разнесён на две функции

Если тест нельзя написать, то проблемы не существует.

источник

09:35пожаловаться #11

СП

Сергей Предводителев... in Yii Framework 3

Я выше всё написал

источник

09:36пожаловаться #12

Nex Otaku in Yii Framework 3

Закрыл обсуждение? )

источник

09:36пожаловаться #13

СП

Сергей Предводителев... in Yii Framework 3

Пока не вижу что ещё написать

источник

09:37пожаловаться #14

Nex Otaku in Yii Framework 3

Слушай. Я проверил этот issue.

Там две части. На каждую я написал тест. Первый тест проходит на уже существующем коде, фиксов не требуется. В issue ошибка.

Вторая часть описанная в issue, тест не проходит.

Но это фиксится в четыре строки кода, добавляем в set ту же логику что присутствует в get. И всё замечательно начинает работать, и старые тесты и новые.

Сейчас PR сделаю.

источник

09:39пожаловаться #15

СП

Сергей Предводителев... in Yii Framework 3

Да тесты тут не причём. Вопрос в логике.

источник

09:40пожаловаться #16

СП

Сергей Предводителев... in Yii Framework 3

Проблема вся вот в такой ситуации:

$array = [
  'a.b' => ['c' => 1],
  'a' => ['b' => ['c' => 2]],
  'a' => ['b.c' => 3],
];

Разве очевидно, что getVaue($array, 'a.b.c') должен вернуть?
Мне кажется, что нет.

источник

09:41пожаловаться #17

АН

Александр Некрасов... in Yii Framework 3

Сергей Предводителев

Проблема вся вот в такой ситуации:

$array = [
  'a.b' => ['c' => 1],
  'a' => ['b' => ['c' => 2]],
  'a' => ['b.c' => 3],
];

Разве очевидно, что getVaue($array, 'a.b.c') должен вернуть?
Мне кажется, что нет.

ну вроде раздлелили на 2 метода, или я ошибаюсь ?, 1 работает с . другой как обычно по ключам

источник

09:42пожаловаться #18

СП

Сергей Предводителев... in Yii Framework 3

Александр Некрасов

ну вроде раздлелили на 2 метода, или я ошибаюсь ?, 1 работает с . другой как обычно по ключам

@nex_otaku топит за то, чтобы НЕ разделять и оставить как есть. И проблемы никакой нет. Всё это надуманость и усложнение.

источник

09:43пожаловаться #19

АН

Александр Некрасов... in Yii Framework 3

ну для 1 метода, должно вернуть a.b.c, если нету то a => [b => [c] а потом уже если нету, то a => 'b.c' вроде

источник

09:44пожаловаться #20