Почему метод array.push в цикле while не добавляет значения в массив?

Question

[Xsader]

Почему в этом случае:

var a =[]
var tns = [{type: 1}, {type: 2}] // может быть больше двух объектов, ниже необходимо вывести все
var i = 0
while(tns[i] == !undefined){a.push([i]['type']); i++} //вывод всех значений type массива с объектами
console.log(a)

console.log выводит пустой массив, и как реализовать задуманное?

Comments

[DanKud]

Во-первых, что это вообще такое:

while(tns[i] == !undefined){a.push([i]['type']); i++}

сравните с тем как должно быть, чтобы работало:

while(tns[i] !== undefined){a.push(tns[i]['type']); i++}

Во-вторых, как вам уже ответили, это все делается через .map() в одну строчку.

[Xsader]

DanKud, Ваш вариант, к сожалению, тоже выводит пустой массив.
Через .map попробую.

[DanKud]

Xsader, конкретно для того примера, который вы написали, все работает - https://codepen.io/anon/pen/YbmjaE

[Daeamon]

while(typeof tns[i] != 'undefined')

Answer 1

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

const a = tns.map(el => el.type);

По вашему коду:
1. Условие tns[i] == !undefined возвращает false
2. Выражение [i]['type'] возвращает undefined.

Comments

[Islam Ibakaev]

это красиво, но для автора вряд ли полезно

[Xsader]

Спасибо большое за удобное решение, но немного запутался насчёт моего кода.
Почему-то a.push([0]['type']) добавляет значение в массив, а a.push([i]['type']) нет, хотя i = 0?

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

Xsader, вот исправленная версия вашего кода:

var a =[];
var tns = [{type: 1}, {type: 2}]; 
var i = 0;

while(tns[i]) {
  a.push(tns[i]['type']);
  i++;
}

console.log(a);

Answer 2

[Islam Ibakaev]

попробуй так
while(i < tns.length){a.push(tns[i]['type']); i++}

Comments

[Xsader]

Спасибо. Ответы, как Ваш так и Антона отметил решениями, оба подходят)

Answer 3

[Александр Баранчугов]

1. Условие в цикле у тебя не верное. Получается что ты сравниваешь элемент массива с!undefined
Для твоего случая правильно условие было бы такое: tns[i] !== undefined.
НО! Следует учитывать, что в середине массива могут встречать empty Элементы

2. Посмотри что ты добавляешь в массив a
правильно будет так: a.push(tns[i]['type']
в частности пропустил tns
[i]['type'] = получить свойство type массива [i]

3. Я бы сократил этот кусок кода до следующего:

a = tns.map( ({type}) => 
  type
);

Answer 4

[Николай Чуприк]

Параметр цикла i где-то строчный, где-то заглавной буквой. В JS это разные идентификаторы.

Comments

[Xsader]

Прошу прощения, была опечатка лишь в вопросе. На практике везде буква строчная, но код всё равно не работает(

[Николай Чуприк]

a.push(tns[i]['type']). Это такая у вас ошибка, но тут уже Антон Спирин изящнее решение написал.