Здравейте!
https://github.com/ToniDimitrov/SoftUni-Homeworks-High-Quality-Code/tree/master/11.%20Unit-Testing-Homework Това е моето домашно.
Бих се радвал на съвети и критики за именоването на тестовете, самото им действие, и разбира се варианти, които съм пропуснал да проверя, тъй като най-вероятно съм забравил няколко.
Здравей,
Личи си, че си се постарал над задачата, но аз желая да предложа някои подобрения. Първо използвай полето "dynamicList" с "this"-a отпред("this.dynamicList"). Второ тестове са свързани само ако листа съдържа стойностни типове и нямаш нито един за референтни. Например какво трябва да стане, ако добавиш към такъв лист null? Какво става, ако ползваш Contains метода и като параметър пак подадеш null? Другото е, че забелязах на едно място използваш: "Assert.IsTrue", а пък на други места- "Аssert.AreEqual", ползвайки булева стойност отвън. Идеята ми е да ползваш едно от двете неща поне за консистентност. А като последно мисля, че да добавиш някакви съобщения към Assert-тите няма да е излишно.
Поздрави! :)
Видеото от предишната инстанция на курса е доста полезно https://www.youtube.com/watch?v=JgAQLZHWoYE, а ето го и моето домашно https://github.com/contracts89/HQC-Unit-Testing-Homework
Първо - благодаря ти много за feedback-а. Доста ми помогна.
Второ - мисля, че оправих повечето неща. Единствено не съм сигурен за референтните типове, дали трябва да се направят отново всичките тестове с референтен тип или проверката за null стига.
Цък - Домашното
Поздрави!! :)
Добре де, ако object-а ни е string? Тогава можем да добавим null и следователно можем да проверим дали списъкът съдържа null. Или аз нещо се омотах?
Ми в оригиналния вариант на класа, можем да, и това не е особено правилно според мен. Аз го промених и в методите, които приемат променливи от тип Т (Аdd, Remove, IndexOf и Contains), добавих един if, който хвърля exception, ако се подаде null.
Надявам се, че успях да ти помогна. :)