DataHandler updated
authorMatthias <balmm1@bfh.ch>
Tue, 8 Dec 2015 10:36:26 +0000 (11:36 +0100)
committerMatthias <balmm1@bfh.ch>
Tue, 8 Dec 2015 10:36:26 +0000 (11:36 +0100)
src/main/java/ch/bfh/ti/srs/datahandler/DataHandler.java
src/main/java/ch/bfh/ti/srs/datahandler/DataHandlerForTests.java
src/main/java/ch/bfh/ti/srs/noTest/DataHandlerForExcel.java

index c45fe73..89fc75e 100644 (file)
@@ -43,6 +43,14 @@ public interface DataHandler {
        public boolean addUser(User user) throws Exception;
 
        /**
+        * Edit a User with the same Name.
+        * @param User
+        *            the new User 
+        * @return a boolean whether the user was edited or not
+        */
+       public boolean editUser(User user);
+       
+       /**
         * Finds the resourceName in the Datacontext and returns the Resource object
         * or null if not found
         * @param resourceName
@@ -132,13 +140,44 @@ public interface DataHandler {
         */
        public void refresh(boolean discardChanges) throws Exception;
 
+       /**
+        * Edit a Reservation with the same ID.
+        * @param reservation
+        *            the new Reservation 
+        * @return a boolean whether the reservation was edited or not
+        */
        public boolean editReservation(Reservation reservation);
 
+       /**
+        * deletes a Reservation from the datastorage
+        * @param reservation
+        *            the Reservation to remove
+        * @return a boolean whether the reservation was deleted or not
+        */
        public boolean deleteReservation(Reservation reservation);
-       
+
+       /**
+        * Edit a Resource with the same ID.
+        * @param resource
+        *            the new Resource 
+        * @return a boolean whether the resource was edited or not
+        */
        public boolean editResource(Resource resource);
-       
+
+       /**
+        * deletes a Resource from the datastorage
+        * @param resource
+        *            the Resource to remove
+        * @return a boolean whether the resource was deleted or not
+        */
        public boolean deleteResource(Resource resource);
 
+       /**
+        * get all the Resources from the datastorage
+        * 
+        * @return a set of Resources
+        */
        public Set<Resource> getAllResources();
 }
+
+
index 55a142c..7c76508 100644 (file)
@@ -196,6 +196,9 @@ public class DataHandlerForTests implements DataHandler {
                        throw new Exception("No implementation on DataHandlerForTests possible");
        }
 
+       /**
+        * {@inheritDoc}
+        */
        @Override
        public boolean editReservation(Reservation reservation) {
                if (reservation == null) {
@@ -209,6 +212,9 @@ public class DataHandlerForTests implements DataHandler {
                return false;
        }
 
+       /**
+        * {@inheritDoc}
+        */
        @Override
        public boolean deleteReservation(Reservation reservation) {
                if (reservation == null) {
@@ -221,22 +227,56 @@ public class DataHandlerForTests implements DataHandler {
                }
        }
 
+       /**
+        * {@inheritDoc}
+        */
        @Override
        public Set<Resource> getAllResources() {
 
                return resourceList;
        }
 
+       /**
+        * {@inheritDoc}
+        */
        @Override
        public boolean editResource(Resource resource) {
-               // TODO Auto-generated method stub
+               if (resource == null) {
+                       throw new IllegalArgumentException("Resource object must not be null!");
+               }
+               if (findResource(resource.getName()) != null) {
+                       resourceList.remove(findResource(resource.getName()));
+                       resourceList.add(resource);
+                       return true;
+               }
                return false;
        }
 
+       /**
+        * {@inheritDoc}
+        */
        @Override
        public boolean deleteResource(Resource resource) {
-               // TODO Auto-generated method stub
-               return false;
+               if (resource == null) {
+                       throw new IllegalArgumentException("Resource object must not be null!");
+               } else if (resourceList.contains(resource)) {
+                       resourceList.remove(resource);
+                       return true;
+               } else {
+                       return false;
+               }
        }
 
+       @Override
+       public boolean editUser(User user) {
+               if (user == null) {
+                       throw new IllegalArgumentException("User object must not be null!");
+               }
+               if (findUser(user.getName()) != null) {
+                       userList.remove(findUser(user.getName()));
+                       userList.add(user);
+                       return true;
+               }
+               return false;
+       }
 }
index c530d87..01d308b 100644 (file)
@@ -218,4 +218,10 @@ public class DataHandlerForExcel implements DataHandler {
                // TODO Auto-generated method stub
                return false;
        }
+
+       @Override
+       public boolean editUser(User user) {
+               // TODO Auto-generated method stub
+               return false;
+       }
 }