|
Сообщение от danik.js
|
|
Вобще я бы не стал делать на регулярках. Нужно писать парсер
|
Это,я даже не знаю,что такое.Так,что не вариант.
|
Сообщение от danik.js
|
Согласен. В таком коде подругому и не получится. Он не поддерживаемый. В дальнейшем просто будет дальше обрастать костылями.
Сообщение от dima***
function unique
Че за муть в этой функции? Откуда у тебя квадратная матрица? У тебя вложенные массивы идут с двумя (иногда одним) элементами. А j у тебя зашкаливает почти всегда. Вот тебе и undefined.
|
Совсем страшный код?
Даже если переделать на:
function unique(arr) {
var obj = {};
for(var i=0; i<arr.length; i++) {
var str = arr[i];
obj[str] = true; // запомнить строку в виде свойства объекта
}
return Object.keys(obj); // или собрать ключи перебором для IE<9
}
он всё равно будет неподдерживаемым?Ещё где "костыли"?Что ещё лучше переделать.
Спасибо за критику.