Функция не работает при $(window).resize(). В чём ошибка?

Question

[Андрей]

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

Итак, есть классическая функция, которая уравнивает высоту двух колонок:

function setEqualHeight(columns)
{var tallestcolumn = 0;
columns.each(
function(){
currentHeight = $(this).height();
if(currentHeight > tallestcolumn)
{tallestcolumn = currentHeight;}});
columns.height(tallestcolumn);}

При открытии страницы, функция отрабатывает отлично:

$(document).ready(function() {
    setEqualHeight($(".b-container > div"));});

$(window).resize(function () {
	setEqualHeight($(".b-container > div"));});

Но при ресайзе страницы (при изменении высоты одной из колонок) функция не работает. При тестировании было выявлено, что проблема в самой функции setEqualHeight.

Поскольку я junior в javascript, не могу понять в чём проблема. Подскажите пожалуйста!

Answer 1

[Николай Антал]

попробуйте добавить columns.removeAttr('style'); перед проходом цикла. Не таботает так-как первый раз цикл прогоняет колонки, и принудительно ставит высоту. Второй раз проганяя цикл, высота уже у всех одинаковая (в инлайн-стилях жестко прописана высота)
А для тестирования используйте хотябы console.log(). В данном случае проверять в каждой итерации значение tallestcolumn уже бы дало понять в чем проблемма

Comments

[Андрей]

спасибо большое, получилось! только работает медленно как-то)

[Николай Антал]

попробуйте заменить each на обычный цыкл for, скорости чуть чуть прибавиться, но врят ли существенно

Answer 2

[Alexey Svirschevskiy]

Ваш код сложно читать, попробуйте придерживаться стандартов.
https://github.com/airbnb/javascript
learn.javascript.ru/coding-style

Answer 3

[Vitaly Musin]

Рабочий код jsfiddle.net/vmpartner/3ZULx/1/

Comments

[Андрей]

Уважаемый, у вас код тоже не работает. Попробуйти окно браузера с контентом сузить, и посмотреть на блоки, когда контент выходит за их пределы снизу. Мне помог @hermit931 с его комментарием выше, и проблема решилась.

[Vitaly Musin]

@escalade Спасибо, поправил код. Сделал удаление через css, так будет правильнее.

Answer 4

[Андрей]

@rezzzet дело в том, что это классический скрипт создания колонок одинаковой высоты.
код взял вот отсюда: habrahabr.ru/post/64173/

Comments

[Николай Антал]

ну и где вы там заметили ресайз?

[Андрей]

ресайз сам добавил и в этом есть необходимость, так как блоки меняют высоту при изменеии размеров браузера

[Николай Антал]

Вот именно, что в исходном варианте ресайза нет, поэтому для ресайза изменение высоты не работает.