Button funktioniert nicht mit HTML und Javascript

Question

Ich versuche, einige Tasten zu machen, und sie funktionieren nicht. Ich bin mir sehr sicher, dass ich mit den Knöpfen alles richtig gemacht habe. Ich denke, es könnte etwas mit den Javascript-Funktionen zu tun haben, die sie aufrufen, daher habe ich auch diesen Teil meines Codes hinzugefügt. Wenn Sie meinen vollständigen Code sehen möchten, gehen Sie zu Github in meinem Profil TheNumnut unter Wars-Of-Shares.

<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> 

Wenn jemand mein Problem lösen kann, werde ich es sehr schätzen. Mein vollständiger Code wird auf dem Link angehängt: https://github.com/TheNumnut/Wars-of-Shares/blob/master/Market/index.html

UPDATE: Die Schaltfläche öffnet einen neuen Link und Warnungen. Allerdings wird es sagen: "Sie haben NaN-Aktien". Und jedes Mal, wenn ich darauf klicke, wird kein Element zu den Freigaben hinzugefügt.

Answer 1

Problem 1: Ihr Knopf onClick verwendet 4 Anführungszeichen. Statt

tun

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

entweder entkommen die Anführungszeichen \" anstelle von " oder verwenden ' verwenden.

Problem 2: Ihre JS-Funktion OpenTab() fehlt eine schließende geschweifte Klammer.

Problem 3: Ihre Variable shares ist keine Nummer. Sie erhalten den ganzzahligen Teil des Werts return von setCookie(), der keine Werte zurückgibt. (Ich sehe auch nicht, wie sich das Setzen von Cookies auf Aktien bezieht). Fügen Sie eine return yourValueHere hinzu oder legen Sie sie manuell fest.