Sonar refactorings

This commit is contained in:
Sven Kubiak 2022-02-04 16:01:22 +01:00
parent d9cde1f429
commit 91b771355b

View File

@ -64,13 +64,13 @@ public class OpenClient {
.POST(HttpRequest.BodyPublishers.ofString(params.toString())) .POST(HttpRequest.BodyPublishers.ofString(params.toString()))
.build(); .build();
PushoverResponse pushoverResponse = PushoverResponse.create().isSuccessful(false); var pushoverResponse = PushoverResponse.create().isSuccessful(false);
try { try {
HttpResponse<String> response = client.send(request, HttpResponse.BodyHandlers.ofString()); HttpResponse<String> response = client.send(request, HttpResponse.BodyHandlers.ofString());
pushoverResponse pushoverResponse
.httpStatus(response.statusCode()) .httpStatus(response.statusCode())
.response(response.body()) .response(response.body())
.isSuccessful((response.statusCode() == 200) ? true : false); .isSuccessful(response.statusCode() == 200);
} catch (IOException | InterruptedException e) { } catch (IOException | InterruptedException e) {
Thread.currentThread().interrupt(); Thread.currentThread().interrupt();
throw new JPushoverException("Pushover Login failed", e); throw new JPushoverException("Pushover Login failed", e);
@ -163,13 +163,13 @@ public class OpenClient {
.POST(HttpRequest.BodyPublishers.ofString(params.toString())) .POST(HttpRequest.BodyPublishers.ofString(params.toString()))
.build(); .build();
PushoverResponse pushoverResponse = PushoverResponse.create().isSuccessful(false); var pushoverResponse = PushoverResponse.create().isSuccessful(false);
try { try {
HttpResponse<String> response = client.send(request, HttpResponse.BodyHandlers.ofString()); HttpResponse<String> response = client.send(request, HttpResponse.BodyHandlers.ofString());
pushoverResponse pushoverResponse
.httpStatus(response.statusCode()) .httpStatus(response.statusCode())
.response(response.body()) .response(response.body())
.isSuccessful((response.statusCode() == 200) ? true : false); .isSuccessful(response.statusCode() == 200);
} catch (IOException | InterruptedException e) { } catch (IOException | InterruptedException e) {
Thread.currentThread().interrupt(); Thread.currentThread().interrupt();
throw new JPushoverException("Failed to delete messages", e); throw new JPushoverException("Failed to delete messages", e);
@ -237,7 +237,7 @@ public class OpenClient {
.POST(HttpRequest.BodyPublishers.ofString(params.toString())) .POST(HttpRequest.BodyPublishers.ofString(params.toString()))
.build(); .build();
PushoverResponse pushoverResponse = PushoverResponse.create().isSuccessful(false); var pushoverResponse = PushoverResponse.create().isSuccessful(false);
try { try {
HttpResponse<String> response = client.send(request, HttpResponse.BodyHandlers.ofString()); HttpResponse<String> response = client.send(request, HttpResponse.BodyHandlers.ofString());
pushoverResponse pushoverResponse
@ -258,7 +258,7 @@ public class OpenClient {
* @return true if close was successful, false otherwise * @return true if close was successful, false otherwise
*/ */
public boolean close() { public boolean close() {
boolean closed = false; var closed = false;
if (webSocket != null) { if (webSocket != null) {
webSocket.sendClose(WebSocket.NORMAL_CLOSURE, "ok"); webSocket.sendClose(WebSocket.NORMAL_CLOSURE, "ok");
closed = webSocket.isInputClosed() && webSocket.isOutputClosed(); closed = webSocket.isInputClosed() && webSocket.isOutputClosed();