From 03362219ee5a0cb74fcd37acfc6e9381019e1ff5 Mon Sep 17 00:00:00 2001 From: kallicain Date: Wed, 8 Feb 2023 19:08:18 -0600 Subject: [PATCH 1/4] pushing up --- index.html | 20 ++++++++++++++++---- scripts.js | 6 ++++-- 2 files changed, 20 insertions(+), 6 deletions(-) diff --git a/index.html b/index.html index 19d0dee..e6265dd 100644 --- a/index.html +++ b/index.html @@ -3,9 +3,9 @@ Tic Tac Toe - + - + @@ -20,11 +20,23 @@

Welcome to Tic Tac Toe

- - + + + + + + + + + + + + + + diff --git a/scripts.js b/scripts.js index ba09e74..5f17c60 100644 --- a/scripts.js +++ b/scripts.js @@ -44,7 +44,7 @@ const addMarker = (id) => { console.log(`Therefore, a "${currentMarker}" should be placed in the square with the id: ${id}`) // @TODO-2: Build a line of code that will set the innerHTML property of the element that was clicked to the "currentMarker" - + document.getElementById(id).innerHTML = currentMarker // @TODO-2.5: MIX & MATCH, You will need the following pieces of code to build that line: // = currentMarker // .getElementById(id) @@ -65,11 +65,13 @@ const addMarker = (id) => { // This "changeMarker" function changes "X" to "O" in the "currentMarker" variable or "O" to "X" const changeMarker = () => { +console.log('BEFORE currentMarker',currentMarker) if(currentMarker === "X"){ currentMarker = "O" } else { currentMarker = "X" } + console.log('AFTER currentMarker',currentMarker) } @@ -86,7 +88,7 @@ const resetBoard = () => { // @TODO-3: To make your "Restart" button work you'll need to build a line of code here that: // collects all of the "td" elements into an HTML Collection: https://www.w3schools.com/jsref/dom_obj_htmlcollection.asp - + const squares = document.getElementsByTagName("TD") // @TODO-3.5: MIX & MATCH, You will need the following pieces of code to build that line: // squares // .getElementsByTagName("TD") From f6f1c5506bee989b953c53430a9090819c1068e8 Mon Sep 17 00:00:00 2001 From: kallicain Date: Wed, 22 Feb 2023 20:25:21 -0600 Subject: [PATCH 2/4] added functions for checking winner --- index.html | 36 ++++++----- scripts.js | 187 +++++++++++++++++++++++++++-------------------------- 2 files changed, 115 insertions(+), 108 deletions(-) diff --git a/index.html b/index.html index e6265dd..a460c57 100644 --- a/index.html +++ b/index.html @@ -1,11 +1,11 @@ - + Tic Tac Toe - - - + + + @@ -14,29 +14,35 @@

Welcome to Tic Tac Toe

Get ready to play!

- + - - - + + + - - - + + + - - - + + + -
diff --git a/scripts.js b/scripts.js index 5f17c60..68498c9 100644 --- a/scripts.js +++ b/scripts.js @@ -1,108 +1,109 @@ -// *********************** -// INSTRUCTIONS -// *********************** +let currentMarker = "X"; +let board = [ + ["", "", ""], + ["", "", ""], + ["", "", ""], +]; -// 1. Read the code below one block at a time. -// 2. Look for the @TODOs, and figure out how to fix them. - // next to each @TODO you will find tasks that need to be finished - -// The variable will change from X to O based on what player turn it is. We need to hold this so we can place an X or O on the board when they're clicked. -let currentMarker = 'X' - - - - -// this "handleClick" function is called when a box is clicked. Here, "element" will hold the same value as "this" does in the HTML. -// "this" is a special word in JS but "element" could have been "thing" or "el" or whatever we wanted it to be as long as we use it again in the "console.log" statement const handleClick = (element) => { + console.log(`The element you clicked on has an id: ${element.id}`); - // this uses the "log" method on the "console" to log out the element's id so we can see it with our human eyes - console.log(`The element you clicked on has an id: ${element.id}`) - - // this next line prevents an X being changed to an O or an O being changed to an X by... - // checking to see if the square clicked has anything in it, if not continue - if(!document.getElementById(element.id).innerHTML){ - addMarker(element.id) + if (!document.getElementById(element.id).innerHTML) { + addMarker(element.id); } -} - - - - - +}; - - - - - -// this function places the "currentMarker" inside the HTML element that was clicked and calls the "changeMarker" function. const addMarker = (id) => { + console.log(`*** The current marker is: ${currentMarker}. ***`); + console.log( + `Therefore, a "${currentMarker}" should be placed in the square with the id: ${id}` + ); + const row = parseInt(id.charAt(0)); + console.log("row", row); + const column = parseInt(id.charAt(2)); + console.log("column", column); + board[row][column] = currentMarker; + document.getElementById(id).innerHTML = currentMarker; + + checkForWin(); +}; - // @TODO-1: Open the console tab in your Chrome Inspector Tool and click on the top-left square to see what's logged to the console. - console.log(`*** The current marker is: ${currentMarker}. ***`) - console.log(`Therefore, a "${currentMarker}" should be placed in the square with the id: ${id}`) - - // @TODO-2: Build a line of code that will set the innerHTML property of the element that was clicked to the "currentMarker" - document.getElementById(id).innerHTML = currentMarker - // @TODO-2.5: MIX & MATCH, You will need the following pieces of code to build that line: - // = currentMarker - // .getElementById(id) - // document - // .innerHTML - - changeMarker() -} - - - - - - - - - - -// This "changeMarker" function changes "X" to "O" in the "currentMarker" variable or "O" to "X" const changeMarker = () => { -console.log('BEFORE currentMarker',currentMarker) - if(currentMarker === "X"){ - currentMarker = "O" + console.log("BEFORE currentMarker", currentMarker); + if (currentMarker === "X") { + currentMarker = "O"; } else { - currentMarker = "X" + currentMarker = "X"; } - console.log('AFTER currentMarker',currentMarker) -} - - - - - - - - + console.log("AFTER currentMarker", currentMarker); +}; +const checkForWin = () => { + if (horizontalWin() || verticalWin() || diagonalWin()) { + window.alert(`Player ${currentMarker} won!`); + } else { + changeMarker(); + } +}; + +const horizontalWin = () => { + if ( + (board[0][0] == "X" && board[0][1] == "X" && board[0][2] == "X") || + (board[0][0] == "O" && board[0][1] == "O" && board[0][2] == "O") + ) { + return true; + } else if ( + (board[1][0] == "X" && board[1][1] == "X" && board[1][2] == "X") || + (board[1][0] == "O" && board[1][1] == "O" && board[1][2] == "O") + ) { + return true; + } else if ( + (board[2][0] == "X" && board[2][1] == "X" && board[2][2] == "X") || + (board[2][0] == "O" && board[2][1] == "O" && board[2][2] == "O") + ) { + return true; + } +}; + +const verticalWin = () => { + if ( + (board[0][0] == "X" && board[1][0] == "X" && board[2][0] == "X") || + (board[0][0] == "O" && board[1][0] == "O" && board[2][0] == "O") + ) { + return true; + } else if ( + (board[0][1] == "X" && board[1][1] == "X" && board[2][1] == "X") || + (board[0][1] == "O" && board[1][1] == "O" && board[2][1] == "O") + ) { + return true; + } else if ( + (board[0][2] == "X" && board[1][2] == "X" && board[2][2] == "X") || + (board[0][2] == "O" && board[1][2] == "O" && board[2][2] == "O") + ) { + return true; + } +}; + +const diagonalWin = () => { + if ( + (board[0][0] == "X" && board[1][1] == "X" && board[2][2] == "X") || + (board[0][0] == "O" && board[1][1] == "O" && board[2][2] == "O") + ) { + return true; + } else if ( + (board[2][2] == "X" && board[1][1] == "X" && board[1][1] == "X") || + (board[2][2] == "O" && board[1][1] == "O" && board[1][1] == "O") + ) { + return true; + } +}; -// This "resetBoard" function is called when the user clicks on the "Restart" button. const resetBoard = () => { - - // @TODO-3: To make your "Restart" button work you'll need to build a line of code here that: - // collects all of the "td" elements into an HTML Collection: https://www.w3schools.com/jsref/dom_obj_htmlcollection.asp - const squares = document.getElementsByTagName("TD") - // @TODO-3.5: MIX & MATCH, You will need the following pieces of code to build that line: - // squares - // .getElementsByTagName("TD") - // = - // document - // const - - // loops over the HTML Collection of TDs and clears out the Xs and Os - for (i=0; i < squares.length; i++) { + const squares = document.getElementsByTagName("TD"); - // will log out the id of each square as it loops over them. - console.log(squares[i].id) + for (i = 0; i < squares.length; i++) { + console.log(squares[i].id); - // sets the innerHTML to null to replace the "X" or "O" - squares[i].innerHTML = null - } -} \ No newline at end of file + squares[i].innerHTML = null; + } +}; From e90b8b6b87fe2997db5df63e00a0a5e27a06f941 Mon Sep 17 00:00:00 2001 From: kallicain Date: Wed, 22 Feb 2023 20:30:54 -0600 Subject: [PATCH 3/4] changes --- scripts.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts.js b/scripts.js index 68498c9..cd3dbe7 100644 --- a/scripts.js +++ b/scripts.js @@ -91,8 +91,8 @@ const diagonalWin = () => { ) { return true; } else if ( - (board[2][2] == "X" && board[1][1] == "X" && board[1][1] == "X") || - (board[2][2] == "O" && board[1][1] == "O" && board[1][1] == "O") + (board[2][2] == "X" && board[1][1] == "X" && board[0][0] == "X") || + (board[2][2] == "O" && board[1][1] == "O" && board[0][0] == "O") ) { return true; } From 4b733e9be92a14105a8b0046d282eb2edb4a7ea8 Mon Sep 17 00:00:00 2001 From: kallicain Date: Wed, 22 Feb 2023 20:50:20 -0600 Subject: [PATCH 4/4] finished reset --- scripts.js | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/scripts.js b/scripts.js index cd3dbe7..ad838a2 100644 --- a/scripts.js +++ b/scripts.js @@ -99,7 +99,15 @@ const diagonalWin = () => { }; const resetBoard = () => { + + board = [ + ["", "", ""], + ["", "", ""], + ["", "", ""], + ]; + const squares = document.getElementsByTagName("TD"); + for (i = 0; i < squares.length; i++) { console.log(squares[i].id);