From 710747d00b4b6d92d739d72f8564e60da7014c9a Mon Sep 17 00:00:00 2001 From: Thomas Beyer Date: Tue, 28 Mar 2023 11:28:43 +0200 Subject: [PATCH] RED-6411 - optimize method --- .../v1/server/redaction/utils/EntitySearchUtils.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/redaction-service-v1/redaction-service-server-v1/src/main/java/com/iqser/red/service/redaction/v1/server/redaction/utils/EntitySearchUtils.java b/redaction-service-v1/redaction-service-server-v1/src/main/java/com/iqser/red/service/redaction/v1/server/redaction/utils/EntitySearchUtils.java index ff22105f..20e8e9f0 100644 --- a/redaction-service-v1/redaction-service-server-v1/src/main/java/com/iqser/red/service/redaction/v1/server/redaction/utils/EntitySearchUtils.java +++ b/redaction-service-v1/redaction-service-server-v1/src/main/java/com/iqser/red/service/redaction/v1/server/redaction/utils/EntitySearchUtils.java @@ -289,14 +289,14 @@ public final class EntitySearchUtils { } } + private boolean isOneARecommendationAndTheOtherEntity(Entity entityOne, Entity entityTwo) { - private boolean isOneARecommendationAndTheOtherEntity (Entity found, Entity existing) { - - return existing.getEntityType().equals(EntityType.RECOMMENDATION) && found.getEntityType().equals(EntityType.ENTITY) || existing.getEntityType() - .equals(EntityType.ENTITY) && found.getEntityType().equals(EntityType.RECOMMENDATION); + var entityTypeOne = entityOne.getEntityType(); + var entityTypeTwo = entityTwo.getEntityType(); + return entityTypeTwo.equals(EntityType.RECOMMENDATION) && entityTypeOne.equals(EntityType.ENTITY) + || entityTypeTwo.equals(EntityType.ENTITY) && entityTypeOne.equals(EntityType.RECOMMENDATION); } - public void addEntitiesIgnoreRank(Set entities, Set found) { // HashSet keeps old value but we want the new. entities.removeAll(found);