В чем разница?

[Antoshka007]

Привет!

Не могу понять в чем разница, между вот этими двумя записями:
 

getItems: function () {        return [].concat(this._items);    }

getItems: function () {        return this._items;    }

 

Это что-то вроде преобразования к массиву, если this._items не массив?

 

 

 

[afdw]

Если this._items был не массивом, то [].concat(this._items) вернёт [this._items] (массив из одного элемента). Но зачем такое поведение нужно?

[Great Rash]

Но зачем такое поведение нужно?

Из названия метода ясно, что он должен возвращать всегда несколько итемов. 

[afdw]

 

Но зачем такое поведение нужно?

Из названия метода ясно, что он должен возвращать всегда несколько итемов. 

 

Но и по _items ясно, что в этой переменной должны храниться всегда несколько итемов.

Edited July 1, 2015 by afdw

[nerv]

Но зачем такое поведение нужно?

хороший вопрос. Допустим

this._items = null;

тогда код

getItems: function () {    return [].concat(this._items);}

будет возвращать

[null]

лично мне такое поведение кажется не очень логичным. Более разумным вижу вариант

getItems: function () {    return this._items || [];}

[Great Rash]

Не соглашусь. Например у нас в this._items должен быть массив объектов. .getIems() всегда должна возвращать массив объектов, т.к. typeof null выдаст 'object', то [null] - это логичное значение.

[nerv]

Great Rash, on 01 Jul 2015 - 2:58 PM, said:

т.к. typeof null выдаст 'object', то [null] - это логичное значение

это официально признанный баг, который не фиксится для обратной совместимости

[null].length === 1;[].length === 0;

[afdw]

Не понял, о чём речь:

[null].length === 1; //true[].length === 1; //false

Вполне логичное поведение, где баг?

[Great Rash]

Тоже не понимаю в чем баг. Почему [null].length должна быть 0? Например в PHP:

$arr = array(null);echo count($arr); // 1

[nerv]

Не понял, о чём речь

у меня опечатка была, исправил в посте выше

речь о том, как определить пустое множество в случае с

[null] // length = 1

это менее очевидно, чем в случае с

[] // length = 0

[afdw]

[null]

Но это - не множество и не пустое.

[nerv]

Но это - не множество и не пустое

если не пустое, то полное

если не множество, то ж*па?)

итого: полная + ж*па)

ЧТО ТОГДА?

Тоже не понимаю в чем баг

т.к. typeof null выдаст 'object'

[afdw]

если не множество, то ж*па?

 

Нет, массив.

[Great Rash]

Что ты этим хочешь сказать, я не понимаю? Ну [null] - массив, состоящий из одного элемента, который null, в чём проблема?

[nerv]

Нет, массив.

ответ из серии "яблоко - это яблоко, потому, что яблоко"

тогда что для тебя множество?

Что ты этим хочешь сказать, я не понимаю? Ну [null] - массив, состоящий из одного элемента, который null, в чём проблема?

http://jsfiddle.net/kqp13mh9/1/

[afdw]

множество

 

Ну в множестве элементы не имеют индекса и не могут повторяться, разве нет?

Множества в JS тоже есть, называются они Set.

[Great Rash]

Оторванный от реальности пример короче. Хз как оно на самом деле, но я так полагаю, что где-то дальше в коде ожидается, что .getItems() вернёт массив, и она его успешно возвращает. Возможно дальше в коде вот прям позарез нужен именно массив и программист не хочет проверять является ли полученный объект массивом или нет, возможно он сразу запускает цикл и уже внутри цикла проверяет чо там лежит внутри.

[Antoshka007]

Оу, как много споров)
Всем спасибо за активность!

[nerv]

Множества в JS тоже есть, называются они Set.

ага, и функций в js до arrow funciton тоже не было)

Оторванный от реальности пример короче.

куда еще реальней. Тебе сервер null вернул, а ту его в items зафигачил

Возможно дальше в коде вот прям позарез нужен именно массив и программист не хочет проверять является ли полученный объект массивом или нет, возможно он сразу запускает цикл и уже внутри цикла проверяет чо там лежит внутри

или ты не читал мои посты

или не смотрел мою ссылку

или и то и другое

Это что-то вроде преобразования к массиву, если this._items не массив?

покажи сеттер. Есть предположение, что это вместо

this._items.slice();

[Antoshka007]

покажи сеттер. Есть предположение, что это вместо

Вот статья, где я это нашел: https://alexatnet.com/articles/model-view-controller-mvc-javascript

Сетер, я так понимаю, это - http://joxi.ru/4AkZEpBuJwk52q