I don't feel there is a reason for this to be out here - fillSubsystemList() could be createSubsystems() and produce the list, and it can be local to the method. Probably doesn't make much difference.