Karte und RegEx-Seltsamkeit

Question

Ich mache eine Kata, die eine Caesar-Chiffre-Zeichenfolge in lesbaren Text dekodiert. Ich benutze RegEx innerhalb einer Map, um Sonderzeichen zu finden und überspringe sie, aber die Ausgabe ist flockig, wenn ich zwei oder mehr Sonderzeichen nebeneinander, ',' oder ':)' habe. Es scheint einige Sonderzeichen zu überspringen.

Kann jemand erklären, was vor sich geht?

Ich habe den ChangeCharCode Funktionscode nicht enthalten, weil ich denke, dass das Problem in meiner Karte ist.

function decodeString(string) { 
    const stringArr = string.toLowerCase().split(''); 
    const specialCharacters = /[ .,\/#!$%\^&\*;:{}=\-_`~()]/g; 
    const codeOfX = 'x'.charCodeAt(0); 
    const codeOfLastLetter = stringArr[stringArr.length - 1].charCodeAt(0); 
    const codeShift = codeOfX - codeOfLastLetter; 

    return stringArr.map((elem) => { 
    // Special character treatment 
    return specialCharacters.test(elem) === true ? elem : changecharCode(elem, codeShift); 
}).join('').toUpperCase(); 
    } 

function changecharCode (letter, codeShift) { 
    const currentCode = letter.charCodeAt(0); 
    // Uppercase letters 
    if ((currentCode >= 65) && (currentCode <= 90)) 
    return letter = String.fromCharCode(((currentCode - 65 + codeShift) % 26) + 65); 
    // Lowercase letters 
    else if ((currentCode >= 97) && (currentCode <= 122)) 
    return letter = String.fromCharCode(((currentCode - 97 + codeShift) % 26) + 97); 
} 

decodeString(' :) ') => ') ' 
decodeString(', ') => ',' 

Answer 1

Es ist das gleiche Problem wie why-does-my-javascript-regex-test-give-alternating-results. Nehmen wir an, dass das Problem wegen des Flags g aufgetreten ist, das einen globalen Übereinstimmungsstatus beibehält. Die Lösung in Ihrem Fall soll sein, ein g Flag zu entfernen, sobald Ihre Funktion Zeichen nacheinander ausführt, dann ist das g Flag unnötig.

Answer 2

Das Problem ist in Ihrer changecharCode Funktion: Wenn ein Zeichencode nicht innerhalb der zwei Bereiche ist, die getestet werden, dann gibt die Funktion nichts zurück, d. H. . Die join, die Sie später machen, wird eine leere Zeichenkette für jeden undefined Wert erzeugen, so dass Sie nichts in der Ausgabe sehen.

Wenn Sie hinzufügen, ein Finale zu changecharCode:

return ' '; // or whatever character you want here 

Dann ist der Ausgang die gleiche Anzahl von Zeichen, die als Eingang haben.

Answer 3

Entfernen Sie die globale Flagge am Ende der Regex, haben Sie ein Zeichen in einer Zeit zu gehen:

function decodeString(string) { 
 
    const stringArr = string.toLowerCase().split(''); 
 
    const specialCharacters = /[ .,\/#!$%\^&\*;:{}=\-_`~()]/; 
 
    //            here ___^ 
 
    const codeOfX = 'x'.charCodeAt(0); 
 
    const codeOfLastLetter = stringArr[stringArr.length - 1].charCodeAt(0); 
 
    const codeShift = codeOfX - codeOfLastLetter; 
 

 
    return stringArr.map((elem) => { 
 
    // Special character treatment 
 
    return specialCharacters.test(elem) === true ? elem : changecharCode(elem, codeShift); 
 
}).join('').toUpperCase(); 
 
    } 
 

 
function changecharCode (letter, codeShift) { 
 
    const currentCode = letter.charCodeAt(0); 
 
    // Uppercase letters 
 
    if ((currentCode >= 65) && (currentCode <= 90)) 
 
    return letter = String.fromCharCode(((currentCode - 65 + codeShift) % 26) + 65); 
 
    // Lowercase letters 
 
    else if ((currentCode >= 97) && (currentCode <= 122)) 
 
    return letter = String.fromCharCode(((currentCode - 97 + codeShift) % 26) + 97); 
 
} 
 

 
console.log('>'+decodeString(' :) ')+'<'); 
 
console.log('>'+decodeString(', ')+'<');