Удалить undefined элементы из массива

Question

У меня есть таблица, где я извлекаю данные и добавляю их в массив массивов. Проблемы в том, что если одна из ячеек таблицы пуста, она появляется в массиве как «неопределенная». Я попытался использовать if, если последний элемент не определен, если так использовать .pop(), элемент должен быть удален. Я все еще получаю неопределенные элементы. Вот мой код и жить demo

HTML:

<table id="contactlisttable"> 
    <tr> 
     <th>Name</th> 
     <th>Title</th> 
     <th>Phone</th> 
    </tr> 
    <tr> 
     <td class="contactlist contactlistlastfirst">Joey</td> 
     <td class="contactlist contactlisttitle">webdesigner</td> 
     <td class="contactlist contactlistphone"></td> 
    </tr> 
    <tr> 
     <td class="contactlist contactlistlastfirst">Anthony</td> 
     <td class="contactlist contactlisttitle">webdesigner</td> 
     <td class="contactlist contactlistphone">5555555</td> 
    </tr> 
</table> 

---

JavaScript:

//IE9+ compatable solution 
$(function(){ 
    var results = [], row; 
    $('#contactlisttable').find('th, td').each(function(){ 
     if(!this.previousElementSibling){ //New Row? 
      row = []; 
      results.push(row); 
      if($(this) === 'undefined'){//Remove undefined elements 
       row.pop(); 
      } 
     } 
     row.push(this.textContent || this.innerText); //Add the values (textContent is standard while innerText is not)  
    }); 
    console.log(results); 
}); 

---

Answer 1

jsFiddle Demo

Вместо того, чтобы делать условные операторы, просто воспользоваться вашей HTML структуры. Сначала выберите по строкам таблицы, а затем переведите дочерний элемент td или th. Вы также можете использовать jQuery's text вместо того, чтобы выполнять обнаружение функции. Текст jQuery будет более надежным.

var results = []; 
$('#contactlisttable tr').each(function(){ 
var row = []; 
$(this).find('td,th').each(function(){ 
    row.push($(this).text()); 
}); 
results.push(row); 
}); 
console.log(results); 

Answer 2

Чтобы знать, если что-то не определено как раз не сравнить с "undefined", использование TYPEOF().

Так что вы хотите сделать:

if (typeof(this) === "undefined") 

Answer 3

Вместо того, чтобы нажимать и выскакивают, если он не соответствует, не давите на первом месте.

Обновление с вашего jsfiddle:

//IE9+ compatable solution 
$(function(){ 
    var results = [], row; 
    $('#contactlisttable').find('th, td').each(function(){ 
     if(!this.previousElementSibling && typeof(this) != 'undefined'){ //New Row? 
      row = []; 
      results.push(row); 
     } 
     row.push(this.textContent || this.innerText); //Add the values (textContent is standard while innerText is not)  
    }); 
    console.log(results); 
}); 

Answer 4

Вы можете добавить компактный метод, как в подчеркиванием и lodash ...

Array.prototype.compact = function() { 
    return this.filter(function(x){ 
     return x !== undefined; 
    }); 
} 

console.log([1,2, '', undefined, 'e', undefined].compact()); // [ 1, 2, '', 'e' ] 

Вы, вероятно, следует добавить проверку на родной реализации compact как ну, так как вы переопределяете собственный прототип с этим.

Или просто

Answer 5

Вы также можете избежать добавления Неопределенные (или фактически пустые) элементы таким образом:

$('#contactlisttable').find('th, td').each(function(){ 
    if(!this.previousElementSibling){ //New Row? 
     row = []; 
     results.push(row); 
    } 
    if(!(this.textContent == '')){ 
     row.push(this.textContent || this.innerText);    
    } 
});