Add support for command line arguments

[jkarder]

There should be a way to pass and handle command line arguments in HeuristicLab, e.g. an argument like /start:Optimizer could automatically start the Optimizer.

[jkarder]

r8563:

• created argument handling infrastructure
• added StartArgument

[jkarder]

It is now possible to automatically start applications using the StartArgument. The syntax is as follows:

• /start:ApplicationName

Examples:

• /start:Optimizer
• /start:"Persistence Configuration"

[gkronber]

We have a very old ticket for this #45 (which I have just closed as duplicate).

[jkarder]

Replying to gkronber:

We have a very old ticket for this #45 (which I have just closed as duplicate).

Oh, I missed that one. Thanks!

[ascheibe]

Thanks very much for implementing this, here are my reviewing comments:

• The white space in the regex should be replaced with \s
• GetArguments in the ArgumentHandling class should throw an exception if the argument is not valid or null so that the user gets notified if there is a problem with the argument
• The StartToken string should be moved to the StartArgument class so that everything concerning this argument is in one place
• Maybe this Token string should also be in the interface as all arguments will need one
• Please remove the public StarterForm(string appName) method from the StarterForm class as it is not used anywhere

[jkarder]

r8677: refactored argument handling infrastructure based on the changes suggested by ascheibe in comment:7:ticket:1926

[jkarder]

Thanks for the review.

[ascheibe]

Thanks, looks good!

[ascheibe]

I'm reassigning this ticket to mkommend as he also wanted to have a look at it.

[abeham]

I wondered if we could have a way to hide the starter. Can you omit showing the starter if the /start command is given? I think we don't really need to show it in this case.

[mkommend]

Reviewing comments:

• Argument.GetHashCode() is strangely implemented. Normally it should reflect the implementation of Equals.
• A better name for the folder containing the files would be CommandLineArguments or CommandLineArgumentHandling. ArgumentHandling is little bit too generic IMHO.
• Why was it decided to implement command line parsing manually and not use existing libraries, that provide more functionality? (e.g., ​http://stackoverflow.com/questions/491595/best-way-to-parse-command-line-arguments-in-c)
• Please address the comment by abeham.

[jkarder]

r8748:

• addressed the suggestion by abeham in comment:13:ticket:1926
• applied the changes suggested by mkommend in comment:14:ticket:1926
• refactored argument handling infrastructure

[jkarder]

Thanks for the detailed review. It is now possible to hide the starter by using the HideStarterArgument (/hideStarter).

[jkarder]

r8749: made generic type parameter in ICommandLineArgument<T> invariant

[jkarder]

r8818:

• added OpenArgument
• refactored argument handling infrastructure

[jkarder]

All valid command line arguments are now passed to any started application through the PluginManager and the ApplicationManager to enable specific handling. As a result it is now possible to load saved HeuristicLab files via command line arguments. A new instance of the Optimizer will be started automatically and the files will be opened. I also attached the registry entries​ needed to associate HeuristicLab files with HeuristicLab. Keep in mind that you probably have to change the specified path to the HeuristicLab executable.

[ascheibe]

r8819 updated slave app to work with the changes of r8818

[jkarder]

r8820: updated DataImporter and MetaOptimization branches to work with the changes of r8818

[mkommend]

Replying to jkarder:

r8818:

• added OpenArgument
• refactored argument handling infrastructure

In r8818 you introduced a command line property in the IMainForm interface, which should be removed as it is not needed and and command line arguments should not be handled be a form object.

[jkarder]

r8821: reverted the changes addressed by mkommend in comment:29:ticket:1926

[jkarder]

Replying to mkommend:

Replying to jkarder:

r8818:

• added OpenArgument
• refactored argument handling infrastructure

In r8818 you introduced a command line property in the IMainForm interface, which should be removed as it is not needed and and command line arguments should not be handled be a form object.

Thanks for the hint.

[jkarder]

r8934: refactored arguments

[mkommend]

Thx for implementing this feature. I have no further reviewing comments.