Refactored UserExistsInteractor so that it satisfies single responsibility principle

Author: Madhavan7

src/main/java/screens/user_registration_screen/UserRegistrationUI.java

@@ -81,7 +81,7 @@ public void getUserCredentials(){
     }
 
     public static void main(String[] args){
-        Database testDB = new UserDatabase(new File("newAccounts"));
+        Database testDB = new UserDatabase(new File("newAccounts2"));
         UserLoginInputBoundary userLoginInteractor = new UserLoginInteractor(testDB);
         UserVerificationOutputBoundary loginUI = new UserLoginUI(userLoginInteractor);
         UserVerificationInputBoundary verificationInteractor = new UserVerificationInteractor(testDB, loginUI);

src/main/java/use_cases/user_registration_use_cases/UserExistsInteractor.java

@@ -5,34 +5,32 @@
 import java.util.Random;
 
 public class UserExistsInteractor implements UserExistsInputBoundary{
-    private final createMailMan mailManFactory;
-    //May need to refactor this using facade design pattern since this class has too many responsibilities.
+    private final VerificationCodeDeliveryManager verCodeDeliveryManager;
     Database database;
     UserExistsOutputBoundary existsOutputBoundary;
 
-    private ISendVerificationCode codeMailMan;
-
     public UserExistsInteractor(Database database, UserExistsOutputBoundary existsOutputBoundary, createMailMan mailMan){
         this.database = database;
         this.existsOutputBoundary = existsOutputBoundary;
-        this.mailManFactory = mailMan;
+        //The responsibility of dealing with verification is passed onto this class
+        this.verCodeDeliveryManager = new VerificationCodeDeliveryManager(mailMan);
     }
     @Override
     public void register(String username, String password, String email) {
         if(!database.UserExists(username, email)){
-            int code = new Random().nextInt(12312341);
+            int code = this.verCodeDeliveryManager.getVerCode();
             existsOutputBoundary.getCode(code);
             existsOutputBoundary.getUserCredentials(username, password, email);
             existsOutputBoundary.getVerificationCredentials();
-            codeMailMan.sendVerificationCode(email, code);
+            this.verCodeDeliveryManager.deliverCode(email);
         }else{
             existsOutputBoundary.presentUserExistsMessage();
         }
     }
 
     @Override
     public void setCodeDeliveryMethod(String type) {
-        this.codeMailMan = mailManFactory.getVerificationMethod(type);
+        this.verCodeDeliveryManager.setMailMan(type);
     }
 
     @Override

src/main/java/use_cases/user_registration_use_cases/VerificationCodeDeliveryManager.java

@@ -0,0 +1,23 @@
+package use_cases.user_registration_use_cases;
+
+import java.util.Random;
+
+public class VerificationCodeDeliveryManager {
+    private final int code = new Random().nextInt(134134);
+    private final createMailMan mailManFactory;
+    private ISendVerificationCode mailMan;
+    public VerificationCodeDeliveryManager(createMailMan mailManFactory){
+        this.mailManFactory = mailManFactory;
+        //By default, the type will be email.
+        this.setMailMan("Email");
+    }
+    public void setMailMan(String type){
+        this.mailMan = mailManFactory.getVerificationMethod(type);
+    }
+    public void deliverCode(String email){
+        this.mailMan.sendVerificationCode(email, this.code);
+    }
+    public int getVerCode(){
+        return this.code;
+    }
+}