nice

im a mean person so I will proceed to brutally critique your code now

1. NEVER NEVER NEVER NEVER NEVER NEVER NEVER NEVER USE INNERHTML - It directly sets the HTML of elements which means malicious users can insert <script>s into your application if you were making something like a chatroom. use innerText instead
2. nobody uses localStorage.getItem just use localStorage[item] or localStorage.item also you don't need to check if something === null (btw kudos for you for using === and not ==) just do if (!something). you can take advantage of these truthy and falsy values which evaluate to true and false, as their name suggests and they are very useful for writing neat code in js
3. instead of having giant if else chains you can use ternary which is useful. your first block from lines 2-6 can be condensed into this: let score = localStorage.score ? + localStorage.score : 0 (you can use + instead of Number() to convert strings to numbers as explained in my post). ternary works a ? (do something if a is true) : (do something if a is false) and you can also use it for assignment x = a ? (assign if a is true) : (assign if a is false) or if you are even bigger brain you can use let score = + localStorage.score || 0 (this works because || will do the first thing, and if the first thing isnt true or truthy it will do the second thing)
4. please try to put spaces in between brackets everytime I see }else{ I die a little bit inside :(
5. you should always use let and const for declaring variables its a better practice since 1) they are both block scoped and 2) they provide a very very slight performance advantage. block scoped means that they only apply to the block they are in or the segment of the code between the curly braces. if I were to use var, it would still exist: if (true) { var a = "hi" } // a still exists here! but this wouldn't be the case with let. it's mainly just a good practice since almost nobody uses var anymore, it has a lot of scoping issues and its weird.
6. never declare variables without let or const it makes them properties of window and immediately global just very weird
7. dont spam document.getElementById() you can assign a variable or something
8. when doing string concatonation you should use template literals, they are just much nicer: `hello ${this} is my message` you can insert any valid javascript expression inside ${} its just like templating as the name suggests
   good luck on any future projects you hope to make with javascript!

Y E S

code go fish!

EXPLOIT

Spam click the cacti

setInterval(()=>{document.getElementById("cactus").click() });

Usage:

Just paste it in console ;)

Change cacti per sec

function changePerSec(amount){localStorage.perSec=amount;window.location.reload();}

Usage:

changePerSec(1000000) - This will change per sec to 1 million!

Change cacti per click

function changePerClick(amount){localStorage.perClick=amount;window.location.reload();}

Usage

changePerClick(1000000) - This will change per click to 1 million!

Change cacti amount

function changeCacti(amount){localStorage.score=amount;window.location.reload();}

Usage

changeCacti(1000000) - This will change cacti to 1 million!

All Together

setInterval(()=>{document.getElementById("cactus").click() });function changePerSec(amount){localStorage.perSec=amount;window.location.reload();}function changePerClick(amount){localStorage.perClick=amount;window.location.reload();}function changeCacti(amount){localStorage.score=amount;window.location.reload();}

hmm... time for an EXPLOIT!!!!

Awesome!

ummmmm are you sure about these upgrade increments?

Hope this becomes as popular as my simple dungeon!

BY THE WAY Chezcoder - It was very easy to hack

hehe i have touchscreen pc

make it so that everytime you click the cactus, it moves its arm like your pfp

this is op

Just a suggestion, my friend made a cookie clicker thing, and one good thing about it was that if you opened it in another tab, the tab would show how many Cookies you had, maybe add it?

Love your game

xdotool be like:

nice game

How do you change the website logo thing

Amazing! Much quicker with an autoclicker at 2500 clicks per second ;-;

cacti: 1,170,083,288

Cacti per click: 20,102

Cacti per second: 22,681

Like a boss.

this is great!!!
I realized I left my tabs open overnight so when I woke up, I had 3 billion+ cacti. :)

I find this easier than cookie clicker

This is cool! I got to 10,000,000 and then gave up, I would love to see some more options in the store

Amazing!!!

WOAH, I left yesterday.. and now....

Well, time to spam buy stuff...

:O The cactus moves now! Awesome!

I just got on after some sleep, and I now have a billion cacti :)

I love the pfp!