Merge pull request #16 from tsarnadelis/giorgos

 Merge branch Giorgos to main, fix maintainability index

Author: tsarnadelis

service/LicensePlateService.js

@@ -8,48 +8,37 @@
  * body LicensePlate License plate object to update
  * no response value expected for this operation
  **/
-
-
 exports.modifyPlate = function(body) {
   return new Promise(function(resolve, reject) {
 
- 
-    
     // Μια ήδη καταχωρημένη πινακίδα μέσα στο σύστημα
     var existingPlates = {
       "licensePlate": "AKH1314", // όνομα πινακίδας
       "id": 15, // το id της πινακίδας
-      
     };
-
-    if (!body.licensePlate || body.licensePlate === "" ) {
-      //αν το licensePlate δεν υπάρχει στο request body ή ισούται με το κενό string τότε έχω σφάλμα με κωδικό σφάλματος 400
-      const error = new Error("license Plate does not exist");
-      error.response = { statusCode: 400 };
-      reject(error);
-      return;
-    }
-
-    // Ο έλεγχος αυτός έγινε comment διότι εκτελείται από το swagger. Συνεπώς, τον κάνω comment ώστε να μην μου χαλάει το
-    //coverage του modifyPlate
-    // if (typeof body.licensePlate !== "string"){
-    //   //αν το licensePlate έχει μη έγκυρη τιμή τότε έχω σφάλμα με κωδικό σφάλματος 400
-    //   const error = new Error("Invalid licenseplate");
-    //   error.response = { statusCode: 400 };
-    //   reject(error);
-    //   return;
-    // }
 
+    //Ορίζω κανόνες εγκυρότητας για τα fields της πινακίδας.
+    //Αν το licensePlate δεν υπάρχει στο request body ή ισούται με το κενό string τότε έχω σφάλμα με κωδικό σφάλματος 400.
+    //Επίσης, αν το id έχει μη έγκυρη τιμή τότε έχω σφάλμα με κωδικό σφάλματος 400.
+    // Έτσι, μειώνω το πλήθος των reject(error).
+    const validations = [
+      { key: 'licensePlate', rule: (value) => value && value !== "", errorMessage: "license Plate does not exist" },
+      { key: 'id', rule: (value) => Number.isInteger(value) && value > 0, 
+        errorMessage: "Invalid id: must be a positive integer." }
+    ];
 
-    if (!Number.isInteger(body.id) || body.id < 0) {
-      //αν το id έχει μη έγκυρη τιμή τότε έχω σφάλμα με κωδικό σφάλματος 400
-      const error = new Error("Invalid id: must be a positive integer.");
-      error.response = { statusCode: 400 };
-      reject(error);
-      return;
+    // Έλεγχος για σφάλματα στα fields της πινακίδας
+    for (const { key, rule, errorMessage } of validations) {
+      const value = body[key];
+      if (!rule(value)) { // Αν η τιμή δεν ικανοποιεί τον κανόνα εγκυρότητας (rule), τότε υπάρχει σφάλμα
+        const error = new Error(errorMessage);
+        error.response = { statusCode: 400 }; // Επιστρέφει κωδικό 400
+        reject(error); 
+        return;
+      }
     }
 
-   //Αν η dummy πινακίδα που είναι ήδη καταχωρημένη μέσα στο σύστημα έχει ίδιο id με εκείνο του request body και διαφορετικό
+    //Αν η dummy πινακίδα που είναι ήδη καταχωρημένη μέσα στο σύστημα έχει ίδιο id με εκείνο του request body και διαφορετικό
    // όνομα πινακίδας με εκείνο του request body, τότε συμβαίνει με επιτυχία η τροποποίηση της πινακίδας. Αν τα γνωρίσματα της 
    // dummy καταχωρημένης πινακίδας και του request body ταυτίζονται , τότε δεν τροποποιώ την πινακίδα αλλά αντίθετα διατηρώ 
    // τα "παλιά" της στοιχεία.Τέλος, αν όλα τα γνωρίσματα της dummy πινακίδας διαφέρουν με τα αντίστοιχα γνωρίσματα του request 
@@ -72,14 +61,13 @@ exports.modifyPlate = function(body) {
     // Handle non-existent ID (η πινακίδα δεν υπάρχει μέσα στο σύστημα)
     const isNonExistent = existingPlates.id !== body.id;
     if (isNonExistent) {
-        const error = new Error("License plate doesn't exist.");
+        const error = new Error("License plate doesn't exist."); // Η πινακίδα που επιθυμώ να τροποποιήσω ΔΕΝ βρέθηκε.
         error.response = { statusCode: 404 }; // Επιστρέφει κωδικό σφάλματος 404.
-        //Δηλαδή, η πινακίδα που επιθυμώ να τροποποιήσω ΔΕΝ βρέθηκε.
         reject(error);
         return;
     }
 
-    resolve();  // Return the updated plate object
+    resolve();
   });
 };
 
@@ -92,42 +80,28 @@ exports.modifyPlate = function(body) {
  * no response value expected for this operation
  **/
 exports.registerPlate = function(body) {
-  return new Promise((resolve, reject) => { 
+  return new Promise((resolve, reject) => {
+    //Ορίζω κανόνες εγκυρότητας για τα fields της πινακίδας.
+    //Αν το id έχει μη έγκυρη τιμή τότε έχω σφάλμα με κωδικό σφάλματος 400.
+    //Αν το licensePlate δεν υπάρχει στο request body ή ισούται με το κενό string τότε έχω σφάλμα με κωδικό σφάλματος 400.
+    // Έτσι, μειώνω το πλήθος των reject(error).
+    const validations = [
+      { key: 'id', rule: (value) => Number.isInteger(value) && value > 0, errorMessage: 'Invalid id: must be a positive integer.' },
+      { key: 'licensePlate', rule: (value) => value && value !== "", errorMessage: 'licenseplate does not exist' }
+    ];
 
-    // Μια ήδη καταχωρημένη πινακίδα μέσα στο σύστημα
-    var existingPlates = {
-      "licensePlate": "AKH1314", // το όνομα της πινακίδας
-      "id": 15, // το id της πινακίδας
-      
-    };
-
-    
-    if (!Number.isInteger(body.id) || body.id < 0) {
-      //αν το id έχει μη έγκυρη τιμή τότε έχω σφάλμα με κωδικό σφάλματος 400
-      const error = new Error("Invalid id: must be a positive integer.");
-      error.response = { statusCode: 400 };
-      reject(error);
-      return;
-    }
-     
-    
-    if (!body.licensePlate || body.licensePlate === "") {
-      //αν το licensePlate δεν υπάρχει στο request body ή ισούται με το κενό string τότε έχω σφάλμα με κωδικό σφάλματος 400
-      const error = new Error("licenseplate does not exist");
-      error.response = { statusCode: 400 };
-      reject(error);
-      return;
+    // Έλεγχος για σφάλματα στα fields της πινακίδας
+    for (const { key, rule, errorMessage } of validations) {
+      const value = body[key];
+      if (!rule(value)) { // Αν η τιμή δεν ικανοποιεί τον κανόνα εγκυρότητας (rule), τότε υπάρχει σφάλμα
+        const error = new Error(errorMessage);
+        error.response = { statusCode: 400 }; // Επιστρέφει κωδικό 400
+        reject(error); 
+        return;
+      }
     }
-  
-    // Ο έλεγχος αυτός έγινε comment διότι εκτελείται από το swagger. Συνεπώς, τον κάνω comment ώστε να μην μου χαλάει το
-    //coverage του registerPlate
-    // if (typeof body.licensePlate !== "string"){
-    //   //αν το licensePlate έχει μη έγκυρη τιμή τότε έχω σφάλμα με κωδικό σφάλματος 400
-    //   const error = new Error("Invalid licenseplate");
-    //   error.response = { statusCode: 400 };
-    //   reject(error);
-    //   return;
-    // }
+
     resolve();
-    });
+  });
 };
+

service/SpotOwnerService.js

@@ -1,80 +1,43 @@
-'use strict';
-
-
 /**
  * Add a new spot owner
  * FR15: The system administrator must be able to add a spot owner to the system
  *
  * body SpotOwner Spot owner object to add
  * no response value expected for this operation
  **/
-
-
 exports.addSpotOwner = function(body) {
-  return new Promise((resolve, reject) =>{
-    //SpotOwner = Ιδιοκτήτης θέσεων πάρκινγκ
-    var existingSpotOwners = {
-      "id":1,  // το id του ιδιοκτήτη θέσεων
-      "idNumber": "AK1234", // ο αριθμός ταυτότητας του ιδιοκτήτη θέσεων
-      "name":"John Doe", // ο ονοματεπώνυμο του ιδιοκτήτη  
-      "email":"johhdoe@gmail.com", // το email του ιδιοκτήτη
-      "phone":"1234567890",// το τηλέφωνο του ιδιοκτήτη
-      "spots":[] // οι θέσεις του ιδιοκτήτη
-    };
-     
+  return new Promise((resolve, reject) => {
 
-    if (!body.id || body.id < 0) {
-      //αν το id έχει μη έγκυρη τιμή τότε έχω σφάλμα με κωδικό σφάλματος 400
-      const error = new Error("Invalid id: must be a positive integer.");
-      error.response = { statusCode: 400 };
-      reject(error);
-      return;
-    }
-
-    if (!body.idNumber || typeof body.idNumber !== "string") {
-      //αν το idNumber έχει μη έγκυρη τιμή τότε έχω σφάλμα με κωδικό σφάλματος 400
-      const error = new Error("Invalid idNumber: must be a string.");
-      error.response = { statusCode: 400 };
-      reject(error);
-      return;
-    }
-
-    if (!body.name || typeof body.name !== "string") {  
-      //αν το name έχει μη έγκυρη τιμή τότε έχω σφάλμα με κωδικό σφάλματος 400
-      const error = new Error("Invalid name: must be a string.");
-      error.response = { statusCode: 400 };
-      reject(error);
-      return;
-    }
-
-    if (!body.email || typeof body.email !== "string") {
-
-      //αν το email έχει μη έγκυρη τιμή τότε έχω σφάλμα με κωδικό σφάλματος 400
-      const error = new Error("Invalid email: must be a string.");
-      error.response = { statusCode: 400 };
-      reject(error);
-      return;
-    }
+    //idNumber = αριθμός ταυτότητας
+    //spots = μια λίστα που περιέχει τις θέσεις που ανήκουν στον ιδιοκτήτη θέσεων.
 
-    if (!body.phone) {
-
-      //αν το phone δεν υπάρχει στο request body τότε έχω σφάλμα με κωδικό σφάλματος 400
-      const error = new Error("No phone.");
-      error.response = { statusCode: 400 };
-      reject(error);
-      return;
-    }
-
-    if (!body.spots) {
-
-      //αν το spots δεν υπάρχει στο request body τότε έχω σφάλμα με κωδικό σφάλματος 400
-      const error = new Error("No spots.");
-      error.response = { statusCode: 400 };
-      reject(error);
-      return;
+    // Ορίζω κανόνες εγκυρότητας για κάθε field του Ιδιοκτήτη θέσεων.
+    //Αν το id δεν είναι θετικός ακέραιος τότε έχω σφάλμα με κωδικό 400.
+    //Αν το idNumber δεν είναι string τότε έχω σφάλμα με κωδικό 400. Το ίδιο ισχύει και για τα name και email.
+    //Αν το phone δεν υπάρχει στο request body τότε έχω σφάλμα με κωδικό σφάλματος 400. Το ίδιο ισχύει και για το spots.
+    const validations = [
+      { key: 'id', rule: (value) => value > 0 && Number.isInteger(value), errorMessage: 'Invalid id: must be a positive integer.' },
+      { key: 'idNumber', rule: (value) => typeof value === 'string' && value !== undefined 
+        && value !== null && value.trim() !== '', errorMessage: 'Invalid idNumber: must be a string.' },
+      { key: 'name', rule: (value) => typeof value === 'string' && value !== undefined 
+        && value !== null && value.trim() !== '', errorMessage: 'Invalid name: must be a string.' },
+      { key: 'email', rule: (value) => typeof value === 'string' && value !== undefined 
+        && value !== null && value.trim() !== '', errorMessage: 'Invalid email: must be a string.' },
+      { key: 'phone', rule: (value) => value !== undefined, errorMessage: 'No phone.' },
+      { key: 'spots', rule: (value) => value !== undefined, errorMessage: 'No spots.' },
+    ];
+
+    // Έλεγχος για σφάλματα στα fields του Ιδιοκτήτη θέσεων
+    for (const { key, rule, errorMessage } of validations) {
+      const value = body[key];
+      if (!rule(value)) { // Αν η τιμή δεν ικανοποιεί τον κανόνα εγκυρότητας (rule), τότε υπάρχει σφάλμα
+        const error = new Error(errorMessage);
+        error.response = { statusCode: 400 }; // Επιστρέφει κωδικό 400
+        reject(error);
+        return;
+      }
     }
 
     resolve();
-    });
-
-}
+  });
+};