evahorackova / j1-lekce07 Goto Github PK

Ahoj Evo,

testy máš hezky a důkladně zpracované, gratuluju. Koukni jen na těchto pár drobností prosím:

1. Zde validuješ metodu getPocetJmen(), což je ale zbytečné, protože jsi ji už validovala v předchozím testu (na řádku 45). Možná jsi zamýšlela použít metodu getSeznamJmen() a otestovat, že vrátí stejný počet jmen (pomocí následného volání metody size() -- v tom případě bys zde napsala:

assertEquals(37, svatky.getSeznamJmen().size());

2. Tyto řádky jsou nadbytečné, metodu getPocetJmen(), už jsi úspěšně validovala ve třetím testu (na řádcích 45 a 46).
   

   j1-lekce07/src/test/java/cz/czechitas/lekce7/SvatkyTest.java

   Line 58 in a40e9c4

   assertNotEquals(50, svatky.getPocetJmen());

   
   

   j1-lekce07/src/test/java/cz/czechitas/lekce7/SvatkyTest.java

   Line 58 in a40e9c4

   assertNotEquals(50, svatky.getPocetJmen());

3. Všechny pomocné komentáře, které jsou po finálním dokončení aplikace už nadbytečné (třeba proto, že sloužily jako úkolovník, co všechno je třeba ještě naprogramovat), tak ty je vhodné před finálním commitem odmazat. Pokud bys o některou z těch informací nechtěla z nějakého důvodu přijít, je lepší rozšířit JavaDoc (tj. komentář na řádcích nad samotnou metodou, např. 15 až 17, 26 až 28 apod.) I to ale bývá nadbytečné vzhledem k tomu, že všechny změny kódu jsou přehledně uloženy v github repu a lze se k nim kdykoli znovu vrátit (bylo-li by třeba).
   Příklady:
   

   j1-lekce07/src/test/java/cz/czechitas/lekce7/SvatkyTest.java

   Line 31 in a40e9c4

   //Otestovat, že najde v seznamu existující jméno a nenajde neexistující jméno

   
   

   j1-lekce07/src/test/java/cz/czechitas/lekce7/SvatkyTest.java

   Line 43 in a40e9c4

   //Otestovat, že vrací počet jmen, která máme v seznamu

   
   

   j1-lekce07/src/test/java/cz/czechitas/lekce7/SvatkyTest.java

   Line 54 in a40e9c4

   //Zkontrolovat, že seznam jmen má správný počet položek

   
   

   j1-lekce07/src/test/java/cz/czechitas/lekce7/SvatkyTest.java

   Line 66 in a40e9c4

   //Otestuje, že je jméno v seznamu svátků a že má přiřazen správný den

   
   

   j1-lekce07/src/test/java/cz/czechitas/lekce7/SvatkyTest.java

   Line 79 in a40e9c4

   //Otestuje, že je jméno v seznamu svátků a že má přiřazen správný den

   
   

   j1-lekce07/src/test/java/cz/czechitas/lekce7/SvatkyTest.java

   Line 105 in a40e9c4

   //Zkontrolovat, že po smazání bude počet svátků odpovídat novému počtu

Jinak paráda, skvělá práce. Díky!