Javascript - 配列をループする

Question

概要：新しい画像リンクを含む画像リンクを含む配列を更新する必要があります。同時に、以前にアップロードされたすべての画像を配列に保存しています。私の問題は、これを行う際に以前の画像リンクが結合されることです。以下の例。配列を修正するためにコードをどのように変更すればよいですか？助けてくれてありがとう。

var allimages = [] 

    var allCurrentImages = req.body.oldimages 
    //this pulls all the previous image links 

     if (allCurrentImages && allCurrentImages.length > 2){ 
     for (i=0;i<allCurrentImages.length;i++){ 
     allimages.push(allCurrentImages[i]); 
     } 
    } 

    if (filepath && filepath.length > 2){ 
    allimages.push(filepath); 
    } 

PROBLEM

は、ここで問題です。 var allCurrentImagesに2つの画像がある場合、私はボディを要求しているので、配列はそれらを1つのアイテムにまとめます。

images[0] = uploads/598f4cc,uploads/53eew2w 
images[1] = uploads/7wusjw2w 

それはこのように見える必要があります：

images[0] = uploads/598f4cc 
images[1] = uploads/53eew2w 
images[2] = uploads/7wusjw2w 

だから私は何とかそれをプッシュする前に、別の部分にreq.body.oldimagesを分割する必要がある3枚の画像がある場合には、このようになります。配列。 （私は思う。）どんな助けや助言も大歓迎です！

Answer 1

問題はMongooseモデルの "oldimages"は配列でしたが、EJSで印刷すると文字列として出力されました。 EJSを印刷しないようにしてこの問題を回避し、foundListings.currentimagesから配列を取り出しました。

ありがとうございました。

Listings.findById(req.params.id, function(err, foundListings){ 

    var allimages = [] 

    var allCurrentImages = foundListings.currentimages; 
    console.log('all images' + allCurrentImages) 


     if (allCurrentImages){ 
     for (i=0;i<allCurrentImages.length;i++){ 
     allimages.push(allCurrentImages[i]); 
     } 
    } 

    if (filepath && filepath.length > 2){ 
    allimages.push(filepath); 
    } 

}); 

Answer 2

変更し、この中に、単にあなたのコード、文字列のすべての文字をループし、それがさらに2つの以上のイメージのために働くべきであるにも取り除く：

allimages.concat(allCurrentImages.split(',')); 

Answer 3

あなたが前にそれを分割することができます。

var allCurrentImagesTmp = req.body.oldimages 
var allCurrentImages = []; 

for (i=0;i<allCurrentImagesTmp .length;i++){ 
    allCurrentImages.concat(allCurrentImagesTmp[i].split(",")); 
} 
... 
// your code 

Answer 4

req.body.oldimagesは文字列の配列ですか？それはあり得ることらしいから、それ以外の場合は

allimages.push(allCurrentImages[i].split(',')); 

：これまで

allimages.push(allCurrentImages[i]); 

：もしそうなら、あなたはこのことから、あなたのコードに行を変更することで、あなたが探しているものを達成することができるはずですあなたは、具体的カンマを探しているとあなたの利点にその情報を使用して、より正確な方法を試みることができる1つの長い文字列、：

var CurrentImages = allCurrentImages; // Use temp variable to protect original 
var CommaIndex = CurrentImages.indexOf(','); // Find index of first comma 
while (CommaIndex>0) { // If there is no comma present, indexOf returns -1 
    allimages.push(CurrentImages.substring(0, CommaIndex-1)); // Push the first image path to allimages 
    CurrentImages = CurrentImages.substring(CommaIndex+1, CurrentImages.length-1); // Grab the rest of the string after the first comma 
    CommaIndex = CurrentImages.indexOf(','); // Check for another comma 
} 
allimages.push(CurrentImages); // This pushes the final one after the last comma - or the only one if there was no comma. 

Answer 5

ハムを私はあなたの目的については本当にわからない、私はそのいくつかの時間を理解します...あなたは、文字列&いつか配列を持って、あなたは別の配列の先頭に要素を追加したい私はこれを行うには正しい&適切な方法はな単純なものだと思う：

let allimages = [] 

let allCurrentImages = req.body.oldimages.split(','); 
//Split by coma 

allimages = allimages.concat(allCurrentImages); 
// attention contact return the concat array so you have to set it to a variable. 

このコードは動作するはずですが、場合にのみ、あなたがこれを制御したいならば、画像には "、"がありません。フロントエンドの正規表現でバックエンド&を防ぐ必要があります。