There SOOOOOO MUUUUUCHH DUUUPLICAAAATE COOOOODE. An obvious improvement is to factor out the "one per building methods" into a single "displayBuilding(String buildingCode)" method that would work off of a HashMap to get the correct information?
Similarly, it seems crazy to have a separate JLabel object reference for each building, rather than putting them into an ArrayList or HashMap, and then iterating through.
Try to get rid of everything that is iterating over building name---and replace it with appropriate data structures.
vincenicoara
commented
8 years ago
There SOOOOOO MUUUUUCHH DUUUPLICAAAATE COOOOODE. An obvious improvement is to factor out the "one per building methods" into a single "displayBuilding(String buildingCode)" method that would work off of a HashMap to get the correct information?
Similarly, it seems crazy to have a separate JLabel object reference for each building, rather than putting them into an ArrayList or HashMap, and then iterating through.
Try to get rid of everything that is iterating over building name---and replace it with appropriate data structures.
200 points