Warum kann diese Funktion dieses bestimmte Array nicht verarbeiten?

Question

Die folgende Funktion findet Wortpaare, die mehr als einmal in einem Array vorkommen, und kombiniert sie dann zu einem einzelnen Array-Element. Es funktioniert auf dem Beispiel-Array, schlägt aber fehl, wenn ich versuche, dies in mein Projekt zu implementieren (das dynamisch ein Array aus Scraping-Webseiten erstellt).

Funktion:

function combineCommon(arr) { 
    var dictionary = {}; 
    for (var a = 0; a < arr.length - 1; a++) { 
    var A = arr[a]; 
    if (dictionary[A] == void 0) { 
     dictionary[A] = []; 
    } 
    dictionary[A].push(arr[a + 1]); 
    } 
    var res = []; 
    for (var index = 0; index < arr.length; index++) { 
    var element = arr[index]; 
    var pass = false; 
    if (dictionary[element].length > 1) { 
     if (dictionary[element] 
     .some(function(a) { 
      return a != dictionary[element][0]; 
     }) == false) { 
     pass = true; 
     } 
    } 
    if (pass) { 
     res.push(arr[index] + " " + dictionary[element][0]); 
     index++; 
    } else { 
     res.push(arr[index]); 
    } 
    } 
    return res; 
} 
console.log(combineCommon(arr)); 

Array, das funktioniert:

var arr = ["john", "smith", "says", "that", "a", "lock", "smith", "can", "open", "the", "lock", "unlike", "john", "smith"]; 

Array das nicht funktioniert:

var arr = ['Social', 'care', 'fund', 'fails', 'to', 'reduce', 'pressure', 'on', 'hospital', 'beds', 'Court', 'questions', 'whether', 'US', 'travel', 'ban', 'is', 'anti', 'Muslim', 'Police', 'pay', 'out', 'at', 'least', '£195m', 'to', 'informants', 'in', 'five', 'years', 'Brexit', 'rebellion', 'avoided', 'after', 'meaningful', 'vote', 'offer', 'FA', 'reforms', 'Chairman', 'Greg', 'Clarke', 'to', 'quit', 'if', 'government', 'does', 'not', 'back', 'plans', 'Louisiana', 'tornadoes', 'The', 'whole', 'house', 'fell', 'apart', 'Uncertainty', 'over', '30', 'hours', 'free', 'childcare', 'say', 'councils', 'Uncertainty', 'over', '30', 'hours', 'free', 'childcare', 'say', 'councils', 'Hans', 'Rosling', 'Data', 'visionary', 'and', 'educator', 'dies', 'aged', '68', 'Dakota', 'Access', 'Pipeline', 'to', 'win', 'US', 'Army', 'permit', 'for', 'completion'] 

Hier ist ein jsfiddle zu demonstrieren. Warum funktioniert das zweite Array nicht?

Answer 1

Beim Erstellen des Wörterbuchs scheint ein Fehler durch einen Fehler zu sein, der dazu führt, dass das letzte Wort des Arrays nicht zum Wörterbuch hinzugefügt wird. Das erste Beispiel funktioniert, weil das letzte Wort ("Smith") bereits zuvor im Array enthalten ist.

soll die dritte Linie for (var a = 0; a < arr.length; a++) { sein, das heißt:

function combineCommon(arr) { 
var dictionary = {}; 
    for (var a = 0; a < arr.length; a++) { 
    var A = arr[a]; 
    if (dictionary[A] == void 0) { 
     dictionary[A] = []; 
    } 
    dictionary[A].push(arr[a + 1]); 
    } 
    var res = []; 
    for (var index = 0; index < arr.length; index++) { 
    var element = arr[index]; 
    var pass = false; 
    if (dictionary[element].length > 1) { 
     if (dictionary[element] 
     .some(function(a) { 
      return a != dictionary[element][0]; 
     }) == false) { 
     pass = true; 
     } 
    } 
    if (pass) { 
     res.push(arr[index] + " " + dictionary[element][0]); 
     index++; 
    } else { 
     res.push(arr[index]); 
    } 
    } 
    return res; 
} 
console.log(combineCommon(arr)); 

Answer 2

Sie haben keine undefinierte Prüfung bei der Zeilennummer 18 durchgeführt:

mit unten aktualisiert

jsfiddle:

if (typeof dictionary[element] !== 'undefined' && dictionary[element].length > 1) { 
     if (dictionary[element] 
     .some(function(a) { 
      return a != dictionary[element][0]; 
     }) == false) { 
     pass = true; 
     } 
    }