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

ну ситуация была немного другая

аля
const a = {...b}

я был уверен что если это не падает значит b всегда присуствует, окказалось что b может быть undefined, класика )

источник

11:44пожаловаться #4

Stas G in Frontend UA

cудя по всему, b может быть вообще любым

источник

11:44пожаловаться #5

Stas G in Frontend UA

я и сам это не держу в голове

источник

11:44пожаловаться #6

Eugene Obrezkov in Frontend UA

Dmytro Dovhan

да, в рантайме у тебя даже ошибка может быть соответствующая по типу Destructuring что-то там undefined, не помню)

источник

11:45пожаловаться #7

Eugene Obrezkov in Frontend UA

в общем, жить с предположением что b всегда есть опасно)

источник

11:45пожаловаться #8

Evgen in Frontend UA

Eugene Obrezkov

да, в рантайме у тебя даже ошибка может быть соответствующая по типу Destructuring что-то там undefined, не помню)

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

источник

11:49пожаловаться #9

Evgen in Frontend UA

ни для undefined, ни для null

источник

11:49пожаловаться #10

Eugene Obrezkov in Frontend UA

не не, {...undefined} вернёт пустой объект - ошибки не должно быть

источник

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

Stas G in Frontend UA

и в фф не падает, наверное и по спеке не должно

источник

11:50пожаловаться #12

Eugene Obrezkov in Frontend UA

это я вспомнил за другую ошибку, когда раньше писали вот эти страшные { data: {something: {foo}}}

источник

11:50пожаловаться #13

Evgen in Frontend UA

вот если формировать массив, то там 100% должен быть итерируемый объект

источник

11:50пожаловаться #14

Roman V in Frontend UA

Dmytro Dovhan

Встречал такой паттерн, это должно работать с undefined и спредом в объект без ошибок, удобно опциональные дополнительные параметры разворачивать, иначе нужна была бы проверка на то что это не undefined, лишние строчки кода. Причем с массивами такое не прокатывает, будет ошибка.

источник

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

Alexey Shvayka in Frontend UA

Dmytro Dovhan

а может кто объяснить зачем так сделано?

const a = {...undefined}
a
> {}

всегда был уверен что для спреда объекта собственно нужен объект

чтобы соответствовать семантике Object.assign

источник

13:42пожаловаться #16

Dmytro Dovhan in Frontend UA

Alexey Shvayka

чтобы соответствовать семантике Object.assign

спасибо, не подумал об этом

источник

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

Alexey Shvayka in Frontend UA

ответ автора: https://github.com/tc39/proposal-object-rest-spread/issues/59#issuecomment-356798479

nit: 100% соответствия Object.assign нет т.к. spread использует [[DefineOwnProperty]] вместо [[Set]]

GitHub

spread with null · Issue #59 · tc39/proposal-object-rest-spread

I know spread is used for array liked data. But I found that null also support spread. Such as: let result = {...null} is workd. But, null is not array liked data.

источник

14:19пожаловаться #18

Вт

Ві тя in Frontend UA

Stas G

думаю вопрос о том, почему нет ошибки в этом случае

будет ошибка

источник

16:47пожаловаться #19