[CS2103T-W13-4] TutorPro

src/main/java/seedu/address/commons/core/Messages.java

@@ -39,6 +39,13 @@ public class Messages {
     public static final String MESSAGE_EXAMS_LISTED_OVERVIEW = "%d exams from %d students listed: \n%s";
     public static final String MESSAGE_ALL_EXAMS_LISTED_OVERVIEW = "%d exams from all students listed:\n%s";
     public static final String MESSAGE_NO_EXAM_FOUND = "No exam is found!";
+    public static final String MESSAGE_EXAM_NOT_COMPLETED = "Exam is not yet completed, a grade cannot be assigned!";
+    public static final String MESSAGE_EXAM_UPDATED_SUCCESS = "Exam %s of student %s is updated to:\n"
+            + "Exam name: %s\n"
+            + "Start Time: %s\n"
+            + "End Time: %s\n"
+            + "Weightage: %s" + "%%" + "\n"
+            + "Grade: %s\n";
     public static final String MESSAGE_HOMEWORK_UPDATED_SUCCESS = "Homework %s of student %s is updated to:\n"
             + "Homework name: %s\n"
             + "Deadline: %s\n";

src/main/java/seedu/address/logic/commands/exam/CreateExamCommand.java

@@ -7,7 +7,6 @@
 import static seedu.address.logic.parser.CliSyntax.PREFIX_STARTTIME;
 import static seedu.address.model.Model.PREDICATE_SHOW_ALL_STUDENTS;
 
-import java.time.Duration;
 import java.time.LocalDateTime;
 import java.util.List;
 
@@ -17,6 +16,7 @@
 import seedu.address.logic.commands.exceptions.CommandException;
 import seedu.address.model.Model;
 import seedu.address.model.student.Exam;
+import seedu.address.model.student.Grade;
 import seedu.address.model.student.NamePredicate;
 import seedu.address.model.student.Student;
 
@@ -45,12 +45,15 @@ public class CreateExamCommand extends Command {
     private final LocalDateTime endTime;
     private final NamePredicate predicate;
     private final List<String> names;
+    private final Double weightage;
+    private final Grade grade;
+
 
     /**
      * Creates a CreateExamCommand to add the specified exam to the specified student.
      */
     public CreateExamCommand(List<String> names, NamePredicate predicate, String examDescription,
-                             LocalDateTime startTime, LocalDateTime endTime) {
+                             LocalDateTime startTime, LocalDateTime endTime, Double weightage, Grade grade) {
         requireNonNull(predicate);
         requireNonNull(examDescription);
         requireNonNull(startTime);
@@ -60,6 +63,9 @@ public CreateExamCommand(List<String> names, NamePredicate predicate, String exa
         this.endTime = endTime;
         this.predicate = predicate;
         this.names = names;
+        this.weightage = weightage;
+        this.grade = grade;
+
     }
 
     @Override
@@ -91,21 +97,11 @@ public CommandResult execute(Model model) throws CommandException {
 
         List<Student> studentList = model.getFilteredStudentList();
 
-        if (startTime.isBefore(LocalDateTime.now())) {
-            throw new CommandException("start time must be in the future.");
-        }
-
-        if (startTime.isAfter(endTime)) {
-            throw new CommandException(Messages.MESSAGE_INVALID_EXAM_TIME);
-        }
-
-        if (Duration.between(startTime, endTime).toMinutes() < 30 || Duration.between(startTime,
-            endTime).toHours() > 3) {
-            throw new CommandException(Messages.MESSAGE_INVALID_EXAM_DURATION);
+        if (endTime.isAfter(LocalDateTime.now()) && grade != null) {
+            throw new CommandException(Messages.MESSAGE_EXAM_NOT_COMPLETED);
         }
 
-        //Todo: currently weightage is 0 for convenience, implement this where possible
-        Exam exam = new Exam(examDescription, startTime, endTime, 0d, null);
+        Exam exam = new Exam(examDescription, startTime, endTime, weightage, grade);
 
         try {
             for (Student student : studentList) {

src/main/java/seedu/address/logic/commands/exam/UpdateExamCommand.java

@@ -2,13 +2,14 @@
 
 import static java.util.Objects.requireNonNull;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_ENDTIME;
+import static seedu.address.logic.parser.CliSyntax.PREFIX_EXAM;
+import static seedu.address.logic.parser.CliSyntax.PREFIX_GRADE;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_INDEX;
-import static seedu.address.logic.parser.CliSyntax.PREFIX_LESSON;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_NAME;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_STARTTIME;
+import static seedu.address.logic.parser.CliSyntax.PREFIX_WEIGHT;
 import static seedu.address.model.Model.PREDICATE_SHOW_ALL_STUDENTS;
 
-import java.time.Duration;
 import java.time.LocalDateTime;
 import java.util.List;
 import java.util.Optional;
@@ -20,6 +21,7 @@
 import seedu.address.logic.commands.exceptions.CommandException;
 import seedu.address.model.Model;
 import seedu.address.model.student.Exam;
+import seedu.address.model.student.Grade;
 import seedu.address.model.student.NamePredicate;
 import seedu.address.model.student.Student;
 
@@ -34,18 +36,23 @@ public class UpdateExamCommand extends Command {
         + "Parameters: "
         + PREFIX_NAME + "STUDENT_NAME "
         + PREFIX_INDEX + "EXAM_INDEX "
-        + PREFIX_LESSON + "EXAM_NAME "
+        + PREFIX_EXAM + "EXAM_NAME "
         + PREFIX_STARTTIME + "START TIME"
-        + PREFIX_ENDTIME + "END TIME\n"
+        + PREFIX_ENDTIME + "END TIME"
+        + PREFIX_WEIGHT + "WEIGHTAGE"
+        + PREFIX_GRADE + "GRADE\n"
         + "Example: " + COMMAND_WORD + " "
         + PREFIX_NAME + "John Doe "
         + PREFIX_INDEX + "1 "
-        + PREFIX_LESSON + "Math Exam ";
+        + PREFIX_EXAM + "Math Exam ";
 
     private final Index index;
     private final Optional<String> examName;
     private final Optional<LocalDateTime> startTime;
     private final Optional<LocalDateTime> endTime;
+    private final Optional<Double> weightage;
+    private final Optional<Grade> grade;
+
     private final NamePredicate predicate;
     private final List<String> names;
 
@@ -60,7 +67,8 @@ public class UpdateExamCommand extends Command {
      */
     public UpdateExamCommand(List<String> names, Index index, NamePredicate predicate,
                                Optional<String> examName, Optional<LocalDateTime> startTime,
-                               Optional<LocalDateTime> endTime) {
+                               Optional<LocalDateTime> endTime, Optional<Double> weightage,
+                               Optional<Grade> grade) {
         requireNonNull(predicate);
         requireNonNull(index);
 
@@ -70,6 +78,8 @@ public UpdateExamCommand(List<String> names, Index index, NamePredicate predicat
         this.startTime = startTime;
         this.endTime = endTime;
         this.names = names;
+        this.weightage = weightage;
+        this.grade = grade;
     }
 
     @Override
@@ -109,22 +119,28 @@ public CommandResult execute(Model model) throws CommandException {
         if (newStartTime.isAfter(newEndTime)) {
             throw new CommandException(Messages.MESSAGE_INVALID_EXAM_TIME);
         }
-
-        if (Duration.between(newStartTime, newEndTime).toMinutes() < 30 || Duration.between(newStartTime,
-            newEndTime).toHours() > 3) {
-            throw new CommandException(Messages.MESSAGE_INVALID_EXAM_DURATION);
+        if (newEndTime.isAfter(LocalDateTime.now()) && grade.isPresent()) {
+            throw new CommandException(Messages.MESSAGE_EXAM_NOT_COMPLETED);
         }
-        Exam newExam = new Exam(newExamName, newStartTime, newEndTime);
+        Double newWeightage = this.weightage.orElse(examToUpdate.getWeightage());
+        Grade newGrade = this.grade.orElse(examToUpdate.getGrade());
+
+        //if (Duration.between(newStartTime, newEndTime).toMinutes() < 30 || Duration.between(newStartTime,
+        //    newEndTime).toHours() > 3) {
+        //    throw new CommandException(Messages.MESSAGE_INVALID_EXAM_DURATION);
+        //}
+        Exam newExam = new Exam(newExamName, newStartTime, newEndTime, newWeightage, newGrade);
+
         try {
             student.setExam(index.getZeroBased(), newExam);
         } catch (Exception e) {
             throw new CommandException(e.getMessage());
         }
 
         return new CommandResult(
-            String.format(Messages.MESSAGE_LESSON_UPDATED_SUCCESS, index.getOneBased(),
+            String.format(Messages.MESSAGE_EXAM_UPDATED_SUCCESS, index.getOneBased(),
                 student.getName().getFirstName(),
-                newExamName, newStartTime, newEndTime));
+                newExamName, newStartTime, newEndTime, newWeightage, newGrade));
     }
 
     @Override

src/main/java/seedu/address/logic/commands/exam/ViewExamCommand.java

@@ -129,15 +129,15 @@ public CommandResult execute(Model model) throws CommandException {
 
         // Loop through each student and add their lesson to the string builder
         for (Student student : studentList) {
-            List<Exam> lessonList = student.getFilteredExamList(examDatePredicate, examNamePredicate,
+            List<Exam> examList = student.getFilteredExamList(examDatePredicate, examNamePredicate,
                 donePredicate);
-            if (!lessonList.isEmpty()) {
+            if (!examList.isEmpty()) {
                 sb.append(student.getName().fullName).append(":\n");
 
-                numOfLessons += lessonList.size();
+                numOfLessons += examList.size();
 
-                for (int i = 0; i < lessonList.size(); i++) {
-                    sb.append(i + 1).append(". ").append(lessonList.get(i)).append("\n");
+                for (int i = 0; i < examList.size(); i++) {
+                    sb.append(i + 1).append(". ").append(examList.get(i)).append("\n");
                 }
 
                 sb.append("--------------------------------------------------\n");

src/main/java/seedu/address/logic/parser/CliSyntax.java

@@ -16,6 +16,8 @@ public class CliSyntax {
     public static final Prefix PREFIX_HOMEWORK = new Prefix("homework/");
     public static final Prefix PREFIX_DEADLINE = new Prefix("deadline/");
     public static final Prefix PREFIX_EXAM = new Prefix("exam/");
+    public static final Prefix PREFIX_WEIGHT = new Prefix("weightage/");
+    public static final Prefix PREFIX_GRADE = new Prefix("grade/");
 
     public static final Prefix PREFIX_STATUS = new Prefix("status/");
     public static final Prefix PREFIX_INDEX = new Prefix("index/");

src/main/java/seedu/address/logic/parser/ParserUtil.java

@@ -23,6 +23,7 @@
 import seedu.address.logic.parser.exceptions.ParseException;
 import seedu.address.model.student.Address;
 import seedu.address.model.student.Email;
+import seedu.address.model.student.Grade;
 import seedu.address.model.student.Homework;
 import seedu.address.model.student.Name;
 import seedu.address.model.student.Phone;
@@ -392,4 +393,43 @@ public static void checkNotNullNames(List<String> names) throws ParseException {
             }
         }
     }
+
+    /**
+     * parses a string and returns a double representing the percentage weightage
+     * @param weight string to parse
+     * @return Double percentage
+     * @throws ParseException
+     */
+    public static double parseWeightage(String weight) throws ParseException {
+        if (!weight.matches("^[0-9]+(?:\\.[0-9]+)?%?$")) {
+            throw new ParseException("Weightage is in an invalid format!");
+        }
+        weight = weight.replace("%", ""); //removes % sign if it exists
+        Double res = null;
+        try {
+            res = Double.parseDouble(weight);
+        } catch (NumberFormatException e) {
+            throw new ParseException("unexpected error occurred when parsing weightage", e);
+        }
+        return res;
+    }
+
+    /**
+     * parses a string and returns a Grade object representing it
+     * @param grade string to parse
+     * @return Grade representationn
+     * @throws ParseException
+     */
+    public static Grade parseGrade(String grade) throws ParseException {
+        if (!grade.matches("^[0-9]+/[0-9]+$")) {
+            throw new ParseException("Grade is in an invalid format!");
+        }
+        Grade res;
+        try {
+            res = new Grade(Double.parseDouble(grade.split("/")[0]), Double.parseDouble(grade.split("/")[1]));
+        } catch (IllegalArgumentException e) {
+            throw new ParseException(e.getMessage());
+        }
+        return res;
+    }
 }

src/main/java/seedu/address/logic/parser/exam/CreateExamCommandParser.java

@@ -4,8 +4,10 @@
 import static seedu.address.commons.core.Messages.MESSAGE_INVALID_COMMAND_FORMAT;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_ENDTIME;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_EXAM;
+import static seedu.address.logic.parser.CliSyntax.PREFIX_GRADE;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_NAME;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_STARTTIME;
+import static seedu.address.logic.parser.CliSyntax.PREFIX_WEIGHT;
 
 import java.time.LocalDateTime;
 import java.util.ArrayList;
@@ -19,6 +21,7 @@
 import seedu.address.logic.parser.ParserUtil;
 import seedu.address.logic.parser.Prefix;
 import seedu.address.logic.parser.exceptions.ParseException;
+import seedu.address.model.student.Grade;
 import seedu.address.model.student.NamePredicate;
 
 /**
@@ -35,7 +38,8 @@ public CreateExamCommand parse(String args) throws ParseException {
         requireNonNull(args);
 
         ArgumentMultimap argMultimap =
-                ArgumentTokenizer.tokenize(args, PREFIX_NAME, PREFIX_EXAM, PREFIX_STARTTIME, PREFIX_ENDTIME);
+                ArgumentTokenizer.tokenize(args, PREFIX_NAME, PREFIX_EXAM, PREFIX_STARTTIME, PREFIX_ENDTIME,
+                        PREFIX_WEIGHT, PREFIX_GRADE);
 
         if (!arePrefixesPresent(argMultimap, PREFIX_NAME, PREFIX_EXAM, PREFIX_STARTTIME, PREFIX_ENDTIME)
                 || !argMultimap.getPreamble().isEmpty()) {
@@ -52,6 +56,18 @@ public CreateExamCommand parse(String args) throws ParseException {
         LocalDateTime endTime = ParserUtil.parseDeadline(argMultimap.getValue(PREFIX_ENDTIME).get());
         List<String> nameKeywords = argMultimap.getAllValues(PREFIX_NAME);
 
+        Double weightage = null;
+        //if the weightage of an exam is provided, store the weight
+        if (argMultimap.getValue(PREFIX_WEIGHT).isPresent()) {
+            weightage = ParserUtil.parseWeightage(argMultimap.getValue(PREFIX_WEIGHT).get());
+        }
+
+        Grade grade = null;
+        //if the grade of an exam is provided, store the grade
+        if (argMultimap.getValue(PREFIX_GRADE).isPresent()) {
+            grade = ParserUtil.parseGrade(argMultimap.getValue(PREFIX_GRADE).get());
+        }
+
         // for all the names, trim the name and only take the first word
         for (int i = 0; i < nameKeywords.size(); i++) {
             String name = nameKeywords.get(i);
@@ -69,7 +85,7 @@ public CreateExamCommand parse(String args) throws ParseException {
         names = nameKeywords;
 
         return new CreateExamCommand(names, new NamePredicate(nameKeywords),
-                examDescription, startTime, endTime);
+                examDescription, startTime, endTime, weightage, grade);
     }
 
     /**

src/main/java/seedu/address/logic/parser/exam/UpdateExamCommandParser.java

@@ -4,9 +4,11 @@
 import static seedu.address.commons.core.Messages.MESSAGE_INVALID_COMMAND_FORMAT;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_ENDTIME;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_EXAM;
+import static seedu.address.logic.parser.CliSyntax.PREFIX_GRADE;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_INDEX;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_NAME;
 import static seedu.address.logic.parser.CliSyntax.PREFIX_STARTTIME;
+import static seedu.address.logic.parser.CliSyntax.PREFIX_WEIGHT;
 
 import java.time.LocalDateTime;
 import java.util.ArrayList;
@@ -22,6 +24,7 @@
 import seedu.address.logic.parser.ParserUtil;
 import seedu.address.logic.parser.Prefix;
 import seedu.address.logic.parser.exceptions.ParseException;
+import seedu.address.model.student.Grade;
 import seedu.address.model.student.NamePredicate;
 
 /**
@@ -40,11 +43,13 @@ public UpdateExamCommand parse(String args) throws ParseException {
 
         ArgumentMultimap argMultimap =
             ArgumentTokenizer.tokenize(args, PREFIX_NAME, PREFIX_INDEX, PREFIX_EXAM, PREFIX_STARTTIME,
-                PREFIX_ENDTIME);
+                PREFIX_ENDTIME, PREFIX_WEIGHT, PREFIX_GRADE);
 
         if ((!arePrefixesPresent(argMultimap, PREFIX_NAME, PREFIX_INDEX, PREFIX_EXAM)
             && !arePrefixesPresent(argMultimap, PREFIX_NAME, PREFIX_INDEX, PREFIX_STARTTIME)
-            && !arePrefixesPresent(argMultimap, PREFIX_NAME, PREFIX_INDEX, PREFIX_ENDTIME))
+            && !arePrefixesPresent(argMultimap, PREFIX_NAME, PREFIX_INDEX, PREFIX_ENDTIME)
+            && !arePrefixesPresent(argMultimap, PREFIX_NAME, PREFIX_INDEX, PREFIX_WEIGHT)
+            && !arePrefixesPresent(argMultimap, PREFIX_NAME, PREFIX_INDEX, PREFIX_GRADE))
             || !argMultimap.getPreamble().isEmpty()) {
             throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT,
                 UpdateExamCommand.MESSAGE_USAGE));
@@ -70,10 +75,10 @@ public UpdateExamCommand parse(String args) throws ParseException {
 
         if (nameKeywords.size() > 1) {
             throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT,
-                "Only one name is allowed for update homework command."));
+                "Only one name is allowed for update exam command."));
         }
 
-        // if homework name is not present, set it to null, else parse it
+        // if exam name is not present, set it to null, else parse it
         Optional<String> examName = Optional.empty();
         if (argMultimap.getValue(PREFIX_EXAM).isPresent()) {
             examName = Optional.of(argMultimap.getValue(PREFIX_EXAM).get());
@@ -90,8 +95,18 @@ public UpdateExamCommand parse(String args) throws ParseException {
             endTime = Optional.of(ParserUtil.parseEndTime(argMultimap.getValue(PREFIX_ENDTIME).get()));
         }
 
+        Optional<Double> weight = Optional.empty();
+        if (argMultimap.getValue(PREFIX_WEIGHT).isPresent()) {
+            weight = Optional.of(ParserUtil.parseWeightage(argMultimap.getValue(PREFIX_WEIGHT).get()));
+        }
+
+        Optional<Grade> grade = Optional.empty();
+        if (argMultimap.getValue(PREFIX_GRADE).isPresent()) {
+            grade = Optional.of(ParserUtil.parseGrade(argMultimap.getValue(PREFIX_GRADE).get()));
+        }
+
         return new UpdateExamCommand(names, index, new NamePredicate(nameKeywords),
-            examName, startTime, endTime);
+            examName, startTime, endTime, weight, grade);
     }
 
     /**

src/main/java/seedu/address/model/student/Exam.java

@@ -17,28 +17,8 @@ public class Exam {
     private final LocalDateTime startTime;
     private final LocalDateTime endTime;
     private final Double weightage;
-    private Grade grade;
+    private final Grade grade;
 
-    /**
-     * Creates a new Exam with the given description, start time, end time.
-     *
-     * @param description The description of the exam.
-     * @param startTime   The start time of the exam.
-     * @param endTime     The end time of the exam.
-     */
-    public Exam(String description, LocalDateTime startTime, LocalDateTime endTime) {
-        Objects.requireNonNull(description);
-        Objects.requireNonNull(startTime);
-        Objects.requireNonNull(endTime);
-        if (startTime.isAfter(endTime)) {
-            throw new IllegalArgumentException("Start time cannot be after end time.");
-        }
-        this.description = description;
-        this.startTime = startTime;
-        this.endTime = endTime;
-        this.weightage = null;
-        this.grade = null;
-    }
 
     /**
      * Creates a new Exam with the given description, start time, end time, weightage, and grade.
@@ -60,8 +40,8 @@ public Exam(String description, LocalDateTime startTime, LocalDateTime endTime,
         }
 
         if (weightage != null) {
-            if (weightage < 0 || weightage > 1) {
-                throw new IllegalArgumentException("Weightage must be between 0 and 1.");
+            if (weightage < 0 || weightage > 100) {
+                throw new IllegalArgumentException("Weightage must be a percentage between 0 and 100.");
             }
         }
 
@@ -109,6 +89,13 @@ public Double getWeightage() {
         return weightage;
     }
 
+    public String getStringWeightage() {
+        if (weightage == null) {
+            return "Undefined";
+        }
+        return String.format("%.2f", weightage) + "%";
+    }
+
     /**
      * Returns the duration of the exam in minutes.
      *
@@ -125,12 +112,16 @@ public int getDurationInMinutes() {
      * @throws UnsupportedOperationException If the exam is not finished.
      */
     public Grade getGrade() {
-        //        if (!status.equals(ExamStatus.Finished)) {
-        //            throw new UnsupportedOperationException("Exam is not finished and does not have a grade.");
-        //        }
         return grade;
     }
 
+    public String getStringGrade() {
+        if (grade == null) {
+            return "Undefined";
+        }
+        return grade.toString();
+    }
+
     /**
      * Returns true if both exams have the same description and start time.
      * This defines a weaker notion of equality between two exams.
@@ -145,7 +136,34 @@ public boolean isSameExam(Exam otherExam) {
 
         return otherExam != null
                 && otherExam.getDescription().equals(getDescription())
-                && otherExam.getStartTime().equals(getStartTime());
+                && otherExam.getStartTime().equals(getStartTime())
+                && otherExam.getEndTime().equals(getEndTime())
+                && compareWeightages(otherExam.getWeightage(), getWeightage())
+                && compareGrades(otherExam.getGrade(), getGrade());
+    }
+
+    private boolean compareWeightages(Double first, Double second) {
+        if (first == null && second == null) {
+            return true;
+        } else {
+            if (first == null || second == null) {
+                return false;
+            } else {
+                return Double.compare(first, second) == 0;
+            }
+        }
+    }
+
+    private boolean compareGrades(Grade first, Grade second) {
+        if (first == null && second == null) {
+            return true;
+        } else {
+            if (first == null || second == null) {
+                return false;
+            } else {
+                return first.equals(second);
+            }
+        }
     }
 
     /**
@@ -199,9 +217,9 @@ public String toString() {
                 .append(" End Time: ")
                 .append(getEndTime().format(formatter))
                 .append(" Weightage: ")
-                .append(getWeightage())
+                .append(getStringWeightage())
                 .append(" Grade: ")
-                .append(getGrade());
+                .append(getStringGrade());
         return builder.toString();
     }
 }

src/main/java/seedu/address/model/student/Grade.java

@@ -20,15 +20,31 @@ public class Grade {
      * @param fullMarks the maximum score that could be obtained
      * @throws IllegalArgumentException if the score is negative or greater than full marks
      */
-    public Grade(double score, double fullMarks) {
+    public Grade(double score, double fullMarks) throws IllegalArgumentException {
         if (score < 0 || score > fullMarks) {
-            throw new IllegalArgumentException("Invalid grade score");
+            throw new IllegalArgumentException("Invalid score, score has to be positive and lesser than the "
+                    + "full marks achievable");
+        }
+        if (fullMarks <= 0) {
+            throw new IllegalArgumentException("Full marks must be positive!");
         }
 
         this.score = score;
         this.fullMarks = fullMarks;
     }
 
+    //Dummy code that needs to be included for json-related classes to work, I don't know why, todo understand this
+    //https://stackoverflow.com/questions/7625783/jsonmappingexception-no-suitable-constructor-
+    // found-for-type-simple-type-class
+
+    /**
+     * dummy code
+     */
+    public Grade() {
+        score = 0;
+        fullMarks = 0;
+    }
+
     /**
      * Returns the score obtained.
      * The score is guaranteed to be non-negative and less than or equal to the full marks.

src/main/java/seedu/address/model/student/Student.java

@@ -309,14 +309,14 @@ public void setLesson(Integer target, Lesson editedLesson) {
         lessonsList.setLesson(target, editedLesson);
     }
 
-    public void setExam(Integer target, Exam editedLesson) {
-        requireAllNonNull(target, editedLesson);
+    public void setExam(Integer target, Exam editedExam) {
+        requireAllNonNull(target, editedExam);
         //        UniqueExamList tempExamList = examList;
-        //        tempExamList.setExam(target, editedLesson);
+        //        tempExamList.setExam(target, editedExam);
         //        if (!(tempExamList.validExams())) {
         //            throw new ConflictingExamsException();
         //        }
-        examList.setExam(target, editedLesson);
+        examList.setExam(target, editedExam);
     }
 
     /**

src/main/java/seedu/address/model/student/UniqueExamList.java

@@ -11,10 +11,8 @@
 import javafx.collections.FXCollections;
 import javafx.collections.ObservableList;
 import seedu.address.model.student.exceptions.ConflictingExamsException;
-import seedu.address.model.student.exceptions.ConflictingLessonsException;
 import seedu.address.model.student.exceptions.DuplicateEntryException;
 import seedu.address.model.student.exceptions.DuplicateExamsException;
-import seedu.address.model.student.exceptions.DuplicateLessonException;
 import seedu.address.model.student.exceptions.EntryNotFoundException;
 
 /**
@@ -49,7 +47,7 @@ public boolean contains(Exam toCheck) {
      *
      * @param toAdd the Exam to be added
      */
-    public void add(Exam toAdd) throws DuplicateLessonException, ConflictingLessonsException {
+    public void add(Exam toAdd) throws DuplicateExamsException, ConflictingExamsException {
         requireNonNull(toAdd);
         if (contains(toAdd)) {
             throw new DuplicateExamsException();
@@ -94,9 +92,6 @@ public void remove(Exam toRemove) {
         }
     }
 
-    public boolean validExmas() {
-        return examsAreUnique(internalList);
-    }
 
     /**
      * Replaces the contents of this list with {@code replacement}.

src/test/java/seedu/address/model/student/ExamTest.java

@@ -25,14 +25,14 @@ public void constructor_validInputs_success() {
     public void constructor_startTimeAfterEndTime_throwsIllegalArgumentException() {
         LocalDateTime startTime = LocalDateTime.of(2023, 3, 31, 13, 0);
         LocalDateTime endTime = LocalDateTime.of(2023, 3, 31, 12, 59);
-        assertThrows(IllegalArgumentException.class, () -> new Exam("Midterm", startTime, endTime, 0.4, null));
+        assertThrows(IllegalArgumentException.class, () -> new Exam("Midterm", startTime, endTime, 40d, null));
     }
 
     @Test
     public void constructor_invalidWeightage_throwsIllegalArgumentException() {
         LocalDateTime startTime = LocalDateTime.of(2023, 3, 31, 13, 0);
         LocalDateTime endTime = LocalDateTime.of(2023, 3, 31, 14, 59);
-        assertThrows(IllegalArgumentException.class, () -> new Exam("Midterm", startTime, endTime, 1.4, null));
+        assertThrows(IllegalArgumentException.class, () -> new Exam("Midterm", startTime, endTime, 104d, null));
     }
 
     @Test