diff --git a/SabreTools.Library/DatFiles/Filter.cs b/SabreTools.Library/DatFiles/Filter.cs index 4049a872..5728c12b 100644 --- a/SabreTools.Library/DatFiles/Filter.cs +++ b/SabreTools.Library/DatFiles/Filter.cs @@ -728,74 +728,5 @@ namespace SabreTools.Library.DatFiles } #endregion - - #region Static methods - - /// - /// Get the machine type from a string - /// - /// Machine type as a string - /// A machine type based on the input - public static MachineType GetMachineTypeFromString(string gametype) - { - MachineType machineType = MachineType.NULL; - switch (gametype.ToLowerInvariant()) - { - case "none": - machineType |= MachineType.None; - break; - case "bios": - machineType |= MachineType.Bios; - break; - case "dev": - case "device": - machineType |= MachineType.Device; - break; - case "mech": - case "mechanical": - machineType |= MachineType.Mechanical; - break; - default: - Globals.Logger.Warning("{0} is not a valid type", gametype); - break; - } - - return machineType; - } - - /// - /// Get the item status from a string - /// - /// Item status as a string - /// An item status based on the input - public static ItemStatus GetStatusFromString(string status) - { - ItemStatus itemStatus = ItemStatus.NULL; - switch (status.ToLowerInvariant()) - { - case "none": - itemStatus |= ItemStatus.None; - break; - case "good": - itemStatus |= ItemStatus.Good; - break; - case "baddump": - itemStatus |= ItemStatus.BadDump; - break; - case "nodump": - itemStatus |= ItemStatus.Nodump; - break; - case "verified": - itemStatus |= ItemStatus.Verified; - break; - default: - Globals.Logger.Warning("{0} is not a valid status", status); - break; - } - - return itemStatus; - } - - #endregion } } diff --git a/SabreTools.Library/Tools/Utilities.cs b/SabreTools.Library/Tools/Utilities.cs index c56eb37f..dd3a1268 100644 --- a/SabreTools.Library/Tools/Utilities.cs +++ b/SabreTools.Library/Tools/Utilities.cs @@ -362,6 +362,52 @@ namespace SabreTools.Library.Tools } } + /// + /// Get MachineType value from input string + /// + /// String to get value from + /// MachineType value corresponding to the string + public static MachineType GetMachineType(string gametype) + { + switch (gametype?.ToLowerInvariant()) + { + case "none": + default: + return MachineType.None; + case "bios": + return MachineType.Bios; + case "dev": + case "device": + return MachineType.Device; + case "mech": + case "mechanical": + return MachineType.Mechanical; + } + } + + /// + /// Get ItemStatus value from input string + /// + /// String to get value from + /// ItemStatus value corresponding to the string + public static ItemStatus GetItemStatus(string status) + { + switch (status?.ToLowerInvariant()) + { + case "none": + default: + return ItemStatus.None; + case "good": + return ItemStatus.Good; + case "baddump": + return ItemStatus.BadDump; + case "nodump": + return ItemStatus.Nodump; + case "verified": + return ItemStatus.Verified; + } + } + /// /// Replace accented characters /// diff --git a/SabreTools/SabreTools.cs b/SabreTools/SabreTools.cs index 33d400a7..d9a48da2 100644 --- a/SabreTools/SabreTools.cs +++ b/SabreTools/SabreTools.cs @@ -740,7 +740,7 @@ namespace SabreTools break; case "-gt": case "--game-type": - filter.MachineTypes |= Filter.GetMachineTypeFromString(args[++i]); + filter.MachineTypes |= Utilities.GetMachineType(args[++i]); break; case "-gz": case "--gz": @@ -759,7 +759,7 @@ namespace SabreTools break; case "-is": case "--status": - filter.ItemStatuses |= Filter.GetStatusFromString(args[++i]); + filter.ItemStatuses |= Utilities.GetItemStatus(args[++i]); break; case "-md5": case "--md5": @@ -790,11 +790,11 @@ namespace SabreTools break; case "-ngt": case "--not-gtype": - filter.NotMachineTypes |= Filter.GetMachineTypeFromString(args[++i]); + filter.NotMachineTypes |= Utilities.GetMachineType(args[++i]); break; case "-nis": case "--not-status": - filter.NotItemStatuses |= Filter.GetStatusFromString(args[++i]); + filter.NotItemStatuses |= Utilities.GetItemStatus(args[++i]); break; case "-nmd5": case "--not-md5": @@ -1026,7 +1026,7 @@ namespace SabreTools break; case "-gt": case "--game-type": - filter.MachineTypes |= Filter.GetMachineTypeFromString(argsplit[1]); + filter.MachineTypes |= Utilities.GetMachineType(argsplit[1]); break; case "-gz": case "--gz": @@ -1045,7 +1045,7 @@ namespace SabreTools break; case "-is": case "--status": - filter.ItemStatuses |= Filter.GetStatusFromString(argsplit[1]); + filter.ItemStatuses |= Utilities.GetItemStatus(argsplit[1]); break; case "-md5": case "--md5": @@ -1076,11 +1076,11 @@ namespace SabreTools break; case "-ngt": case "--not-gtype": - filter.NotMachineTypes |= Filter.GetMachineTypeFromString(argsplit[1]); + filter.NotMachineTypes |= Utilities.GetMachineType(argsplit[1]); break; case "-nis": case "--not-status": - filter.NotItemStatuses |= Filter.GetStatusFromString(argsplit[1]); + filter.NotItemStatuses |= Utilities.GetItemStatus(argsplit[1]); break; case "-nmd5": case "--not-md5":