From f458ed64c5e00baae1ea3fa193649663352a3ad4 Mon Sep 17 00:00:00 2001 From: PIRADATA <16529503+piradata@users.noreply.github.com> Date: Tue, 9 Aug 2022 04:59:51 -0300 Subject: [PATCH] Remove code that is no more necessary for validating .strobe selector cheat checkResults functions already prevents cheating with ".strobe" selector by comparing `$(".table").html() == ruleTable.html()` because `ruleTable.find(rule).addClass("strobe");` will never find anything if dependent on the .strobe class as `ruleTable.find(".strobe").removeClass("strobe");` removes the class right before --- js/restaurant.js | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/js/restaurant.js b/js/restaurant.js index 999b8f7..f3f8bd7 100644 --- a/js/restaurant.js +++ b/js/restaurant.js @@ -362,11 +362,6 @@ function resetTable(){ function fireRule(rule) { - // prevent cheating - if(rule === ".strobe") { - rule = null; - } - $(".shake").removeClass("shake"); $(".strobe,.clean,.shake").each(function(){ @@ -410,7 +405,7 @@ function fireRule(rule) { } if(ruleSelected.length == levelSelected.length && ruleSelected.length > 0){ - win = checkResults(ruleSelected,levelSelected,rule); + win = checkResults(rule); } if(win){ @@ -519,7 +514,7 @@ function winGame(){ resetTable(); } -function checkResults(ruleSelected,levelSelected,rule){ +function checkResults(rule){ var ruleTable = $(".table").clone(); ruleTable.find(".strobe").removeClass("strobe"); ruleTable.find(rule).addClass("strobe");