Joselobo753 / js-tp2-2-Rolingcode

js-tp2-2-Rolingcode
0 stars 0 forks source link

Ejercicio 5 #4

Open ezeamin opened 7 months ago

ezeamin commented 7 months ago

Acá te recomiendo lo mismo que en issue anterior. vocales no indica acción, me costó entender que era una funcion y no una variable. Sería, en todo caso, contarVariables. Además, después asignas un número a vocal, que gramáticamente tampoco tiene sentido.

 let vocal = vocales(cadena);
 let espa = espacios(cadena);

¿Qué vendria a ser espacios++?

switch(cadena[i]){
  // ...
  case " ":
    espacios++;
  // ...
}

Un switch para una sola opción es como usar una raqueta de tenis para jugar al ping pong, te sobra.

function espacios(cadena) {
    let espaci = 0;
    for (let i = 0; i < cadena.length; i++) {
      switch (cadena[i]) {
        case " ":
          espaci++;
        default:
          break;
      }
      return espaci;
    }
  }

Revisar esta respuesta para el siguiente código:

for (let i = 0; i < cadena.length; i++) {
      if (cadena[i] === cadena[i].toUpperCase()) {
        contarMay++;
      } else if (cadena[i] === cadena[i].toLowerCase()) {
        contarMin++;
      }
    }