From d4ac80ef5cab34da12fc8e19ec0230434d581da4 Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 16:15:49 +0100 Subject: [PATCH 01/19] test auf korrekte Berechnung des Dreieckarrays --- src/test/c/Jason/test_ASCII_art.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/src/test/c/Jason/test_ASCII_art.c b/src/test/c/Jason/test_ASCII_art.c index 5f88ae5..bac784c 100644 --- a/src/test/c/Jason/test_ASCII_art.c +++ b/src/test/c/Jason/test_ASCII_art.c @@ -103,5 +103,16 @@ void test_korrekteBerechnungAufMaximaleBreiteWeihnachtsbaum() { int actualResult = max_breite(6); //assert + TEST_ASSERT_EQUAL_INT(expectedResult, actualResult); +} + +void test_korrekteBerechnungAufMaximaleGroesseDesDreieck() { + //arrange + int expectedResult = 65; + + //act + int actualResult = berechnungDerDreieckArraygroesse(10); + //assert + TEST_ASSERT_EQUAL_INT(expectedResult, actualResult); } \ No newline at end of file From 94486d7e2afa91acbf68a59f35ab16e5c363ec1f Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 16:23:54 +0100 Subject: [PATCH 02/19] =?UTF-8?q?Test=20der=20Dreieckberechnung=20auf=20se?= =?UTF-8?q?hr=20gro=C3=9Fe=20Zahlen?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/test/c/Jason/test_ASCII_art.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/src/test/c/Jason/test_ASCII_art.c b/src/test/c/Jason/test_ASCII_art.c index bac784c..d6af16c 100644 --- a/src/test/c/Jason/test_ASCII_art.c +++ b/src/test/c/Jason/test_ASCII_art.c @@ -114,5 +114,16 @@ void test_korrekteBerechnungAufMaximaleGroesseDesDreieck() { int actualResult = berechnungDerDreieckArraygroesse(10); //assert + TEST_ASSERT_EQUAL_INT(expectedResult, actualResult); +} + +void test_korrekteBerechnungAufMaximaleGroesseDesDreiecksBeiGrossenZahlen() { + //arrange + int expectedResult = 50015000; + + //act + int actualResult = berechnungDerDreieckArraygroesse(10000); + //assert + TEST_ASSERT_EQUAL_INT(expectedResult, actualResult); } \ No newline at end of file From 8e9d66b8559db3fb76962465761e887a4e54ff52 Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 16:41:06 +0100 Subject: [PATCH 03/19] =?UTF-8?q?Test=20auf=20korrekte=20Ausgabe=20des=20D?= =?UTF-8?q?reiecks=20inklusive=20fix=20des=20bef=C3=BCllens?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/main/c/Jason/ASCII_art.c | 2 +- src/test/c/Jason/test_ASCII_art.c | 14 ++++++++++++++ 2 files changed, 15 insertions(+), 1 deletion(-) diff --git a/src/main/c/Jason/ASCII_art.c b/src/main/c/Jason/ASCII_art.c index 9242f53..c0dd61c 100644 --- a/src/main/c/Jason/ASCII_art.c +++ b/src/main/c/Jason/ASCII_art.c @@ -305,7 +305,7 @@ void befuellenDesDreiecksMitZeilenumbruechen(char arr[], int gesamtzeilen, int a stelleFuerZeilenumbruch += (zeile+2); } - arr[arraygroesse] = '\0'; + arr[arraygroesse -1] = '\0'; } void raketeBegruessung(){ diff --git a/src/test/c/Jason/test_ASCII_art.c b/src/test/c/Jason/test_ASCII_art.c index d6af16c..c0edbf7 100644 --- a/src/test/c/Jason/test_ASCII_art.c +++ b/src/test/c/Jason/test_ASCII_art.c @@ -126,4 +126,18 @@ void test_korrekteBerechnungAufMaximaleGroesseDesDreiecksBeiGrossenZahlen() { //assert TEST_ASSERT_EQUAL_INT(expectedResult, actualResult); +} + +void test_wirdKleinesDreieckKorrektBefuellt(void){ + // arrange + char expectedMessage[] = "X\nXX"; + + // act + char message[6]; + befuellenDesDreiecksMitX(message, 5); + befuellenDesDreiecksMitZeilenumbruechen(message, 2, 5); + + + // assert + TEST_ASSERT_EQUAL_STRING(expectedMessage, message); } \ No newline at end of file From ae4db7e22e6250560d37e57805c698f98bb7a090 Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 16:47:07 +0100 Subject: [PATCH 04/19] Kleine Bugfixes und Erweiterung Des Quadrats --- src/main/c/Jason/ASCII_art.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/main/c/Jason/ASCII_art.c b/src/main/c/Jason/ASCII_art.c index c0dd61c..341015d 100644 --- a/src/main/c/Jason/ASCII_art.c +++ b/src/main/c/Jason/ASCII_art.c @@ -11,7 +11,7 @@ int ASCII_Art(){ auswahlDerArt(auswahl); - printf("Vielen Dank, dass sie sich ASCII-Art angesehen haben\n" + printf("\nVielen Dank, dass sie sich ASCII-Art angesehen haben\n" "Wollen Sie weitere ASCII-Art sehen?\n" "(1) Ja\n" "(2) Nein\n" @@ -97,6 +97,8 @@ void rakete(){ "Eingbe:"); } } while(hoehe < 3); + + printf("\nComing soon ;)\n"); } From 617cd2bbdd345df5339b2d958e90d5c0af271372 Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 16:55:20 +0100 Subject: [PATCH 05/19] refactoring: optimierung von Funktionsnamen in Testdatei --- src/test/c/Jason/test_ASCII_art.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/c/Jason/test_ASCII_art.c b/src/test/c/Jason/test_ASCII_art.c index c0edbf7..97cebb1 100644 --- a/src/test/c/Jason/test_ASCII_art.c +++ b/src/test/c/Jason/test_ASCII_art.c @@ -95,7 +95,7 @@ void test_erscheintDieAbfrageFuerWeihnachtsbaum() { TEST_ASSERT_EQUAL_INT(expectedResult, actualResult); } -void test_korrekteBerechnungAufMaximaleBreiteWeihnachtsbaum() { +void test_korrekteBerechnungAufMaximaleBreiteDesWeihnachtsbaums() { //arrange int expectedResult = 10; @@ -106,7 +106,7 @@ void test_korrekteBerechnungAufMaximaleBreiteWeihnachtsbaum() { TEST_ASSERT_EQUAL_INT(expectedResult, actualResult); } -void test_korrekteBerechnungAufMaximaleGroesseDesDreieck() { +void test_korrekteBerechnungAufMaximaleGroesseDesDreiecks() { //arrange int expectedResult = 65; From 7ae27c18bfeac3ddbaa058042d8f74e20ae8c4f7 Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 17:01:07 +0100 Subject: [PATCH 06/19] refactoring: Sprachliche Anpassungen --- src/main/c/Jason/ASCII_art.c | 4 ++-- src/main/c/Jason/ASCII_art.h | 2 +- src/test/c/Jason/test_ASCII_art.c | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/main/c/Jason/ASCII_art.c b/src/main/c/Jason/ASCII_art.c index 341015d..d77a9a6 100644 --- a/src/main/c/Jason/ASCII_art.c +++ b/src/main/c/Jason/ASCII_art.c @@ -5,7 +5,7 @@ int ASCII_Art(){ int weitermachen = 0; do { - printWelcomeMessage(); + Willkommensnachricht(); int auswahl; scanf("%d", &auswahl); auswahlDerArt(auswahl); @@ -102,7 +102,7 @@ void rakete(){ } -int printWelcomeMessage(){ +int Willkommensnachricht(){ printf("Willkommen bei ASCII-Art!\n"); printf("Wählen Sie aus welche ASCII-Art Sie sehen wollen!\n"); printf("(1) Quadrat\n"); diff --git a/src/main/c/Jason/ASCII_art.h b/src/main/c/Jason/ASCII_art.h index 4917774..62fa5f6 100644 --- a/src/main/c/Jason/ASCII_art.h +++ b/src/main/c/Jason/ASCII_art.h @@ -2,7 +2,7 @@ #define ASCII_ART_H int ASCII_Art(); -int printWelcomeMessage(); +int Willkommensnachricht(); void auswahlDerArt(int); void quadrat(); int frageNachGroesse(); diff --git a/src/test/c/Jason/test_ASCII_art.c b/src/test/c/Jason/test_ASCII_art.c index 97cebb1..d263a0b 100644 --- a/src/test/c/Jason/test_ASCII_art.c +++ b/src/test/c/Jason/test_ASCII_art.c @@ -9,12 +9,12 @@ void tearDown(void){ } -void test_doesWelcomeMessageWork() { +void test_funktioniertDieWillkommensnachricht() { //arrange int expectedResult = 0; //act - int actualResult = printWelcomeMessage(); + int actualResult = Willkommensnachricht(); //assert TEST_ASSERT_EQUAL_INT(expectedResult, actualResult); From 5458590d66fcea3f6c2fd3749e33024e68abf57a Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 17:12:27 +0100 Subject: [PATCH 07/19] =?UTF-8?q?Test=20auf=20korrekte=20Baumkrome=20hinzu?= =?UTF-8?q?gef=C3=BCgt?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/test/c/Jason/test_ASCII_art.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/src/test/c/Jason/test_ASCII_art.c b/src/test/c/Jason/test_ASCII_art.c index d263a0b..cd7e2e5 100644 --- a/src/test/c/Jason/test_ASCII_art.c +++ b/src/test/c/Jason/test_ASCII_art.c @@ -138,6 +138,21 @@ void test_wirdKleinesDreieckKorrektBefuellt(void){ befuellenDesDreiecksMitZeilenumbruechen(message, 2, 5); + // assert + TEST_ASSERT_EQUAL_STRING(expectedMessage, message); +} + +void test_wirdKleinesBaumkroneKorrektBefuelld(void){ + // arrange + char expectedMessage[] = " + \n+++"; + + // act + char message[9]; + befuellenDesWeihnachtsbaumsMitLeerzeichen(message, 8); + befuellenDesWeihnachtsbaumsMitPlus(message, 4, (3-1)); + befuellenDesWeihnachtsbaumsMitZeilenumbruechen(message, 3, 4); + + // assert TEST_ASSERT_EQUAL_STRING(expectedMessage, message); } \ No newline at end of file From 48451c2a733e863cc65c25bd88a5b867091d17a7 Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 17:19:58 +0100 Subject: [PATCH 08/19] =?UTF-8?q?refactoring:=20Erkl=C3=A4rungen=20von=20T?= =?UTF-8?q?ests=20hinzugef=C3=BCgt?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/test/c/Jason/test_ASCII_art.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/test/c/Jason/test_ASCII_art.c b/src/test/c/Jason/test_ASCII_art.c index cd7e2e5..2eaad76 100644 --- a/src/test/c/Jason/test_ASCII_art.c +++ b/src/test/c/Jason/test_ASCII_art.c @@ -46,6 +46,8 @@ void test_stringAbgleich(void){ // arrange char expectedMessage[] = "XXXXXXX"; + // Dieser Test überprüft, ob das Feld für die Baumkrone korrekt mit X befüllt wird + // act char message[9]; befuellenDesArraysMitX(message, 8); @@ -59,6 +61,8 @@ void test_stringAbgleichMitLeerzeichen(void){ // arrange char expectedMessage[] = "X X X X X X X X X"; + // Hier wird überprüft ob die Baumkrone wie erwartet mit X und Leerzeichen befüllt wird + // act char message[19]; befuellenDesArraysMitX(message, 18); From 3d8bff17010753cd2c049422c72b3634a3df6e3b Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 17:23:56 +0100 Subject: [PATCH 09/19] =?UTF-8?q?refactoring:=20Erkl=C3=A4rung=20von=20Tes?= =?UTF-8?q?ts=20und=20Verbesserung=20der=20Lesbarkeit?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/test/c/Jason/test_ASCII_art.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/test/c/Jason/test_ASCII_art.c b/src/test/c/Jason/test_ASCII_art.c index 2eaad76..a864f5d 100644 --- a/src/test/c/Jason/test_ASCII_art.c +++ b/src/test/c/Jason/test_ASCII_art.c @@ -125,6 +125,8 @@ void test_korrekteBerechnungAufMaximaleGroesseDesDreiecksBeiGrossenZahlen() { //arrange int expectedResult = 50015000; + // Hier sollte auf overflow überprüft werden bei großen Zahlen + //act int actualResult = berechnungDerDreieckArraygroesse(10000); //assert @@ -136,8 +138,10 @@ void test_wirdKleinesDreieckKorrektBefuellt(void){ // arrange char expectedMessage[] = "X\nXX"; + // act char message[6]; + befuellenDesDreiecksMitX(message, 5); befuellenDesDreiecksMitZeilenumbruechen(message, 2, 5); @@ -152,6 +156,7 @@ void test_wirdKleinesBaumkroneKorrektBefuelld(void){ // act char message[9]; + befuellenDesWeihnachtsbaumsMitLeerzeichen(message, 8); befuellenDesWeihnachtsbaumsMitPlus(message, 4, (3-1)); befuellenDesWeihnachtsbaumsMitZeilenumbruechen(message, 3, 4); From 9a9d78193941e1b9889985782345fbccc1cde17a Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 17:39:41 +0100 Subject: [PATCH 10/19] refactoring: Verbesserung der Lesbarkeit --- src/main/c/Jason/ASCII_art.c | 11 +++++++---- src/main/c/Jason/ASCII_art.h | 4 ++-- src/test/c/Jason/test_ASCII_art.c | 8 ++++---- 3 files changed, 13 insertions(+), 10 deletions(-) diff --git a/src/main/c/Jason/ASCII_art.c b/src/main/c/Jason/ASCII_art.c index d77a9a6..86e114b 100644 --- a/src/main/c/Jason/ASCII_art.c +++ b/src/main/c/Jason/ASCII_art.c @@ -6,6 +6,7 @@ int ASCII_Art(){ int weitermachen = 0; do { Willkommensnachricht(); + int auswahl; scanf("%d", &auswahl); auswahlDerArt(auswahl); @@ -30,8 +31,8 @@ void quadrat(){ int arraySize = berechnungDerLaengeDesQuadratArrays(groesse); char Quadrat[arraySize]; befuellenDesArraysMitX(Quadrat, arraySize); - befuelleArrayMitLeerzeichen(Quadrat, arraySize); - befuelleArrayMitZeilenumbruechen(Quadrat,groesse); + befuelleDasArrayMitLeerzeichen(Quadrat, arraySize); + befuelleDasArrayMitZeilenumbruechen(Quadrat,groesse); printf("\n%s", Quadrat); @@ -158,7 +159,7 @@ void befuellenDesArraysMitX(char quadraT[], int laenge){ } -void befuelleArrayMitLeerzeichen(char quadraT[], int laenge){ +void befuelleDasArrayMitLeerzeichen(char quadraT[], int laenge){ for (int i = 0; i < laenge; i++){ @@ -166,10 +167,12 @@ void befuelleArrayMitLeerzeichen(char quadraT[], int laenge){ quadraT[i] = ' '; } } + + // Das \0 muss Extra platziert werden um einen korrekten Ablauf sicherzustellen quadraT[laenge-1] = '\0'; } -void befuelleArrayMitZeilenumbruechen(char quadraT[], int laenge){ +void befuelleDasArrayMitZeilenumbruechen(char quadraT[], int laenge){ int zeile = 2 * laenge; int laengeGesamt = zeile * laenge; diff --git a/src/main/c/Jason/ASCII_art.h b/src/main/c/Jason/ASCII_art.h index 62fa5f6..7624d56 100644 --- a/src/main/c/Jason/ASCII_art.h +++ b/src/main/c/Jason/ASCII_art.h @@ -8,8 +8,8 @@ void quadrat(); int frageNachGroesse(); int berechnungDerLaengeDesQuadratArrays(int); void befuellenDesArraysMitX(char[], int); -void befuelleArrayMitLeerzeichen(char[],int); -void befuelleArrayMitZeilenumbruechen(char[],int); +void befuelleDasArrayMitLeerzeichen(char[],int); +void befuelleDasArrayMitZeilenumbruechen(char[],int); void weihnachtsbaum(); int abfrageDerHoehe(); int max_breite(int); diff --git a/src/test/c/Jason/test_ASCII_art.c b/src/test/c/Jason/test_ASCII_art.c index a864f5d..fed9eac 100644 --- a/src/test/c/Jason/test_ASCII_art.c +++ b/src/test/c/Jason/test_ASCII_art.c @@ -66,7 +66,7 @@ void test_stringAbgleichMitLeerzeichen(void){ // act char message[19]; befuellenDesArraysMitX(message, 18); - befuelleArrayMitLeerzeichen(message, 18); + befuelleDasArrayMitLeerzeichen(message, 18); // assert @@ -80,8 +80,8 @@ void test_stringAbgleichMitLeerzeichenUndZeilenumbruch(void){ // act char message[19]; befuellenDesArraysMitX(message, 18); - befuelleArrayMitLeerzeichen(message, 18); - befuelleArrayMitZeilenumbruechen(message, 3); + befuelleDasArrayMitLeerzeichen(message, 18); + befuelleDasArrayMitZeilenumbruechen(message, 3); // assert @@ -156,7 +156,7 @@ void test_wirdKleinesBaumkroneKorrektBefuelld(void){ // act char message[9]; - + befuellenDesWeihnachtsbaumsMitLeerzeichen(message, 8); befuellenDesWeihnachtsbaumsMitPlus(message, 4, (3-1)); befuellenDesWeihnachtsbaumsMitZeilenumbruechen(message, 3, 4); From 808e47f9a9631c6c441496dc0df0e5242418af8c Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 17:44:06 +0100 Subject: [PATCH 11/19] refactoring: Anpassung Namensgebung --- src/main/c/Jason/ASCII_art.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/main/c/Jason/ASCII_art.c b/src/main/c/Jason/ASCII_art.c index 86e114b..59a45ea 100644 --- a/src/main/c/Jason/ASCII_art.c +++ b/src/main/c/Jason/ASCII_art.c @@ -71,19 +71,21 @@ void weihnachtsbaum(){ void dreieck(){ dreieckWillkommensnachricht(); - int hoehe; + int hoeheDreieck; + + // Eingabe Auf Definitions do{ - scanf("%d", &hoehe); - if(hoehe < 2){ + scanf("%d", &hoeheDreieck); + if(hoeheDreieck < 2){ printf("Es wurde ein Wert eingegeben, der kleiner ist als 2. Bitte geben Sie erneut eine Zahl ein!\n" "Eingbe:"); } - } while(hoehe < 2); + } while(hoeheDreieck < 2); - int arraygroesse = berechnungDerDreieckArraygroesse(hoehe); + int arraygroesse = berechnungDerDreieckArraygroesse(hoeheDreieck); char dreieck[arraygroesse]; befuellenDesDreiecksMitX(dreieck, arraygroesse); - befuellenDesDreiecksMitZeilenumbruechen(dreieck, hoehe, arraygroesse); + befuellenDesDreiecksMitZeilenumbruechen(dreieck, hoeheDreieck, arraygroesse); printf("%s", dreieck); } From e996ac3049b6b4731a9943eef5020c301cfde0ca Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 17:46:02 +0100 Subject: [PATCH 12/19] refactoring: Anpassung --- src/main/c/Jason/ASCII_art.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/main/c/Jason/ASCII_art.c b/src/main/c/Jason/ASCII_art.c index 59a45ea..cc65c64 100644 --- a/src/main/c/Jason/ASCII_art.c +++ b/src/main/c/Jason/ASCII_art.c @@ -92,14 +92,14 @@ void dreieck(){ void rakete(){ raketeBegruessung(); - int hoehe; + int hoeheRakete; do{ - scanf("%d", &hoehe); - if(hoehe < 3){ + scanf("%d", &hoeheRakete); + if(hoeheRakete < 3){ printf("Es wurde ein Wert eingegeben, der kleiner ist als 3. Bitte geben Sie erneut eine Zahl ein!\n" "Eingbe:"); } - } while(hoehe < 3); + } while(hoeheRakete < 3); printf("\nComing soon ;)\n"); } From 16e60eadf94700e877f5690ff0d430c75a3ed3f5 Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 17:49:25 +0100 Subject: [PATCH 13/19] =?UTF-8?q?refactoring:=20Vert=C3=A4ndlicherer=20Var?= =?UTF-8?q?iablenname?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/main/c/Jason/ASCII_art.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/main/c/Jason/ASCII_art.c b/src/main/c/Jason/ASCII_art.c index cc65c64..311787f 100644 --- a/src/main/c/Jason/ASCII_art.c +++ b/src/main/c/Jason/ASCII_art.c @@ -218,18 +218,18 @@ void befuellenDesWeihnachtsbaumsMitPlus(char arr[], int maxBreite, int zeilen){ int arrayGroesse = maxBreite * zeilen; int mitteDesBaums = maxBreite / 2; - int mengeX = 1; + int mengeDerXInDerZeile = 1; for(int i = zeilen; i > 0; i--){ int position = mitteDesBaums -1; - while(position < mitteDesBaums-1 + mengeX){ + while(position < mitteDesBaums-1 + mengeDerXInDerZeile){ arr[position++] = '+'; } mitteDesBaums += (maxBreite -1); - mengeX += 2; + mengeDerXInDerZeile += 2; } arr[arrayGroesse] = '\0'; } From 30dddda26ed4d1edda612323e16446f0c8e234aa Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 17:52:22 +0100 Subject: [PATCH 14/19] =?UTF-8?q?refactoring:=20Lesbarkeit=20erh=C3=B6ht?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/main/c/Jason/ASCII_art.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/main/c/Jason/ASCII_art.c b/src/main/c/Jason/ASCII_art.c index 311787f..9be847b 100644 --- a/src/main/c/Jason/ASCII_art.c +++ b/src/main/c/Jason/ASCII_art.c @@ -26,13 +26,13 @@ int ASCII_Art(){ void quadrat(){ frageNachGroesse(); - int groesse; - scanf("%d", &groesse); - int arraySize = berechnungDerLaengeDesQuadratArrays(groesse); + int LaengeDerZeilen; + scanf("%d", &LaengeDerZeilen); + int arraySize = berechnungDerLaengeDesQuadratArrays(LaengeDerZeilen); char Quadrat[arraySize]; befuellenDesArraysMitX(Quadrat, arraySize); befuelleDasArrayMitLeerzeichen(Quadrat, arraySize); - befuelleDasArrayMitZeilenumbruechen(Quadrat,groesse); + befuelleDasArrayMitZeilenumbruechen(Quadrat,LaengeDerZeilen); printf("\n%s", Quadrat); From 5f1aa9b84205c0323cacb134fcfaf291ef4e6f98 Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 17:56:49 +0100 Subject: [PATCH 15/19] refactoring: Verbesserung vom Variablennamen hoehe --- src/main/c/Jason/ASCII_art.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/main/c/Jason/ASCII_art.c b/src/main/c/Jason/ASCII_art.c index 9be847b..2a8793a 100644 --- a/src/main/c/Jason/ASCII_art.c +++ b/src/main/c/Jason/ASCII_art.c @@ -41,24 +41,24 @@ void quadrat(){ void weihnachtsbaum(){ abfrageDerHoehe(); - int hoehe; + int hoeheVomBaumInklusiveStamm; do{ // Diese Schleifen erfragen user input solange bis korrekte Werte eingegeben werden - scanf("%d", &hoehe); - if (hoehe < 3){ + scanf("%d", &hoeheVomBaumInklusiveStamm); + if (hoeheVomBaumInklusiveStamm < 3){ printf("\nDer Weihnachtsbaum muss mindestens 3 Zeilen groß sein!\n"); printf("Bitte geben Sie erneut einen Wert ein der mindestens 3 lautet:"); } - }while(hoehe < 3); + }while(hoeheVomBaumInklusiveStamm < 3); - int maxBreiteDesWeihnachtsbaums = max_breite(hoehe); + int maxBreiteDesWeihnachtsbaums = max_breite(hoeheVomBaumInklusiveStamm); int mitte_baum = mitteDesBaums(maxBreiteDesWeihnachtsbaums); - int groesseDesArrays = (hoehe -1) * maxBreiteDesWeihnachtsbaums; + int groesseDesArrays = (hoeheVomBaumInklusiveStamm -1) * maxBreiteDesWeihnachtsbaums; char weihnachtsbaum[groesseDesArrays]; befuellenDesWeihnachtsbaumsMitLeerzeichen(weihnachtsbaum, groesseDesArrays); - befuellenDesWeihnachtsbaumsMitPlus(weihnachtsbaum, maxBreiteDesWeihnachtsbaums, (hoehe-1)); - befuellenDesWeihnachtsbaumsMitZeilenumbruechen(weihnachtsbaum, hoehe, maxBreiteDesWeihnachtsbaums); + befuellenDesWeihnachtsbaumsMitPlus(weihnachtsbaum, maxBreiteDesWeihnachtsbaums, (hoeheVomBaumInklusiveStamm-1)); + befuellenDesWeihnachtsbaumsMitZeilenumbruechen(weihnachtsbaum, hoeheVomBaumInklusiveStamm, maxBreiteDesWeihnachtsbaums); char stamm[maxBreiteDesWeihnachtsbaums]; befuellenDesStammarraysMitLeerzeichen(stamm, maxBreiteDesWeihnachtsbaums); stammAnRichtigerStelleEinsetzen(stamm, maxBreiteDesWeihnachtsbaums); From f1e35e502ce4271769beacb6355e671421c1fd69 Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 18:01:11 +0100 Subject: [PATCH 16/19] refactoring: Rechtschreibfehler gefixt --- src/main/c/Jason/ASCII_art.c | 4 ++-- src/main/c/Jason/ASCII_art.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/main/c/Jason/ASCII_art.c b/src/main/c/Jason/ASCII_art.c index 2a8793a..f20cfc9 100644 --- a/src/main/c/Jason/ASCII_art.c +++ b/src/main/c/Jason/ASCII_art.c @@ -38,7 +38,7 @@ void quadrat(){ } -void weihnachtsbaum(){ +void Weihnachtsbaum(){ abfrageDerHoehe(); int hoeheVomBaumInklusiveStamm; @@ -122,7 +122,7 @@ void auswahlDerArt(int pick){ quadrat(); break; case (2): - weihnachtsbaum(); + Weihnachtsbaum(); break; case (3): diff --git a/src/main/c/Jason/ASCII_art.h b/src/main/c/Jason/ASCII_art.h index 7624d56..71da3b7 100644 --- a/src/main/c/Jason/ASCII_art.h +++ b/src/main/c/Jason/ASCII_art.h @@ -10,7 +10,7 @@ int berechnungDerLaengeDesQuadratArrays(int); void befuellenDesArraysMitX(char[], int); void befuelleDasArrayMitLeerzeichen(char[],int); void befuelleDasArrayMitZeilenumbruechen(char[],int); -void weihnachtsbaum(); +void Weihnachtsbaum(); int abfrageDerHoehe(); int max_breite(int); int mitteDesBaums(int); From 49ca583fb1bfeee4b0dd96c994761865a4dcb84f Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Mon, 5 Feb 2024 18:35:11 +0100 Subject: [PATCH 17/19] refactoring: Funktionsname --- src/main/c/Jason/ASCII_art.c | 4 ++-- src/main/c/Jason/ASCII_art.h | 2 +- src/test/c/Jason/test_ASCII_art.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/main/c/Jason/ASCII_art.c b/src/main/c/Jason/ASCII_art.c index f20cfc9..d2cf2a5 100644 --- a/src/main/c/Jason/ASCII_art.c +++ b/src/main/c/Jason/ASCII_art.c @@ -52,7 +52,7 @@ void Weihnachtsbaum(){ } }while(hoeheVomBaumInklusiveStamm < 3); - int maxBreiteDesWeihnachtsbaums = max_breite(hoeheVomBaumInklusiveStamm); + int maxBreiteDesWeihnachtsbaums = max_breiteDesBaums(hoeheVomBaumInklusiveStamm); int mitte_baum = mitteDesBaums(maxBreiteDesWeihnachtsbaums); int groesseDesArrays = (hoeheVomBaumInklusiveStamm -1) * maxBreiteDesWeihnachtsbaums; char weihnachtsbaum[groesseDesArrays]; @@ -193,7 +193,7 @@ int abfrageDerHoehe(){ return 0; } -int max_breite(int zeilen){ +int max_breiteDesBaums(int zeilen){ int maximaleBreiteEinerZeile; maximaleBreiteEinerZeile = zeilen + (zeilen - 2); diff --git a/src/main/c/Jason/ASCII_art.h b/src/main/c/Jason/ASCII_art.h index 71da3b7..55b1691 100644 --- a/src/main/c/Jason/ASCII_art.h +++ b/src/main/c/Jason/ASCII_art.h @@ -12,7 +12,7 @@ void befuelleDasArrayMitLeerzeichen(char[],int); void befuelleDasArrayMitZeilenumbruechen(char[],int); void Weihnachtsbaum(); int abfrageDerHoehe(); -int max_breite(int); +int max_breiteDesBaums(int); int mitteDesBaums(int); void befuellenDesWeihnachtsbaumsMitLeerzeichen(char[],int); void befuellenDesWeihnachtsbaumsMitPlus(char[], int, int); diff --git a/src/test/c/Jason/test_ASCII_art.c b/src/test/c/Jason/test_ASCII_art.c index fed9eac..3c13a2d 100644 --- a/src/test/c/Jason/test_ASCII_art.c +++ b/src/test/c/Jason/test_ASCII_art.c @@ -104,7 +104,7 @@ void test_korrekteBerechnungAufMaximaleBreiteDesWeihnachtsbaums() { int expectedResult = 10; //act - int actualResult = max_breite(6); + int actualResult = max_breiteDesBaums(6); //assert TEST_ASSERT_EQUAL_INT(expectedResult, actualResult); From f82781daaaeee0f7068811910ad368a671b96b08 Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Tue, 6 Feb 2024 09:30:08 +0100 Subject: [PATCH 18/19] =?UTF-8?q?test=20ob=20Stammarray=20vom=20Baum=20kor?= =?UTF-8?q?rekt=20bef=C3=BCllt=20wird?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/test/c/Jason/test_ASCII_art.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/src/test/c/Jason/test_ASCII_art.c b/src/test/c/Jason/test_ASCII_art.c index 3c13a2d..52a5894 100644 --- a/src/test/c/Jason/test_ASCII_art.c +++ b/src/test/c/Jason/test_ASCII_art.c @@ -162,6 +162,21 @@ void test_wirdKleinesBaumkroneKorrektBefuelld(void){ befuellenDesWeihnachtsbaumsMitZeilenumbruechen(message, 3, 4); + // assert + TEST_ASSERT_EQUAL_STRING(expectedMessage, message); +} + +void test_wirdBaumstammAnRichtigerStelleEingestz(void){ + // arrange + char expectedMessage[] = " + "; + + // act + char message[7]; + + befuellenDesStammarraysMitLeerzeichen(message, 6); + stammAnRichtigerStelleEinsetzen(message, 6); + + // assert TEST_ASSERT_EQUAL_STRING(expectedMessage, message); } \ No newline at end of file From e0b16560352bdf6ed58ce00eaa49f3a4cf76ed39 Mon Sep 17 00:00:00 2001 From: Jason Peters Date: Tue, 6 Feb 2024 09:32:51 +0100 Subject: [PATCH 19/19] refactoring: Rechtschreibfehler --- src/test/c/Jason/test_ASCII_art.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/c/Jason/test_ASCII_art.c b/src/test/c/Jason/test_ASCII_art.c index 52a5894..227fb8d 100644 --- a/src/test/c/Jason/test_ASCII_art.c +++ b/src/test/c/Jason/test_ASCII_art.c @@ -166,7 +166,7 @@ void test_wirdKleinesBaumkroneKorrektBefuelld(void){ TEST_ASSERT_EQUAL_STRING(expectedMessage, message); } -void test_wirdBaumstammAnRichtigerStelleEingestz(void){ +void test_wirdBaumstammAnRichtigerStelleEingesetzt(void){ // arrange char expectedMessage[] = " + ";