Skip to main content
cuerpo editado
Origen Enlace
Mariano
  • 24.5k
  • 20
  • 76
  • 105

Ahora bien, para coincidir con una letra, conviene usar una clase de caracteres (que es un poco más eficiente, y más sencilla de escribir y de leer). Sería lo mismo que:

capitalized = results.replace(/[aeiu]/g, function(match){
    return match.toUpperCase();
});

Nótese también que el modificador /i es innecesario (no hace falta reemplazar las vocales en mayúscula por su misma letra).

Además, podría haberse puesto como un IF dentro del reemplazo anterior (con a-z). Si ya estábaestaba reemplazando todas las letras, sólo era necesario ver si era una vocal ahí y devolverla en mayúsculas. No creo que esté bueno estar llamando a un nuevo replace() para cada condición.

Ahora bien, para coincidir con una letra, conviene usar una clase de caracteres (que es un poco más eficiente, y más sencilla de escribir y de leer). Sería lo mismo que:

capitalized = results.replace(/[aeiu]/g, function(match){
    return match.toUpperCase();
});

Nótese también que el modificador /i es innecesario (no hace falta reemplazar las vocales en mayúscula por su misma letra).

Además, podría haberse puesto como un IF dentro del reemplazo anterior (con a-z). Si ya estába reemplazando todas las letras, sólo era necesario ver si era una vocal ahí y devolverla en mayúsculas. No creo que esté bueno estar llamando a un nuevo replace() para cada condición.

Ahora bien, para coincidir con una letra, conviene usar una clase de caracteres (que es un poco más eficiente, y más sencilla de escribir y de leer). Sería lo mismo que:

capitalized = results.replace(/[aeiu]/g, function(match){
    return match.toUpperCase();
});

Nótese también que el modificador /i es innecesario (no hace falta reemplazar las vocales en mayúscula por su misma letra).

Además, podría haberse puesto como un IF dentro del reemplazo anterior (con a-z). Si ya estaba reemplazando todas las letras, sólo era necesario ver si era una vocal ahí y devolverla en mayúsculas. No creo que esté bueno estar llamando a un nuevo replace() para cada condición.

se añadieron 294 caracteres en el cuerpo
Origen Enlace
Mariano
  • 24.5k
  • 20
  • 76
  • 105

Ahora bien, para coincidir con una letra, conviene usar una clase de caracteres (que es un poco más eficiente, y más sencilla de escribir y de leer). Sería lo mismo que:

capitalized = results.replace(/[aeiu]/g, function(match){
    return match.toUpperCase();
});

Nótese también que el modificador /i es innecesario (no hace falta reemplazar las vocales en mayúscula por su misma letra).

Además, podría haberse puesto como un IF dentro del reemplazo anterior (con a-z). Si ya estába reemplazando todas las letras, sólo era necesario ver si era una vocal ahí y devolverla en mayúsculas. No creo que esté bueno estar llamando a un nuevo replace() para cada condición.

Ahora bien, para coincidir con una letra, conviene usar una clase de caracteres (que es un poco más eficiente, y más sencilla de escribir y de leer). Sería lo mismo que:

capitalized = results.replace(/[aeiu]/g, function(match){
    return match.toUpperCase();
});

Nótese también que el modificador /i es innecesario (no hace falta reemplazar las vocales en mayúscula por su misma letra).

Ahora bien, para coincidir con una letra, conviene usar una clase de caracteres (que es un poco más eficiente, y más sencilla de escribir y de leer). Sería lo mismo que:

capitalized = results.replace(/[aeiu]/g, function(match){
    return match.toUpperCase();
});

Nótese también que el modificador /i es innecesario (no hace falta reemplazar las vocales en mayúscula por su misma letra).

Además, podría haberse puesto como un IF dentro del reemplazo anterior (con a-z). Si ya estába reemplazando todas las letras, sólo era necesario ver si era una vocal ahí y devolverla en mayúsculas. No creo que esté bueno estar llamando a un nuevo replace() para cada condición.

Origen Enlace
Mariano
  • 24.5k
  • 20
  • 76
  • 105

Copio los fragmentos de tu código y voy describiendo:

function LetterChanges(str) { 
    var results = "", vowels = "aeiou", capitalized = "", char = "", znZ = ""; 
    
    znZ = str.replace(/z|Z/gi, function(i){
       return "a"; // Reemplaza la z o Z por a
    });

Tal cual describiste, reemplaza las z o Z por a.

Esta forma de reemplazar se repite en todo el código. La función String.prototype.replace() permite pasar una función para el reemplazo (un callback). Esa función, recibe como parámetro la coincidencia del regex (que se asigna a la variable i, y espera el valor por el cual reemplazarlo como valor devuelto por la función ("a" en este caso).

De todas formas, esto es lo mismo que:

znZ = str.replace( /z/gi, "a");

porque al usar el modificador /i ya está ignorando mayúsculas/minúsculas, y claramente no es necesario pasar una función que siempre devuelva el mismo resultado.



znZ.replace(/[a-z]/gi, function(i){ return results += String.fromCharCode(i.charCodeAt() + 1); }); // Ahora si encuentra una letra entre a-z('i' - > insensitivo) la reemplazara por su proximo charCode(lo hace sumandole 1), pero esto no debería funcionar porque ya reemplazo la z en la anterior funcion, no?

Es correcto, incrementa todas las letras. Y también es correcto que ya reemplazó la z, por lo que tanto las z y las a del texto original van a ser reemplazadas por b... Asumo que es un descuido de quien lo programó.

A tener en cuenta, el resultado de la función replace() es descartado. En cambio, se usa la variable results dentro de la función. De esa forma, se está ignorando cualquier caracter que no esté en el rango a-z.



    capitalized = results.replace(/a|e|i|i|u/gi, function(i){
      return i.toUpperCase();
    }); // Sé que aqui las quiere colocar en mayusculas pero no entiendo como lo hace, mas bien no entiendo como funciona la expresión regular.

De nuevo, usando un callback, por lo que la función es llamada cada vez por cada coincidencia de /a|e|i|i|u/gi (o sea, en este caso, por cada letra). Y dentro de la función, la variable i (pasada como parámetro) recibe esa vocal. La función luego devuelve esa vocal en mayúsculas.

En regex, una | implica alternancia, o sea que actúa como un OR. Por lo que la expresión se puede interpretar como: a o e o i o i (sí, probablemente sea un error de tipeo del programador, por lo que esta segunda "i" es innecesaria o debería reemplazarse por "o") o u.

Ahora bien, para coincidir con una letra, conviene usar una clase de caracteres (que es un poco más eficiente, y más sencilla de escribir y de leer). Sería lo mismo que:

capitalized = results.replace(/[aeiu]/g, function(match){
    return match.toUpperCase();
});

Nótese también que el modificador /i es innecesario (no hace falta reemplazar las vocales en mayúscula por su misma letra).



    return capitalized; //Devuelve la string transformada
}

Y así devuelve el texto modificado... ¿Para qué? Ni idea. No es que una función como esta vaya a ofrecer algún grado de encriptación ni mucho menos... Parece más bien un juego, alterando las letras (con algún que otro error de por medio).