package seedu.taskboss.model.task;

import seedu.taskboss.commons.exceptions.IllegalValueException;

/**
 * Represents a Task's name in TaskBoss.
 * Guarantees: immutable; is valid as declared in {@link #isValidName(String)}
 */
public class Name {

    public static final String MESSAGE_NAME_CONSTRAINTS =
            "Task names should only contain alphanumeric characters,"
            + " special characters and spaces, and it should not be blank";

    /*
     * The first character of the information must not be a whitespace,
     * otherwise " " (a blank string) becomes a valid input.
     */
//    public static final String NAME_VALIDATION_REGEX = "[\\p{Alnum}][\\p{Alnum}|\\s|!|"
//            + "|_|-|+|$|#|@|%|^|*|?+/||<|>|`|~|&]*";

    public final String fullName;

    /**
     * Validates given name.
     *
     * @throws IllegalValueException if given name string is invalid.
     */
    public Name(String name) throws IllegalValueException {
        assert name != null;
        String trimmedName = name.trim();
        if (!isValidName(trimmedName)) {
            throw new IllegalValueException(MESSAGE_NAME_CONSTRAINTS);
        }
        this.fullName = trimmedName;
    }

    /**
     * Returns true if a given string is a valid person name.
     */
    public static boolean isValidName(String test) {
        if ("".equals(test)) {
            return false;
        }
        return true;
    }


    @Override
    public String toString() {
        return fullName;
    }

    @Override
    public boolean equals(Object other) {
        return other == this // short circuit if same object
                || (other instanceof Name // instanceof handles nulls
                && this.fullName.equals(((Name) other).fullName)); // state check
    }

    @Override
    public int hashCode() {
        return fullName.hashCode();
    }

}

@@ -446,16 +446,66 @@
  blank%22;
+%0A    public static final String EMPTY_STRING = %22%22;
 %0A%0A    /*
@@ -1389,86 +1389,41 @@
     
-if (%22%22.equals(test)) %7B%0A            return false;%0A        %7D%0A        return true
+return !test.equals(EMPTY_STRING)
 ;%0A  

improve code quality