DataHandler update
authorMatthias <balmm1@bfh.ch>
Mon, 23 Nov 2015 10:30:34 +0000 (11:30 +0100)
committerMatthias <balmm1@bfh.ch>
Mon, 23 Nov 2015 10:30:34 +0000 (11:30 +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 2728ef8..79fa790 100644 (file)
@@ -35,9 +35,10 @@ public interface DataHandler {
         * Adds a user to the Datacontext from a given User object
         * @param user
         *            User object
+        * @return returns a boolean whether the user was made or not
         * @throws Exception
         */
-       public void addUser(User user) throws Exception;
+       public boolean addUser(User user) throws Exception;
 
        /**
         * Finds the resourceName in the Datacontext and returns the Resource object
@@ -53,9 +54,10 @@ public interface DataHandler {
         * Adds a resource to the Datacontext from a given Recource object
         * @param resource
         *            Resource object
+        * @return returns a boolean whether the resource was made or not
         * @throws Exception
         */
-       public void addResource(Resource resource) throws Exception;
+       public boolean addResource(Resource resource) throws Exception;
 
        /**
         * Finds the reservation by the id param in the Datacontext and returns the
@@ -104,9 +106,11 @@ public interface DataHandler {
         * Adds a reservation to the Datacontext from a given Reservation object
         * @param reservation
         *            Reservation object
+        *            
+        * @return returns a boolean whether the reservation was made or not
         * @throws Exception
         */
-       public void addReservation(Reservation reservation) throws Exception;
+       public boolean addReservation(Reservation reservation) throws Exception;
        
        /**
         * Saves the current changes on the local data to the datastorage
index 916047c..c23c5eb 100644 (file)
@@ -43,12 +43,17 @@ public class DataHandlerForTests implements DataHandler {
         * {@inheritDoc}
         */
        @Override
-       public void addUser(User user) throws Exception {
-               if (user == null)
+       public boolean addUser(User user) throws Exception {
+               if (user == null){
                        throw new IllegalArgumentException("User object must not be null!");
-               if (userList.contains(user))
-                       throw new IllegalArgumentException("User already  exists!");
-               userList.add(user);
+               }
+               else if (userList.contains(user)){
+                       return false;
+               }
+               else{
+                       userList.add(user);
+                       return true;
+               }
        }
 
        /**
@@ -67,12 +72,17 @@ public class DataHandlerForTests implements DataHandler {
         * {@inheritDoc}
         */
        @Override
-       public void addResource(Resource resource) throws Exception {
-               if (resource == null)
+       public boolean addResource(Resource resource) throws Exception {
+               if (resource == null){
                        throw new IllegalArgumentException("Resource object must not be null!");
-               if (resourceList.contains(resource))
-                       throw new IllegalArgumentException("Resource already  exists!");
-               resourceList.add(resource);
+               }
+               else if (resourceList.contains(resource)){
+                       return false;
+               }
+               else{
+                       resourceList.add(resource);
+                       return true;
+               }
        }
 
        /**
@@ -148,12 +158,17 @@ public class DataHandlerForTests implements DataHandler {
         * {@inheritDoc}
         */
        @Override
-       public void addReservation(Reservation reservation) throws Exception {
-               if (reservation == null)
+       public boolean addReservation(Reservation reservation) throws Exception {
+               if (reservation == null){
                        throw new IllegalArgumentException("Reservation object must not be null!");
-               if (resourceList.contains(reservation))
-                       throw new IllegalArgumentException("Reservation already  exists!");
-               reservationList.add(reservation);
+               }
+               else if(resourceList.contains(reservation)) {
+                       return false;
+               }
+               else{
+                       reservationList.add(reservation);
+                       return true;
+               }
        }
 
        /**
index e38e2a9..b25377b 100644 (file)
@@ -51,11 +51,14 @@ public class DataHandlerForExcel implements DataHandler {
        }
 
        @Override
-       public void addUser(User user) throws Exception {
+       public boolean addUser(User user) throws Exception {
                if (findUser(user.getUserName()) != null)
                        throw new Exception("User already exists!");
+               else{
                insertNewUser(user);
                updateUserList();
+               return true;
+               }
        }
 
        private void updateUserList() throws Exception {
@@ -127,15 +130,15 @@ public class DataHandlerForExcel implements DataHandler {
        }
 
        @Override
-       public void addResource(Resource resource) throws Exception {
+       public boolean addResource(Resource resource) throws Exception {
                // TODO Auto-generated method stub
-
+               return false;
        }
 
        @Override
-       public void addReservation(Reservation reservation) throws Exception {
+       public boolean addReservation(Reservation reservation) throws Exception {
                // TODO Auto-generated method stub
-
+               return false;
        }
 
        @Override