Показать сообщение отдельно
  #1 (permalink)  
Старый 12.10.2014, 19:22
Аватар для Armen
Кандидат Javascript-наук
Отправить личное сообщение для Armen Посмотреть профиль Найти все сообщения от Armen
 
Регистрация: 28.06.2009
Сообщений: 120

Как улучшить сортировку массива?
Здравствуйте, подскажите пожалуйста.
Вот написал код, который принимает в качестве параметра массив. Ячейки этого массива - объекты с параметром locale.
Задача в следующем:
найти повторение языков, то есть, если у нас есть в списке 2 и более английских(en_CA , en_US - определяем по префиксу en), то мы берем все совпадения языков и записываем их, каждое, в свойство RC.Lang.Common.LANGUAGE_LIST
в противном случае, если язык не повторяется, то записываем его RC.Lang.Common.LANGUAGE_LABELS.
Вот код ниже
var test =  [{locale:"en_US"}, {locale:"en_GB"}, {locale:"ru"}];

function getArrayLanguages3(array) {
    var languageStore = array.slice();

    var getShortLang = function (str) {
        return str.match(/[a-z]+/)[0];
    };

    for (var langLabelIndex = 0; langLabelIndex < array.length; langLabelIndex ++) {
        var langLabel = array[langLabelIndex].locale;

        languageStore[langLabelIndex] = RC.Lang.Common.LANGUAGE_LABELS[langLabel];
    }

    for (var langListIndexNow = 0; langListIndexNow < array.length; langListIndexNow ++) {
        var shortLangListsNow =  getShortLang(array[langListIndexNow].locale),
            langListNow = array[langListIndexNow].locale;

        for (var langListIndexNext = langListIndexNow + 1; langListIndexNext < array.length; langListIndexNext ++) {
            var shortLangListsNext = getShortLang(array[langListIndexNext].locale),
                langListNext = array[langListIndexNext].locale;

            if (shortLangListsNow == shortLangListsNext) {
                languageStore[langListIndexNow] = RC.Lang.Common.LANGUAGE_LIST[langListNow];
                languageStore[langListIndexNext] = RC.Lang.Common.LANGUAGE_LIST[langListNext];
            }
        }
    }

    return languageStore;
}

console.log(getArrayLanguages3(test));


ВОпрос: как можно улучшить этот код, например, как избавится от лишних циклов? Особенно от первого? Может стоит вынести все циклы в отдельную функцию, как один цикл и потом передавать в функцию нужные параметры?
Спасибо.
Ответить с цитированием