Sabe o que é engraçado? Na parte subjetiva, o erro vai se perpetuando tanto que fica melhor assim :)

Por exemplo:

principalmente se outras pessoas forem dar manutenção no código ("Por que não usou um for?")

As pessoas usam tanto errado, que "ninguém" mais pergunta por que não usou o for :D O que obviamente, estritamente o map() continua sendo a pior solução, mas as pessoas param de questionar. E aí vem a frase que sempre uso: quando você treina o erro é ele que fará sempre. E vai brigar que ele é o certo.

Para quem faz questão de uma linha:

for (const n of array) console.log(n * 2);

Ah, mas é mais longo. Objetivo do código não é ter menos caracteres, é de produzir o resultado certo da melhor forma possível. Esta forma é semanticamente mais correta e mais eficiente.

Com forEach() não é tão errado, mas ainda é menos eficiente, e ainda é uma desnecessidade. O que não é necessário nunca deveria ser usado.

Os outros exemplos mostram como usar + map() facilmente se torna mais comploicado e contém armadilhas. Já viu os malabarismos que o pessoal faz porque precisava de um continue que obviamente não tem no map()?

Espero ter ajudado.

Farei algo que muitos pedem para aprender a programar corretamente, gratuitamente. Para saber quando, me segue nas suas plataformas preferidas. Quase não as uso, não terá infindas notificações (links aqui).