लूप

Question

के लिए ब्रेक के बिना सरणी के माध्यम से लूपिंग और आइटम को हटाने के बाद, मेरे पास लूप के लिए निम्न है, और जब मैं किसी आइटम को निकालने के लिए splice() का उपयोग करता हूं, तो मुझे लगता है कि 'सेकंड' अपरिभाषित है। मैं जांच सकता हूं कि यह अपरिभाषित है, लेकिन मुझे लगता है कि ऐसा करने के लिए शायद एक और शानदार तरीका है। इच्छा है कि किसी आइटम को आसानी से हटाएं और आगे बढ़ें।

for (i = 0, len = Auction.auctions.length; i < len; i++) { 
    auction = Auction.auctions[i]; 
    Auction.auctions[i]['seconds'] --; 
    if (auction.seconds < 0) { 
     Auction.auctions.splice(i, 1); 
    }   
} 

Answer 1

जब आप .splice() करते हैं तो सरणी को फिर से अनुक्रमित किया जा रहा है, जिसका अर्थ है कि जब आप हटा दिए जाते हैं तो आप एक इंडेक्स पर छोड़ देंगे, और आपके कैश किए गए .length अप्रचलित हैं।

इसे ठीक करने के लिए, आप या तो एक .splice() के बाद i घटती करने, या बस रिवर्स में पुनरावृति आवश्यकता होगी ...

var i = Auction.auctions.length 
while (i--) { 
    ... 
    if (...) { 
     Auction.auctions.splice(i, 1); 
    } 
} 

इस तरह फिर से अनुक्रमण में अगले आइटम को प्रभावित नहीं करता पुनरावृत्ति, चूंकि अनुक्रमण केवल वर्तमान बिंदु से वस्तुओं को ऐरे के अंत तक प्रभावित करता है, और पुनरावृत्ति में अगला आइटम वर्तमान बिंदु से कम है।

Answer 2

लंबाई हर बार लूप के माध्यम से करने के बजाय सिर्फ शुरू में, उदा, इस तरह आप सीमा से अधिक नहीं होगा पुनः गणना करें, .:

for (i = 0; i < Auction.auctions.length; i++) { 
     auction = Auction.auctions[i]; 
     Auction.auctions[i]['seconds'] --; 
     if (auction.seconds < 0) { 
      Auction.auctions.splice(i, 1); 
      i--; //decrement 
     } 
} 

।

संपादित करें: यदि कथन में कमी शामिल है।

Answer 3

हालांकि अपने प्रश्न सरणी पर दोहराया जा रहा है और नहीं के बारे में तत्वों (अलावा कुछ अन्य प्रसंस्करण के लिए) को बाहर निकालने कुशलता से तत्वों को हटाने के बारे में है, मुझे लगता है कि एक ऐसी ही स्थिति में अगर यह पर पुनर्विचार करना चाहिए।

इस दृष्टिकोण की एल्गोरिदमिक जटिलता O(n^2) स्प्लिस फ़ंक्शन के रूप में है और लूप के लिए दोनों सरणी पर पुनरावृत्ति (स्प्लिस फ़ंक्शन सबसे खराब मामले में सरणी के सभी तत्वों को स्थानांतरित करता है)। इसके बजाय आप केवल आवश्यक तत्वों को नए सरणी में धक्का दे सकते हैं और फिर उस सरणी को वांछित चर (जिसे केवल पुनरावृत्त किया गया था) पर असाइन करें।

var newArray = []; 
for (var i = 0, len = Auction.auctions.length; i < len; i++) { 
    auction = Auction.auctions[i]; 
    auction.seconds--; 
    if (!auction.seconds < 0) { 
     newArray.push(auction); 
    } 
} 
Auction.auctions = newArray; 

या सरणी कार्य करता है और तीर फ़ंक्शन का उपयोग एक लाइन पर यह सब फिट करने के लिए:

Auction.auctions = Auction.auctions.filter(auction => --auction.seconds >=0); 

Answer 4

Auction.auction = Auction.auctions.filter(function(el) { 
    return --el["seconds"] > 0; 
}); 

Answer 5

यह एक बहुत आम मुद्दा है। समाधान पीछे की ओर पाश के लिए है:

for (var i = Auction.auctions.length - 1; i >= 0; i--) { 
    Auction.auctions[i].seconds--; 
    if (Auction.auctions[i].seconds < 0) { 
     Auction.auctions.splice(i, 1); 
    } 
} 

अगर आप उन्हें अंत के बंद पॉपिंग रहे हैं, क्योंकि सूचकांक संरक्षित किया जाएगा के रूप में आप पीछे की ओर जाना यह कोई फर्क नहीं पड़ता।

Answer 6

जब पाशन newArray में एक सरणी रिले करने का प्रयास करें:

var auctions = Auction.auctions; 
var auctionIndex; 
var auction; 
var newAuctions = []; 

for (
    auctionIndex = 0; 
    auctionIndex < Auction.auctions.length; 
    auctionIndex++) { 

    auction = auctions[auctionIndex]; 

    if (auction.seconds >= 0) { 
    newAuctions.push(
     auction); 
    }  
} 

Auction.auctions = newAuctions; 

Answer 7

यहाँ जोड़ के समुचित उपयोग के लिए एक और उदाहरण है। यह उदाहरण 'सरणी' से 'विशेषता' को हटाने के बारे में है।

for (var i = array.length; i--;) { 
    if (array[i] === 'attribute') { 
     array.splice(i, 1); 
    } 
} 

Answer 8

एक और सरल उपाय एक बार एक सरणी तत्वों को पचाने के लिए:

while(Auction.auctions.length){ 
    // From first to last... 
    var auction = Auction.auctions.shift(); 
    // From last to first... 
    var auction = Auction.auctions.pop(); 

    // Do stuff with auction 
} 

Answer 9

for (i = 0, len = Auction.auctions.length; i < len; i++) { 
    auction = Auction.auctions[i]; 
    Auction.auctions[i]['seconds'] --; 
    if (auction.seconds < 0) { 
     Auction.auctions.splice(i, 1); 
     i--; 
     len--; 
    } 
} 

Answer 10

तुम बस के माध्यम से देखने के लिए और shift()

Answer 11

का उपयोग आप उपयोग कर ES6 + ई कर रहे हैं कर सकते हैं - क्यों सिर्फ का उपयोग नहीं Array.filter विधि?

Auction.auctions = Auction.auctions.filter((auction) => { 
    auction['seconds'] --; 
    return (auction.seconds > 0) 
}) 

ध्यान दें कि फिल्टर यात्रा के दौरान सरणी तत्व को संशोधित केवल वस्तुओं के लिए काम करता है और आदिम मूल्यों की सरणी के लिए काम नहीं करेगा।