Braces aren't required here but I think it makes it easier to read and is defensive to changes later. Also I think it is preferable to use !validationSet.isEmpty() in place of validationSet.size() == 0 incase the underlying set list doesn't hold onto the size and it has to count it. In this case it doesn't matter because it is backed by an arraylist but if we shouldn't require that