From 05c18a66cefa8282f71c801c63db9b4a161bcb15 Mon Sep 17 00:00:00 2001 From: ADato88 Date: Wed, 9 Feb 2022 11:07:08 +0100 Subject: [PATCH] =?UTF-8?q?refactoring=20von=20ErstelleFarbigeSpezialKarte?= =?UTF-8?q?n=20und=20zus=C3=A4tztliche=20Tests=20zu=20ErstelleFarbigeSpezi?= =?UTF-8?q?alKartenTest?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../MiniGames/Client/ViewModel/Uno.cs | 19 +++++-------------- BlazorSolution/MiniGamesTests/UnoTest.cs | 10 +++++++--- 2 files changed, 12 insertions(+), 17 deletions(-) diff --git a/BlazorSolution/MiniGames/Client/ViewModel/Uno.cs b/BlazorSolution/MiniGames/Client/ViewModel/Uno.cs index 5855d13..9cff5de 100644 --- a/BlazorSolution/MiniGames/Client/ViewModel/Uno.cs +++ b/BlazorSolution/MiniGames/Client/ViewModel/Uno.cs @@ -135,13 +135,13 @@ namespace MiniGames.Client.ViewModel stapelModel.KartenModels = new(); for (int f = 1; f <= 4; f++) { - EinzelneKarteErstellen(stapelModel, f, 0); + EinzelneKarteErstellen(stapelModel, f, 0, false); for (int k = 1; k <= 9; k++) { for (int a = 0; a < 2; a++) { - EinzelneKarteErstellen(stapelModel, f, k); + EinzelneKarteErstellen(stapelModel, f, k, false); } } @@ -150,7 +150,7 @@ namespace MiniGames.Client.ViewModel return stapelModel; } - private static void EinzelneKarteErstellen(StapelModel stapelModel, int farbenIndex, int kartenbezeichungIndex) + private static void EinzelneKarteErstellen(StapelModel stapelModel, int farbenIndex, int kartenbezeichungIndex, bool spezial) { var kartenbezeichnung = (Kartenbezeichnung)kartenbezeichungIndex; var kartenfarbe = (Farbe)farbenIndex; @@ -159,7 +159,7 @@ namespace MiniGames.Client.ViewModel { Name = kartenbezeichnung.ToString(), Farbe = kartenfarbe.ToString(), - Spezial = false + Spezial = spezial } ); } @@ -174,16 +174,7 @@ namespace MiniGames.Client.ViewModel { for (int i = 0; i < 2; i++) { - var _kartenbezeichnung = (Kartenbezeichnung)k; - var kartenfarbe = (Farbe)f; - stapelModel.KartenModels.Add( - new KartenModel - { - Name = _kartenbezeichnung.ToString(), - Farbe = kartenfarbe.ToString(), - Spezial = true - } - ); + EinzelneKarteErstellen(stapelModel, f, k, true); } } } diff --git a/BlazorSolution/MiniGamesTests/UnoTest.cs b/BlazorSolution/MiniGamesTests/UnoTest.cs index b1e532c..4cfcc02 100644 --- a/BlazorSolution/MiniGamesTests/UnoTest.cs +++ b/BlazorSolution/MiniGamesTests/UnoTest.cs @@ -403,13 +403,17 @@ namespace MiniGamesTests [Theory] [InlineData(Kartenbezeichnung.PlusZweiKarte, Farbe.Gelb, true, 6)] - [InlineData(Kartenbezeichnung.RetoureKarte, Farbe.Rot, true, 6)] - [InlineData(Kartenbezeichnung.AussetzenKarte, Farbe.Gelb, true, 6)] [InlineData(Kartenbezeichnung.PlusZweiKarte, Farbe.Blau, true, 6)] [InlineData(Kartenbezeichnung.PlusZweiKarte, Farbe.Gruen, true, 6)] + [InlineData(Kartenbezeichnung.PlusZweiKarte, Farbe.Rot, true, 6)] [InlineData(Kartenbezeichnung.RetoureKarte, Farbe.Gelb, true, 6)] + [InlineData(Kartenbezeichnung.RetoureKarte, Farbe.Blau, true, 6)] + [InlineData(Kartenbezeichnung.RetoureKarte, Farbe.Gruen, true, 6)] + [InlineData(Kartenbezeichnung.RetoureKarte, Farbe.Rot, true, 6)] + [InlineData(Kartenbezeichnung.AussetzenKarte, Farbe.Gelb, true, 6)] + [InlineData(Kartenbezeichnung.AussetzenKarte, Farbe.Blau, true, 6)] [InlineData(Kartenbezeichnung.AussetzenKarte, Farbe.Gruen, true, 6)] - [InlineData(Kartenbezeichnung.PlusZweiKarte, Farbe.Rot, true, 6)] + [InlineData(Kartenbezeichnung.AussetzenKarte, Farbe.Rot, true, 6)] private void ErstelleFarbigeSpezialKartenTest(Kartenbezeichnung kartenbezeichnung, Farbe farbe, bool spezial, int anzahl) { //arrange