2016-11-18 6 views
-1

ボタンをいくつか作ろうとしていて、うまくいきません。私はボタンですべてを正しく済ませたと確信しています。私はそれが彼らが呼び出すJavascript関数と関係があるかもしれないと思っているので、私はコードのその部分も追加しました。 私の完全なコードを表示したい場合は、Wows-Of-Sharesの下の私のプロフィールTheNumnutでGithubに行きます。ボタンが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株を持っていると言います。 Not-a-number。共有を非番号に設定する場所を見つけて修正します。 – ceejayoz

答えて

1

問題1:ボタンonClickは4つの引用符を使用します。代わりに

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

をすることのいずれか"または使用'の代わりに\"を使用して引用符をエスケープします。

問題2: JS関数OpenTab()には中かっこがありません。

問題3:変数sharesは数字ではありません。 return値の整数部分がsetCookie()になっていますが、値は返されません。 (私はクッキーの設定がどのように株式に関連するか見ていない)。 return yourValueHereを追加するか、手動で設定してください。

+0

私のギターに行ってください。あなたが言ったことは、そうではない私の完全なコードだと仮定しています。私のギターを見てください。 – Elodin

+0

しました。もともと私はより多くの問題を抱えていましたが、完全なコードでは問題ではないと見て、それらを削除しました。また、次回はフルコードのみを投稿します。抜粋はいくつかの部分が欠けていると考えて誤解を招いています。 – k97513

関連する問題