added virtual destructor and std::move() for string constructor parameter for optimization (for more details look out for comment in file)

Author: piyush26c

rider.hpp

@@ -2,14 +2,21 @@
 #include "common.hpp"
 
 class Rider {
-	string name;
-	RATING rating;
+    string name;
+    RATING rating;
 public:
-	Rider(string pName, RATING pRating) : name(pName), rating(pRating) {}
-	string getRiderName() {
-		return name;
-	}
-	RATING getRating() {
-		return rating;
-	}
-}; 
+	/* Moving (std::move()) a string involves transferring ownership of the internal data from the source to the destination, which is generally much faster than copying the data.
+	This is particularly beneficial for objects like std::string that manage dynamically allocated memory. */
+    Rider(string pName, RATING pRating) : name(std::move(pName)), rating(pRating) {
+    }
+
+    const string& getRiderName() const {
+        return name;
+    }
+
+    RATING getRating() const {
+        return rating;
+    }
+
+    virtual ~Rider() = default; // Added virtual destructor
+};