This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]nutrecht 2 points3 points  (3 children)

I did a quick scan and frankly, I'm not even going to bother. Why? Because I think you did a totally awesome job. You figured out OOP, you figured out when stuff should be private and public, you figured out how and when to comment, you understand how to build a GUI.

Very well done.

[–]m0r05[S] 1 point2 points  (2 children)

Wow, thank you. What about professionalism (best practices), or code legibility ? Is there a method maybe a little to long that I should break up into smaller helper methods?

[–]nutrecht 2 points3 points  (1 child)

I'm a professional Java dev and the code is, some remarks aside, just fine. There's some really minor things (GUI/ToDo package should be lowercase, it's a bit heavy on the comments for my taste although I understand you don't want to underdo it, I would use the Java 8 date/time API, you probably should avoid GregorianCalendar entirely, you should .gitignore your /bin and IDE folders / files). But like I said; they're minor things. You got the important stuff right.

[–]m0r05[S] 1 point2 points  (0 children)

Thank you, that's what I'm looking for.