From 134727bafa6bb5493d1ff2e1e1637353f1ba4735 Mon Sep 17 00:00:00 2001
From: ADato88 <da-maschine@hotmail.de>
Date: Wed, 26 Jan 2022 17:48:16 +0100
Subject: [PATCH] refactor naming

---
 BlazorSolution/MiniGames/Client/ViewModel/IUno.cs | 2 +-
 BlazorSolution/MiniGames/Client/ViewModel/Uno.cs  | 5 ++---
 BlazorSolution/MiniGamesTests/UnoTest.cs          | 4 ++--
 3 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/BlazorSolution/MiniGames/Client/ViewModel/IUno.cs b/BlazorSolution/MiniGames/Client/ViewModel/IUno.cs
index 7355372..3910e49 100644
--- a/BlazorSolution/MiniGames/Client/ViewModel/IUno.cs
+++ b/BlazorSolution/MiniGames/Client/ViewModel/IUno.cs
@@ -8,7 +8,7 @@ namespace MiniGames.Client.ViewModel
         StapelModel ZiehKartenStapel { get; set; }
 
         void ZiehKartenStapelZusammenstellen();
-        bool IstAblageStabelFarbeGleichZuLegendeKarte(string neueKarteFarbe);
+        bool IstAblageStapelFarbeGleichZuLegendeKarte(string neueKarteFarbe);
         List<KartenModel> ZiehEineKarte(List<KartenModel> zuZiehendeHand);
     }
 }
\ No newline at end of file
diff --git a/BlazorSolution/MiniGames/Client/ViewModel/Uno.cs b/BlazorSolution/MiniGames/Client/ViewModel/Uno.cs
index 04078d9..1663f19 100644
--- a/BlazorSolution/MiniGames/Client/ViewModel/Uno.cs
+++ b/BlazorSolution/MiniGames/Client/ViewModel/Uno.cs
@@ -10,7 +10,7 @@ namespace MiniGames.Client.ViewModel
         public StapelModel KartenAblageStapel { get; set; } = new();
         public StapelModel ZiehKartenStapel { get; set; } = new();
 
-        public bool IstAblageStabelFarbeGleichZuLegendeKarte(string neueKarteFarbe)
+        public bool IstAblageStapelFarbeGleichZuLegendeKarte(string neueKarteFarbe)
         {
             if (KartenAblageStapel.KartenModels.LastOrDefault().Farbe.Equals(neueKarteFarbe))
             {
@@ -51,8 +51,7 @@ namespace MiniGames.Client.ViewModel
                 case _plusZweiKarte:
                     for (int i = 0; i < 2; i++)
                     {
-                        _naechstenHand.Add(ZiehKartenStapel.KartenModels.FirstOrDefault());
-                        ZiehKartenStapel.KartenModels.RemoveAt(0);
+                        _naechstenHand = ZiehEineKarte(_naechstenHand);
                     }
                     break;
                 case _retoureKarte:
diff --git a/BlazorSolution/MiniGamesTests/UnoTest.cs b/BlazorSolution/MiniGamesTests/UnoTest.cs
index 05f8b75..13b49ac 100644
--- a/BlazorSolution/MiniGamesTests/UnoTest.cs
+++ b/BlazorSolution/MiniGamesTests/UnoTest.cs
@@ -74,7 +74,7 @@ namespace MiniGamesTests
         [InlineData(Farbe.Blau, Farbe.Blau, true)]
         [InlineData(Farbe.Gelb, Farbe.Rot, false)]
         [InlineData(Farbe.Gruen, Farbe.Gruen, true)]
-        private void IstAblageStabelFarbeGleichZuLegendeKarteTest(Farbe ablegeStapelKarteFarbe, Farbe neueKarteFarbe, bool erwartet)
+        private void IstAblageStapelFarbeGleichZuLegendeKarteTest(Farbe ablegeStapelKarteFarbe, Farbe neueKarteFarbe, bool erwartet)
         {
             //arrange
             UnoRegeln.ZiehKartenStapelZusammenstellen();
@@ -86,7 +86,7 @@ namespace MiniGamesTests
             UnoRegeln.KartenAblageStapel.KartenModels.Add(BeispielKarte("Testkarten Name", _ablegeStabelKarteFarbe, false));
 
             //act
-            var erhalten = UnoRegeln.IstAblageStabelFarbeGleichZuLegendeKarte(_neueKarteFarbe);
+            var erhalten = UnoRegeln.IstAblageStapelFarbeGleichZuLegendeKarte(_neueKarteFarbe);
 
             //assert
             Assert.Equal(_erwartet, erhalten);