Skip to main content
se añadieron 918 caracteres en el cuerpo
Origen Enlace
Yussef
  • 23.4k
  • 4
  • 12
  • 29

Tienes 2 problemas:

1.- this.route.snapshot.paramMap.get('listaId') puede entregar un valor nulo o string tal como lo indica la documentación

Entonces tu tienes:

const listaId = this.route.snapshot.paramMap.get('listaId');

donde listaId será del tipo null | string, en cambio tu función obtenerLista acepta como argumento el tipo string | number.

2.- El otro problema es que obtenerLista no está retornando algo, es decir es vacio void

Para corregir el primer problema basta con agregar el null como un posible tipo del argumento de la función y el segundo problema se resuelve con agregar return

obtenerLista( id: string | number | null){

  id = Number(id);
  return this.listas.find( listaData => {
      return listaData.id === id;    
  }id);
}

Por último debes tenerLlegado a este punto te encuentras con un par de dilemas:

  • Si id==null entonces al convertirlo a número es cero, ya que Number(null)==0
  • Que el id ingresado no exista en el array, entonces el find retornará undefined

Este es un problema muy comun en cuentala programación. Tu podrias simplemente retornar null o undefined para estos casos y hacer que Number(lista acepte esos tipos -> lista: Lista | null) ==| 0undefined;, lo cual ahora será un posible caso pero esto solo arrastra el problema y eventualmente en algún lado debes validarlo. Por lo que puedes desear manejar esa condiciónOtra forma sería arrojar una excepción, como por ejemplo retornando un array vacioalgo así:

obtenerLista( id: string | number | null) : Lista {
  if(id==nullid == null) returnthrow [];new Error("Id es nulo");

  id = Number(id);
 let returnresp = this.listas.find( listaData => listaData.id === id);

 if(resp) return resp;

 throw new Error("Item no está en el array");
}

Y luego en AgregarPage

  lista: Lista;

  constructor(private deseosService: DeseosService,
              private route: ActivatedRoute) { 

    const listaId return= listaDatathis.idroute.snapshot.paramMap.get('listaId');

 === id;  try{
      this.lista = this.deseosService.obtenerLista( listaId ); 
    }
    catch(err){
       console.error(err);
       //Aquí tomas acciones para alertar que ha ocurrido un problema, puede ser un mensaje al usuario
    }
}

Tienes 2 problemas:

1.- this.route.snapshot.paramMap.get('listaId') puede entregar un valor nulo o string tal como lo indica la documentación

Entonces tu tienes:

const listaId = this.route.snapshot.paramMap.get('listaId');

donde listaId será del tipo null | string, en cambio tu función obtenerLista acepta como argumento el tipo string | number.

2.- El otro problema es que obtenerLista no está retornando algo, es decir es vacio void

Para corregir el primer problema basta con agregar el null como un posible tipo del argumento de la función y el segundo problema se resuelve con agregar return

obtenerLista( id: string | number | null){

  id = Number(id);
  return this.listas.find( listaData => {
      return listaData.id === id;    
  });
}

Por último debes tener en cuenta que Number(null) == 0, lo cual ahora será un posible caso. Por lo que puedes desear manejar esa condición, como por ejemplo retornando un array vacio

obtenerLista( id: string | number | null){
  if(id==null) return [];

  id = Number(id);
  return this.listas.find( listaData => {
      return listaData.id === id;    
  });
}

Tienes 2 problemas:

1.- this.route.snapshot.paramMap.get('listaId') puede entregar un valor nulo o string tal como lo indica la documentación

Entonces tu tienes:

const listaId = this.route.snapshot.paramMap.get('listaId');

donde listaId será del tipo null | string, en cambio tu función obtenerLista acepta como argumento el tipo string | number.

2.- El otro problema es que obtenerLista no está retornando algo, es decir es vacio void

Para corregir el primer problema basta con agregar el null como un posible tipo del argumento de la función y el segundo problema se resuelve con agregar return

obtenerLista( id: string | number | null){

  id = Number(id);
  return this.listas.find( listaData => listaData.id === id);
}

Llegado a este punto te encuentras con un par de dilemas:

  • Si id==null entonces al convertirlo a número es cero, ya que Number(null)==0
  • Que el id ingresado no exista en el array, entonces el find retornará undefined

Este es un problema muy comun en la programación. Tu podrias simplemente retornar null o undefined para estos casos y hacer que lista acepte esos tipos -> lista: Lista | null | undefined; pero esto solo arrastra el problema y eventualmente en algún lado debes validarlo. Otra forma sería arrojar una excepción, algo así:

obtenerLista( id: string | number | null) : Lista {
 if(id == null) throw new Error("Id es nulo");

 id = Number(id);
 let resp = this.listas.find( listaData => listaData.id === id);

 if(resp) return resp;

 throw new Error("Item no está en el array");
}

Y luego en AgregarPage

  lista: Lista;

  constructor(private deseosService: DeseosService,
              private route: ActivatedRoute) { 

    const listaId = this.route.snapshot.paramMap.get('listaId');

    try{
      this.lista = this.deseosService.obtenerLista( listaId ); 
    }
    catch(err){
       console.error(err);
       //Aquí tomas acciones para alertar que ha ocurrido un problema, puede ser un mensaje al usuario
    }
}
Origen Enlace
Yussef
  • 23.4k
  • 4
  • 12
  • 29

Tienes 2 problemas:

1.- this.route.snapshot.paramMap.get('listaId') puede entregar un valor nulo o string tal como lo indica la documentación

Entonces tu tienes:

const listaId = this.route.snapshot.paramMap.get('listaId');

donde listaId será del tipo null | string, en cambio tu función obtenerLista acepta como argumento el tipo string | number.

2.- El otro problema es que obtenerLista no está retornando algo, es decir es vacio void

Para corregir el primer problema basta con agregar el null como un posible tipo del argumento de la función y el segundo problema se resuelve con agregar return

obtenerLista( id: string | number | null){

  id = Number(id);
  return this.listas.find( listaData => {
      return listaData.id === id;    
  });
}

Por último debes tener en cuenta que Number(null) == 0, lo cual ahora será un posible caso. Por lo que puedes desear manejar esa condición, como por ejemplo retornando un array vacio

obtenerLista( id: string | number | null){
  if(id==null) return [];

  id = Number(id);
  return this.listas.find( listaData => {
      return listaData.id === id;    
  });
}