- changed status to resolved
Threading for loading a file is improperly implemented
Issue #105
closed
The Loadworker thread from TVController and ClusterController both just call a load method of the TVModel. This load method in return creates an object of NewModelLoader which is used to load data from a file into the TVModel.
The process contains a lot of GUI code that is not handled properly for SwingWorker threads. Static calls to the Views and Swing components within TVFrame shoudl not happen within that class. SwingWorkers use publish() and process() to safely update labels or progress bars. NewModelLoader should be adapted to that.
Comments (2)
-
reporter -
reporter - changed status to closed
- Log in to comment
Loading reduced to one SwingWorker, GUI updating properly implemented through publish() and process(). Also reactivated Observer pattern between TreeViewFrame and TVController