]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-6990 remove useless using to Map#contains
authorSébastien Lesaint <sebastien.lesaint@sonarsource.com>
Tue, 15 Dec 2015 18:02:17 +0000 (19:02 +0100)
committerSébastien Lesaint <sebastien.lesaint@sonarsource.com>
Thu, 17 Dec 2015 09:44:05 +0000 (10:44 +0100)
server/sonar-server/src/main/java/org/sonar/server/computation/duplication/DuplicationRepositoryImpl.java

index 3a37dab05b1a6d86e131b551429ff80e9e9a27a2..bd3eeb32363a34d531cf4a6e4f4622a48557e168 100644 (file)
@@ -52,8 +52,9 @@ public class DuplicationRepositoryImpl implements DuplicationRepository {
   public Set<Duplication> getDuplications(Component file) {
     checkFileComponentArgument(file);
 
-    if (duplicationsByComponentUuid.containsKey(file.getUuid())) {
-      return from(duplicationsByComponentUuid.get(file.getUuid()).getDuplicates())
+    Duplications duplications = duplicationsByComponentUuid.get(file.getUuid());
+    if (duplications != null) {
+      return from(duplications.getDuplicates())
         .transform(DuplicatesEntryToDuplication.INSTANCE)
         .toSet();
     }
@@ -231,8 +232,9 @@ public class DuplicationRepositoryImpl implements DuplicationRepository {
     private final Map<TextBlock, Duplicates> duplicatesByTextBlock = new HashMap<>();
 
     public Duplicates getOrCreate(TextBlock textBlock) {
-      if (duplicatesByTextBlock.containsKey(textBlock)) {
-        return duplicatesByTextBlock.get(textBlock);
+      Duplicates duplicates = duplicatesByTextBlock.get(textBlock);
+      if (duplicates != null) {
+        return duplicates;
       }
       Duplicates res = new Duplicates();
       duplicatesByTextBlock.put(textBlock, res);