wouterfennis / CodeRefactorFW

For ASP Assignment
0 stars 1 forks source link

Twee if statements kan naar een enkele #18

Open vincent92 opened 8 years ago

vincent92 commented 8 years ago

In de ClientServlet.java

Vanaf regel 51 wordt 2 keer gekeken of administrator is deze kan in een enkele if

// If currentUser is administrator refresh autocomplete that is required to choose socialworker 
        if (currentUser instanceof Administrator) {
            req.getSession().setAttribute("users", OverviewController.getInstance().autoComplete(currentUser));
        }

        // Check if currentuser is administrator or socialworker to load correct page
        if (currentUser instanceof Administrator) {
            PAGE_CLIENT_OVERVIEW = "/administrator/" + PAGE_CLIENT_OVERVIEW;
            PAGE_CLIENT_ADD_EDIT = "/administrator/" + PAGE_CLIENT_ADD_EDIT;
        }