Refactor HelloModel to enhance async message/file sending logic and improve NtfyMessageDto constructors for clarity and consistency. Clean up unused dependencies and imports.

Author: annikaholmqvist94

pom.xml

@@ -35,17 +35,6 @@
             <version>${mockito.version}</version>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>tools.jackson.core</groupId>
-            <artifactId>jackson-databind</artifactId>
-            <version>3.0.1</version>
-        </dependency>
-        <dependency>
-            <groupId>org.wiremock</groupId>
-            <artifactId>wiremock</artifactId>
-            <version>4.0.0-beta.15</version>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>org.openjfx</groupId>
             <artifactId>javafx-controls</artifactId>

src/main/java/com/example/HelloFX.java

@@ -1,6 +1,6 @@
 package com.example;
 
-import io.github.cdimascio.dotenv.Dotenv;
+
 import javafx.application.Application;
 import javafx.fxml.FXMLLoader;
 import javafx.scene.Parent;

src/main/java/com/example/HelloModel.java

@@ -13,7 +13,10 @@
 
 import java.io.File;
 
+import java.io.FileNotFoundException;
+import java.io.IOException;
 import java.util.*;
+import java.util.concurrent.CompletableFuture;
 
 
 /**
@@ -54,32 +57,28 @@ private static void runOnFx(Runnable task) {
     }
 
     /**
-     * Sends the current text message to the Ntfy server via the connection.
-     * The message is added to the local list before sending.
+     * Sends the current text message asynchronously.
+     * The network call is moved off the FX thread. UI updates happen via runOnFx.
      */
     public void sendMessage() {
-        String message = messageToSend.get();
-        if (message != null && !message.trim().isEmpty()) {
-
-            // 1. Skapa den lokala DTO:n (med isLocal = true)
-            NtfyMessageDto localMessage = new NtfyMessageDto(
-                    UUID.randomUUID().toString(),
-                    System.currentTimeMillis() / 1000L,
-                    "message",
-                    currentTopic.get(),
-                    message.trim(),
-                    true // Markera som lokalt skickat
-            );
-
-            // 2. Lägg till i listan (UI-uppdatering) PÅ RÄTT TRÅD
-            runOnFx(() -> messages.add(localMessage));
-
-            // 3. Skicka meddelandet via anslutningen
-            connection.send(message, currentTopic.get());
-
-            // 4. Rensa meddelandefältet efter skickning
-            messageToSend.set("");
+        final String message = messageToSend.get();
+        if (message == null || message.trim().isEmpty()) {
+            System.err.println("Cannot send empty message.");
+            return;
         }
+
+        // 1. Markera meddelandet som lokalt skickat och lägg till i listan (UI-uppdatering)
+        final NtfyMessageDto localSentDto = new NtfyMessageDto(message, currentTopic.get(), "text", true);
+        runOnFx(() -> {
+            messages.add(localSentDto);
+            messageToSend.set(""); // Rensa inmatningsfältet
+        });
+
+        // 2. Utför nätverksanropet på en bakgrundstråd
+        CompletableFuture.runAsync(() -> {
+            // Nätverksanropet sker här i bakgrunden
+            connection.send(message, currentTopic.get());
+        });
     }
 
     /**
@@ -89,27 +88,19 @@ public void sendMessage() {
      */
     public void sendFile() {
         File file = fileToSend.get();
-        if (file != null) {
-
-            // 1. Skapa den lokala DTO:n för fil (ofta med tom message)
-            NtfyMessageDto localFileMessage = new NtfyMessageDto(
-                    UUID.randomUUID().toString(),
-                    System.currentTimeMillis() / 1000L,
-                    "file", // Använd "file" event om det är en fil
-                    currentTopic.get(),
-                    "Fil skickad: " + file.getName(), // Detta meddelande visas bara i logik, CellFactory hanterar visning
-                    true
-            );
-
-            // 2. Lägg till i listan (UI-uppdatering) PÅ RÄTT TRÅD
-            runOnFx(() -> messages.add(localFileMessage));
-
-            // 3. Skicka filen
-            connection.sendFile(file, currentTopic.get());
+        final NtfyMessageDto localSentDto = new NtfyMessageDto(file.getName(), currentTopic.get(), "file", true);
+        runOnFx(() -> {
+            messages.add(localSentDto);
+            fileToSend.set(null); // Rensa filbilagan i UI
+        });
 
-            // 4. Rensa filbilagan efter skickning
-            fileToSend.set(null);
-        }
+        // 2. Utför nätverksanropet på en bakgrundstråd
+
+        // 2. Utför nätverksanropet på en bakgrundstråd
+        CompletableFuture.runAsync(() -> {
+            // Nätverksanropet sker här i bakgrunden
+            connection.sendFile(file, currentTopic.get());
+        });
     }
 
     /**

src/main/java/com/example/NtfyConnection.java

@@ -1,7 +1,7 @@
 package com.example;
 
 import java.io.File;
-import java.io.FileNotFoundException;
+
 import java.util.function.Consumer;
 
 public interface NtfyConnection {
@@ -14,9 +14,9 @@ public interface NtfyConnection {
 
     void connect(String topic, Consumer<NtfyMessageDto> messageHandler);
 
-    //public boolean send(String message);
+
 
     public void receive(Consumer<NtfyMessageDto> messageHandler);
 
-    //public boolean sendFile(File file) throws FileNotFoundException;
+
 }

src/main/java/com/example/NtfyMessageDto.java

@@ -3,20 +3,33 @@
 
 
 
-public record NtfyMessageDto(String id, long time, String event, String topic, String message, boolean isLocal) {
+public record NtfyMessageDto(String id, long time, String event, String topic, String type, String message, boolean isLocal) {
 
-    public NtfyMessageDto(String id, long time, String event, String topic, String message) {
-        this(id, time, event, topic, message, false);
+    // Sekundär konstruktor för mottagna meddelanden (isLocal = false)
+    public NtfyMessageDto(String id, long time, String event, String topic, String type, String message) {
+        this(id, time, event, topic, type, message, false);
     }
 
-    public NtfyMessageDto(String message) {
-        this(null, 0, "message", null, message, true);
+    /**
+     * Konstruktor för att skapa ett lokalt skickat meddelande (som visas i listan innan bekräftelse).
+     * @param message Den huvudsakliga meddelandetexten eller filnamnet.
+     * @param topic Den aktuella topicen.
+     * @param type Typ av innehåll ("text" eller "file").
+     * @param isLocal Alltid true för lokalt skickade meddelanden.
+     */
+    public NtfyMessageDto(String message, String topic, String type, boolean isLocal) {
+        // Fix: Vi använder 0L för att explicit kasta 0 som en long
+        this(null, 0L, "message", topic, type, message, isLocal);
     }
 
+    // Kort konstruktor för att bara skicka en meddelandetext (kanske inte används men fixas för konsekvens)
+    public NtfyMessageDto(String message) {
+        this(null, 0L, "message", null, "text", message, true);
+    }
 
     @Override
     public String toString(){
-            return message;
+        return message;
     }
 
 }

src/test/java/com/example/NtfyConnectionSpy.java

@@ -1,7 +1,6 @@
 package com.example;
 
 import java.io.File;
-import java.io.FileNotFoundException;
 import java.util.function.Consumer;
 
 public class NtfyConnectionSpy implements NtfyConnection {