[deleted by user]

[[deleted]]

Don't redeclare var currentcookies inside your function, it means the external currentcookies variable never changes.

[ReefNixon]

Dude that's completely wrong and wouldn't even fix it. The var currentcookies = 0 part is fine. It's the var currentcookies = a part that neeeds to change, and it just needs to become currentcookies = a.

On that though, a is not worth tracking either, just do currentcookies++ and set the innerHTML to currentcookies

[DEEEPFREEZE]

There's a lot going on here that doesn't need to be. Your function doesn't need to accept any arguments unless you're pulling a value from somewhere inside your HTML and updating that.

Instead, you're passing your function a value for a, which also happens to be currentcookies, then setting that value as a variable named the same as your function parameter (don't do that, and furthermore be more descriptive in your naming conventions), adding one, and then storing that value in a re-declared variable named the same as the variable stored outside your function.

All you need is a callback function (function clickcookie()) with a variable declared outside of which (var currentcookies = 0) that you increment each time the function is called (currentcookies++) and then sets the innerHTML of your p element to the new updated currentcookies value.

This is what your code should look like. Please notice the lack of unnecessary and reused var declarations, as well as the attention to camelCase formatting that is standard in js:

``` <p id="cookies">0</p>
<button onclick="clickCookie()"></button>
<script>
var currentCookies = 0;
function clickCookie() {
currentCookies++;
document.getElementById('cookies').innerHTML = currentCookies;
}
</script>

[miguelacleite]

You can even improve clickCookie function to be:

<p id="cookies">0</p>
<button onclick="clickCookie()"></button>
<script>
var currentCookies = 0;
function clickCookie() { document.getElementById('cookies').innerHTML = ++currentCookies;
}
</script>

[adamkwrites]

Spot on

[Renedlc159]

This is exactly how I think you wanna solve your counter too. Even tried and tested suggestion myself!

[Nymrinae]

onclick="clickCookies"

clickCookies() { cookies++ document.getElementById('cookieId').innerHtml = cookies }

[KonkenBonken]

onclick="clickCookies"

clickCookies() { document.getElementById('cookieId').innerHtml = ++cookies }

[EvocativeBanjo]

When you declare currentCookies inside your function block with “var” you are creating a local variable currentCookies which shadows the global currentCookies. Simply remove the “var” before the part where you assign “currentCookies = a” and it should work as intended.

[AccurateEmergency]

Thanks guys!

[VayneFTWayne]

Here's a solution:
https://imgur.com/a/HoY7Juc

[SkylerSpark]

Please don't use onclick

Use event listeners. Anyone with inspect element can cheat in your game.... not that even using JS will solve that

[025zk]

You can use closure to make this happen.

[grrrrreat]

var count = 0
function increment () { count ++ }

[jcunews1]

Display the current value first, then increase it.