From d4ba21c443ab350b2178b9c986012cb0dec0929c Mon Sep 17 00:00:00 2001
From: Volker Wollmann <64246526+VolkerWollmann@users.noreply.github.com>
Date: Fri, 24 Jul 2020 08:09:01 +0200
Subject: [PATCH] String Constant Mapping unecessary

---
 .../Commands/SelectColorCommand.cs            |  2 +-
 .../MyMasterMind/Controls/CodeField.xaml      | 13 ++++----
 .../Interfaces/MasterMindEnums.cs             | 31 -------------------
 3 files changed, 8 insertions(+), 38 deletions(-)

diff --git a/MyMasterMind/MyMasterMind/Commands/SelectColorCommand.cs b/MyMasterMind/MyMasterMind/Commands/SelectColorCommand.cs
index f87f189..9ded1ba 100644
--- a/MyMasterMind/MyMasterMind/Commands/SelectColorCommand.cs
+++ b/MyMasterMind/MyMasterMind/Commands/SelectColorCommand.cs
@@ -21,7 +21,7 @@ public bool CanExecute(object parameter)
 
 		public void Execute(object parameter)
 		{
-			CodeField.SetColor(MyMasterMindConstants.GetCodeColor((string)parameter));
+			CodeField.SetColor((MyMasterMindCodeColors)parameter);
 		}
 
 		public SelectColorCommand(CodeField codeField )
diff --git a/MyMasterMind/MyMasterMind/Controls/CodeField.xaml b/MyMasterMind/MyMasterMind/Controls/CodeField.xaml
index 76bf13e..32778e7 100644
--- a/MyMasterMind/MyMasterMind/Controls/CodeField.xaml
+++ b/MyMasterMind/MyMasterMind/Controls/CodeField.xaml
@@ -4,17 +4,18 @@
              xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006" 
              xmlns:d="http://schemas.microsoft.com/expression/blend/2008" 
              xmlns:local="clr-namespace:MyMasterMind.Controls"
+             xmlns:constants="clr-namespace:MyMasterMind.Interfaces" 
              mc:Ignorable="d" 
              d:DesignHeight="450" d:DesignWidth="800">
     <StackPanel Name="CodeFieldStackPanel">
         <StackPanel.ContextMenu>
             <ContextMenu>
-                <MenuItem Header="Red" Background="Red" CommandParameter="RED" Command="{Binding SelectColorCommand, Mode=OneWay}"/>
-                <MenuItem Header="Green" Background="Green" CommandParameter="GREEN" Command="{Binding SelectColorCommand, Mode=OneWay}" />
-                <MenuItem Header="Blue" Background="Blue" CommandParameter="BLUE" Command="{Binding SelectColorCommand, Mode=OneWay}" />
-                <MenuItem Header="Yellow" Background="Yellow" CommandParameter="YELLOW" Command="{Binding SelectColorCommand, Mode=OneWay}" />
-                <MenuItem Header="Magenta" Background="Magenta" CommandParameter="MAGENTA" Command="{Binding SelectColorCommand, Mode=OneWay}" />
-                <MenuItem Header="Cyan" Background="Cyan" CommandParameter="CYAN" Command="{Binding SelectColorCommand, Mode=OneWay}" />
+                <MenuItem Header="Red" Background="Red" CommandParameter="{Binding Source={x:Static constants:MyMasterMindCodeColors.Red}}" Command="{Binding SelectColorCommand, Mode=OneWay}"/>
+                <MenuItem Header="Green" Background="Green" CommandParameter="{Binding Source={x:Static constants:MyMasterMindCodeColors.Green}}" Command="{Binding SelectColorCommand, Mode=OneWay}" />
+                <MenuItem Header="Blue" Background="Blue" CommandParameter="{Binding Source={x:Static constants:MyMasterMindCodeColors.Blue}}" Command="{Binding SelectColorCommand, Mode=OneWay}" />
+                <MenuItem Header="Yellow" Background="Yellow" CommandParameter="{Binding Source={x:Static constants:MyMasterMindCodeColors.Yellow}}" Command="{Binding SelectColorCommand, Mode=OneWay}" />
+                <MenuItem Header="Magenta" Background="Magenta" CommandParameter="{Binding Source={x:Static constants:MyMasterMindCodeColors.Magenta}}" Command="{Binding SelectColorCommand, Mode=OneWay}" />
+                <MenuItem Header="Cyan" Background="Cyan" CommandParameter="{Binding Source={x:Static constants:MyMasterMindCodeColors.Cyan}}" Command="{Binding SelectColorCommand, Mode=OneWay}" />
             </ContextMenu>
         </StackPanel.ContextMenu>
         <Rectangle Name="CodeFieldRectangle" HorizontalAlignment="Left" 
diff --git a/MyMasterMind/MyMasterMind/Interfaces/MasterMindEnums.cs b/MyMasterMind/MyMasterMind/Interfaces/MasterMindEnums.cs
index 42e4b1e..90768f0 100644
--- a/MyMasterMind/MyMasterMind/Interfaces/MasterMindEnums.cs
+++ b/MyMasterMind/MyMasterMind/Interfaces/MasterMindEnums.cs
@@ -48,36 +48,5 @@ public class MyMasterMindConstants
 	{
 		public const int CLOUMNS = 4;
 		public const int ROWS = 10;
-
-		public const string RED = "RED";
-		public const string GREEN = "GREEN";
-		public const string BLUE = "BLUE";
-		public const string YELLOW = "YELLOW";
-		public const string MAGENTA = "MAGENTA";
-		public const string CYAN = "CYAN";
-
-		private static readonly string[] COLORNAMES = { RED, GREEN, BLUE, YELLOW, MAGENTA, CYAN};
-
-		public static MyMasterMindCodeColors GetCodeColor(string color)
-		{
-			List<Tuple<string, MyMasterMindCodeColors>> mapping =
-				new List<Tuple<string, MyMasterMindCodeColors>>()
-				{
-					new Tuple<string, MyMasterMindCodeColors>( RED, MyMasterMindCodeColors.Red ),
-					new Tuple<string, MyMasterMindCodeColors>( GREEN, MyMasterMindCodeColors.Green ),
-					new Tuple<string, MyMasterMindCodeColors>( BLUE, MyMasterMindCodeColors.Blue ),
-					new Tuple<string, MyMasterMindCodeColors>( YELLOW, MyMasterMindCodeColors.Yellow ),
-					new Tuple<string, MyMasterMindCodeColors>( MAGENTA, MyMasterMindCodeColors.Magenta ),
-					new Tuple<string, MyMasterMindCodeColors>( CYAN, MyMasterMindCodeColors.Cyan ),
-
-				};
-
-			if (mapping.Any(m => (m.Item1 == color)))
-			{
-				return mapping.First(m => (m.Item1 == color)).Item2;
-			}
-
-			return MyMasterMindCodeColors.None;
-		}
 	}
 }