See if you can follow this logic. The Task class declares an internal interface named TaskNameListener. This internal interface contains a single methor, taskNameChanged(String argName). There is only one class that implements this interface, TaskFrame.
However, TaskFrame never uses the String argument from this method. If there is a name change the TaskFrame accesses the name from the Task it displays directly. Unless we think there will be other implementations of the TaskNameListener in the future, I recommend we remove the Task.TaskNameListener interface. It is unecessary "code fat". We should do this instead: - Replace the list of Task.TaskNameListeners in each Task object with a list of TaskFrame references. - Leave a no-argument taskNameChanged method on the TaskFrame class. - When a TaskFrame is being constructed to display a Task, add a reference to the TaskFrame to the list of TaskFrames in the Task that will be displayed. This will accomplish the same thing, without the need for the TaskNameListener interface and the taskNameChanged(String argName) method whose String argument is never used. The Sunburned Surveyor ------------------------------------------------------------------------- Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW! Studies have shown that voting for your favorite open source project, along with a healthy diet, reduces your potential for chronic lameness and boredom. Vote Now at http://www.sourceforge.net/community/cca08 _______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
