How can I edit an autoclicker Javascript bookmarklet to allow two autoclickers at the same time?

Question

I am trying to edit a Javascript bookmarklet that creates an autoclicker(using the click event) to allow 2 clickers on the page at the same time. I am using the web version of Visual Studio Code and testing it on the idle game Cookie Clicker.

Sorry for the mess of comments and if it looks bad. This is my first time trying to use JS but I'm able to figure it out because I'm in a programming course using Java.

Here is the entire code:

//javascript: ...[removed to look cleaner]
//^original^

//GOAL: I want two autoclickers to be able to run at the same time
//SOLUTION(maybe): change if statement checking if there is already an autoclicker

// javascript:
var DELAY = 1;
var autoClickerStyleElement = document.createElement("style");//here
autoClickerStyleElement.innerHTML="*{cursor: crosshair !important;}";//to
document.body.appendChild(autoClickerStyleElement);//here sets up style/crosshair
function addClicker(e) {
    if(!e.isTrusted) {//if click event is not trusted
        return;}//do nothing
//THIS IS CODE THAT WAS ALREADY HERE
//  if(e.target.classList.contains("auto-clicker-target")) {//if there's already one
//      e.target.classList.remove("auto-clicker-target");}//remove it
//  else {
//      e.target.classList.add("auto-clicker-target");}//add target class to the click event's classList

//THIS IS AN EDITED VERSION OF THE CODE
    if(e.target.classList.contains("auto-clicker-target1") && !(e.target.classList.contains("auto-clicker-target2"))) {//if there's 1st & not 2nd
        e.target.classList.add("auto-clicker-target2");}//add a 2nd
    //if 1st and 2nd exist, remove both
    else if(e.target.classList.contains("auto-clicker-target1") && e.target.classList.contains("auto-clicker-target2")){//if it has both
        e.target.classList.remove("auto-clicker-target1");//remove 1st
        e.target.classList.remove("auto-clicker-target2");}//remove 2nd

    //my code compacted: ...[removed to look cleaner]
    //rewritten full: ...[removed to look cleaner]

    //WHY NOT WORK?
    //MAIN EDITED CODE END

    document.body.removeChild(autoClickerStyleElement);//make crosshair disappear
    document.body.removeEventListener("click", addClicker);//add clicker
    e.preventDefault();
    autoClick(e.target);//run autoclick where crosshair is clicked
    }//end of addClicker
function autoClick(element) {
    if(element.classList.contains("auto-clicker-target1")) {//i just duped this part and changed the name
        element.click();//simulates click
        setTimeout(function()//set delay for click
        { autoClick(element); }, DELAY);
    }//ADDED:
    if(element.classList.contains("auto-clicker-target2")) {
        element.click();
        setTimeout(function()
        { autoClick(element); }, DELAY);
    }

}
document.body.addEventListener("click", addClicker, 0);//waits for my click to add autoclicker

This is the code I am specifically focusing on:

//THIS IS CODE THAT WAS ALREADY HERE
//  if(e.target.classList.contains("auto-clicker-target")) {//if there's already one
//      e.target.classList.remove("auto-clicker-target");}//remove it
//  else {
//      e.target.classList.add("auto-clicker-target");}//add target class to the click event's classList

I tried changing it to this:

//THIS IS AN EDITED VERSION OF THE CODE
    if(e.target.classList.contains("auto-clicker-target1") && !(e.target.classList.contains("auto-clicker-target2"))) {//if there's 1st & not 2nd
        e.target.classList.add("auto-clicker-target2");}//add a 2nd
    //if 1st and 2nd exist, remove both
    else if(e.target.classList.contains("auto-clicker-target1") &&
e.target.classList.contains("auto-clicker-target2")){//if it has both
        e.target.classList.remove("auto-clicker-target1");//remove 1st
        e.target.classList.remove("auto-clicker-target2");}//remove 2nd

I big screwed up somewhere because now it won't even show the crosshair. Please help me fix this.

How can I edit an autoclicker Javascript bookmarklet to allow two autoclickers at the same time?

Answers (1)

Related Questions