2017-01-18 3 views
1

У меня есть три кнопки внизу, а кнопка с пометкой +1, должна увеличивать кошелек на один. Кнопка с пометкой Transfer money to Bank должна переместить сумму кошелька в банк и сбросить кошелек на 0. Но ничего не вызывается или не работает. Я абсолютно застрял. Пожалуйста помоги.Я не могу понять, почему мой код не выполняется

<html> 
    <head> 
     <title>Area 52</title> 
    </head> 
    <body> 
     <h1><center>Area 52</center></h1> 
     <p><center><p Id="name">User</p><p>Is currently logged on</p></center><p> 
    <script> 
     var money=0 
     var bank=0 
     var Money_handler(money,bank,op){ 
     alert("!") 
     if (op=="add"){ 
      money+=1 
      document.getElementById('wallet').innerHTML = money; 
      return money; 
     } 
     if (op=="bank"){ 
      bank=bank+money 
      document.getElementById('bank').innerHTML = bank; 
      money=0 
      document.getElementById('wallet').innerHTML = money; 
      return money 
     } 

     } 
     var namer=function(){ 
     var x=prompt("What is your name?"); 
     document.getElementById('name').innerHTML = x; 
     } 
     </script> 
    <center> 
     <button type="button" onclick=money=Money_handler(money,bank,"add")>+1</button> 
     <button type="button" onclick=namer()>Enter username</button> 
     <button type="button" onclick=Money_handler(money,bank,"bank")>Transfer money to Bank</button> 
     <p Id="wallet">Wallet: $0</p> 
     <p Id="bank">Bank: $0</p> 
    </center> 
    </body> 
</html> 
+4

У вас много синтаксических ошибок и недостающих полуколоний и т. Д. Хорошим местом для начала является использование javascript-консоли вашего браузера, и это покажет вам много из этих проблем. Вы также должны переключиться на текстовый редактор, созданный для записи JS, он выделит проблемы синтаксиса для вас. –

+1

Посмотрите на консоль ошибок вашего браузера. Код имеет несколько синтаксических ошибок. – David

+0

'var Money_handler (money, bank, op) {' ... '}' недействителен. Используйте [консоль браузера] (http://webmasters.stackexchange.com/q/8525) (нажмите 'F12') и прочитайте ошибки. Используйте [JSHint] (http://jshint.com/), чтобы немедленно найти проблемы с вашим кодом. – Xufox

ответ

0

Вы просто использовать вар вместо функции:

function Money_handler(money,bank,op){ 

EDIT: При обновлении денег вы забыли остальную часть строки:

.innerHTML = "Wallet: $"+money; 

You пришлось изменить это на всех трех местах с кошельком/банком.

+0

ах, lol я так догнал другие части кода, я полностью забыл добавить функциональную часть, ура! – Mental

+0

Если вы хотите использовать 'var', вы также можете сделать' var money_handler = function (money, bank, op) {...} 'Или вы могли бы также использовать новые лямбда-функции:' var money_handler = (деньги, банк , op) => {...} 'Ваш выбор. – user3685285

+0

Я знал об этом изменении, я не потрудился положить его, пока я не смог исправить исходную проблему – Mental

0

Это должно сработать. Помимо ключевого слова function вы также пропустили кавычки для значения атрибута onclick.

<html> 
 
<head> 
 
    <title>Area 52</title> 
 
</head> 
 
<body> 
 
    <h1><center>Area 52</center></h1> 
 
    <p><center><p Id="name">User</p><p>Is currently logged on</p></center><p> 
 
<script> 
 
var money = 0 
 
var bank = 0 
 
var Money_handler= function(money, bank, op) { 
 
    alert("!") 
 
    if (op == "add") { 
 
    money += 1 
 
    document.getElementById('wallet').innerHTML = money; 
 
    return money; 
 
    } 
 
    if (op == "bank") { 
 
    bank = bank + money 
 
    document.getElementById('bank').innerHTML = bank; 
 
    money = 0 
 
    document.getElementById('wallet').innerHTML = money; 
 
    return money 
 
    } 
 

 
} 
 
var namer = function() { 
 
    var x = prompt("What is your name?"); 
 
    document.getElementById('name').innerHTML = x; 
 
} 
 
</script> 
 
<center> 
 
    <button type="button" onclick="Money_handler(money,bank,'add');">+1</button> 
 
    <button type="button" onclick="namer()">Enter username</button> 
 
    <button type="button" onclick="Money_handler(money,bank,'bank')">Transfer money to Bank</button> 
 
    <p Id="wallet">Wallet: $0</p> 
 
    <p Id="bank">Bank: $0</p> 
 
</center> 
 
</body> 
 
</html>

Примечание: я только исправил синтаксических недостатки. Я думаю, вы должны работать над логикой.

Смежные вопросы