Для начала я бы использовал карту, а не список при просмотре набора результатов для поиска сотрудников по id перед обновлением роли.
Map<Integer, Employee> empMap = employeeList.stream().collect(Collectors.toMap(e -> e.getId(), e));
while (rs.next) {
Employee e = empMap.get(rs.getLong("ID"));
if (e != null) {
String role = rs.getString("RoleName");
if (e.getRoleName() == null) {
e.setRoleName(role));
} else {
e.setRoleName(e.getRole() + ", " + role);
}
}
}
Если возможно, лучше добавить роль.Назовите функциональность в методе класса Employee
public void addRoleName(String role) {
if (this.role == null) {
this.role = role;
} else {
this.role += ", " + role;
}
}
, что упрощает цикл
while (rs.next) {
Employee e = empMap.get(rs.getLong("ID"));
if (e != null) {
e.addRoleName(rs.getString("RoleName"));
}
}