From 45c93ce079d338023c3d192f92036e5ccf5531d7 Mon Sep 17 00:00:00 2001 From: Lukasz Baldyga Date: Tue, 20 Oct 2020 10:05:38 +0100 Subject: [PATCH] Fixed bug in helper function where the function wouldn't work with single commands --- .../SettingsAndHelperFunctions.cs | 27 ++++++++++++------- 1 file changed, 18 insertions(+), 9 deletions(-) diff --git a/Advanced Software Engineering/Advanced Software Engineering/SettingsAndHelperFunctions.cs b/Advanced Software Engineering/Advanced Software Engineering/SettingsAndHelperFunctions.cs index 67af0f4..0bbe841 100644 --- a/Advanced Software Engineering/Advanced Software Engineering/SettingsAndHelperFunctions.cs +++ b/Advanced Software Engineering/Advanced Software Engineering/SettingsAndHelperFunctions.cs @@ -94,18 +94,27 @@ namespace Advanced_Software_Engineering { //split the command from the parameters string[] parameters = Strip(text).Split(new char[] { " "[0] }, 2); - //set command var - string command = Strip(parameters[0]); - commandAndParameters["command"] = new string[] { command }; + //Check there is content + if (parameters.Length > 0) { + //set command var + string command = Strip(parameters[0]); + commandAndParameters["command"] = new string[] { command }; + } - //seperate all of the parameters by a comma - parameters = parameters[1].Split(","[0]); + //If there is nothing split after the first split, why bother doing anything? + else if (parameters.Length > 1) { + //seperate all of the parameters by a comma + parameters = parameters[1].Split(","[0]); - //Remove spaces around parameters - parameters = StripStringArray(parameters).ToArray(); + //Remove spaces around parameters + parameters = StripStringArray(parameters).ToArray(); - //set parameter list - commandAndParameters["parameters"] = parameters; + //We don't want parameters if there aren't any + if (!(parameters.Length == 1 && parameters[0] == "")) { + //set parameter list + commandAndParameters["parameters"] = parameters; + } + } return commandAndParameters; }