wouterfennis/borg_calendar

BORG Calendar - A Desktop Calendar and Task Tracking System. Language: Java Swing

borg_calendar's Issues

Code review van de refactoring

Bekijk of de refactoring technieken goed zijn uitgevoerd en of er nog dingen moeten worden opgelost

Verplaats speciale dag logica van Day naar SpecialDay klasse

De if'jes moet naar SpecialDay worden omgezet naar één IF zodat deze dynamisch kan worden afgevraagd met instance variabelen

De holiday int omzetten naar een boolean

Op dit moment lijkt het er op dat de int op de volgende regel gebruikt wordt als een boolean
In de applicatie wordt de int namelijk soms op 1 gezet en verder wordt er niks mee gedaan.
Dus kan worden omgezet naar een boolean

Regel ~367
if (month == 0 && day == 1) { hol.setText(Resource.getResourceString("New_Year's_Day")); ret.setHoliday(1); }

Regel ~417
private int holiday; // set to indicate if any appt in the list is a

Extract Constants

Example:

Line 230: hol.setColor("black");

Aanmaken nieuwe Class SpecialDay

Voor het verplaatsen van de logica om te bepalen of een dag een speciale dag is moet er een nieuwe klasse worden aangemaakt naast de "Day" Class.

Genaamd "SpecialDay"

Voeg JavaDoc toe of update het bestaande

Er was al wat JavaDoc aanwezig binnen de applicatie. Dit moet worden nagekeken en zonodig worden aangevuld.

Voor onze nieuwe methodes binnen SpecialDay moet ook JavaDoc worden toegevoegd

Renaming Methods in Day.java

Example:

Line 410: private static int nthdom(int year, int month, int dayofweek, int week)

Structuur van Day.java

De attributen staan op de verkeerde posities, en er moet gekeken worden naar de volgorde van de methodes

Renaming variables in Day.java

Example
Line 215: Day ret = new Day();

Met SonarLint/Qube de applicatie nalopen of er nog bijzondere issues zijn

Het is belangrijk om te kijken of er nog vreemde issues aanwezig waren. Nadat wij een groot gedeelte hebben gerefactored