Bug fixes and improvements

RedDragon0293 · Apr 27, 2024 · e296a9b · e296a9b
1 parent 16092bb
commit e296a9b
Showing 1 changed file with 13 additions and 8 deletions.
diff --git a/src/main/java/cn/reddragon/eportal/HelloController.java b/src/main/java/cn/reddragon/eportal/HelloController.java
@@ -47,14 +47,14 @@ public void updateUI() {
  if (Authenticator.type == null) {
  sb.append("(...)");
  } else
- for (LoginType it : LoginType.values()) {
- if (Objects.equals(it, Authenticator.type)) {
- sb.append('(');
- sb.append(it.displayName);
- sb.append(')');
- break;
+ for (LoginType it : LoginType.values()) {
+ if (Objects.equals(it, Authenticator.type)) {
+ sb.append('(');
+ sb.append(it.displayName);
+ sb.append(')');
+ break;
+ }
  }
- }
  } else {
  button.setText("Login");
  sb.append("Offline");
@@ -73,8 +73,9 @@ protected void onLoginButtonClick() {
  button.setText("Logout");
  } else if (button.getText().equals("Logout")) {
  new Thread(() -> {
+ HttpURLConnection connection = null;
  try {
- HttpURLConnection connection = Authenticator.logout();
+ connection = Authenticator.logout();
  if (connection == null) {
  Platform.runLater(() -> {
  resultText.setText("Error!");
@@ -90,6 +91,10 @@ protected void onLoginButtonClick() {
  } catch (Exception e) {
  e.printStackTrace();
  Platform.runLater(() -> resultText.setText(e.getMessage()));
+ } finally {
+ if (connection != null) {
+ connection.disconnect();
+ }
  }
  Platform.runLater(() -> button.setDisable(false));
  }).start();