Some updates to ip scanner

src/se/hal/plugin/netscan/LocalNetworkDevice.java

@@ -6,36 +6,34 @@ import se.hal.intf.HalEventData;
 import se.hal.struct.devicedata.SwitchEventData;
 import zutil.ui.Configurator;
 
-import java.net.InetAddress;
-
 /**
  * Created by Ziver on 2016-10-02.
  */
-public class NetworkDevice implements HalEventConfig {
+public class LocalNetworkDevice implements HalEventConfig {
 
     @Configurator.Configurable("IP Address")
-    private String ip;
+    private String host;
 
 
 
-    public NetworkDevice() { }
+    public LocalNetworkDevice() { }
-    public NetworkDevice(String hostAddress) {
+    public LocalNetworkDevice(String hostAddress) {
-        this.ip = hostAddress;
+        this.host = hostAddress;
     }
 
 
-    public String getIp() {
+    public String getHost() {
-        return ip;
+        return host;
     }
 
     @Override
     public String toString(){
-        return "IP: "+ip;
+        return "Host: "+ host;
     }
     @Override
     public boolean equals(Object obj){
-        if (obj instanceof NetworkDevice)
+        if (obj instanceof LocalNetworkDevice)
-            return ip != null && ip.equals(((NetworkDevice) obj).ip);
+            return host != null && host.equals(((LocalNetworkDevice) obj).host);
         return false;
     }
 

src/se/hal/plugin/netscan/NetScanController.java

@@ -25,7 +25,7 @@ public class NetScanController implements HalEventController, HalAutoScannableCo
 
     private ScheduledExecutorService executor;
     private HalEventReportListener listener;
-    private ArrayList<NetworkDevice> devices = new ArrayList<>();
+    private ArrayList<LocalNetworkDevice> devices = new ArrayList<>();
 
 
 
@@ -56,13 +56,12 @@ public class NetScanController implements HalEventController, HalAutoScannableCo
 
     @Override
     public void run() {
-        try{
+        try(MultiCommandExecutor executor = new MultiCommandExecutor();){
-            MultiCommandExecutor executor = new MultiCommandExecutor();
             for (int i = 0; i < devices.size(); i++) {
-                NetworkDevice device = devices.get(i);
+                LocalNetworkDevice device = devices.get(i);
                 if (listener != null) {
-                    logger.fine("Pinging ip: "+device.getIp());
+                    logger.fine("Pinging ip: "+device.getHost());
-                    boolean online = InetScanner.isReachable(InetAddress.getByName(device.getIp()), executor);
+                    boolean online = InetScanner.isReachable(device.getHost(), executor);
                     listener.reportReceived(device, new SwitchEventData(online, System.currentTimeMillis()));
                 }
             }
@@ -75,7 +74,7 @@ public class NetScanController implements HalEventController, HalAutoScannableCo
         logger.fine("Detected ip: "+ip.getHostAddress());
         if (listener != null)
             listener.reportReceived(
-                    new NetworkDevice(ip.getHostAddress()),
+                    new LocalNetworkDevice(ip.getHostAddress()),
                     new SwitchEventData(true, System.currentTimeMillis()));
     }
 
@@ -83,8 +82,8 @@ public class NetScanController implements HalEventController, HalAutoScannableCo
 
     @Override
     public void register(HalEventConfig event) {
-        if (event instanceof NetworkDevice)
+        if (event instanceof LocalNetworkDevice)
-            devices.add((NetworkDevice) event);
+            devices.add((LocalNetworkDevice) event);
     }
     @Override
     public void deregister(HalEventConfig event) {

src/se/hal/plugin/netscan/plugin.json

@@ -3,6 +3,6 @@
 	"name": "Network Scanner",
 	"interfaces": [
 		{"se.hal.intf.HalAutoScannableController": "se.hal.plugin.netscan.NetScanController"},
-		{"se.hal.intf.HalEventConfig": "se.hal.plugin.netscan.NetworkDevice"}
+		{"se.hal.intf.HalEventConfig": "se.hal.plugin.netscan.LocalNetworkDevice"}
 	]
 }