Question

Is there a way that I can simplify this code:

var topic_html = obj.$form.find("#select-topic").html();
var topic_val = obj.$form.find("#select-topic").val();
var topic_text = obj.$form.find("#select-topic option:selected").text();

I know it’s fairly clean but is there a way that I can avoid the three instances of obj.$form.find.

Answer 1

You don’t need find the select each time, cache it in a variable.

var topic = obj.$form.find("#select-topic");
var topic_html = topic.html();
var topic_val = topic.val();
var topic_text = topic.find("option:selected").text();