Threading for loading a file is improperly implemented

Issue #105 closed
Christopher Keil repo owner created an issue

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)

  1. Christopher Keil reporter

    Loading reduced to one SwingWorker, GUI updating properly implemented through publish() and process(). Also reactivated Observer pattern between TreeViewFrame and TVController

  2. Log in to comment