From c65d15c086cb61f41bb12bbca1fa18c2f950c43b Mon Sep 17 00:00:00 2001
From: manuelmanuntag96 <MANUELMANUNTAG15@GMAIL.COM>
Date: Wed, 14 Oct 2020 15:46:55 +0800
Subject: [PATCH] Implemented Logging for LogInCommand.java

---
 .gitignore                                    |  3 ++
 .../java/seedu/duke/command/AddCommand.java   |  2 +-
 .../java/seedu/duke/command/LogInCommand.java | 33 +++++++++++++++++--
 3 files changed, 35 insertions(+), 3 deletions(-)

diff --git a/.gitignore b/.gitignore
index f69985ef1f..fd1d7b1a2f 100644
--- a/.gitignore
+++ b/.gitignore
@@ -15,3 +15,6 @@ bin/
 
 /text-ui-test/ACTUAL.txt
 text-ui-test/EXPECTED-UNIX.TXT
+AddCommand.log
+LogInCommand.log.1
+LogInCommand.log
diff --git a/src/main/java/seedu/duke/command/AddCommand.java b/src/main/java/seedu/duke/command/AddCommand.java
index 3b34ef19b8..45425b9029 100644
--- a/src/main/java/seedu/duke/command/AddCommand.java
+++ b/src/main/java/seedu/duke/command/AddCommand.java
@@ -26,7 +26,7 @@ public AddCommand(String input) {
         super(input);
     }
 
-    private static Logger logger = Logger.getLogger("LogAddCommand");
+    private static Logger logger = Logger.getLogger("LogLogInCommand");
 
     @Override
     public void execute(UserList users, Ui ui, User nowUser/*, Storage storage*/) throws DukeException {
diff --git a/src/main/java/seedu/duke/command/LogInCommand.java b/src/main/java/seedu/duke/command/LogInCommand.java
index 14b3dfe410..35d8e18da2 100644
--- a/src/main/java/seedu/duke/command/LogInCommand.java
+++ b/src/main/java/seedu/duke/command/LogInCommand.java
@@ -8,6 +8,13 @@
 import seedu.duke.user.User;
 import seedu.duke.user.UserList;
 
+import java.io.IOException;
+import java.util.logging.FileHandler;
+import java.util.logging.Level;
+import java.util.logging.LogManager;
+import java.util.logging.Logger;
+import java.util.logging.SimpleFormatter;
+
 /**
  * Adds an event to the task list.
  */
@@ -17,18 +24,22 @@ public LogInCommand(String input) {
         super(input);
     }
 
+    private static Logger logger = Logger.getLogger("LogAddCommand");
+
     @Override
     public void execute(UserList users, Ui ui, User nowUser/*, Storage storage*/) throws DukeException {
         boolean doesExist = false;
+        setupInputLogger();
         
         String[] parsedInputs = input.split(" /", 2);
-        for (int i = 1; i < users.getTotalUserCount(); i++) {
+        for (int i = 1; i <= users.getTotalUserCount(); i++) {
             if (parsedInputs[0].equals((users.getUser(i)).getName())) {
                 if (parsedInputs[1].equals((users.getUser(i)).getPassWord())) {
                     currentUser = users.getUser(i);
                     //System.out.println(parsedInputs[0] + " == " + (users.getUser(i)).getName());
                     doesExist = true;
                 } else {
+                    logger.log(Level.WARNING,"Wrong Password" + "\n");
                     throw new DukeException("Wrong Password");
                 }
             }
@@ -39,11 +50,29 @@ public void execute(UserList users, Ui ui, User nowUser/*, Storage storage*/) th
             User newUser = new User(parsedInputs[0], parsedInputs[1]);
             currentUser = newUser;
             users.addUser(newUser);
+
+            logger.log(Level.INFO,"User Name successfully added:  " + parsedInputs[0]);
+            logger.log(Level.INFO,"Password successfully added:  " + parsedInputs[1]  + "\n");
         }
         ui.greetUser(currentUser);
         isLogIn = true;
 
-
+        logger.log(Level.INFO, "end of processing");
+        
         //storage.write(tasks);
     }
+
+    private void setupInputLogger() {
+        LogManager.getLogManager().reset();
+        logger.setLevel(Level.ALL);
+        try {
+            FileHandler fh = new FileHandler("LogInCommand.log",true);
+            fh.setLevel(Level.INFO);
+            fh.setFormatter(new SimpleFormatter());
+            logger.addHandler(fh);
+
+        } catch (IOException e) {
+            logger.log(Level.SEVERE,"File logging not working");
+        }
+    }
 }