More iterations [for (Role role : roles)] can be avoid from isAdministrator() and isSuperuser() methods of User java class by using contains() of Set<> collection.

In [ org.jboss.hal.config.User.java ] , isAdministrator() and isSuperuser() methods contains the loop for checking  'ADMINISTRATOR' and 'SUPER_USER' . So, we can avoid the loop by using the collection contains() method in the code.

-----------------------------

  for (Role role : roles) {

if (ADMINISTRATOR.equals(role))

}

--------------------------------------

Replacing the old code with the given line of code:

--------------------------------------

 if(roles.contains(ADMINISTRATOR)) { return true; } else { return false; }

--------------