View Code? Open in Web Editor
NEW
This project forked from mikeberger /borg_calendar
BORG Calendar - A Desktop Calendar and Task Tracking System. Language: Java Swing
Home Page: https://mikeberger.github.io/borg_calendar/
Java 83.69%
HTML 13.80%
PHP 2.45%
Shell 0.01%
Haskell 0.04%
borg_calendar's People
borg_calendar's Issues
Het is belangrijk om te kijken of er nog vreemde issues aanwezig waren. Nadat wij een groot gedeelte hebben gerefactored
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
De attributen staan op de verkeerde posities, en er moet gekeken worden naar de volgorde van de methodes
Example:
Line 410: private static int nthdom(int year, int month, int dayofweek, int week)
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"
Bekijk of de refactoring technieken goed zijn uitgevoerd en of er nog dingen moeten worden opgelost
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
Example
Line 215: Day ret = new Day();
De if'jes moet naar SpecialDay worden omgezet naar één IF zodat deze dynamisch kan worden afgevraagd met instance variabelen
Example:
Line 230: hol.setColor("black");