Почему не работает вывод максимального значения массива?

Question

[SpeakeazyYT1]

Имеется следующий код:

$('#year').on('change', function() {
        $('#result').html('');
            a = [];
        var maxleng = getMax(a, "UF_LENGTH");
        $.getJSON("api.php", {
                action: "getAkb"
            })
            .done(function(data) {
                $.each(data, function(i, item) {
                    if (item.UF_TYPE == 'A') a.push(item);
                });
                if (a.length > 0) {
                    result += '<h4>Europe</h4>';
                  
                    console.log("Наибольшая длина: " + maxleng.UF_LENGTH);
                  
                    $.each(a, function(i, item) {
                        result += item.UF_LENGTH + ' x ' + item.UF_WIDTH + ' x ' + item.UF_HEIGHT + ' ' + item.UF_CAPACITY + 'Ah ' + item.UF_CAPACITY + 'EN V' + '<br>';
                    });
                }
                $('#result').html(result);
            });
    });

Всё что должно интересовать - это переменная maxleng, которая занимается выводом максимального значения по ключу UF_LENGTH в массиве "a".

Функция getMax:

function getMax(arr, prop) {
    var max;
    for (var i=0 ; i<arr.length ; i++) {
        if (!max || parseInt(arr[i][prop]) > parseInt(max[prop]))
            max = arr[i];
    }
    return max;
}

В .done(function(data) я поставил console.log("Наибольшая длина: " + maxleng.UF_LENGTH); чтобы вывести максимальное значение по ключу json из массива "a".
Возникла проблема, когда я попробовал исполнить свой скрипт, в консоли вылезла ошибка: Uncaught TypeError: Cannot read property 'UF_LENGTH' of undefined.
Что я сделал в коде не так?

Массив данных примерно такой:

[
  {"UF_LENGTH":"1"},
  {"UF_LENGTH":"2"},
  {"UF_LENGTH":"3"}
]

Answer 1

[Rsa97]

Так вы значение переменной maxleng вычислили когда массив был пустой. Естественно, вернулось пустое значение, в котором нет поля UF_LENGTH.
К тому же, в ваших данных JSON не определено поле UF_TYPE, значит при фильтрации в массив a ничего не попадёт.

Comments

[SpeakeazyYT1]

Я использовал решение человека ниже - console.log("Наибольшая длина: " + getMax(a, "UF_LENGTH"));
При этом у меня ошибки не возникло, но в консоли выводится [object Object]

Answer 2

[Андрей Гончаров]

У вас переменная maxleng определилась и посчиталась уже в 4-ой строчке. Вам надо вызывать функцию getMax уже с заполненным массивом "a". Т.е. там где вы выводите в консоль надо и считать, а не ДО заполнения массива.

var maxleng = getMax(a, "UF_LENGTH");
console.log("Наибольшая длина: " + maxleng.UF_LENGTH);

Comments

[SpeakeazyYT1]

В этом случае в консоли теперь нет ошибки, но и значения тоже нет, вот что вывелось - [object Object]

[SpeakeazyYT1]

Может быть надо как-то взаимодействовать с $.each(a, function(i, item) {
У меня используется этот код для вывода всех значений JSON, пробежавшись по всему массиву, ну что-то типа foreach в php.

[Андрей Гончаров]

SpeakeazyYT1,
Тогда попробуй так, на сколько я разобрался:

var maxleng = getMax(a, "UF_LENGTH");
console.log("Наибольшая длина: " + maxleng.UF_LENGTH);

Т.е. строчку var maxleng = getMax(a, "UF_LENGTH"); перенеси перед выводом в консоль

[SpeakeazyYT1]

Всё заработало, огромное спасибо.