Respondiendo a tu pregunta específica
"¿Esta función me devuelve un error alguien me podría explicar porqué?"
Antes de nada, el código que aportas está incompleto ya que si copio y pego directamente en un archivo .js, éste falla por estructura.
Entiendo que al faltar la palabra reservada "function" y ver el this
, hablamos de un método que has extraido de una clase/objeto y expuesto aquí. Como comentario para futuras ocasiones te recomiendo agregar tódo el código o realizar pseudocódigo lo mas fiel al problema para facilitar la respuesta, en este caso al ser algo simple no es mucho problema, de otra forma son "errores adicionales".
Si partimos de que el código debería ser algo similar a:
class my_service {
contenido = [
{id: 1, condicion: 'alquilar'},
{id: 2, condicion: 'otra_cosa'},
{id: 3, condicion: 'diferente_cosa'},
{id: 4, condicion: 'alquilar'},
{id: 5, condicion: 'alquilar'},
{id: 6, condicion: 'algo_distinto'}
];
getalquiler() {
this.contenido.forEach(obj => {
if (obj.condicion === 'alquilar'){
return obj;
}
});
}
}
DEBUG Y ANÁLISIS
No existe ningún problema en cuanto a ejecución, es decir funciona correctamente, sin embargo no realiza lo que buscas concretamente.
Analicemos juntos el código y que es lo que está sucediendo.
1.- Acceso propiedad del objeto
Cuando situas condicion
, entiendo que es el atributo de tus objetos en ese array de datos, es decir la propiedad de ese objeto que almacena la palabra que buscas 'alquilar', si no es así, entonces fíjate en el ejemplo, ya que estás accediendo a otra propiedad distinta.
2.- Hay datos de tipo 'alquilar' en tu base de datos
Por otra parte, si no existe ningún objeto que sea de alquilar
entonces digamos que tu condición if
, no va a encontrar nada.
3.- Si existen datos y los encontramos ¿A dónde van con el return?
Es decir, retornas pero el error está en el foreach
, ya que no es una función que te permita retornar y almacenar como lo estás haciendo.
Para esta finalidad existen otras funciones específicas de JS y por esta razón tu función deolvería como mucho "undefined" como puedes observar en la ejecución ejemplo final.
Si sitúas un return console.log(obj)
en tu función, podrás observar por consola está correcta, es decir encuentra y llega a mostrar el objeto encontrado / filtrado pero no se retorna ni almacena "en ninguna parte".
Ahora, he mencionado algo interesante filtrado
, quiero invitarte a investigar las funciones nativas de javascript que te ahorran tiempo, esfuerzo y "reinventar la rueda" por así decirlo.
SOLUCIÓN
Puedes ahorrarte usar foreach
+ if
+ return
+ {}
, de la siguiente forma:
getalquiler() {
return this.contenido.filter((obj) => obj.condicion === 'alquilar');
}
Traducido a lenguaje humano sería:
"Método getalquiler
que retorna la "operación" de filtrado de tu array de objetos cuando el objeto cumpla la condición de 'alquilar'."
La operación que buscamos es filter
, que significa directamente filtrar.
Resumiendo su funcionamiento, realiza un bucle sobre tu array de objetos (como un foreach), los cuales va retornando en la variable obj
("magia interna de JS"). Ahora la propia función filter retorna un array con aquellos objetos que cumplan la condición de filtrado directamente,tan solo debes igualar la operación a una nueva variable.
ENRIQUECIMIENTO ADICIONAL
Puedes leer mas en detalle:
https://developer.mozilla.org/es/docs/Web/JavaScript/Referencia/Objetos_globales/Array/filter
Además, te sugiero mirar "map, find, findIndexOff, fill, some, every"
FUNCIONAL COMPARATIVO
Aquí presento el ejemplo funcional:
class my_service {
contenido = [
{id: 1, condicion: 'alquilar'},
{id: 2, condicion: 'otra_cosa'},
{id: 3, condicion: 'diferente_cosa'},
{id: 4, condicion: 'alquilar'},
{id: 5, condicion: 'alquilar'},
{id: 6, condicion: 'algo_distinto'}
];
getalquiler_new() {
return this.contenido.filter((obj) => obj.condicion === 'alquilar');
}
getalquiler_old() {
return this.contenido.forEach(obj => {
if (obj.condicion === 'alquilar'){
return obj;
}});
}
}
const servicio = new my_service();
console.log(servicio.getalquiler_new());
console.log(servicio.getalquiler_old());
forEach
, SIEMPRE vas a obtenerundefined
, que es lo único que devuelveforEach
tal y como indica la documentación del método... Cómo pretendes que funcione unreturn
dentro de un método QUE SE DEBE EJECUTAR PARA CADA OBJETO? Cual devolvería, el primero que encuentre? El segundo? El último? Para eso deberías usarfilter
, tal como te indica @AlbertoSiurob en su respuesta