Ladet immer letzte werte,warum?

Noobie_Chan

Hy,

habe folgendes problem, wenn ich auf einen kreis drücke (2-8, 1 ist noch nicht fertig[summe2-8]) soll er mir unten kästchen anzeigen (was er auch tut) und weiter unten soll er mit werte übernehmen (was er auch tut aber immer "letzte" nimmt (werte vom kreis 8). Ich finde nicht heraus warum er immer den letzten nimmt.

Bitte um hilfe!

https://codepen.io/Noobie_Chan/pen/qBdjwWQ?editors=1010

Sempervivum

Der Kern der Sache scheint mir dieses zu sein:

Code

            function statsValue(pos) {

                nameValue[pos];
                console.log('2');
                console.log(pos);

            };

Ich vermute, Du willst die Einträge aus nameValue in die Felder unten übertragen. nameValue ist jedoch keine Funktion und die Anweisungen, die darin stehen, werden nicht ausgeführt durch nameValue[pos], sondern nur ein Mal, wenn Du diese Struktur definierst. Vermutlich würde dies tun, was Du vor hast:

Code

             const nameValue = {
                knightsvalue: {
                    hp: 1520472,
                    Atk: 17632,
                    // ebenso alle weiteren Eintraege aendern
                    document.getElementById('P_Def').innerText = 7832,
                    document.getElementById('M_Def').innerText = 6096,
                    document.getElementById('Mp_Recovery_Atk').innerText = 210,
                    document.getElementById('Crit').innerText = 150,
                    document.getElementById('Acc').innerText = 100,
                    document.getElementById('Cc_Resist').innerText = 150,
                    document.getElementById('Penetration').innerText = 150,
                    document.getElementById('P_Dodge').innerText = 250,
                    document.getElementById('M_Dodge').innerText = 250,
                    document.getElementById('Atk_Speed').innerText = 1000
                },
                warriorsvalue: {
                    // alle Elemente wie oben aendern
                    document.getElementById('Hp').innerText = 1291376,
                    document.getElementById('Atk').innerText = 20032,
                    document.getElementById('P_Def').innerText = 6528,
                    document.getElementById('M_Def').innerText = 7832,
                    document.getElementById('Mp_Recovery_Atk').innerText = 235,
                    document.getElementById('Acc').innerText = 100,
                    document.getElementById('Crit').innerText = 150,
                    document.getElementById('Penetration').innerText = 150,
                    document.getElementById('P_Dodge').innerText = 100,
                    document.getElementById('M_Dodge').innerText = 100,
                    document.getElementById('Atk_Speed').innerText = 1000,
                    document.getElementById('Cc_Resist').innerText = 150
                },
            };

Alles anzeigen

Und dann in der Funktion:

Code

            function statsValue(pos) {

                //nameValue[pos];
                var obj = nameValue[pos]
                for (var property in obj) {
                    if (obj.hasOwnProperty(property)) {
                        document.getElementById(property).innerText = obj[property];
                    }
                }
                console.log('2');
                console.log(pos);

            };

Alles anzeigen

Ungetestet, versuche es mal so.

Sempervivum

PS: Jetzt getestet und war noch fehlerhaft. Die Funktion muss so aussehen:

Code

            function statsValue(pos) {
                //nameValue[pos];
                var obj = nameValue[pos + 'value'];
                for (var property in obj) {
                    if (obj.hasOwnProperty(property)) {
                        document.getElementById(property).innerText = obj[property];
                    }
                }
                console.log('2');
                console.log(pos);
            };

Alles anzeigen

Noobie_Chan

danke für deine hilfe, es funktioniert genau so wie es soll.

Meine erste idee war eh so:

Code

const nameValue = {
    knightsValue: {
      Hp: 1520472,
      Atk: 17632,
      P_Def: 7832,
      M_Def: 6096,
      Mp_Recovery_Atk: 210,
      Crit: 150,
      Acc: 100,
      Cc_Resist: 150,
      Penetration: 150,
      P_Dodge: 250,
      P_Dodge: 250,
      Atk_Speed: 1000
    },
};

Alles anzeigen

hatte probleme mit der schreibweise wie man es schreibt. wusste nicht das ich es einfach so machen kann.

das ist nur neu:

Code

hasOwnProperty(property)  // woher kommt das (property)?

muss ich mir mal genau anschauen, ist ein vergleich ob es gleich ist? (true, false)?

warum verwendest du hier var? (hab "gelernt" let zu benutzen, var in ausnahmefällen, "größerer scope-breich" trifft das hier zu?)

Sempervivum

Was var, let und const betrifft, hast Du vollkommen Recht, ich hatte den Code von hier:

https://zellwk.com/blog/looping-through-js-objects/

und darauf nicht so geachtet.

Die Sache mit hasOwnProperty ist hier beschrieben:

https://wiki.selfhtml.org/wiki/JavaScrip…ung_in_Schleife

Jetzt mitmachen!