Boundaries: Added PortList class to encapsulate BitSet and thus have a

clear boundary to the outside world that uses the result

1 files changed, 8 insertions, 5 deletions

diff --git a/src/main/java/com/it_jaros/jscanner/ScanResult.java b/src/main/java/com/it_jaros/jscanner/ScanResult.java
index 85d9a36..3eb4d0d 100644
--- a/src/main/java/com/it_jaros/jscanner/ScanResult.java
+++ b/src/main/java/com/it_jaros/jscanner/ScanResult.java
@@ -1,19 +1,22 @@
 package com.it_jaros.jscanner;
 
-import java.util.*;
+import java.util.BitSet;
+import java.util.Collections;
+import java.util.List;
+import java.util.Map;
 
 public record ScanResult(
         String host,
-        BitSet openPorts,
-        BitSet filteredPorts,
+        PortList openPorts,
+        PortList filteredPorts,
         Map<Integer, ServiceType> bannerRecognition,
         List<ScanFailure> errors
 ) {
     public static ScanResult empty(String host) {
         return new ScanResult(
                 host,
-                new BitSet(PortRange.MAX_PORT),
-                new BitSet(PortRange.MAX_PORT),
+                new PortList(new BitSet(PortRange.MAX_PORT)),
+                new PortList(new BitSet(PortRange.MAX_PORT)),
                 Collections.emptyMap(),
                 Collections.emptyList()
         );