From cffd5fb6200e7b79f8a37a19ea2efba01dfa7360 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Raphael=20Elstr=C3=B6m?= Date: Fri, 10 Feb 2023 12:34:11 +0100 Subject: [PATCH] =?UTF-8?q?refactoring:=20Alle=20Dateien=20=C3=BCberarbeit?= =?UTF-8?q?et=20auf=20bessere=20Lesbarkeit,=20rauswurf=20von=20unnoetigen?= =?UTF-8?q?=20Zeilen,=20und=20Kommentare=20zur=20erkennung=20und=20lesbark?= =?UTF-8?q?eit?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- vier_gewinnt/CSS/style.css | 4 ---- vier_gewinnt/JS/allSpaltenArrays.js | 1 - vier_gewinnt/JS/gameReset.js | 3 +-- vier_gewinnt/JS/gewinnErmitteln.js | 1 - vier_gewinnt/JS/gewinnErmitteln_mock.js | 1 - vier_gewinnt/JS/naechsteFreierPlatz.js | 1 - vier_gewinnt/JS/regeln.js | 1 - vier_gewinnt/JS/spielerWechsel.js | 1 - vier_gewinnt/JS/steinSetzen.js | 8 ++++---- vier_gewinnt/JS/steineInReihe.js | 5 +---- vier_gewinnt/JS/steineInSpalte.js | 5 +---- vier_gewinnt/JS/vierDiagonalLinks.js | 7 ++++++- vier_gewinnt/JS/vierDiagonalRechts.js | 4 ++++ vier_gewinnt/JS/vierInReihe.js | 2 ++ vier_gewinnt/JS/vierInSpalte.js | 2 ++ 15 files changed, 21 insertions(+), 25 deletions(-) diff --git a/vier_gewinnt/CSS/style.css b/vier_gewinnt/CSS/style.css index a34bbd8..df1cbe2 100644 --- a/vier_gewinnt/CSS/style.css +++ b/vier_gewinnt/CSS/style.css @@ -28,8 +28,4 @@ td{ td:hover{ background-color: rgb(80, 80, 80); -} - -button{ - } \ No newline at end of file diff --git a/vier_gewinnt/JS/allSpaltenArrays.js b/vier_gewinnt/JS/allSpaltenArrays.js index 62e4a69..1b9f0e6 100644 --- a/vier_gewinnt/JS/allSpaltenArrays.js +++ b/vier_gewinnt/JS/allSpaltenArrays.js @@ -3,6 +3,5 @@ function allSpaltenArrays() { for (let i = 1; i <= 7; i++) { allSpaltenArray.push(steinInSpalte(i)) } - return allSpaltenArray } \ No newline at end of file diff --git a/vier_gewinnt/JS/gameReset.js b/vier_gewinnt/JS/gameReset.js index f3357ef..1671b24 100644 --- a/vier_gewinnt/JS/gameReset.js +++ b/vier_gewinnt/JS/gameReset.js @@ -2,8 +2,7 @@ function gameReset(winner) { if (winner === 1 || winner === 2) { for (let i = 1; i<= 7; i++){ for (let j = 1; j <= 6; j++) { - document.getElementById(j + "/" + i).style.backgroundColor = "white"; //<-- muss beim testen auskommentiert sein, da Node.js HTML nicht initialisiert und somit für den Test einen Fehler gibt. Man hätte auch einen Mock machen können bin aber gerade zu faul xD. - + document.getElementById(j + "/" + i).style.backgroundColor = "white"; //<-- muss beim testen auskommentiert sein, da Node.js HTML nicht initialisiert und somit für den Test einen Fehler gibt. Man hätte auch einen Mock machen können bin aber gerade zu faul xD. } } return 1; diff --git a/vier_gewinnt/JS/gewinnErmitteln.js b/vier_gewinnt/JS/gewinnErmitteln.js index d8c642f..e34b416 100644 --- a/vier_gewinnt/JS/gewinnErmitteln.js +++ b/vier_gewinnt/JS/gewinnErmitteln.js @@ -8,5 +8,4 @@ function gewinnErmitteln(spalte,reihe) { }else{ return 0; } - } \ No newline at end of file diff --git a/vier_gewinnt/JS/gewinnErmitteln_mock.js b/vier_gewinnt/JS/gewinnErmitteln_mock.js index 4240db7..c26eb19 100644 --- a/vier_gewinnt/JS/gewinnErmitteln_mock.js +++ b/vier_gewinnt/JS/gewinnErmitteln_mock.js @@ -7,6 +7,5 @@ function gewinnErmittelnMock() { if (vierInSpalte([0, 0, 0, 0, 0, 0]) === 2 || vierInReihe([0, 0, 0, 0, 0, 0, 0]) === 2 || vierDiagonalRechts([[0, 0, 0, 0, 0, 0], [1, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0], [1, 1, 2, 2, 0, 0], [1, 2, 2, 0, 0, 0], [1, 2, 0, 0, 0, 0], [2, 0, 0, 0, 0, 0]]) === 2 || vierDiagonalLinks([[0, 0, 0, 0, 0, 0], [1, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0], [1, 1, 2, 2, 0, 0], [1, 2, 2, 0, 0, 0], [1, 2, 0, 0, 0, 0], [2, 0, 0, 0, 0, 0]]) === 2) { return 2; } - } module.exports= gewinnErmittelnMock; \ No newline at end of file diff --git a/vier_gewinnt/JS/naechsteFreierPlatz.js b/vier_gewinnt/JS/naechsteFreierPlatz.js index eb430a1..80bb3d5 100644 --- a/vier_gewinnt/JS/naechsteFreierPlatz.js +++ b/vier_gewinnt/JS/naechsteFreierPlatz.js @@ -3,7 +3,6 @@ function naechsterFreierPlatz(arr) { if (arr[elem] === 0) { return elem+1; } - } } module.exports= naechsterFreierPlatz; \ No newline at end of file diff --git a/vier_gewinnt/JS/regeln.js b/vier_gewinnt/JS/regeln.js index de165db..38173af 100644 --- a/vier_gewinnt/JS/regeln.js +++ b/vier_gewinnt/JS/regeln.js @@ -6,5 +6,4 @@ function regelAnzeige() { document.getElementById("regelButton").innerHTML = "Regeln anzeigen"; document.getElementById("regeln").innerHTML = ""; } - } \ No newline at end of file diff --git a/vier_gewinnt/JS/spielerWechsel.js b/vier_gewinnt/JS/spielerWechsel.js index 6777d2b..632c1e1 100644 --- a/vier_gewinnt/JS/spielerWechsel.js +++ b/vier_gewinnt/JS/spielerWechsel.js @@ -5,5 +5,4 @@ function spielerWechsel() { }else{ document.getElementById("aktuellerSpieler").innerHTML = 1; } - } \ No newline at end of file diff --git a/vier_gewinnt/JS/steinSetzen.js b/vier_gewinnt/JS/steinSetzen.js index 2e863b6..4c07b0f 100644 --- a/vier_gewinnt/JS/steinSetzen.js +++ b/vier_gewinnt/JS/steinSetzen.js @@ -1,6 +1,8 @@ function steinSetzen(reihe, spalte) { var aktuellerSpieler = document.getElementById("aktuellerSpieler").innerHTML; var color; + var reihe = naechsterFreierPlatz(steinInSpalte(spalte)); + if (aktuellerSpieler == 1) { color = "green"; @@ -8,10 +10,8 @@ function steinSetzen(reihe, spalte) { color = "red"; } - var reihe = naechsterFreierPlatz(steinInSpalte(spalte)); - document.getElementById(reihe + "/" + spalte).style.backgroundColor= color; - gameReset(gewinnErmitteln(spalte, reihe)); - spielerWechsel(); + gameReset(gewinnErmitteln(spalte, reihe)); + spielerWechsel(); } \ No newline at end of file diff --git a/vier_gewinnt/JS/steineInReihe.js b/vier_gewinnt/JS/steineInReihe.js index d4ffd3c..d50dd1e 100644 --- a/vier_gewinnt/JS/steineInReihe.js +++ b/vier_gewinnt/JS/steineInReihe.js @@ -16,8 +16,5 @@ function steinInReihe(reiheImFeld) { reihe.push(0); } } - - return reihe; - - + return reihe; } \ No newline at end of file diff --git a/vier_gewinnt/JS/steineInSpalte.js b/vier_gewinnt/JS/steineInSpalte.js index 5ba41d2..66679f8 100644 --- a/vier_gewinnt/JS/steineInSpalte.js +++ b/vier_gewinnt/JS/steineInSpalte.js @@ -16,9 +16,6 @@ function steinInSpalte(spalte) { zeile.push(0); } } - - return zeile; - - + return zeile; } //module.exports = steinInSpalte; \ No newline at end of file diff --git a/vier_gewinnt/JS/vierDiagonalLinks.js b/vier_gewinnt/JS/vierDiagonalLinks.js index 18e8c79..b303326 100644 --- a/vier_gewinnt/JS/vierDiagonalLinks.js +++ b/vier_gewinnt/JS/vierDiagonalLinks.js @@ -2,9 +2,14 @@ function vierDiagonalLinks(allSpaltenArray) { var Gewinner =0; for (let i = 3; i <= allSpaltenArray.length; i++) { for (let j = 0; j <= allSpaltenArray[0].length -3; j++) { + + //Spieler Gruen if (allSpaltenArray[j][i]===1 && allSpaltenArray[j+1][i-1]===1 && allSpaltenArray[j+2][i-2]===1 && allSpaltenArray[j+3][i-3]===1) { Gewinner = 1; - }if (allSpaltenArray[j][i]===2 && allSpaltenArray[j+1][i-1]===2 && allSpaltenArray[j+2][i-2]===2 && allSpaltenArray[j+3][i-3]===2) { + } + + //Spieler Rot + if (allSpaltenArray[j][i]===2 && allSpaltenArray[j+1][i-1]===2 && allSpaltenArray[j+2][i-2]===2 && allSpaltenArray[j+3][i-3]===2) { Gewinner = 2; } } diff --git a/vier_gewinnt/JS/vierDiagonalRechts.js b/vier_gewinnt/JS/vierDiagonalRechts.js index 81d1a29..e25ffa3 100644 --- a/vier_gewinnt/JS/vierDiagonalRechts.js +++ b/vier_gewinnt/JS/vierDiagonalRechts.js @@ -2,9 +2,13 @@ function vierDiagonalRechts(allSpaltenArray) { var Gewinner =0; for (let i = 0; i <= allSpaltenArray.length -3; i++) { for (let j = 0; j <= allSpaltenArray[0].length -3; j++) { + + //Spieler Gruen if (allSpaltenArray[j][i]===1 && allSpaltenArray[j+1][i+1]===1 && allSpaltenArray[j+2][i+2]===1 && allSpaltenArray[j+3][i+3]===1) { Gewinner = 1; } + + //Spieler Rot if (allSpaltenArray[j][i]===2 && allSpaltenArray[j+1][i+1]===2 && allSpaltenArray[j+2][i+2]===2 && allSpaltenArray[j+3][i+3]===2) { Gewinner = 2; } diff --git a/vier_gewinnt/JS/vierInReihe.js b/vier_gewinnt/JS/vierInReihe.js index c62735b..243c0f7 100644 --- a/vier_gewinnt/JS/vierInReihe.js +++ b/vier_gewinnt/JS/vierInReihe.js @@ -1,11 +1,13 @@ function vierInReihe(arr){ var Gewinner = 0; + //Spieler Gruen for (let i = 0; i < arr.length - 3; i++) { if (arr[i] === 1 && arr[i + 1] === 1 && arr[i + 2] === 1 && arr[i + 3] === 1) { Gewinner = 1; } } + //Spieler Rot for (let i = 0; i < arr.length - 3; i++) { if (arr[i] === 2 && arr[i + 1] === 2 && arr[i + 2] === 2 && arr[i + 3] === 2) { diff --git a/vier_gewinnt/JS/vierInSpalte.js b/vier_gewinnt/JS/vierInSpalte.js index a900485..8611398 100644 --- a/vier_gewinnt/JS/vierInSpalte.js +++ b/vier_gewinnt/JS/vierInSpalte.js @@ -1,11 +1,13 @@ function vierInSpalte(arr){ var Gewinner = 0; + //Spieler Gruen for (let i = 0; i < arr.length - 3; i++) { if (arr[i] === 1 && arr[i + 1] === 1 && arr[i + 2] === 1 && arr[i + 3] === 1) { Gewinner = 1; } } + //Spieler Rot for (let i = 0; i < arr.length - 3; i++) { if (arr[i] === 2 && arr[i + 1] === 2 && arr[i + 2] === 2 && arr[i + 3] === 2) {