ConsoleUtility Refactor

[ghost]

• Simplified several methods
• Added a check to prevent a user from entering an empty string into the AskForString method
• Updated documentation throughout to be more concise and detailed
• Renamed the the logging methods to be more explicit
• In the AskYesAndNoQuestion method the user can now also enter yes and no explicitly

[CreepPork]

AskToEnterString should have an option to enter an empty string. For example, game arguments can be empty and for the future the private key path can be empty so it doesn’t build with a key.

[ghost]

Yeah I don't know why I thought that.

[CreepPork]

Perhaps message print methods should be shortened to PrintInfo, PrintError or to stay in C# conventions (like the Console class): WriteInfo, WriteError, etc.

[ghost]

I did ShowErrorMessage as it was more explicit in that it would be print to the console and would receive a user defined string to output.

Write to me sounds like something that you would use if you were writing more to a stream that was not being used for simple console output. (Like a log file on the filesystem)

[ghost]

AskToEnterString should have an option to enter an empty string. For example, game arguments can be empty and for the future the private key path can be empty so it doesn’t build with a key.

Resolved

[CreepPork]

ShowErrorMessage to me seems more like a dialog box that shows up not a console output. I think that it's quite unnecessary to include the message part in the method name as it would be quite obvious.

I'd prefer PrintInfo and so on.

[ghost]

ShowErrorMessage to me seems more like a dialog box that shows up not a console output. I think that it's quite unnecessary to include the message part in the method name as it would be quite obvious.

I'd prefer PrintInfo and so on.

Resolved, personally I think its better to be explicit even if it is obvious as someone might not get it but its your codebase so i'll abide by your conventions. (I did PrintErrorMessage not show as that does sound like a dialog prompt)

[CreepPork]

Looks fine but needs testing.