Как исправить код, чтобы он заработал?

Question

[dmitrii000]

У меня есть код, который должен при выпадение числа 1, 2 или 3 пополнять переменную cash на 200, но он не работает. Что нужно исправить?

<div class="container-balance">
    <h3><script>
        let cash = 10000
        document.write("Ваш баланс:" + cash + "руб")
    </script></h3>
</div>
<div class="container-spin">
    <script>
        let cash = 10000

        function Random(max) {
            return Math.cell(Math.random() * max)
        }

        let max = Random(9)

        function spin() {
            Random(9)
            if (max == 1 || max == 2 || max == 3) {
            cash = cash + 200
            }
        }
    </script>	
    <button onclick="Random()"><img src="img/spin (2).jpg" alt="spin" height="98px" 
    width="100px"></button>
</div>

Answer 1

[Сергей Соколов]

HTML отдельно, JS отдельно.
Слушателей событий добавлять через addEventListener()
Забыть про document.write(). Вместо этого использовать element.innerText = "Хабр"; например.
Обновлять запись баланса понадобится из двух разных мест: в начале, и при каждом выигрыше. Поэтому – в функцию.

Comments

[dmitrii000]

Я недавно начал изучать js и не совсем понимаю, что значит фраза html отдельно, js отдельно, надо в теге script указать ссылку на отдельный файл с кодом?

[Сергей Соколов]

dmitrii000, Да, лучше всего в отдельном файле .js держать все скрипты, и подгружать их в самом конце страницы, перед закрывающим тегом </body>
Или там же, в конце страницы, после всей разметки, в тегах <script>

Но не в самой разметке, как у вас в вопросе document.write, и script перед кнопкой.

[dmitrii000]

Сергей Соколов,
в левом нижнем прямоугольнике должен появляться текст с балансом, до того, как я поменял код, он был, после он исчез,
что нужно поменять

[Сергей Соколов]

dmitrii000, не знаю. Смотрите, есть ли ошибки в консоли, например.

Answer 2

[Crone01]

function spin() {
            //вот тут
            max = Random(9)
            if (max == 1 || max == 2 || max == 3) {
            cash = cash + 200
            }
        }

Comments

[Crone01]

ну и тут ты должен вызывать не Random() а spin()

<button onclick="Random()"><img src="img/spin (2).jpg" alt="spin" height="98px"

[Crone01]

после чего должно быть какое-то действие, вывод этого самого cash куда-нибудь
например
console.log(cash)
после того как пополнишь на 200

[dmitrii000]

не помогло

[dmitrii000]

Crone01, Куда вставить console.log в функцию?