fiprojects / telemaniacs Goto Github PK

• getOccurrences - gets all occurrences of the transmission, incl. occurrences in history
• getUpcomingOccurrences - gets all upcoming occurrences of the transmission
• getVotings - gets all votings for the transmission
• getAverageVoting - gets average numeric voting for the transmission

Issue

Create DAO for Voting entity

Bude obsahovat metodu getSchedule, která zabaluje pouze volání služby
Dále bude obsahovat metodu getUserSchedule, která spojí služby UserProfileService a ScheduleService, získá oblíbené kanály uživatele a vrací pro něj TV program jeho oblíbených kanálů
Naopak nebude obsahovat metodu getEndDate

Create a voting entity according to class diagram

Changes should be persisted in the two services (they aren't recently)

• followChannel - add channel to user favorites
• unfollowChannel - remove channel from user favorites
• getFavoriteChannels - get all users favorite channels

• authenticate - vezme uživatelské jméno, password hash a vrací boolean, zda je kombinace platná, nebo ne
• isAdmin - vrací boolean, jestli je uživatel administrátor nebo ne

Datum by mělo jít specifikovat od i od/do.

• Enum ageAvailability by ještě mohl obsahovat položku UNRESTRICTED (null object pattern)
• @table(name = "Transmissions")
• Typo: lenght -> length
• CRUD update by měl ověřovat parametr user na null, při volání transmission.getId() by jinak mohlo dojít k NullPointerEx
• CRUD update může vracet void (aby to bylo jednotné s create)
• ConstraintViolationException, IllegalArgumentException jsou unchecked exceptions, není třeba uvádět throws (zjednoduší se i kontrakt), stačí očekávané výjimky uvést do JavaDocu
• Kolekce by se měly vracet read-only

Update

• Přidal jsem do Channel vztah k TransmissionOccurrence

Nízká priorita

• Sjednotit JavaDoc: Created by -> @author
• Do JavaDocu by bylo dobré doplnit vždy alespoň účel třídy/enumu

• CRUD update by měl ověřovat parametr user na null, při volání user.getId() by jinak mohlo dojít k NullPointerEx
• CRUD update může vracet void (aby to bylo jednotné s create)
• ConstraintViolationException, IllegalArgumentException jsou unchecked exceptions, není třeba uvádět throws (zjednoduší se i kontrakt), stačí očekávané výjimky uvést do JavaDocu
• Kolekce by se měly vracet read-only

Nízká priorita

• Neměl by mít passwordHash anotaci @column(nullable = false)?
• U atributu age by šla použít validace na @min
• Regulární výraz pro e-mail není úplně správný
• Konstruktory entity by šly vyfaktorovat do společné metody (DRY)
• Pojmenování favouriteProgram mixuje US a UK EN, bylo by dobré sjednotit, nebo pojmenovat favouriteTransmissions (více výstižné)

Create base IntelliJ IDEA project

Create unit test for Voting entity

• Episode.equals()

  • napr. podmienka 'if ((name == null && episode.getName() != null)
    || !name.equals(episode.getName()))' - ak sa this.name == null a episode.getName() != null tak ti to spadne na NPE
  • to sa tyka vsetkych podmienok v tej metode

• low priority:
  Episode - best practice je pristupovat k fieldom triedy cez getry, preto by som upravil equals ,hashcode, toString

• velmi male detaily:

  • Episode.equals() - 'if (other == null || !(other instanceof Episode)) return false;' - instanceof počítá s null hodnotou, preto je prva podmienka zbytočná
  • EpisodeDaoImpl.update() - chekovat episode.getId() != null ?

• len otázky:

  • Episode.isRerun - ako to planuješ využívat? myslím že tak ako to je navrhnuté bude pre každé vysielanie nová Episode a nenapadá ma, k čomu by sa tento parameter mohol využit

Modify other entities (according to class diagram) to work with Voting entity

• followTransmission
• unfollowTransmission
• getFavoriteTransmissionsByUser
• getUpcomingFavoriteTransmissionsByUser - get upcoming favorite transmissions (in some reasonable time span)

• Implementace equals: (1) mělo by se použít instanceof místo getClass() != o.getClass(), (2) místo Objects.equals(getName(), channel.getName()) lze použít name.equals(...)
• Kolekce by se měly vracet read-only
• JoinColumn u episodes asi nebude potřeba (?)
• CRUD update: zkontrolovat existenci editované položky v databázi

Nízká priorita

• Do JavaDocu by bylo dobré doplnit vždy alespoň účel třídy/enumu
• Jmenné konvence: @table(name = "Channels")