From a44ee95d4bdbf7b6b36ca4b3cf999e7fe9885434 Mon Sep 17 00:00:00 2001
From: ADato88 <da-maschine@hotmail.de>
Date: Wed, 19 Jan 2022 10:35:40 +0100
Subject: [PATCH] refactoring vom IsStone Code

---
 .../ViewModel/SchereSteinPapierEchseSpock.cs  | 12 +++++-----
 .../SchereSteinPapierEchseSpockModel.cs       |  2 +-
 .../SchereSteinPapierEchseSpockTest.cs        | 23 +++++++++++++++----
 3 files changed, 25 insertions(+), 12 deletions(-)

diff --git a/BlazorSolution/MiniGames/Client/ViewModel/SchereSteinPapierEchseSpock.cs b/BlazorSolution/MiniGames/Client/ViewModel/SchereSteinPapierEchseSpock.cs
index 82abade..639e688 100644
--- a/BlazorSolution/MiniGames/Client/ViewModel/SchereSteinPapierEchseSpock.cs
+++ b/BlazorSolution/MiniGames/Client/ViewModel/SchereSteinPapierEchseSpock.cs
@@ -7,17 +7,17 @@ namespace MiniGames.Client.ViewModel
     {
         SchereSteinPapierEchseSpockModel RegelModel = new();
 
-        public string SpielerStein(int hand, int gegner)
+        public string SpielerStein(string spielerHand, string gegnerHand)
         {
-            if(hand == 0 && (gegner == 1 || gegner == 3))
+            if(spielerHand.Equals(RegelModel.hand[0]) && (gegnerHand.Equals(RegelModel.hand[1]) || gegnerHand.Equals(RegelModel.hand[3])))
             {
-                return RegelModel.erg[0].ToString();
-            } else if(hand == 0 && (gegner == 2 || gegner == 4))
+                return RegelModel.erg[0];
+            } else if(spielerHand.Equals(RegelModel.hand[0]) && (gegnerHand.Equals(RegelModel.hand[2]) || gegnerHand.Equals(RegelModel.hand[4])))
             {
-                return RegelModel.erg[1].ToString();
+                return RegelModel.erg[1];
             } else
             {
-                return RegelModel.erg[2].ToString();
+                return RegelModel.erg[2];
             }
         }
     }
diff --git a/BlazorSolution/MiniGames/Shared/Models/SchereSteinPapierEchseSpockModel.cs b/BlazorSolution/MiniGames/Shared/Models/SchereSteinPapierEchseSpockModel.cs
index fecc91c..39d4741 100644
--- a/BlazorSolution/MiniGames/Shared/Models/SchereSteinPapierEchseSpockModel.cs
+++ b/BlazorSolution/MiniGames/Shared/Models/SchereSteinPapierEchseSpockModel.cs
@@ -11,8 +11,8 @@ namespace MiniGames.Shared.Models
         public string[] hand = new[]
         {
             "Stein",
-            "Papier",
             "Schere",
+            "Papier",
             "Echse",
             "Spock"
         };
diff --git a/BlazorSolution/MiniGamesTests/SchereSteinPapierEchseSpockTest.cs b/BlazorSolution/MiniGamesTests/SchereSteinPapierEchseSpockTest.cs
index 40a8646..bd28dd7 100644
--- a/BlazorSolution/MiniGamesTests/SchereSteinPapierEchseSpockTest.cs
+++ b/BlazorSolution/MiniGamesTests/SchereSteinPapierEchseSpockTest.cs
@@ -12,19 +12,32 @@ namespace MiniGamesTests
     public class SchereSteinPapierEchseSpockTest
     {
         SchereSteinPapierEchseSpock Spiel = new();
+        SchereSteinPapierEchseSpockModel SpieleModel = new();
+        private const string stein = "Stein";
+        private const string schere = "Schere";
+        private const string papier = "Papier";
+        private const string echse = "Echse";
+        private const string spock = "Spock";
 
+        private const string gewonnen = "Gewonnen";
+        private const string verloren = "Verloren";
+        private const string unentschieden = "Unentschieden";
 
         [Theory]
-        [InlineData(0, 0, "Unentschieden")]
-        public void IsStone(int hand, int gegnerHand, string erwartet)
+        [InlineData(stein, stein, unentschieden)]
+        [InlineData(stein, schere, gewonnen)]
+        [InlineData(stein, papier, verloren)]
+        [InlineData(stein, echse, gewonnen)]
+        [InlineData(stein, spock, verloren)]
+        public void IsStone(string hand, string gegner, string erwartet)
         {
             //arrange
-            int handId = hand;
-            int gegnerId = gegnerHand;
+            string spielerHand = hand;
+            string gegnerHand = gegner;
             string erwartetesErg = erwartet;
 
             //act
-            string getestetesErg = Spiel.SpielerStein(handId, gegnerId);
+            string getestetesErg = Spiel.SpielerStein(spielerHand, gegnerHand);
 
             //assert
             Assert.Equal(erwartetesErg, getestetesErg);