Sanitise inputs

[nus-se-bot]

Different person name can have other same parameters For example, I can add add n/John Doe p/98765432 e/johnd@example.com a/John street, block 123, #01-01 add n/George Tan p/98765432 e/johnd@example.com a/John street, block 123, #01-01 add n/Betty Grace p/98765432 e/johnd@example.com a/John street, block 123, #01-01

Suggested Improvement the person object has many fields. You need to compare each value to check whether it is identical. If it is, I don't think you're suppose to add it.

if(person1.getPhoneNumber == person2.getPhoneNumber) { DoNotAdd(); }

Additional context

---

Reported by: @jeraldtsy Severity: High

_{[original: nus-cs2103-AY1819S2/pe-dry-run#139]}

[jwl1997]

actually, we never change this part from the original addressbook4.... and it uses name+phone+email to uniquely identify a person...