Si bien este no se ve como el mejor código del mundo ni mucho menos, la idea que tuve al realizar este código fue no solo preservar la estructura que ya poseias, si no que ademas funcionara y se extendiera su funcionamiento.
La cosa es que el codigo que poseias tenia algunos errores, uno de ellos es que estabas tratando de retornar una nueva función la cual de por si ya se encontraba mal construida, esto por supuesto generaba errores, si quisieramos corregir eso para que almenos funcione el código, se veria asi:
var objContains = function(obj, prop, value) {
if (obj[prop] === value) {
return true
} else if (typeof obj[prop] === 'object') {
//this es un puntero, y se refiere al contexto actual, en este caso nuestra función
//Podemos ejecutar nuestra función llamandola con this
return this(obj[prop], prop, value)
}
}
Ahora bien, el codigo que poseias, tiene un problema, y es que solo estarias buscando coincidencias en el primer nivel del objeto y con suerte en el segundo nivel del objeto (si llega a existir), ya que no estas iterando el objeto para ir buscando dentro de las propiedades del objeto.
Por lo que propongo el siguiente código, en el cual dentro de el se explica que es lo que se hace.
const user = {
id: 6,
email: "[email protected]",
infoPersonal: {
nombre: "Homero Simpson",
direccion: {
calle: "Avenida Siempreviva",
numero: 742,
barrio: "Springfield",
estado: "Massachusetts",
},
}
}
var objContains = function(obj, prop, value, index = 0){
//Para no realizar demasiadas iteraciones miramos primero si
//En el objeto o sub-objeto existe la propiedad buscada, si existe miramos si
//El objeto en esa propiedad posee nuestro valor, de ser asi entonces
//directamente lo retornamos
if(prop in obj){
if(obj[prop] === value){
return true;
}
}
//Obtenemos solo las keys del objeto en un array
const keys = Object.keys(obj).slice(index);
//Recorremos las propiedades con un for a fin de no desperdiciar memoria
//Y para hacer mas eficiente el proceso
for(let i = 0; i < keys.length; i++){
//Valor actual del objeto en una de sus keys
const curr = obj[keys[i]];
//Retorna true si el valor es encontrado con la clave proporcionada
if(curr === value) {
return true
}else if(typeof curr === 'object'){
//Retornamos la operacion de realizar una recursion del sub-objeto,
//A la vez, aprovechamos de que si la operacion en el sub-objeto devuelve
//false, entonces seguimos recorriendo las demas propiedades del
//Objeto principal hasta encontrar una coincidencia
return objContains(curr, prop, value, 0) ||
objContains(obj, prop, value, ++index);
}else{
//Seguimos con la recursion del objeto actual en caso de que
//el valor no coincida y que ademas el valor no sea del tipo objeto
return objContains(obj, prop, value, ++index);
}
}
//Si al hacer todo el proceso de recursion aun asi no se encuentra el valor
//Entonces damos por sentado de que el valor buscado con su propiedad no existe
return false;
}
console.log(objContains(user, "barrio", "Springfield"));
console.log(objContains(user, "empleo", "Empleado en planta nuclear"));
Notese que las dos ideas principales del código son limitar la recursion para conseguir performance, y ademas que se busque en el objeto cumpleto y en sus sub-objetos (paramos en la primera coincidencia completa), donde la key
y el valor
buscado deben coincidir para ser retornados.
El hecho de que se busque en el objeto completo y en sus sub objetos quizás llegaria a consumir mas memoria y tiempo debido a que busca en todo el objeto la primera coincidencia, sin embargo esto permite que el ciclo principal del objeto no pare en caso de que no encuentre alguna coincidencia en el primer subobjeto, resolviendo el problema de que quizas sí haya una coincidencia, pero ésta no sea tomada en cuenta por no ser el primer objeto evaluado.