Skip to content

PP-WDFT-MEX-ALEJANDRA-OCHOA-ISRAEL-SANCHEZ#619

Closed
AleOchoa wants to merge 1 commit into
ironhack-labs:masterfrom
AleOchoa:master
Closed

PP-WDFT-MEX-ALEJANDRA-OCHOA-ISRAEL-SANCHEZ#619
AleOchoa wants to merge 1 commit into
ironhack-labs:masterfrom
AleOchoa:master

Conversation

@AleOchoa
Copy link
Copy Markdown

@AleOchoa AleOchoa commented Jan 6, 2020

En el bonus 2 sólo consideramos acentos, espacios y los caracteres especiales: ¿ ? ! ¡ ,

Quisimos hacerlo nosotros solos, pero podríamos buscar otras soluciones en google. :)

Copy link
Copy Markdown

@ta-web-mex ta-web-mex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Muy buen ejercicio chicos! Pongan un poco más de atención a los detalles y siempre busquen en google como otros solucionan los problemas de los labs para que su apredizaje no se quede únicamente en la clase, si no que prueben cosas nuevas y se empapen de código en el bootcamp.
Sigan así! 👏

Comment thread js/index.js
const hacker1 = "Ale"
console.log("The driver's name is "+hacker1)
const hacker2 = "Israel"
console.log("The navigator's name is "+hacker1)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aquí dejaron hacker1 como nombre de navigator

Comment thread js/index.js
console.log('The driver has the longest name, it has' +hacker1.length+" characters.")
}
else{
if(hacker1.length==hacker2.length)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cuidado con la doble igualdad, es mejor utilizar triple igualdad si sabes que el tipo de dato es el mismo, como en este caso. No por nada hay mil memes de la doble igualdad ;)
best-js-meme-to-date-2

Comment thread js/index.js
}

// Iteration 3: Loops No newline at end of file
// Iteration 3: Loops
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Para esta iteración pudieron haber usado un método de los strings de JS que se llama localeCompare() que regresa 1, 0 o -1 dependiendo del orden lexicográfico de las palabras o cero si son iguales.

Comment thread js/index.js

let phraseToCheck = "Amor, Roma"
let phrase=phraseToCheck
phraseToCheck=phraseToCheck.replace(" ","")
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Para usar replace() con todos los casos que escribieron línea por línea, pudieron haber utilizado una expresión regular (regex) para incluir todos esos elementos en el primer parámetro del método y reemplazarlos por un espacio. En este momento puede parecer un poco avanzado pero les dejo liga a una introducción a regex de MDN para que le echen un ojo si quieren :)
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions

@ta-web-mex ta-web-mex closed this Jan 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants