JavaScript forEach Sıkıntısı

Sanch · 10 Ağustos 2020

Normalde burada olması gereken şey, kodlardaki belli bir kısmı silmek. Ama bu kodu çalıştırınca, son 5 tanesi hariç hepsine yapıyor. Neden acaba? Normalde hepsini silmesi gerek ama son 5 tanesi olmuyor. Lazım olursa kodları daha detaylı paylaşabilirim.

JavaScript:

taskList.childNodes.forEach(function(item) {
    if (item.nodeType == 1) {
        item.remove();
    }
})

khoomei · 11 Ağustos 2020

Debug etmek gerek, if'ten önce debugger koy adım adım incele. Demek ki son 5 tanesinin nodeType'ı 1 değil.

Sanch · 11 Ağustos 2020

Her defasında bir adet element siliniyor, sonra bir tane atlıyor. Yani mesela [1,2,3,4,5] olsun.
[1,3,5] siliniyor.

5 hariç diye de bir şey yokmuş. Orası yanlış.

Kendimce bir fikrim var,
şimdi elemanlar üzerinde forEach ile gezerken mesela elemanlar[0] değerini alıyor ve siliyor.
[1,2,3,4,5] ===> [2,3,4,5]
Şimdi de eleman[1] değeri silinecek. Ama eleman[1] değeri artık bir sonraki değer olacak. Yani 2 atlanacak.
[2,3,4,5] ===> [2,4,5]
eleman[1] burada 3 değeri oldu ve silindi. Çünkü aradaki değerlerden biri kalktı.

bitwise · 11 Ağustos 2020

=== ile tekrar denemelisin.

Zakramoreas · 11 Ağustos 2020

@Sanch JS kodlarını bilmiyorum ama ben verdiğiniz örnekteki gibi bir şey yapacak olsam ilk önce o kümenin([1, 2, 3, 4, 5])uzunluğunu alırdım ve bu şekilde denerdim. Bunu denerken if bloğunu kaldırıp sıkıntının nerede olduğunu çözmeye çalışırdım.

Sanch · 11 Ağustos 2020

bitwise dedi:
=== ile tekrar denemelisin.

Sonuç aynı.

Zakramoreas dedi:
@Sanch JS kodlarını bilmiyorum ama ben verdiğiniz örnekteki gibi bir şey yapacak olsam ilk önce o kümenin ([1, 2, 3, 4, 5]) uzunluğunu alırdım ve bu şekilde denerdim. Bunu denerken if bloğunu kaldırıp sıkıntının nerede olduğunu çözmeye çalışırdım.

Sıkıntıyı çözdüm zaten. Anlayınca biraz saçma geldi. Muhtemelen yukarıda bahsettiğim şey .

Zakramoreas · 11 Ağustos 2020

@Sanch Çözümünü buraya yazarsanız sonradan bakan arkadaşlarda faydalanır .

The Anorak · 11 Ağustos 2020

Lambda Expression kullana da bilirsiniz.

JavaScript:

taskList.childNodes.forEach((item, index) => {
    if (item.nodeType === ELEMENT_NODE) {
        item.remove();
    }
});

Ancak kodun neden atlayarak ilerlediğini tam olarak anlamadım.

Daniel Saddler · 11 Ağustos 2020

taskList'e JavaScript ile element oluşturuyorsun sanırım, ancak bu fonksiyonu oluşturma işleminden önce çalıştırıyorsun.

Sanch · 11 Ağustos 2020

Zakramoreas dedi:
@Sanch Çözümünü buraya yazarsanız sonradan bakan arkadaşlarda faydalanır .

Çözüm olarak başka bir yöntem kullandım, neden atlayarak gittiğini hâlâ anlamış değilim.
Çözüm için kullandığım kod:

JavaScript:

taskList.innerHTML = "";

O şekilde silmek yerine direk oradaki tüm elemanları sıfırladım ve silinmiş oldu.

Yöntem 2:

JavaScript:

while (taskList.firstChild) {
            taskList.removeChild(taskList.firstChild);
}

Bu şekilde de, eleman olduğu sürece ilk eklenen elemanı silecek, yani eleman olduğu sürece silme işlemi yapacak. Bu şekilde hepsi silinecek.

Önceki kodun neden atlayarak gittiğini bilen bir arkadaşım varsa yardımı çok iyi olur.

Sanch dedi:
Her defasında bir adet element siliniyor, sonra bir tane atlıyor. Yani mesela [1,2,3,4,5] olsun.
[1,3,5] siliniyor.

Kendimce bir fikrim var,
şimdi elemanlar üzerinde forEach ile gezerken mesela elemanlar[0] değerini alıyor ve siliyor.
[1,2,3,4,5] ===> [2,3,4,5]
Şimdi de eleman[1] değeri silinecek. Ama eleman[1] değeri artık bir sonraki değer olacak. Yani 2 atlanacak.
[2,3,4,5] ===> [2,4,5]
eleman[1] burada 3 değeri oldu ve silindi. Çünkü aradaki değerlerden biri kalktı.

Ben böyle düşündüm ama yanlış olabilir.

JavaScript forEach Sıkıntısı

Ayrıntılı düzenleme

Sanch

Kilopat

khoomei

Hectopat

Sanch

Kilopat

bitwise

Gigapat

Zakramoreas

Kilopat

Sanch

Kilopat

Zakramoreas

Kilopat

The Anorak

Terapat

Daniel Saddler

Hectopat

Sanch

Kilopat

Benzer konular