2016-11-18 67 views
-1

我想制作一些按钮,但它们不起作用。我非常肯定,我已经做好了一切与按钮正确。我认为这可能与他们所调用的Javascript函数有关,因此我也添加了我的代码的一部分。 如果您想查看我的完整代码,请转到我的个人资料TheNumnut下的Github下的Wars-Of-Shares。不支持HTML和Javascript的按钮

<button onclick="buyshare()">Buy Share</button> 
<button onclick="OpenTab("https://thenumnut.github.io/Wars-of-Shares/Profile")">Profile</button> 

<script> 
function buyshare() { 
    shares ++; 
    setCookie("shares", shares, 365); 
    alert("You have " + shares + " shares"); 
    window.open("https://thenumnut.github.io/Wars-of-Shares/Market"); 
} 

function OpenTab(x) { 
    window.open("https://thenumnut.github.io/Wars-of-Shares/Profile"); 
</script> 

如果有人能解决我的问题,我将非常感激。我完整的代码附加链接:https://github.com/TheNumnut/Wars-of-Shares/blob/master/Market/index.html

UPDATE: 该按钮并打开新的连接和警报。但是,它会说:“你有NaN股票”。只要我点击它,它就不会添加一个共享。

+0

你在'OpenTab'函数的末尾忘了'}'。顺便说一句,你的浏览器的开发者控制台会告诉你这个(或者至少那个错误在那条线上)。 – ceejayoz

+0

但它不会将股票加1。谢谢,它现在打开新标签和警报。 – Elodin

+0

所以,做一些调试。现在,它说你有NaN股票。不是一个数。找到您将股份设置为非数字并将其修复的位置。 – ceejayoz

回答

1

问题1:您的按钮onClick使用4个引号。而不是做

onclick="OpenTab("https://thenumnut.github.io/Wars-of-Shares/Profile")" 

任逃生使用\"引号代替"或使用'

问题2:你的JS功能OpenTab()缺少右大括号。

问题3:您的变量shares不是一个数字。您得到returnsetCookie()的整数部分,该值不返回任何值。 (我没有看到如何设置cookies与股票有关)。添加return yourValueHere或手动设置。

+0

转到我的Github。你说的是假设这是我的完整代码,而不是。请看看我的Github。 – Elodin

+0

我做到了。我原本有更多的问题,但删除他们,因为看到他们不是完整的代码中的问题。另外,下次只发布完整的代码。摘录是错误的,因为它缺少一些部分。 – k97513