rohancragg/ManyConsole — comment on commit e23839ffb96ec733b30047989e10142d651ce698
File:  line 0
By fschwiet:

This solution assumes that different commands will all have a fixed number of arguments. It seems it could change, say I have a command that takes an arbitrary number of files.

FinishLoadingArguments is kind of awkward, but I'm not sure the approach of breaking it into two overloads is the approach I want to take. What I'm considering is to let each command do it owns parameter handling in a single call, applying the NDesk option set on its own. This adds less value, but it keeps ManyConsole out of the way. The only reason ManyConsole really needs the OptionSet is to show help text to the user, so maybe it should be less involved with the OptionSet otherwise. Perhaps that kind of command class would be a base class of the console class you've extended it to be now.