Ist es besser, Zustandsvariablen zu definieren oder keine Variablen zu verwenden und die Bedingung jedes Mal zu überprüfen?

Question

Ist es besser, eine Bedingung einmal zu prüfen, Variablen zu definieren und diese Variablen zu verwenden?

<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} 

Oder ist es besser, keine Variablen zu verwenden und die Bedingung jedes Mal zu überprüfen, wenn ein Wert ausgegeben werden soll?

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

Answer 1

Wenn Sie gehen jede Variable in vielen Orten in Ihnen Seite, erste Ansatz zeigen (wenn nur ein einziges Mal eine Bedingung überprüfen) wird mehr bevorzugt. Weil Sie damit Doppelarbeit vermeiden können.

Wenn Sie Variablen nur einmal anzeigen, brauchen Sie keine zusätzlichen <c:if und zweiten Ansatz ist in Ordnung. Weil es einfacher und lesbarer ist.

---

Natürlich ist dies nur meine Meinung, aber ich würde es vorziehen, diese Logik-Controller bewegen:

@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"; 
    } 
} 

In diesem Fall wird Ihre Sicht klar und einfach bleiben, und alle Business-Logik wird an einer Stelle (JSP wird übrigens nichts anzeigen, wenn Sie als Wert null übergeben).