Много благодаря колега, сега разбрах защо така става, а не съм валидирал имейла както по принцип би трябвало, понеже в задачата се иска да се провери единствено дали съдържа '@' иначе си прав, че това не е никаква валидация на имейл. Още един път благодаря за бързия отговор!

Имаш грешка при презаписването на ToString. И по-точно когато подаваш Age си написала "this, Age", а не "this.Age".

Също няма нужда от if условието ти за "value.Length < 1", защото то ще влези само ако e 0, а то тогава ще е влязло още в предния ти if за празен стринг и няма да стигне въобще до него.

О! Благодаря ти! :)

То няма кой знае колко специфики на ООП в тази задача, за да объркаш много фундаментално нещата :) Все пак - валидиращите методи, не им е мястото в обекта Person или поне няма нужда да са инстанционни методи. Те не работят с нищо от обекта, а им се подава стойност и връщат нещо. Аз лично бих ги отделил в отделен статичен клас, нека го наречем PersonUtils, ако наистина не могат да се преизползват за друго, освен за Person. :)

Т.е. тези методи или да ги направя статични, или да ги отделя в статичен клас, това ли препоръчваш? Вторият вариант ми се струва доста по-полезен ако реша от други места да ги викам.

Аз в момента съм ги направил и private с идеята, че валидирам данните само при създаване на обект и при промяна отвън. Ако ги изкарам в друг клас не следва ли да ги направя публични? Ако искам като поведение да не се променят нещата, т.е. да останат валидациите private, как да постъпя? Май имаше protected, но става малко сложно, щото трябва да направя Person да наследява тоя клас или... Абе малко навлизам в материя, която ми е мътна. Но като се замисля, що да няма статични публични методи за валидация и да си ги ползва който иска?

Валидацията на имейл може да ти се наложи на много места в апликейшъна, през различно време от това на създаване на обекта, според мен е оправдано да е публичен статичен метод :) Представи си го като метода IsNullOrEmpty от класа String. Той е точно такъв валидационен метод, който е публичен статичен.

Гледай на публичните методи като на API методи. Те са такива, защото ги предоставяш на програмистите (или на себе си, ако се единствения програмист) да ги ползват в апликейшъните си.

Опитай се да осъзнаеш ролята на сетъра. Може би не беше добра идея да се почва със C#, тъй като този syntax sugar не винаги помага да се разберат нещата.

Забелязваш ли как на пропъртито Name си написала if (String.IsNullOrEmpty(value))? Това е правилният начин.

set псевдо-методът представлява съкратено това:

private int age;

public void setAge(int value)

{

    this.age = value;

}

това което правиш ти в set (или в примера по-горе setAge) е следното

private int age;

public void setAge(int value)

{

        if (this.age < 1 || this.age > 100)

}

Т.е. не проверяваш това, което се подава в параметъра (value) а текущата стойност на age. Тя ще е 0, защото не е иницализирана никога. Когато се опиташ да сетнеш възраст, то ще провери - колко е текущата възраст, ще върне 0, което е по-малко от 1 и ще хвърли ексепшън.

Псевдометодите set в C# имат псевдоаргумента value, който е от съответния тип, от който е и полето. Използвай този аргумент, защото това е подадената от потребителя стойност, която искаш да инжектнеш.

Ахаааа, благодаря за подробното разяснение. Мисля, че поне по тази точка малко от малко ми просветна :)

Просто защото може и без this, но е препоръчително да се пише с this тъй като е по-лесно четимо и ясно /така няма съмнение, че става дума за поле/пропърти на конкретния обект от класа, а не някой подаден параметър/.