Кажется, что автор хочет увидеть, как можно переписать функцию в более функциональном стиле. Например, вот так:
function convertText(str, key) {
key = (26 - key) % 26;
return [...str].map(symb => {
const code = symb.charCodeAt(0);
let shift = 0;
if ((65 <= code) && (code <= 90)) {
shift = 65;
} else if ((97 <= code) && (code <= 122)) {
shift = 97;
}
if (shift > 0) {
return String.fromCharCode(shift + ((code - shift + key) % 26));
} else {
return symb;
}
}).join('');
}
alert(convertText('Hmjw uzsfuw dwv eq xjawfv Zwjumdw Hgajgl, xgjewjdq uzawx gx lzw Twdyasf xgjuw, lg tw ugffwulwv oalz lzw Klqdwk uskw', 18));