Varias pautas:
#1:
Por qué agregas la persona y el indice juntos?, es decir, porque haces esto?
push([newPerson, index])
No tiene sentido por donde lo veas, ya que para poder acceder a esa persona requeriras previamente tener un indice, y el indice seria el mismo que el que tienes en tu segunda pocision del arreglo, por lo que seria hacer algo redundante.
#2:
comparar un objeto con otro siempre te dara false
a menos de que los dos objetos sean el mismo o apunten al mismo, por lo que tu condición:
if(arr.indexOf(newPerson) === -1 )
Técnicamente debería cumplirse siempre ya que al comparar 2 objetos totalmente distintos nunca va a encontrarlo en el array.
El problema es que si los objetos son exactamente iguales en cuerpo pero no tienen la misma instancia entonces igual te los añadira, debido a que te los reconocera como un objeto distinto. por ende tu filtro no estaria sirviendo.
#3
: tu ciclo es ineficiente y es innecesario, esto debido a que indexOf
de por si solo ya haria el trabajo del ciclo por ti, por lo que el ciclo es innecesario (esto solo si quieres añadir la nueva persona una sola vez), pues podrias hacer lo siguiente:
if(people.indexOf(newPerson) === -1 ){
people.push([newPerson, index]);
}
En vez de hacer:
people.forEach((curr, index, arr)=>{
if(arr.indexOf(newPerson) === -1 ){
people.push([newPerson, index]);
}
})
Ademas, si tienes esta condicion significa que no quieres que la persona se repita:
if(people.indexOf(newPerson) === -1 ){
people.push([newPerson, index]);
}
Por lo que si me dices que igual necesitas el ciclo, en este caso preguntaria... para qué?, si se supone la idea es no añadir la misma persona mas de una vez.
De todas maneras dejarlo de esta manera:
if(people.indexOf(newPerson) === -1 ){
people.push([newPerson, index]);
}
También estaría errado, pues la condición sea como sea siempre se cumpliría de acuerdo a tu código debido a lo que explique mas arriba sobre instancias y eso.
Lo otro es... en tu codigo donde defines newCharacter
?
ya que estas retornandolo pero no veo donde lo defines:
return newCharacter;
Eso solo generaría un error si no lo tienes definido en tu codigo.
Entonces como soluciono el error?
Comparar dos objetos de manera adecuada puede ser una tarea un tanto ineficiente, pues por esto de las instancias hacer algo tan simple como:
if(Objeto1 == Objeto2){
}
O
if(Objeto1 === Objeto2){
}
O
if(Objeto1.instanceOf(Objeto2)){
}
Todas ellas no te darían el resultado que buscas, algunas personas han inventado métodos como el de convertir ambos objetos en strings primero:
if(Objeto1.toString() === Objeto2.toString()){
}
Y aunque esto si funcione no es lo mas eficiente de realizar segun entiendo.
Por lo que otra opcion seria que crearas tu propio metodo que te enumere las propiedades y los valores y compare cada una de estas con las del otro objeto.
Lo cual sigue siendo ineficiente, incluso mas que convertirlos
simplemente a strings.
Por lo que yo optaria por simplemente convertir ambos a strings
y hacer comparaciones, de esta manera el resultado seria el correcto.