• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

kod javascript nie działa w pełni poprawnie

VPS Starter Arubacloud
0 głosów
632 wizyt
pytanie zadane 22 października 2017 w JavaScript przez mi-20 Stary wyjadacz (13,190 p.)

Witam wszystkich, mam taki kod js:

var editButton = document.getElementById("edit-ad");
var deleteButton = document.getElementById("delete-ad");
var editDivShow = document.getElementById("edit-alert");
var deleteDivShow = document.getElementById("delete-alert");
var closeDivDelete = document.getElementById("no");

editButton.onclick = function(){
    editDivShow.style.display = "block";
}
deleteButton.onclick = function(){
    deleteDivShow.style.display = "block";
}
   closeDivDelete.onclick = function(){
    deleteDivShow.style.display = "none";
    editDivShow.style.display = "none";
}

kod ma zadanie wyświetlać okno potwierdzające wykonanie działania po kliknięciu w odpowiedni przycisk, wszystko działa prawidłowo, ale nie zupełnie.

Kod ma potwierdzać usunięcie lub edycję postu pobranego z bazy danych. Posty pobierane są za pomocą pętli foreach. Kod js wykonuje się prawidłowo tylko dla pierwszego wyświetlonego postu, dla kolejnych nie wykonuje się wgl.

Jak można rozwiązać ten problem?

Pozdrawiam

komentarz 22 października 2017 przez ScriptyChris Mędrzec (190,190 p.)

Posty pobierane są za pomocą pętli foreach

Pokaż kod tej pętli. Zapewne "nadziałeś się" na asynchroniczność JSa.


editButton.onclick

Wszelaką obsługę DOMowych eventów wykonuj za pomocą EventListenerów

editDivShow.style.display

Styluj w CSS, a JSem dodawaj/usuwaj/podmieniaj klasy za pomocą metod dostępnych w property classList. Chyba, że jest to element HTML, który jest np. progressbarem, gdzie w krótkim czasie zmieniasz jego długość/szerokość lub przesuwasz jakiś obiekt po stronie.

komentarz 22 października 2017 przez mi-20 Stary wyjadacz (13,190 p.)

w pętli jest taki zapis:

echo '<div class="vie"><div class="edit" id="edit-ad">Edytuj</div><div id="delete-ad">Usuń</div></div>';

 

komentarz 22 października 2017 przez ScriptyChris Mędrzec (190,190 p.)

Jeśli robisz pętle w PHP, to nie pomogę, bo nie znam PHP. Mogę Ci natomiast podpowiedzieć, że jeśli do przesłania danych z PHP do JSa używasz echo, to robisz to źle (są tematy odnośnie tego na Forum - poczytaj). Zamiast tego użyj AJAX.

komentarz 22 października 2017 przez mi-20 Stary wyjadacz (13,190 p.)

Nie używam echo do przesyłania danych z PHP do JS. Echo używam do wyświetlania elementów HTML pomiędzy tagami <?php ?>, tak jak w tym przypadku.

Zauważyłem że podobny problem do mojego występuje nawet jeśli nie używamy pętli PHP tylko mamy coś takiego:

KOD HTML:

  <div class="fun">
                                    <div class="imp"><a href="auction.php">Kup</a></div>
                                    <div id="imp" class="imp">Licytuj</div>
                                    <div id="imp_in" class="imp-click"><form><input id="pr-l" type="text" placeholder="Wpisz swoją cenę" /> <input id="su-l" type="submit" value=">" title="licytuj"/>
                                    <div id="back" title="anuluj">X</div>
                                        </form></div>
                                </div>
  <div class="fun">
                                    <div class="imp"><a href="auction.php">Kup</a></div>
                                    <div id="imp" class="imp">Licytuj</div>
                                    <div id="imp_in" class="imp-click"><form><input id="pr-l" type="text" placeholder="Wpisz swoją cenę" /> <input id="su-l" type="submit" value=">" title="licytuj"/>
                                    <div id="back" title="anuluj">X</div>
                                        </form></div>
                                </div>
  <div class="fun">
                                    <div class="imp"><a href="auction.php">Kup</a></div>
                                    <div id="imp" class="imp">Licytuj</div>
                                    <div id="imp_in" class="imp-click"><form><input id="pr-l" type="text" placeholder="Wpisz swoją cenę" /> <input id="su-l" type="submit" value=">" title="licytuj"/>
                                    <div id="back" title="anuluj">X</div>
                                        </form></div>
                                </div>

KOD JS:

var wyswietl=document.getElementById("imp");
var wyswietl_pole=document.getElementById("imp_in");
var back=document.getElementById("back");
wyswietl.onclick = function(){
    wyswietl_pole.style.display = "block";
    wyswietl.style.width ="1px";
};
back.onclick=function(){
    wyswietl_pole.style.display = "none";
    wyswietl.style.width ="";
};

W pierwszym elemencie kod działa poprawnie a w drugim nie działa wgl.

Ogólnie problem występuje kiedy mamy kilka takich samych elementów na stronie.

komentarz 22 października 2017 przez ScriptyChris Mędrzec (190,190 p.)

Nadałeś taki sam atrybut id kilku elementom HTML, co jest błędem. Identyfikator powinien być unikalny. Gdy łapiesz element poprzez ID, to pobrany zostanie pierwszy element o danym ID. Żeby zebrać pozostałe elementy, to albo nadaj im jakieś inkrementowalne ID i do każdego (np. w pętli) podpinaj listener (co nie jest zbyt eleganckie) albo zastosuj EventDelegation.

Zaloguj lub zarejestruj się, aby odpowiedzieć na to pytanie.

Podobne pytania

0 głosów
1 odpowiedź 273 wizyt
pytanie zadane 28 stycznia 2023 w JavaScript przez Verone Nowicjusz (120 p.)
0 głosów
3 odpowiedzi 343 wizyt
pytanie zadane 11 września 2022 w JavaScript przez Doge Gaduła (3,320 p.)
0 głosów
3 odpowiedzi 276 wizyt
pytanie zadane 1 września 2015 w JavaScript przez CPP_Newbie Użytkownik (770 p.)

92,452 zapytań

141,262 odpowiedzi

319,081 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...