Лучше ли определять переменные условия или использовать переменные и проверять условие каждый раз?

Question

Лучше проверить условие один раз, определить переменные, а затем использовать эти переменные?

<c:if test="${someCondition}"> 
    <c:set var="varOne" value="1" /> 
    <c:set var="varTwo" value="2" /> 
    <c:set var="varThree" value="3" /> 
    <c:set var="varFour" value="4" /> 
    <c:set var="varFive" value="5" /> 
</c:if> 

${varOne} 
... 
${varTwo} 
... 
${varThree} 
... 
${varFour} 
... 
${varFive} 

Или лучше использовать переменные и проверять условие каждый раз, когда нужно вывести значение?

${someCondition ? "1" : ''} 
... 
${someCondition ? "2" : ''} 
... 
${someCondition ? "3": ''} 
... 
${someCondition ? "4": ''} 
... 
${someCondition ? "5": ''} 

Answer 1

Если вы собираетесь показать каждую переменную во многих местах в вас странице, первый подход (если проверять состояние только один раз) будет более предпочтительным. Потому что это позволяет избежать дублирования кода.

Если вы покажете переменные только один раз, вам не нужны дополнительные <c:if, а второй подход - в порядке. Потому что это проще и читабельнее.

---

Конечно, это только мое мнение, но я предпочел бы переместить эту логику в контроллер:

@Controller 
public class MyController{ 

    @RequestMapping("/my-mapping") 
    public String showView(ModelMap model){ 
     model.addAttribute("varOne", someCondition?1:null); 
     model.addAttribute("varTwo", someCondition?2:null); 
     //etc ... 
     return "view-name"; 
    } 
} 

В этом случае ваше мнение будет оставаться ясным и простым, и все бизнес-логика будет в одном месте (кстати, JSP ничего не покажет, когда вы проходите null в качестве значения).