[imdejv1]

Mam pytanie, w jaki sposób zrobić tutaj JS, aby pobierał nazwę z buttona? Nawet ta nazwa mogłaby być umieszczona w buttonie niekoniecznie pomiędzy zamknięciem ><. Chodzi o to, aby nie trzeba było kolejnych pozycji dodawać w JS tylko żeby on tam czytał z nazwy.

fiddle: http://jsfiddle.net/w5dxb8cg/

#html #js #javascript

[nunczako]

@imdejv1:

Array.from(document.getElementsByTagName('button')).forEach( b=> b.addEventListener('click', () => ass.value += b.innerHTML ))

?

[Makurise]

@imdejv1: Podstawy this się kłaniają.

ass.value += `fass, ${ this.innerHTML }`;

[ferdidurke]

@imdejv1: http://jsfiddle.net/sgo926rj/1/

[Makurise]

@ferdidurke:

function pobierzWartosc()

Weź, bo jeszcze zacznie tworzyć potworki typu isPies ( ͡° ͜ʖ ͡°)

[asdfghjkl]

@imdejv1: http://jsfiddle.net/6bqoax40/11/

[asdfghjkl]

@imdejv1: przede wszystkim id musi by c unikalny w skali dokumentu. Zrób sobie jakiś inny handler po className czy data czy cokolwiek innego.

[ferdidurke]

@Makurise: wybacz mi. mam przeczucie, że to jakieś zadanie do szkoły i nie chcę żeby młodego posądzili o kopiowanie z angielskich stron. ( ͡° ͜ʖ ͡°)

[patrolez]

@imdejv1: id pochodzi od słówka "identificator"/"identity" i przypisanie tego atrybutu powinno być unikalne w całym dokumencie, tj. może wystąpić tylko raz w wynikowym dokumencie HTML w atrybutach.

Jeżeli chcesz np. używać jakiegoś tokena do wybrania obiektów, aby je wystylizować lub móc odpytać w JS, to używaj atrybutu class.

[Makurise]

@imdejv1: Tak będzie najładniej - dwa sposoby: http://jsfiddle.net/uo127fqe/

[Makurise]

@imdejv1: Tylko warto wiedzieć, że Object.values nie jest wspierane przez IE. Jeżeli z jakiegoś powodu musisz wspierać tę przeglądarkę i nie używasz żadnego frameworka, to wtedy przypisujesz eventy w zwykłej pętli for.

[imdejv1]

Niestety zielony jestem mega w tym js itp.

@nunczako: @ferdidurke: dziękuje, przykład od Panów poniżej zadziałał idealnie. No właśnie nie zadanie do szkoły tylko miniformularz zamówień, że nie było sensu w woocommerce sie bawić, a ja już stary pryk po szkole :P

@asdfghjkl: , @Makurise: - Przeogromne dzięki, działa wspaniale jedna i drugie, o to dokładnie chodziło. A co zrobić, żeby powstawał przecinek pomiedzy

[asdfghjkl]

@imdejv1: na przykład tak
http://jsfiddle.net/6bqoax40/13/

Lepsze rozwiązanie niż pierwotnie. Robisz tablice klikniętych buttonów i potem z pomoca join łączysz elementy tablicy i rozdzielasz separatorem

[asdfghjkl]

@imdejv1: tu masz to samo bazujące na kodzie @Makurise bo uproszczony przez co czytelniejszy http://jsfiddle.net/m4pnwue6/1/

[imdejv1]

@asdfghjkl: Wielkie dzięki, obydwa działają, a to dla mnie najważniejsze. Jeszcze przydałby się CLEAN, ale to już jest malo ważne. Jestem ogromnie wdzięczny za pomoc, no jest bajka po prostu 8-)

[asdfghjkl]

@imdejv1: http://jsfiddle.net/m4pnwue6/5/

[imdejv1]

@asdfghjkl: Jest idealnie, dziękuje pięknie i jestem dłużny!

[Makurise]

@imdejv1: @asdfghjkl:

Robisz tablice klikniętych buttonów i potem z pomoca join łączysz elementy tablicy i rozdzielasz separatorem

Rozwiązanie o tyle słabe, że się nie skaluje w żaden sposób. Z każdym kliknięciem wydajność spada - co oczywiście nie będzie widoczne gołym okiem przy nawet kilkuset kliknięciach, ale nie jest to dobra praktyka. Lepiej po prostu sprawdzać czy jest już jakaś wartość wpisana w pole.

http://jsfiddle.net/qrvmzh1n/

[asdfghjkl]

@Makurise: to jest formularz zamówienia. Operowanie na tablicy daje masę dodatkowych możliwości a wydajność jest pomijalna w zastosowaniach opa bo na 99% elementów nie będzie więcej niż 10.
pętla for jest conajmniej kilka razy szybsza niż forEach czy ma to jakieś znaczenie dla opa...nie ( ͡° ͜ʖ ͡°)
Rozwiązania dzielą się na wystarczające i nie warte dodatkowej pracy ( ͡° ͜ʖ ͡°)

[Makurise]

@asdfghjkl: No i widzisz, tutaj zaczynasz bredzić i zakładać z góry różne rzeczy, co jest niebezpieczne w zawodzie programisty ( ͡° ͜ʖ ͡°)

Operowanie na tablicy daje masę dodatkowych możliwości

Daje masę możliwości, o których OP się nawet słowem nie zająknął, a ty wychodzisz z inicjatywą w taki sposób, że narażasz go na bugi. Pamiętaj, że oparł on wyświetlanie się klikniętych przycisków o pole tekstowe, które można dowolnie edytować niezależnie

[imdejv1]

Lepiej po prostu sprawdzać czy jest już jakaś wartość wpisana w pole.

@Makurise: No, ale jak wartość już jest wpisana w polu, a znowu chcemy tą wartość? Tutaj idealnie dla tego mini formularza byłoby gdyby wtedy dodawalo przed nazwą np: 3x, czyli: 3x zapałki, zapalniczka, 7x mydło. Trudne do dodania coś takiego jest? To już pewnie dużo linijek kodu.