Tengo un poco de código en Javascript, que devuelve verdadero cuando obviamente no debería. Para el contexto: tengo un mapa mental y para encontrar las coordenadas más extremas del mapa, recorro todos los nodos y compruebo si este nodo tiene un valor más alto o más bajo que el valor más alto o más bajo actualmente. Al ejecutar el código, el método devuelve valores incorrectos y con console.log() creo que encontré algo extraño, pero no sé cómo solucionarlo.
findMaxX(maxX){ console.log("this.x before test:"+this.x); console.log("maxX before test:"+maxX); if (this.x > maxX){ console.log("this.x:"+this.x); console.log("maxX before:"+maxX); maxX = this.x; console.log("maxX after:"+maxX); } this.children.forEach(element => { maxX = element.findMaxX(maxX); }) return maxX; }Lo que encuentro en la consola es esto:
maxX before test:360 this.x before test:288 maxX before test:360 this.x before test:64 maxX before test:360 this.x:64 maxX before:360 maxX after:64 this.x before test:65 maxX before test:64Parece estar funcionando bien, pero en algún momento la condición parece volverse verdadera para una declaración obviamente falsa. ¿Hay algún error de novato que estoy cometiendo? ¿Dónde puedo buscar errores? Mi código se ve bien para mí y no puedo ver una conexión con el error observado. Estoy feliz por todos los consejos! Gracias
Me parece que this.x y maxX podrían ser cadenas. Y por eso obtienes estos resultados. Aquí puedes ver estos ejemplos (ejecuta este fragmento de código):
let xNumber = 64, yNumber = 360; let xString = "64", yString = "360"; console.log(xNumber > yNumber); console.log(xString > yString);Y lea información sobre la comparación de diferentes tipos en JavaScript aquí
Además, como sugerencia para obtener lo que necesitas (si quieres comparar estas variables como números), te aconsejo convertirlas a Número, por ejemplo de esta forma:
if (parseFloat(this.x) > parseFloat(maxX)) { // here your code }