-2

Tengo el siguiente código para comprobar si la primera letra de una cadena es mayúscula o no. ¿Se puede optimizar más el siguiente algoritmo quizás con el condicional ternario?

'use strict'

var cadena = 'EsAS';

var mayusculas ='ABCDEFGHIJKLMNÑOPQRSTUWVXYZ';

var respuesta=''

for(var i=0;i < mayusculas.length;i++){

    if(cadena[0]==mayusculas[i]){

        respuesta="la primera letra es mayuscula"
    }

}

console.log(respuesta)
1
  • La pregunta se basa principalmente en opiniones pues no indica ningún criterio de optimización, como reducir el número de líneas de código, mejorar el tiempo de ejecución, etc.
    – Rubén
    el 21 oct. 2018 a las 20:32

5 respuestas 5

6

Puedes hacerlo sin buscar tu primera letra en otra cadena haciendo lo siguiente.

function esMayuscula (str) {
  if (str[0] === str[0].toUpperCase()) {
    return 'la primera letra es mayuscula';
  }
  return 'la primera es miniscula';
}
console.log(esMayuscula('EsAs'));
console.log(esMayuscula('esAs'));

Solamente verificas tu primer elemento con el primer elemento siempre en mayúscula.

1
  • 3
    Mas alla que la pregunta es mala, esta es la unica respuesta que parece valer la pena.
    – gbianchi
    el 18 oct. 2018 a las 13:50
0

El operador ternario reduciría código solamente. Puedes usar las funciones que te brinda javascript como indexOf.

Ejemplo:

'use strict'

var cadena = 'EsAS';

var mayusculas ='ABCDEFGHIJKLMNÑOPQRSTUWVXYZ';

var respuesta=''
if (mayusculas.indexOf(cadena[0]) >= 0){
    respuesta="la primera letra es mayuscula"
}

La función indexOf te devuelve la posición donde se encuentra el carácter que buscas en el string mayusculas, si devuelve -1 es que no lo ha encontrado.

Saludos.

0

Si se puede, incluso no hace falta el bucle for. Lo puedes hacer utilizando la function indexOf, es decir comprobar si en la variable mayusculas existe la primera letra de la variable cadena.

let cadena = 'EsAS';

let mayusculas ='ABCDEFGHIJKLMNÑOPQRSTUWVXYZ';

//Operador termario
let respuesta = mayusculas.indexOf(cadena[0]) > -1 ? "la primera letra es mayuscula": ""

console.log(respuesta)
0

MEJORAS

Ayúdate de lo sigueinte para mejorar tu código

  1. en lugar de === para comparar si el valor existe; en ES6 existe inlcudes
  2. ya no uses var sustituyelo por let para variables

Tú código

    let cadena = 'EsAS';
    let mayusculas ='ABCDEFGHIJKLMNÑOPQRSTUWVXYZ';
    
    let respuesta=''
    
    for(let i=0;i < mayusculas.length;i++){
    
        if(cadena[0].includes(mayusculas[i])){
            respuesta = "la primera letra es mayuscula"
        }
    }
    
    console.log(respuesta)

5
  • 2
    'use strict'; es totalmente necesario a menos que declares el código como un módulo (en cuyo caso el modo estricto ya está activado) el 18 oct. 2018 a las 14:09
  • añado documentacion del 'use strict' el 18 oct. 2018 a las 14:50
  • Gracias compañeros por la corrección y referencia pues según de un curso aprendí que ya no era necesario pero no me dijeron el por qué
    – user75901
    el 18 oct. 2018 a las 14:56
  • en el ejemplo que colocastes definistes todas las variables , si quitas una definicion y agregas el use strict te dara error que la variable es indefinida(no declarada) el 18 oct. 2018 a las 15:11
  • Si @ JackNavaRow ahora comprendo mejor su uso gracias pues estaba en un error
    – user75901
    el 18 oct. 2018 a las 15:15
-1

Viendo todas las demás respuestas en negativo igual quise intentarlo y ver si podía dar una respuesta válida. Simplemente cree una función el cual recibe la cadena y siempre esta evaluando el primer caracter de la cadena con sigo misma pero en mayúscula así:

cadena.charAt(0) === cadena.charAt(0).toUpperCase()

'use strict'

const mayuscula = (cadena) => {
  return (cadena.charAt(0) === cadena.charAt(0).toUpperCase()) ? "la primera letra es mayuscula" : "la primera letra NO es mayuscula";
}

let cadena = 'EsAS';
let cadena2 = 'esAS';
console.log(mayuscula(cadena));
console.log(mayuscula(cadena2));

Espero que sea lo que andas buscando.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.