Как спарсить utm-метки в переменные?

Question

[Егор Рублёв]

Почему выдает undefined в переменные utm1 и utm2?

Ссылка к примеру http://жк-зеленоградский.рф/?utm1=test1&utm2=test2

var x = location.search,
    y = x.slice(1),
    z = y.split('&');

$(z).each(function(index){
    var t = z[index] = z[index].split('=');
    return t;
});

var utmName = function(utmName) {
    $(z).each(function(index, utmName){
        if (z[index][0] == utmName) {
            var uName = z[index][1];
            return uName;
        } else {
            var uName;
            return uName;
        }
    });
}

var utm1 = utmName('utm1'),
    utm2 = utmName('utm2');

Answer 1

[Arris]

Передаваемая в each функция - callback, который будет вызываться для каждого элемента коллекции, и в который будут передаваться два параметра - индекс и значение элемента в коллекции.

Это у вас первая ошибка.
А вторая ошибка - из замыкания $(z).each(function(){})); ничего не возвращается. То есть функция сама по себе результат возвращает, но он нигде не сохраняется.

Я сделал так:

var utmName = function(utmName) {
                var result = '';
                $(z).each(function(index){
                    if (z[index][0] == utmName) {
                        result = z[index][1];
                    }
                });
                return result;
            }

Это работает, но я не уверен, что это абсолютно синтаксически корректно.

Comments

[Егор Рублёв]

Спасибо!

[Arris]

Егор Рублёв: Только это нифига не эффективный код :)

function get_wlh_key( wlh_array, key )
{
    var result = null;
    wlh_array.forEach(function(element, index, array) {
        let pair = element.split('=');
        if (pair[0] == key) {
            result = pair[1];
        }
    });
    return result;
}
var 
   ls = location.search.slice(1),
   splitted = ls.split('&');

console.log( get_wlh_key(splitted, 'utm1') );
console.log( get_wlh_key(splitted, 'utm2') );