jQuery get div's height on ready & resize

Question

У меня есть 2 отдельных функции, чтобы получить высоту одного элемента и применить его к его дочерью. Таким образом, в основном функция повторяется, когда документ готов и изменяется размер окна. Есть ли лучший способ сделать это?

$(document).ready(function() { 
    var height = $('#sgn-panel-friendslist-holder').height(); 
    $("#sgn-panel-friendslist-holder #scroll").css("height", height-24) 
}); 


$(window).resize(function() { 
    var height = $('#sgn-panel-friendslist-holder').height(); 
    $("#sgn-panel-friendslist-holder #scroll").css("height", height-24) 
}); 

Answer 1

Ну, всегда есть «различные» способы и подходы к закодировать что-то и получить тот же результат, и в этом случае, чтобы сказать, какая из них «лучше» или «более эффективным» не так уверен как это может быть и в других задачах кодирования и/или сценариев ..

, например: Вы можете взять код дубликата в .ready и .resize события и положить затем в одну функцию, например, так:

$(document).ready(function() { 
    setHeight(); 
}); 

$(window).resize(function() { 
    setHeight(); 
}); 

function setHeight() { 
    var height = $('#sgn-panel-friendslist-holder').height(); 
    $("#sgn-panel-friendslist-holder #scroll").css("height", height-24); 
} 

Но теперь возникает вопрос: действительно ли это «лучше» или «более эффективно»? У других может быть свое мнение, но, на мой взгляд, нет. Это экономит время или сохраняет строки кода? Мое мнение .. Нет. Структурно легче читать, понимать и отлаживать? Мое мнение. №

Итак, там я пойду .. Есть ли «лучший способ»? По моему мнению. Нет, не совсем.

Answer 2

Здесь лучше код:

$(document).ready(function() { 
    $(window).resize(function() { 
     var height = $('#sgn-panel-friendslist-holder').height(); 
     $("#sgn-panel-friendslist-holder #scroll").css("height", height-24) 
    }).trigger('resize'); 
});