Tengo dos métodos similares en cuanto al cuerpo, pero con un número diferente de parámetros y una condición adicional en el interior. Sé que hay una forma de fusionarlos en un solo método usando un predicado, pero no estoy del todo seguro de cómo implementarlo. ¿Cuál es la mejor manera de abordar esto?
public boolean checkIfAllCodesAreUnique(List<String> bsnCodes) { List<Businesscode> codes = ConverterUtil.iterableToList(businessCodeService.findAll()); if(codes != null && !codes.isEmpty() && bsnCodes != null && !bsnCodes.isEmpty()) for (String code : bsnCodes) if (codes.stream().anyMatch(obj -> code.equals(obj.getCode()))) return false; return true; } public boolean checkIfAllCodesAreUnique(List<String> bsnCodes, int idRole) { List<Businesscode> codes = ConverterUtil.iterableToList(businessCodeService.findAll()); if(codes != null && !codes.isEmpty() && bsnCodes != null && !bsnCodes.isEmpty()) for (String code : bsnCodes) if (codes.stream().anyMatch(obj -> code.equals(obj.getCode()) && obj.getId() != idRole)) return false; return true; }public boolean checkIfAllCodesAreUnique(List<String> bsnCodes) { return isAllCodesAreUnique(bsnCodes, businessCode -> true); } public boolean checkIfAllCodesAreUnique(List<String> bsnCodes, int idRole) { return isAllCodesAreUnique(bsnCodes, businessCode -> businessCode.getId() != idRole); } private boolean isAllCodesAreUnique(List<String> bsnCodes, Predicate<Businesscode> checkRole) { List<Businesscode> businessCodes = Optional.ofNullable(ConverterUtil .iterableToList(businessCodeService.findAll())).orElse(List.of()); for (String bsnCode : Optional.ofNullable(bsnCodes).orElse(List.of())) { if (businessCodes.stream() .filter(businessCode -> bsnCode.equals(businessCode.getCode())) .anyMatch(checkRole)) return false; } return true; }Básicamente, el predicado no le permitiría nada específico en el sentido de una interfaz autodeterminable o lo que sea. Probablemente la mejor combinación de los dos sería:
public boolean checkIfAllCodesAreUnique(List<String> bsnCodes, Integer idRole) { List<Businesscode> codes = ConverterUtil.iterableToList(businessCodeService.findAll()); if(codes != null && !codes.isEmpty() && bsnCodes != null && !bsnCodes.isEmpty()) for (String code : bsnCodes) if (codes.stream().anyMatch(obj -> code.equals(obj.getCode()) || (idRole != null && obj.getId() != idRole)) return false; return true; }Y luego pase el segundo parámetro como nulo siempre que no esté disponible.