Как сделать переменную видимой в цикле?

Question

[Ст. Ку.]

Нужно так, чтоб переменная tpbl в цикле была видима:

var tpbl = $(this).prev().prev().prev().prev('blocks');
$("blocks").each(function(){
  if (tpbl.length)
  {
    var top = tpbl.css('top');
    var hei = tpbl.children('scrl').height() - 295 + parseFloat(top);
    $(this).css('top', hei);
  }
});

Как? Спасибо.

UPD.

Answer 1

[nikosias]

Выводы в консоль одинаковы?

var tpbl = $(this).prev().prev().prev().prev('blocks');
console.log(1,tpbl,tpbl.length)
$("blocks").each(function(){
var tpbl = $(this).prev().prev().prev().prev('blocks');
console.log(2,tpbl,tpbl.length)
  if (tpbl.length)
  {
    var top = tpbl.css('top');
    var hei = tpbl.children('scrl').height() - 295 + parseFloat(top);
    $(this).css('top', hei);
  }
});

var tpbl = $(this).prev().prev().prev().prev('blocks');

Comments

[Ст. Ку.]

Нет, не одинаковы. Добавил принтскрин консоли в вопрос.

[nikosias]

Это значит что вы не правильно сформировали цепочку вызовов в tpbl = $(this).prev().prev().prev().prev('blocks'); Попробуйте сделать минимально рабочий пример в https://jsfiddle.net/

[nikosias]

Рано отправилось. Сейчас не понятна структура вашего HTML. Почему взята именно такая цепочка вызовов. По факту отвечая на Ваш вопрос: переменная tpbl видима:) Выкладывайте пример и тогда можно будет переработать код для того чтобы он отвечал поставленной задачи.

[Ст. Ку.]

nikosias: это вот сюда w3o.ru там на блоках клик и скролл. Вроде как у пинтерест грид. не хотелось бы вычислять количество столбцов внутри цикла.

[Ст. Ку.]

nikosias: там в head скрипт.

[nikosias]

Ок, я не совсем понял что вы в этом цикле пытаетесь сделать. Можете сформулировать?

[Ст. Ку.]

nikosias: вычисляет высоту пустоты до верхнего блока и поднимает нижний блок до верхнего. Чтоб определить какой блок над каким, нужно знать количество столбцов, которое зависит от ширины экрана. Вот тут:
switch(true)
{
case (wih > 1920): var tpbl = $(this).prev().prev().prev().prev().prev().prev('block');break;
case (wih > 1440): var tpbl = $(this).prev().prev().prev().prev().prev('block');break;
case (wih > 1024): var tpbl = $(this).prev().prev().prev().prev('block');break;
case (wih > 640): var tpbl = $(this).prev().prev().prev('block');break;
default: var tpbl = $(this).prev().prev('block');
}

[Ст. Ку.]

nikosias: вот этот свитч я не хочу делать внутри цикла, ибо он одинаковый для всех.

[nikosias]

отступ от верхнего блока у вас во всех столбцах одинаковый? В этом случае вам проще использовать $('conteiner block:eq(0)') Это самый первый верхний блок. Или Вы имеете виду от первого блока в линнии и остальные блоки в этой линии ровнять по первому блоку в линии.

[nikosias]

$('conteiner block:eq(0)').position().top

[Ст. Ку.]

nikosias: ну конечно нет. отступ разный. И он наследуется нижним блоком от верхнего + своя разница.

[nikosias]

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

[Ст. Ку.]

nikosias: да, именно это я и делаю в цикле. Но вычисление количество столбцов не должно быть в цикле. Ибо оно одинаковое.

[nikosias]

var count=0;
switch(true)
{
  case (wih > 1920): count = 5;break;
  case (wih > 1440): count = 4;break;
  case (wih > 1024): count = 3;break;
  case  (wih > 640): count = 2;break;
  default: count = 1;
}
var bloks=$('conteiner block');
for(i = 0; i< bloks.length;i++){
  if (i>count){
       var tpbl=bloks[i-count];
       var top = tpbl.css('top'),
       hei = tpbl.children('scrl').height() - 295 + parseFloat(top);
       bloks[i].css('top', hei);
  }
}

Что то типа такого?

[Ст. Ку.]

nikosias: ) пока не знаю сейчас попытаюсь присобачить.

[Ст. Ку.]

nikosias: ну, пока не получилось, но ход правильный, лучше, чем у меня.

Answer 2

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

она и так видима.

Comments

[Ст. Ку.]

Не работает. Если задать переменную внутри цикла - работает.

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

Станислав Кукаев: напишите сверху var a = 1, внутри console.log(a)

[Ст. Ку.]

Да, передает а = 1. Может быть, дело в this? Не понимает вне чего именно this, а?

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

Станислав Кукаев: Вы уверены,что в tbl что-то полезное записывается ? И больше нигде не меняется ни в каких других местах? Нет нигде переменных с таким же названием (по таймеру и т.д.)
Если есть такое же названия,то LET действительно может помочь

[Ст. Ку.]

Sharp: Эта переменная сложно вычисляется, не хотелось бы всовывать её вычисление в цикл.

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

Станислав Кукаев: Вы до цикла проверьте вычислилась ли она правильно?

[Ст. Ку.]

Sharp: всё работает, если внутрь цикла всунуть. А снаружи не работает. По-моему проблема в this.

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

Станислав Кукаев:Переменная видна. Проблема в том,что идет после равно '='. Вы выложили кусок кода,что там в this снаружи цикла не очень ясно, тем более не ясно несколько корректна строчка $(this).prev().prev().prev().prev('blocks');
Внутри цикла в this ссылка на текущий элемент,по которому проходится each

[imdeveloper]

Станислав Кукаев: this - Это ссылка на текущий объект уберите вверху var и используйте в любом месте

[Ст. Ку.]

imdeveloper: вот так взять и убрать? ) не работает без var.

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

Станислав Кукаев: последний раз пишу, переменная видна, устал повторять =)
Проблема в строке
$(this).prev().prev().prev().prev('blocks');
Что она делает я не разбирался,в любом случае в this снаружи и внутри цикла лежат ссылки на разные объекты. Удачи.

[Ст. Ку.]

Sharp: удачи