Как вытащить значение переменной из функции JS?

Question

[Денис Букреев]

Доброго дня.

Надо достать переменную из функции, но что-то идет не так.

function oneHeight()
	{
		var maxHeight = 0;

		$('.reviews-slider__item .review').each(function()
		{
			maxHeight = Math.max(this,maxHeight);
		});

		console.log(maxHeight);
	}

Вот, собственно код.
Я присваивал функции имя, пытался занести в переменную результат функции, но лог нещадно выдавал ошибку: переменная не найдена.
Как достать maxHeight правильно?

UPD: Решение проблемы уместилось в одну строчку maxHeight = Math.max(elHeight,maxHeight);:

function oneHeight()
	{
		var maxHeight = 0,
			element = $('.reviews-slider__item .review');

		element.each(function()
		{
			var elHeight = $(this).height();
			maxHeight = Math.max(elHeight,maxHeight);
		});

		element.height(maxHeight);
	}

Comments

[#алгоптимизируй #отботизируй]

Элементов один или больше? Если один зачем each, если много,то на выходе должен быть массив?

[Денис Букреев]

С Sharp: на выходе должна получиться переменная с максимальным значением высоты, по идее

проверяю каждый элемент... похоже надо вынести определение maxHeight до вызова функции
вроде и моя проблема решится

спасибо!

[Денис Букреев]

С Sharp: нет, не помогло
все равно не нравится ему мой код

[Hyubert]

прочтите , поможет понять , как это самому сделать learn.javascript.ru/closures

[Денис Букреев]

Hyubert: это уже все чуть ли не наизусть в голове
без практики туго идет, вот и практикую

Answer 1

[#алгоптимизируй #отботизируй]

var maxHeight= 0; $([4,5,6,7]).each(function () { maxHeight = Math.max(this,maxHeight)}); console.log(maxHeight);

Переменная должна быть снаружи.
Можно так. Возможно есть красивее решение

-------------------------------------------------
maxHeight = Math.max(this,maxHeight);
this для примера, тут должно быть height

Comments

[#алгоптимизируй #отботизируй]

stackoverflow.com/questions/5052673/jquery-min-max... тут пытаются создать общий метод.

[Денис Букреев]

not a number выдает

[Денис Букреев]

С Sharp: iprom.bookreev.com - применяется к элементам слайдера в блоке "Отзывы"

[#алгоптимизируй #отботизируй]

maxHeight = Math.max($(this).height(),maxHeight); Выше добавил замечание

[Денис Букреев]

С Sharp: понял
скопировал не обдумав
все пошло

[#алгоптимизируй #отботизируй]

$(this).height() не уверен,что выдает не NaN, не в курсе...

[Денис Букреев]

С Sharp: спасибо
иногда кажется, что все же нативный JS часто лучше, понятнее и короче всяких джейквериев
который раз вижу, как в строчку JS умещается много jQ кода

[#алгоптимизируй #отботизируй]

Денис Букреев: https://jsfiddle.net/2s78Lf9r/ Пожалуйста. вместо return x -> x.height

Answer 2

[Rsa97]

var maxHeight = $('.reviews-slider__item .review')
    .toArray()
    .reduce(function(acc, cur) {
        return ($(cur).height() > acc ? $(cur).height() : acc);
    }, -1);
console.log(maxHeight);