Как строго отделить Array от Object?

Question

[newaitix]

var ty={
	o:function(v){
		return!Array.isArray(v)?typeof v==='object':!1;
	},
	a:function(v){
		if(v!=null&&typeof v==='object')
			if(v.length>=0)
				return!0;
		return Array.isArray(v);
	}
}

Я сделал так. Мне кажется, что это оптимальный вариант. А как вы отделяете итерируемые массивы от объекта ?

Answer 1

[Антон Спирин]

Это что троллинг?
Для определения массива достаточно Array.isArray
Для определения объекта:

function isObject(item) {
  return (typeof item === "object" && !Array.isArray(item) && item !== null);
}

Если в вашем коде возникают трудности с определением типа данных могу порекомендовать использовать TypeScript или Flow.

Еще старайтесь писать читаемый код. Использование ни о чем не говорящих аргументов, отсутствие пробелов между операторами и операндами, и !0, !1 вместо true, false не очень хорошая практика.

Comments

[newaitix]

а и в правду. Xто то я фантазирую переделал так

var ty={
	a:function(v){
		return Array.isArray(v);
	},
	o:function(v){
		return Array.isArray(v)?!1:typeof v==='object';
	}
}

[Stalker_RED]

newaitix,

ty.o({}) // -> false

так и задумано?

[newaitix]

Хотя знаете есть ситуация в которой это не так ))
к примеру вот
NodeList(3) [meta, meta, meta]
это массив ? Да это массив.
Но Array.isArray() вернет false

[Stalker_RED]

newaitix, потому что это НЕ массив
https://developer.mozilla.org/ru/docs/Web/API/NodeList
Отличия_NodeList_от_Array

[Антон Спирин]

newaitix, NodeList это не массив.
Для определения объекта достаточно:

function isObject(item) {
  return (typeof item === "object" && !Array.isArray(item) && item !== null);
}

добавил к ответу
В обертке над isArray нет объективной необходимости, как и в самой сущности ty.

[Антон Спирин]

newaitix,
ломаем ваш первый "оптимальный вариант" ty.a
ломаем ваш второй "оптимальный вариант" ty.o

[newaitix]

Антон Спирин, Я буду считать что null это object
по поводу ty.o
ty.a переделал так
v==null?!1:Array.isArray(v);

[Антон Спирин]

newaitix,
1. для выявления массива достаточно Array.isArray
2. NodeList это не массив и там отсутствуют многие методы реализованные в массивах, а на IE отсутствует даже метод forEach
3. Не изобретайте велосипеды, код завязаный на хелперы вроде ty - плохой код.
4. Возможно, вы выбрали неправильный путь для решения задачи, раз у вас возникла необходимость в таких велосипедах.

[newaitix]

Почему NodeList не считаль массивом ? Я не понимаю.

[Антон Спирин]

newaitix, а я не понимаю, как вам вообще взбрело в голову, что NodeList это массив. Читайте спецификации.
NodeList
Array

[newaitix]

Антон Спирин, ну окей. Что вы предлагаете
не обрабатывать NodeList вообще ?
То есть это массив ? нет
Тогда это объект тогда давай сделаем так
NodeList.addEventListener
ой опять ошибка. Как же так ?

[Антон Спирин]

newaitix, почему вы считаете, что addEventListener является методом Object?

Посмотрите этот пример и почитайте о EventTarget.addEventListener

Вам скинули хорошие материалы по NodeList и то почему вы до сих пор считаете эту коллекцию массивом для меня остается загадкой.
Рекомендую вам учить JavaScript с нуля. Думаю, только это может помочь в вашем случае.
Хорошее место для этого https://learn.javascript.ru

[newaitix]

Тогда я не понимаю как работать с NodeList
Вы меня окончательно запутали

Вариант на который вы намекаете не будит работать (т.к. по вашим словам это объект)

var divs=document.getElementsByTagName('div')
for(var i of divs)
console.log(divs[i])//undefined

а вариант который я хочу использовать вы запрещаете использовать

var divs=document.getElementsByTagName('div')
for(var i=0;i<divs.length;i++)
console.log(divs[i])//div element

[newaitix]

Ответьте просто мне на один вопрос: как перебирать элементы находящиеся в NodeList
А затем ответьте себе что такое NodeList.
Я понимаю что это не объект. Javascript очень гибкий и мы можем сделать к примеру вот так

var arr=[];
arr.fn=function(){return!0}
arr.fn();// true

И в этом случае у нас массив содержит метод.
Все очень не однозначно. И NodeList можно воспринимать как массив не смотря на то что он не имеет методов присущих обычным массивам.

[Антон Спирин]

newaitix,

Тогда я не понимаю как работать с NodeList
Вы меня окончательно запутали

NodeList. Читайте там все есть.

Вариант на который вы намекаете не будит работать (т.к. по вашим словам это объект)

а вариант который я хочу использовать вы запрещаете использовать

Я ничего такого не писал и ни на что не намекал.
И да Document.getElementsByTagName() возвращает HTMLCollection, а не NodeList. Это разные коллекции.

ответьте просто мне на один вопрос: как перебирать элементы находящиеся в NodeList

Перейдите по ссылке и почитайте NodeList.

Я понимаю что это не объект.

Array, NodeList, HTMLCollection, Function это все потомки Object. Так что технически в JavaScript все непримитивные сущности являются объектами.

Javascript очень гибкий и мы можем сделать к примеру вот так

В JavaScript можно производить любые действия описанные в спецификации. Печально, что вы вместо того чтобы с ними ознакомиться руководствуетесь своими догадками и домыслами.
Array является потомком объекта. Это всего лишь объект с особыми методами в прототипе и ключом length. Естественно можно производить любые операции с его свойствами.
Этот пример, видимо, должен окончательно разорвать ваши шаблоны.

И NodeList можно воспринимать как массив не смотря на то что он не имеет методов присущих обычным массивам.

Массив это Array. NodeList это коллекция нод.
Еще пример

[newaitix]

Антон Спирин, все что можете насоветовать это читать документацию. Но в документации сказано

Пример
Элементы в NodeList, можно обработать следующим образом:

for (var i = 0; i < myNodeList.length; ++i) {
  var item = myNodeList[i];  // Вызов myNodeList.item(i) необязателен в JavaScript
}

Так можно или нельзя считать его массивом ?

[newaitix]

Антон Спирин,
Сказано

NodeList используется подобно массивам, и потому может возникнуть закономерное желание использовать в нём методы, предоставляемые Array.prototype. Однако NodeList не реализует методы, подобные таковым у Array.

Но я не просил использовать методы для моих массивов. Все что мне нужно это отделить объекты которые можно обрабатывать циклом

arr =[1,2,3]
for(var i=0; i<arr.length; i++){
console.log(arr[i])
}

от тех которые нельзя обработать таким циклом

arr={1:'a'}
for(var i=0; i<arr.length; i++){
console.log(arr[i])
}

А вы вместо помощи занимаетесь нравоучениями.

[Антон Спирин]

newaitix, с чего вам взбрело в голову, что то что можно перебирать оператором for можно считать массивом?
Перебор объекта.
По ссылке, что вам уже раз десять скинули, русским языком написано об отличиях массива и коллекции нод.
NodeList нельзя считать массивом, но можно перебрать циклом for, правда старые браузеры не все реализуют ключ length. Так же NodeList можно сконвертировать в массив.
Но NldeList это не массив, это коллекция.

[newaitix]

Антон Спирин, Понимаете почему это важно ? потому что далее во коду я делаю

var arr={};
if(ty.a(arr))
for(var i=0; i<arr.length; i++){
//...
}

И в этом случае вы предлагаете мне сделать так чтоб ty.a(arr) вернул true.

[newaitix]

Антон Спирин, окей спасибо я понял.

[Антон Спирин]

newaitix,

. Все что мне нужно это отделить объекты которые можно обрабатывать циклом

С этого и надо было начинать.
Если у вас возникла такая необходимость, то вам стоит пересмотреть свои подходы к дизайну и решению задач.

[Антон Спирин]

newaitix,

Понимаете почему это важно ? потому что далее во коду я делаю

Не надо так писать.

[newaitix]

Антон Спирин, Ну почему же плохой?
почему плохой ?
Если в переменную elem падает
document.getElementsByTagName('div')
или в эту же переменную падает
document.getElementById('middle')
То как мне сделать их красными ?
В первом случае это будит

var elem=document.getElementsByTagName('div');
for(var i=0; i<elem.length; i++)
elem[i].style.backgroundColor='rgb(255,0,0)'

Во втором случае мы будим делать так

var elem=document.getElementById('middle')
elem.style.backgroundColor='rgb(255,0,0)'

Верно ?
Вот поэтому я спрашиваю
ty.a(elem) то делаем первый вариант
ty.o(elem) то делаем второй вариант

[Антон Спирин]

newaitix, откуда упадет? Если вы не знаете, что и откуда у вас "падает", вы пишите очень плохой код.
Внедрять в проект велосипеды вроде вашего ty.o, ty.a - плохая идея. Уже писал об этом.

[newaitix]

Антон Спирин, как вы предлагаете обрабатывать код который я привел выше ?
функция для присваивания красного цвета элементам. Может принимать как объект так и массив объектов. А зачем плодить повторяющийся код ?
Я спрошу если элементов много значит значит я присвою поочередно через цикл каждому красный цвет, если элемент один то присвою непосредственно ему без использования цикла.

[Антон Спирин]

newaitix,

как вы предлагаете обрабатывать код который я привел выше ?

функция для присваивания красного цвета элементам. Может принимать как объект так и массив объектов. А зачем плодить повторяющийся код ?

По вашему примеру, вам лишь надо отличать HTMLElement от HTMLCollection для этого достаточно:

if (data instanceof HTMLCollection || data instanceof NodeList) {
  // итерация коллекции
} else {
  // работа с элементом
}

Свой велосипед использовать не надо.

Answer 2

[Павел Корнилов]

newaitix, да, согласен с Антон Спирин , у вас всё запущено.
Постараюсь проще объяснить отличия NodeList от Array.
Они наследуют свои основные свойства совершенно от разных прототипов. И единственными общими прототипами для них являются Object и null.
Для проверки можно также использовать instanceof.

if (obj instanceof Array) console.log('obj isArray!');