Как исправить ошибку Don't make functions within a loop?

Question

[MrSusua]

Здравствуйте!

Как исправить ошибку Don't make functions within a loop. в 20 строке? И почему он ругается на слово jQuery в последней?

(function($, window, document, undefined) {
	'use strict';

	var s = document.body || document.documentElement;
		s = s.style;
	if (s.webkitFlexWrap === '' || s.msFlexWrap === '' || s.flexWrap === '') return true;

	var $list = $('.list'),
		$items = $list.find('.list__item'),
		setHeights = function() {
			$items.css('height', 'auto');

			var perRow = Math.floor($list.width() / $items.width());
			if (perRow === null || perRow < 2) return true;

			for (var i = 0, j = $items.length; i < j; i += perRow) {
				var maxHeight = 0,
					$row = $items.slice(i, i + perRow);

				$row.each(function() {
					var itemHeight = parseInt($(this).outerHeight());
					if (itemHeight > maxHeight) maxHeight = itemHeight;
				});
				$row.css('height', maxHeight);
			}
		};

	setHeights();
	$(window).on('resize', setHeights);
	$list.find('img').on('load', setHeights);

})(jQuery, window, document);

Answer 1

[Петр]

Вы декларируете функцию внутри цикла for.
Для решения нужно просто вынести ее из цикла. Примерно так

var maxHeight = 0;
var fn = function() {
    var itemHeight = parseInt($(this).outerHeight());
    if (itemHeight > maxHeight) maxHeight = itemHeight;
};
for (var i = 0, j = $items.length; i < j; i += perRow){
  maxHeight = 0;
  var $row = $items.slice(i, i + perRow);
  $row.each(fn);
  $row.css('height', maxHeight);
}

Comments

[MrSusua]

Пётр, я только начинаю изучать JavaScript. Не в службу, а в дружбу, как будет выглядеть уже переписанная, полная функция?

[Петр]

MrSusua: Изменил код в ответе. Все остальное не поменялось.

Answer 2

[timfcsm]

в самой ошибке ответ на вопрос, как её исправить.. вынесите объявление функции за цикл. это jshint? в конфиге добавьте в globals 'jQuery'