[JavaScript]Event click w pętli, Coffee script Opcje

[MateuszS]

Hej,

Mam problem z eventem click na buttonie, ktory jest w pętli. Tu fragment kodu (coffee script)

[JAVASCRIPT] pobierz, plaintext 
                    for index of shelfs
                        shelf = shelfs[index]
                        content = 'Empty shelf '+ shelf.getId()
                        $(@).append('<input type="button" style="width: 100%; height: 40px" class="input_button" value="'+content+'" /><br />')
                        button = $(@).find('input').last()
 
                        parent = $(@)
                        button.click -> 
                            console.log(button.val())
[JAVASCRIPT] pobierz, plaintext 

Za kazdym razem wyswietla mi value ostatniego buttona zamiast kolejnych. Wiecie jak to obejsc?

Pozdrawiam

[nospor]

console.log(button.val())
Poniewaz wartosc button jest nadpisywana w petli i na jej koncu zawiera wartosc ostatniego elementu ktory byl do niej przypisany

[Comandeer]

A do jakiego to kodu jest kompilowane? Bo wygląda tak, jakby button był wyciągnięty poza pętlę (CS to lubi) i tym samym każda iteracja ją nadpisuje. W czystym JS po prostu zawartość pętli otacza się w IIFE. W nowym JS (ES6) po prostu wystarczyłoby zadeklarować button przez słowo kluczowe let, a nie var wewnątrz pętli.

[MateuszS]

console.log(button.val())
Poniewaz wartosc button jest nadpisywana w petli i na jej koncu zawiera wartosc ostatniego elementu ktory byl do niej przypisany

Tak, tego się domyśliłem, ale nie jest to u mnie porządane zachowanie

@Comandeer

[JAVASCRIPT] pobierz, plaintext 
for (index in shelfs) {
              shelf = shelfs[index];
              content = 'Empty shelf ' + shelf.getId();
 
              $(this).append('<input type="button" style="width: 100%; height: 40px" class="input_button" value="' + content + '" /><br />');
              button = $(this).find('input').last();
              parent = $(this);
              results.push(button.click(function() {
                console.log(button.val());
 
             }));
[JAVASCRIPT] pobierz, plaintext 

Ten post edytował MateuszS 16.09.2015, 13:13:41

[nospor]

Czemu nei zrobisz poprostu:

console.log(this.value);
? Przeciez pod this masz aktualny element

[MateuszS]

Czemu nei zrobisz poprostu:

console.log(this.value);
? Przeciez pod this masz aktualny element

Ajj moja pomyłka, miał być console.log(shelf.getId()). Z buttonem owszem this działa ale shelf jest nadpisywany.

[nospor]

shelf = shelfs[index];
To jest zmienna globalna i poraz kolejny nadpisywana.

sprobuj:
var shelf = shelfs[index]; //zmienna lokalna

[MateuszS]

Próbowałem zmienić to w .js i to też nie pomogło:

[JAVASCRIPT] pobierz, plaintext 
for (index in shelfs) {
              var shelf = shelfs[index];
              var content = 'Empty shelf ' + shelf.getId();
              $(this).append('<input type="button" style="width: 100%; height: 40px" class="input_button" value="' + content + '" /><br />');
              var button = $(this).find('input').last();
              var parent = $(this);
              results.push(button.click(function() {
                console.log(console.log(shelf.getId());
[JAVASCRIPT] pobierz, plaintext 

Niestety coffee script wyrzuca mi zawsze deklaracje zmiennych "piętro" wyżej i nie bardzo wiem czy da się to jakoś przestawić. Ale tak czy siak niestety nie pomaga.

//edit, moze daloby sie jakos przez parametr clicka przeslac tego shelfa? Bo pewnie onClick w inpucie by przeszlo ale nie jest to eleganckie, moze jakis trigger wywolam z parametrem?

Ten post edytował MateuszS 16.09.2015, 13:40:37

[nospor]

chcesz tylko dobrac sie do tego ID? To dopisuj go jako atrybuj dla inputa i pobieraj na podobnej zasadzie co this.value. I po sprawie

[Comandeer]

Kod

for (index in shelfs) {
    (function(shelf)
    {
        var content = 'Empty shelf ' + shelf.getId();
        $(this).append('<input type="button" style="width: 100%; height: 40px" class="input_button" value="' + content + '" /><br />');
        var button = $(this).find('input').last();
        var parent = $(this);
        button.click(function() {
            console.log(console.log(shelf.getId());
        });
    }(shelfs[index]));
}

?

[MateuszS]

chcesz tylko dobrac sie do tego ID? To dopisuj go jako atrybuj dla inputa i pobieraj na podobnej zasadzie co this.value. I po sprawie

ten shelf jest obiektem, czasami bede potrzebowal rzeczy z "wyzej". Obiektu nie zapisze do atrybuta bez jakiejs serializacji a to chyba nie jest dobre rozwiazanie. Fajnie byloby tego clicka wywolac jakos z parametrem cos jak w onClick

Kod

for (index in shelfs) {
    (function(shelf)
    {
        var content = 'Empty shelf ' + shelf.getId();
        $(this).append('<input type="button" style="width: 100%; height: 40px" class="input_button" value="' + content + '" /><br />');
        var button = $(this).find('input').last();
        var parent = $(this);
        button.click(function() {
            console.log(console.log(shelf.getId());
        });
    }(shelfs[index]));
}

?

to kod JSowy, jak to zapisac w coffee? To zadziala?

[nospor]

ten shelf jest obiektem, czasami bede potrzebowal rzeczy z "wyzej"

To zapisuj index tego obiektu z tablicy. I po sprawie. Mnozysz problemy

[Comandeer]

https://realguess.net/2014/03/03/immediatel...n-coffeescript/ → tak by to wyglądało w CS.

[MateuszS]

https://realguess.net/2014/03/03/immediatel...n-coffeescript/ → tak by to wyglądało w CS.

No wlasnie nie za dobrze mi kompiluje. Początek się zgadza

(shelf) ->

natomiast mam problem z przekazaniem argumentu. Gdy wklepię kod na sztywno do JS to wywala mi

Uncaught TypeError: Cannot read property 'createDocumentFragment' of undefined

[Comandeer]

Prawdę mówiąc nie bardzo wiem czego ten błąd dotyczy… Czyżby chodziło o $.fn.append?