浏览代码

resolved sonarclouds mentioned issues

Jan-Peter Klein 1 年之前
父节点
当前提交
3120751d3e

+ 3 - 0
src/main/java/org/cryptomator/ui/common/HttpHelper.java

@@ -10,6 +10,9 @@ import java.nio.charset.StandardCharsets;
 
 public class HttpHelper {
 
+	private HttpHelper(){
+		throw new IllegalStateException("Utility class");
+	}
 	public static String readBody(HttpResponse<InputStream> response) throws IOException {
 		try (var in = response.body(); var reader = new InputStreamReader(in, StandardCharsets.UTF_8)) {
 			return CharStreams.toString(reader);

+ 10 - 8
src/main/java/org/cryptomator/ui/error/ErrorController.java

@@ -28,6 +28,7 @@ import java.net.http.HttpResponse;
 import java.nio.charset.StandardCharsets;
 import java.util.Comparator;
 import java.util.Map;
+import java.util.Optional;
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.TimeUnit;
@@ -126,9 +127,7 @@ public class ErrorController implements FxController {
 		Clipboard.getSystemClipboard().setContent(clipboardContent);
 
 		copiedDetails.set(true);
-		CompletableFuture.delayedExecutor(2, TimeUnit.SECONDS, Platform::runLater).execute(() -> {
-			copiedDetails.set(false);
-		});
+		CompletableFuture.delayedExecutor(2, TimeUnit.SECONDS, Platform::runLater).execute(() -> copiedDetails.set(false));
 	}
 
 	private void loadHttpResponse(HttpResponse<InputStream> response){
@@ -137,15 +136,18 @@ public class ErrorController implements FxController {
 					new InputStreamReader(response.body(),StandardCharsets.UTF_8),
 					new TypeToken<Map<String,ErrorDiscussion>>(){}.getType());
 
-			if(!map.values().stream().filter(this::isPartialMatchFilter).findFirst().isEmpty()) {
+			if(map.values().stream().anyMatch(this::isPartialMatchFilter)) {
 				Comparator<ErrorDiscussion> comp = this::compareExactMatch;
-				matchingErrorDiscussion = map.values().stream().sorted(comp
+				Optional<ErrorDiscussion> value = map.values().stream().min(comp
 						.thenComparing(this::compareSecondLevelMatch)
 						.thenComparing(this::compareThirdLevelMatch)
 						.thenComparing(this::compareIsAnswered)
-						.thenComparing(this::compareUpvoteCount)
-				).findFirst().get();
-				lookUpSolutionVisibility.set(true);
+						.thenComparing(this::compareUpvoteCount));
+
+				if(value.isPresent()){
+					matchingErrorDiscussion = value.get();
+					lookUpSolutionVisibility.set(true);
+				}
 			}
 		}
 	}

+ 0 - 3
src/main/java/org/cryptomator/ui/error/ErrorDiscussion.java

@@ -8,8 +8,5 @@ public class ErrorDiscussion {
 	Answer answer;
 
 	static class Answer{
-		private String url;
-		private int upvoteCount;
 	}
-
 }

+ 1 - 1
src/test/java/org/cryptomator/ui/error/ErrorControllerTest.java

@@ -97,7 +97,7 @@ class ErrorControllerTest {
 		ErrorDiscussion ed1 = createErrorDiscussion("",0, null);
 		ErrorDiscussion ed2 = createErrorDiscussion("",0, null);
 		int result = errorController.compareIsAnswered(ed1,ed2);
-		Assertions.assertEquals(result, 0);
+		Assertions.assertEquals(0, result);
 	}
 
 	@Test