Variable wird bei einer .each() leer geräumt

[eStar]

Hallo,

ich bin bei meinm script auf ein Problem gestoßen das meine Variable die ich einem Objekt hinzufüge leer geräumt wird.

Ich habe jetzt verschiedene Methoden ausprobiert, das ganze in ein Array verwandeln, statisch eintragungen zu machen, etc. aber immer leider ohne Erfolg.

Ausgangssituation :
Ich habe eine Art bewertungsseite für sich selbst, in dem Bereich geht es um die Sportliche aktivität am Tage. Man kann auswählen wie viele Minuten Mann zb. gejoggt ist und dann eine Selbstbewertung anhand 3er Smiley vornehmen.

Ich wähle eine Minuten Zahl zwischen 0,0-15,15-30 .... 60+ und dann eines der 3 Smiley , ,

Wenn ich das ganze Abspeichere läuft mein Script über mein HTML und sammelt alles ein was ich gemacht habe und fügt das einem Objekt hinzu.

Es fügt Jedoch nur den Wert des ersten Smileys hinzu obwohl mehrere angeklickt sind, in den rest schreibt er eine 0 rein.

//HTML:
<tr data-kategorie="ausdauer"><td colspan="4">Ausdauer
                    <span class="little">(Joggen,Schwimmen)</span></td>
                <td class="zeit-deritaet"><select name="movementtime"><option value="0">0</option><option value="15">0-15</option><option value="30">15-30</option><option value="45">30-45</option><option value="60">45-60</option><option value="u60">60+</option></select></td>
                <td class="bewertungs-smile" colspan="2"><span class="good-smile gut last-smile" data-wert="30"><img src="/theme/common/media/gut.png" width="18"></span>
                    <span class="neutral-smile mittel" data-wert="20"><img src="/theme/common/media/mittel.png" width="18"></span>
                    <span class="bad-smile schlecht active-smiley" data-wert="10"><img src="/theme/common/media/schlecht-aktiv.png" width="18"></span></td>
            </tr>
            <tr data-kategorie="fitness"><td colspan="4">Fitness
                    <span class="little">(Center, Kurs)</span></td>
                <td class="zeit-deritaet"><select name="movementtime"><option value="0">0</option><option value="15">0-15</option><option value="30">15-30</option><option value="45">30-45</option><option value="60">45-60</option><option value="u60">60+</option></select></td>
                <td class="bewertungs-smile" colspan="2"><span class="good-smile gut last-smile active-smiley" data-wert="30"><img src="/theme/common/media/gut-aktiv.png" width="18"></span>
                    <span class="neutral-smile mittel" data-wert="20"><img src="/theme/common/media/mittel.png" width="18"></span>
                    <span class="bad-smile schlecht" data-wert="10"><img src="/theme/common/media/schlecht.png" width="18"></span></td>
            </tr>
            <tr data-kategorie="krafttraining"><td colspan="4">Krafttraining</td>
                <td class="zeit-deritaet"><select name="movementtime"><option value="0">0</option><option value="15">0-15</option><option value="30">15-30</option><option value="45">30-45</option><option value="60">45-60</option><option value="u60">60+</option></select></td>
                <td class="bewertungs-smile" colspan="2"><span class="good-smile gut last-smile" data-wert="30"><img src="/theme/common/media/gut.png" width="18"></span>
                    <span class="neutral-smile mittel active-smiley" data-wert="20"><img src="/theme/common/media/mittel-aktiv.png" width="18"></span>
                    <span class="bad-smile schlecht" data-wert="10"><img src="/theme/common/media/schlecht.png" width="18"></span></td>
            </tr>
            <tr data-kategorie="sport"><td colspan="4">Spor
                    <span class="little">(z.B. Fussball,Tennis)</span></td>
                <td class="zeit-deritaet"><select name="movementtime"><option value="0">0</option><option value="15">0-15</option><option value="30">15-30</option><option value="45">30-45</option><option value="60">45-60</option><option value="u60">60+</option></select></td>
                <td class="bewertungs-smile" colspan="2"><span class="good-smile gut last-smile" data-wert="30"><img src="/theme/common/media/gut.png" width="18"></span>
                    <span class="neutral-smile mittel" data-wert="20"><img src="/theme/common/media/mittel.png" width="18"></span>
                    <span class="bad-smile schlecht" data-wert="10"><img src="/theme/common/media/schlecht.png" width="18"></span></td>
            </tr>
            <tr data-kategorie="beweglichkeit"><td colspan="4">Beweglichkeit
                    <span class="little">(Yoga,TaiChi)</span></td>
                <td class="zeit-deritaet"><select name="movementtime"><option value="0">0</option><option value="15">0-15</option><option value="30">15-30</option><option value="45">30-45</option><option value="60">45-60</option><option value="u60">60+</option></select></td>
                <td class="bewertungs-smile" colspan="2"><span class="good-smile gut last-smile" data-wert="30"><img src="/theme/common/media/gut.png" width="18"></span>
                    <span class="neutral-smile mittel" data-wert="20"><img src="/theme/common/media/mittel.png" width="18"></span>
                    <span class="bad-smile schlecht" data-wert="10"><img src="/theme/common/media/schlecht.png" width="18"></span></td>
            </tr> 

//jQuery:
function getAllValuesMovement(){
    var SmileValues = {};
    $("table.movement-kategorie tr").each(function(){
        $(this).children("td.bewertungs-smile").children().each(function(){
            childClass = $(this).attr("class").split(" ");
            childkategorie = $(this).parent().parent().data("kategorie");
            if($(this).hasClass("active-smiley"))
            {
                wert = childClass[0];
            }else {
                wert = 0;
            }
            zeitWert = $(this).parent().parent().children("td.zeit-deritaet").children().val();
            SmileValues[childkategorie] = { 0 : wert, 1 : zeitWert};
        });
    });
    console.log(SmileValues);
    return SmileValues;
} 

//DOM Ausgabe:

ausdauer
    
    Object { 0="bad-smile", 1="45"}
    
beweglichkeit
    
    Object { 0=0, 1="30"}
    
fitness
    
    Object { 0=0, 1="15"}
    
frischeluft
    
    Object { 0=0, 1="0"}
    
krafttraining
    
    Object { 0=0, 1="0"}
    
sport
    
    Object { 0=0, 1="0"} 

habe ich irgendetwas übersehen?

Viele Grüße und Danke im vorraus

[tkausl]

function getAllValuesMovement(){
    var SmileValues = {};
    $("table.movement-kategorie tr").each(function(){
        $(this).children("td.bewertungs-smile").children().each(function(){
            childClass = $(this).attr("class").split(" ");
            childkategorie = $(this).parent().parent().data("kategorie");
            if($(this).hasClass("active-smiley"))
            {
                wert = childClass[0];
            }else {
                wert = 0;
            }
            zeitWert = $(this).parent().parent().children("td.zeit-deritaet").children().val();
            SmileValues[childkategorie] = { 0 : wert, 1 : zeitWert};
        });
    });
    console.log(SmileValues);
    return SmileValues;
}

Warum nimmst du nicht einfach Checkboxen?

[eStar]

Weil ich das jetzt in allen Templates ändern müsste + mein komplettes script anpassen, dafür hab ich heute leider keine Zeit mehr, zu viel anderen Krams.

Meine Möglichkeit funktioniert doch auch ... nur finde ich den Fehler nicht, wieso er nur den ersten Wert in mein objekt zieht ...

[tkausl]

Meine Möglichkeit funktioniert doch auch ...

Na irgendwie ja nicht, sonst wärst du nicht hier.

http://jsfiddle.net/

[Gast]

ich blick da nicht durch, frage mich aber ob es not tut, dass alle selectboxen den selben namen haben,?

[nikosch]

Wo soll active-smiley denn auch herkommen? Insgesamt ein ziemlich schlechtes Script. Vor allem die Stelle:

wert = childClass[0];

Oder das da:

zeitWert = $(this).parent().parent().children("td.zeit-deritaet").children().val();

[eStar]

Kritik Ok,

Kritik ohne ein Argument, ziemlich bescheiden ...

[Gast]

//OT:
wo fehlt dir jetzt das argument?

[nikosch]

Dein Script ist hochgradig repetitiv, schafft strukturelle Abhängigkeiten und hangelt sich dazu sinnlos durch DOM-Strukturen, nutzt globale Variablen, speichert Daten dort wo sie nicht hingehören (in Klassen), ist ein sprachlicher und semantischer Kauderwelsch... und.. funktioniert halt nicht.