fine-code/DashboardProject

Issues

• The "Clicked" classes - besides the names - could use some refactoring

  #11 opened 5 years ago by IsaacPacht
  0

• Window.displayWindow

  #10 opened 5 years ago by IsaacPacht
  1

• Window class: I think you have a number of methods that are public but don't need to be

  #7 opened 5 years ago by IsaacPacht
  0

• The method name "createWindow" is misleading. It does more than create the window - it displays it.

  #5 opened 5 years ago by IsaacPacht
  0

• Does this compile? Where is the menubar defined? It looks like it should be a field, but I don't see any field definition.

  #8 opened 5 years ago by IsaacPacht
  0

• The names of the "Clicked" classes could use some work. They are "Listeners".

  #9 opened 5 years ago by IsaacPacht
  0

• There's no reason to use a StringBuilder here

  #2 opened 5 years ago by IsaacPacht
  0

• The lists of objects should be factored out into a "Database" class of some kind

  #3 opened 5 years ago by IsaacPacht
  0

• Reaction class

  #4 opened 5 years ago by IsaacPacht
  1

• Fields can be final

  #1 opened 5 years ago by IsaacPacht
  1

• You should probably have a class (probably this Window class) that subclasses JFrame

  #6 opened 5 years ago by IsaacPacht
  0