CRUD Metric (#82)

* Metric: Update implementations

* Add AddMetricCommandParser

* Add AddMetricCommand class

* Implement AddMetricCommandParser

* Update HireLah! add metric functionality

* AddMetricCommand: Fix bugs

* Implement DeleteMetricCommand

* Add ListMetricCommand class

* Add EditMetricCommand class

* Add EditMetricCommandParser

* Implement HireLah! to perform edit metric

* fix checkstyle

* Update CRUD Metric to be done after finalizing session

* Update AddMetricCommand to relax the restriction of metric params

* Fix checksty;e

* Fix checkstyle

* Fix checkstyle

Co-authored-by: CornCobs <[email protected]>

Author: mario7lorenzo

src/main/java/seedu/address/logic/commands/AddMetricCommand.java

@@ -0,0 +1,61 @@
+package seedu.address.logic.commands;
+
+import static java.util.Objects.requireNonNull;
+
+import java.util.List;
+
+import seedu.address.commons.exceptions.IllegalValueException;
+import seedu.address.logic.commands.exceptions.CommandException;
+import seedu.address.model.Model;
+import seedu.address.model.hirelah.AttributeList;
+import seedu.address.model.hirelah.MetricList;
+
+/**
+ * AddMetricCommand describes the behavior of HireLah!
+ * when a user wants to delete the metric from the list.
+ */
+
+public class AddMetricCommand extends AddCommand {
+    public static final String COMMAND_WORD = "metric";
+    public static final String MESSAGE_HAS_NOT_FINALIZED = "The session has not been finalized. Please finalize it"
+            + " before adding metrics.";
+    public static final String MESSAGE_SUCCESS = "New metric added: %1$s";
+    public static final String MESSAGE_USAGE = "new " + COMMAND_WORD + ": Adds an metric to the Metric list.\n"
+            + "Parameters: "
+            + "NAME -a attributePrefix1 -w weight1 -a attributePrefix2 -w weight2\n"
+            + "Example: new " + COMMAND_WORD + " extremeLeader -a lea -w 0.7 -a te -w 0.6";
+
+
+    private final String toAdd;
+    private final List<String> attributePrefixes;
+    private final List<Double> addedWeights;
+
+    /**
+     * Creates an AddMetricCommand to add the specified {@code Metric}
+     */
+
+    public AddMetricCommand(String toAdd, List<String> attributePrefixes, List<Double> addedWeights) {
+        this.toAdd = toAdd;
+        this.attributePrefixes = attributePrefixes;
+        this.addedWeights = addedWeights;
+    }
+
+    @Override
+    public CommandResult execute(Model model) throws CommandException {
+        requireNonNull(model);
+
+        if (!model.isfinalisedInterviewProperties()) {
+            throw new CommandException(MESSAGE_HAS_NOT_FINALIZED);
+        }
+
+        MetricList metrics = model.getMetricList();
+        AttributeList attributes = model.getAttributeList();
+
+        try {
+            metrics.add(toAdd, attributes, attributePrefixes, addedWeights);
+            return new CommandResult(String.format(MESSAGE_SUCCESS, toAdd), ToggleView.METRIC);
+        } catch (IllegalValueException e) {
+            throw new CommandException(e.getMessage());
+        }
+    }
+}

src/main/java/seedu/address/logic/commands/DeleteMetricCommand.java

@@ -0,0 +1,56 @@
+package seedu.address.logic.commands;
+
+import static java.util.Objects.requireNonNull;
+
+import seedu.address.commons.exceptions.IllegalValueException;
+import seedu.address.logic.commands.exceptions.CommandException;
+import seedu.address.model.Model;
+import seedu.address.model.hirelah.Metric;
+import seedu.address.model.hirelah.MetricList;
+
+/**
+ * DeleteMetricCommand describes the behavior when the
+ * client wants to delete an attribute from the list.
+ */
+
+public class DeleteMetricCommand extends DeleteCommand {
+    public static final String COMMAND_WORD = "metric";
+    public static final String MESSAGE_HAS_NOT_FINALIZED = "The session has not been finalized. Please finalize it"
+            + " before deleting metrics.";
+    public static final String MESSAGE_USAGE = COMMAND_WORD
+            + ": Deletes the metric identified by its prefix.\n"
+            + "Parameters: PREFIX\n"
+            + "Example: delete " + COMMAND_WORD + " lea";
+
+    public static final String MESSAGE_DELETE_METRIC_SUCCESS = "Deleted Metric with prefix: %1$s";
+
+    private final String metricPrefix;
+
+    public DeleteMetricCommand(String metricPrefix) {
+        this.metricPrefix = metricPrefix;
+    }
+
+    @Override
+    public CommandResult execute(Model model) throws CommandException {
+        requireNonNull(model);
+        if (!model.isfinalisedInterviewProperties()) {
+            throw new CommandException(MESSAGE_HAS_NOT_FINALIZED);
+        }
+
+        MetricList metrics = model.getMetricList();
+        try {
+            Metric metric = metrics.delete(metricPrefix);
+            return new CommandResult(String.format(MESSAGE_DELETE_METRIC_SUCCESS,
+                    metric), ToggleView.METRIC);
+        } catch (IllegalValueException e) {
+            throw new CommandException(e.getMessage());
+        }
+    }
+
+    @Override
+    public boolean equals(Object other) {
+        return other == this // short circuit if same object
+                || (other instanceof DeleteMetricCommand // instanceof handles nulls
+                && metricPrefix.equals(((DeleteMetricCommand) other).metricPrefix)); // state check
+    }
+}

src/main/java/seedu/address/logic/commands/EditMetricCommand.java

@@ -0,0 +1,59 @@
+package seedu.address.logic.commands;
+
+import static java.util.Objects.requireNonNull;
+
+import java.util.List;
+
+import seedu.address.commons.exceptions.IllegalValueException;
+import seedu.address.logic.commands.exceptions.CommandException;
+import seedu.address.model.Model;
+import seedu.address.model.hirelah.AttributeList;
+import seedu.address.model.hirelah.MetricList;
+
+/**
+ * EditMetricCommand describes the behavior of HireLah!
+ * when a user wants to edit a metric.
+ */
+
+public class EditMetricCommand extends EditCommand {
+    public static final String COMMAND_WORD = "metric";
+    public static final String MESSAGE_HAS_NOT_FINALIZED = "The session has not been finalized. Please finalize it"
+            + " before editing metrics.";
+    public static final String MESSAGE_USAGE = COMMAND_WORD
+            + ": Edits the metric identified by the prefix.\n"
+            + "Parameters: IDENTIFIER NAME [-n NEW_NAME] [-a ATTRIBUTE_1] [-w WEIGHT_1]\n"
+            + "Example: edit " + COMMAND_WORD + " extremeLeadership -n extremeDictatorship -a lea -w 100";
+
+    public static final String MESSAGE_EDIT_METRIC_SUCCESS = "Successfully edited Metric";
+
+    private final String toEdit;
+    private final String updatedName;
+    private final List<String> attributePrefixes;
+    private final List<Double> weightages;
+
+    public EditMetricCommand(String toEdit, String updatedName, List<String> attributePrefixes,
+                             List<Double> weightages) {
+        this.toEdit = toEdit;
+        this.updatedName = updatedName;
+        this.attributePrefixes = attributePrefixes;
+        this.weightages = weightages;
+    }
+
+    @Override
+    public CommandResult execute(Model model) throws CommandException {
+        requireNonNull(model);
+        if (!model.isfinalisedInterviewProperties()) {
+            throw new CommandException(MESSAGE_HAS_NOT_FINALIZED);
+        }
+
+        MetricList metrics = model.getMetricList();
+        AttributeList attributes = model.getAttributeList();
+
+        try {
+            metrics.edit(toEdit, updatedName, attributes, attributePrefixes, weightages);
+            return new CommandResult(MESSAGE_EDIT_METRIC_SUCCESS, ToggleView.METRIC);
+        } catch (IllegalValueException e) {
+            throw new CommandException(e.getMessage());
+        }
+    }
+}

src/main/java/seedu/address/logic/commands/ListMetricCommand.java

@@ -0,0 +1,30 @@
+package seedu.address.logic.commands;
+
+import static java.util.Objects.requireNonNull;
+
+import seedu.address.logic.commands.exceptions.CommandException;
+import seedu.address.model.Model;
+
+/**
+ * ListMetricCommand describes the behavior when the
+ * client wants to list the metrics available.
+ */
+public class ListMetricCommand extends ListCommand {
+    public static final String COMMAND_WORD = "metric";
+    public static final String MESSAGE_SUCCESS = "Here is the list of metrics:";
+    public static final String MESSAGE_USAGE = "list " + COMMAND_WORD + ": List the metrics from the Metric list. "
+            + "Example: list " + COMMAND_WORD;
+
+    /**
+     * Creates a ListMetricCommand to list the {@code Metric}
+     */
+    public ListMetricCommand() {
+
+    }
+
+    @Override
+    public CommandResult execute(Model model) throws CommandException {
+        requireNonNull(model);
+        return new CommandResult(MESSAGE_SUCCESS, ToggleView.METRIC);
+    }
+}

src/main/java/seedu/address/logic/commands/ToggleView.java

@@ -8,5 +8,6 @@ public enum ToggleView {
     INTERVIEWEE,
     QUESTION,
     SESSION,
-    TRANSCRIPT
+    TRANSCRIPT,
+    METRIC
 }

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

@@ -9,6 +9,7 @@
 import seedu.address.logic.commands.AddAttributeCommand;
 import seedu.address.logic.commands.AddCommand;
 import seedu.address.logic.commands.AddIntervieweeCommand;
+import seedu.address.logic.commands.AddMetricCommand;
 import seedu.address.logic.commands.AddQuestionCommand;
 import seedu.address.logic.parser.exceptions.ParseException;
 
@@ -46,6 +47,9 @@ public AddCommand parse(String arguments) throws ParseException {
         case AddQuestionCommand.COMMAND_WORD:
             return new AddQuestionCommand(addArguments.trim());
 
+        case AddMetricCommand.COMMAND_WORD:
+            return new AddMetricCommandParser().parse(addArguments.trim());
+
         default:
             throw new ParseException(MESSAGE_UNKNOWN_COMMAND);
         }

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

@@ -19,7 +19,6 @@ public class AddIntervieweeCommandParser implements Parser<AddIntervieweeCommand
      * @throws ParseException if the user input does not conform the expected format
      */
     public AddIntervieweeCommand parse(String arguments) throws ParseException {
-        System.out.println("'" + arguments + "'");
         ArgumentMultimap argMultimap =
                 ArgumentTokenizer.tokenize(arguments, PREFIX_ALIAS);
 

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

@@ -0,0 +1,50 @@
+package seedu.address.logic.parser;
+
+import static seedu.address.commons.core.Messages.MESSAGE_INVALID_COMMAND_FORMAT;
+import static seedu.address.logic.parser.CliSyntax.PREFIX_ATTRIBUTE;
+import static seedu.address.logic.parser.CliSyntax.PREFIX_WEIGHTAGE;
+
+import java.util.List;
+import java.util.stream.Collectors;
+
+import seedu.address.logic.commands.AddMetricCommand;
+import seedu.address.logic.parser.exceptions.ParseException;
+
+/**
+ * Parses input arguments and creates a new object of type AddMetricCommand
+ */
+public class AddMetricCommandParser implements Parser<AddMetricCommand> {
+    private static final String MESSAGE_INCOMPLETE_ARGUMENT = "Missing attribute and weightage details.\n%s";
+    private static final String MESSAGE_INVALID_WEIGHTAGE_FORMAT = "There is an invalid format of the weightage."
+            + "Please ensure your weightage are in numbers.";
+
+    /**
+     * Parses the given {@code String} of arguments in the context of the AddMetricCommand
+     * and returns an AddMetricCommand object for execution.
+     *
+     * @param arguments the arguments to be parsed
+     * @throws ParseException if the user input does not conform the expected format
+     */
+    public AddMetricCommand parse(String arguments) throws ParseException {
+        ArgumentMultimap argMultimap =
+                ArgumentTokenizer.tokenize(arguments, PREFIX_ATTRIBUTE, PREFIX_WEIGHTAGE);
+
+        if (arguments.equals("")) {
+            throw new ParseException(
+                    String.format(MESSAGE_INVALID_COMMAND_FORMAT, AddMetricCommand.MESSAGE_USAGE));
+        } else if (!argMultimap.arePrefixesPresent(PREFIX_ATTRIBUTE)
+                || !argMultimap.arePrefixesPresent(PREFIX_WEIGHTAGE)) {
+            throw new ParseException(
+                    String.format(MESSAGE_INCOMPLETE_ARGUMENT, AddMetricCommand.MESSAGE_USAGE));
+        } else {
+            List<String> weightages = argMultimap.getAllValues(PREFIX_WEIGHTAGE);
+            try {
+                List<Double> castedWeightages = weightages.stream().map(Double::valueOf).collect(Collectors.toList());
+                return new AddMetricCommand(argMultimap.getPreamble(), argMultimap.getAllValues(PREFIX_ATTRIBUTE),
+                        castedWeightages);
+            } catch (NumberFormatException e) {
+                throw new ParseException(MESSAGE_INVALID_WEIGHTAGE_FORMAT);
+            }
+        }
+    }
+}

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

@@ -53,7 +53,6 @@ private static List<PrefixPosition> findPrefixPositions(String argsString, Prefi
             positions.add(extendedPrefix);
             prefixPosition = findPrefixPosition(argsString, prefix.getPrefix(), prefixPosition);
         }
-
         return positions;
     }
 

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

@@ -11,4 +11,5 @@ public class CliSyntax {
     public static final Prefix PREFIX_NEW = new Prefix("-n");
     public static final Prefix PREFIX_ATTRIBUTE = new Prefix("-a");
     public static final Prefix PREFIX_WEIGHTAGE = new Prefix("-w");
+    public static final Prefix PREFIX_NAME = new Prefix("-n");
 }