#41 Refactor and add Diagrams

mlee97 · Dec 7, 2020 · 6f56d35 · 6f56d35
1 parent f22f9d5
commit 6f56d35
Show file tree

Hide file tree

Showing 7 changed files with 63 additions and 25 deletions.
diff --git a/Diagrams/Class Diagram Delivery 3.png b/Diagrams/Class Diagram Delivery 3.png
diff --git a/Diagrams/Package model.png b/Diagrams/Package model.png
diff --git a/model.uml b/model.uml
@@ -0,0 +1,39 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Diagram>
+  <ID>JAVA</ID>
+  <OriginalElement>smarthomesimulator.model</OriginalElement>
+  <nodes>
+    <node x="917.0" y="0.0">smarthomesimulator.model.SHP</node>
+    <node x="0.0" y="999.0">smarthomesimulator.model.Zone</node>
+    <node x="0.0" y="752.0">smarthomesimulator.model.Door</node>
+    <node x="662.0" y="252.0">smarthomesimulator.model.Profile.Role</node>
+    <node x="963.0" y="752.0">smarthomesimulator.model.Light</node>
+    <node x="384.0" y="752.0">smarthomesimulator.model.RoomTest</node>
+    <node x="0.0" y="0.0">smarthomesimulator.model.Simulator</node>
+    <node x="636.0" y="0.0">smarthomesimulator.model.Profile</node>
+    <node x="686.0" y="752.0">smarthomesimulator.model.ConsoleOutput</node>
+    <node x="369.0" y="0.0">smarthomesimulator.model.Room</node>
+    <node x="192.0" y="752.0">smarthomesimulator.model.Window</node>
+  </nodes>
+  <notes />
+  <edges>
+    <edge source="smarthomesimulator.model.Profile.Role" target="smarthomesimulator.model.Profile">
+      <point x="0.0" y="-125.5" />
+      <point x="0.0" y="101.0" />
+    </edge>
+  </edges>
+  <settings layout="Hierarchic Group" zoom="1.0" x="540.5" y="387.5" />
+  <SelectedNodes>
+    <node>smarthomesimulator.model.SHP</node>
+  </SelectedNodes>
+  <Categories>
+    <Category>Inner Classes</Category>
+    <Category>Properties</Category>
+    <Category>Methods</Category>
+    <Category>Constructors</Category>
+    <Category>Fields</Category>
+  </Categories>
+  <SCOPE>All</SCOPE>
+  <VISIBILITY>private</VISIBILITY>
+</Diagram>
+
diff --git a/smart-home-simulator/src/main/java/smarthomesimulator/interfaces/Observable.java b/smart-home-simulator/src/main/java/smarthomesimulator/interfaces/Observable.java
@@ -1,8 +1,5 @@
 package smarthomesimulator.interfaces;
 
 public interface Observable {
-
-    public void attachObserver(Observer o);
-    public void detachObserver(Observer o);
-    public void notifyObserver(Observable observable);
+    void notifyObserver(Observable observable);
 }
diff --git a/smart-home-simulator/src/main/java/smarthomesimulator/interfaces/Observer.java b/smart-home-simulator/src/main/java/smarthomesimulator/interfaces/Observer.java
@@ -1,5 +1,5 @@
 package smarthomesimulator.interfaces;
 
 public interface Observer {
-    public void update(Observable o);
+    void update(Observable o);
 }
diff --git a/smart-home-simulator/src/main/java/smarthomesimulator/layout/ParseLayout.java b/smart-home-simulator/src/main/java/smarthomesimulator/layout/ParseLayout.java
@@ -26,31 +26,35 @@ public static void parse(String fileName) throws IOException {
             Pattern patternLights = Pattern.compile("Lights:(.*?);");
             Matcher matcherLights = patternLights.matcher(line);
 
-            if (matcherRoomName.find()) {
-                int numOfDoors = 0;
-                int numOfWindows = 0;
-                int numOfLights = 0;
+            matchRooms(matcherRoomName, matcherDoors, matcherWindows, matcherLights);
+        }
+        br.close();
+    }
 
-                String roomName = matcherRoomName.group(1);
+    private static void matchRooms(Matcher matcherRoomName, Matcher matcherDoors, Matcher matcherWindows, Matcher matcherLights) {
+        if (matcherRoomName.find()) {
+            int numOfDoors = 0;
+            int numOfWindows = 0;
+            int numOfLights = 0;
 
-                if (matcherDoors.find()) {
-                    numOfDoors = (Integer.parseInt(matcherDoors.group(1)));
-                }
+            String roomName = matcherRoomName.group(1);
 
-                if (matcherWindows.find()) {
-                    numOfWindows = (Integer.parseInt(matcherWindows.group(1)));
-                }
+            if (matcherDoors.find()) {
+                numOfDoors = (Integer.parseInt(matcherDoors.group(1)));
+            }
 
-                if (matcherLights.find()) {
-                	numOfLights = ((Integer.parseInt(matcherLights.group(1))));
-                }
-
-                Room room = new Room(roomName, numOfDoors, numOfWindows,numOfLights);
+            if (matcherWindows.find()) {
+                numOfWindows = (Integer.parseInt(matcherWindows.group(1)));
+            }
 
-                Simulator.roomsOfHouse.add(room);
+            if (matcherLights.find()) {
+                numOfLights = ((Integer.parseInt(matcherLights.group(1))));
             }
+
+            Room room = new Room(roomName, numOfDoors, numOfWindows,numOfLights);
+            Simulator.roomsOfHouse.add(room);
         }
-        br.close();
     }
 
+
 }
diff --git a/smart-home-simulator/src/main/resources/static/js/shhTab.js b/smart-home-simulator/src/main/resources/static/js/shhTab.js