Primer code review - Fabiola González#23
Conversation
HectorBlisS
left a comment
HectorBlisS
left a comment
There was a problem hiding this comment.
Tienes un gran progreso, pero es importante poner atención a los detalles cómo poner todo en su lugar para que después no se rompa tu programa al mover cosas importantes al final, hazlo lo antes posible.
src/pruebas.js
Outdated
| var letterOriginal= inputText.charAt(i); | ||
| var letter = letterOriginal.toUpperCase(); | ||
| var letterCode = letter.charCodeAt(); | ||
| var letterCodeNumber = parseInt(letterCode); | ||
| if(letterCodeNumber == 32){ |
There was a problem hiding this comment.
Es importante cambiar var por let o const (la mayoría pide const aunque no hay gran diferencia entre let y const es un debate) pero var compromete tu código, ya no lo uses.
src/pruebas.js
Outdated
| }else{ | ||
| codeNumberCiphered = (letterCodeNumber-65+offset)%26+65; | ||
| } | ||
| var cipheredLetter = String.fromCharCode(codeNumberCiphered); |
| codeNumberCiphered = (letterCodeNumber-65+offset)%26+65; | ||
| } | ||
| var cipheredLetter = String.fromCharCode(codeNumberCiphered); | ||
| console.log(letter+" "+letterCode+" "+codeNumberCiphered+" "+cipheredLetter); |
There was a problem hiding this comment.
de preferencia crea una variable que almacene todo esto y despues solo le das al console log la variable
src/style.css
Outdated
| background-color: #2196F3; | ||
| } | ||
|
|
||
| input:focus + .slider { |
There was a problem hiding this comment.
Esto es muy específico y está bien, pero puede fallar, mejor usa el selector de descendente (espacio) en vez del + para que sea más seguro
…ffset and special characters, including ñand space, remain the same.
2 participants
Para el cipher, generé un archivo llamado pruebas.js que contiene todo lo que he trabajado al respecto. Sin embargo, ya he puesto mi función en el objeto cipher del archivo cipher.js
En HTML solo tengo estructurada la maqueta pero aún faltan estilos de css.
¡Gracias!