Como capturar a exceção corretamente de http.request ()?

132

Parte do meu código:

import {Injectable} from 'angular2/core';
import {Http, Headers, Request, Response} from 'angular2/http';
import {Observable} from 'rxjs/Observable';
import 'rxjs/add/operator/map';

@Injectable()
export class myClass {

  constructor(protected http: Http) {}

  public myMethod() {
    let request = new Request({
      method: "GET",
      url: "http://my_url"
    });

    return this.http.request(request)
      .map(res => res.json())
      .catch(this.handleError); // Trouble line. 
                                // Without this line code works perfectly.
  }

  public handleError(error: Response) {
    console.error(error);
    return Observable.throw(error.json().error || 'Server error');
  }

}

myMethod() produz exceção no console do navegador:

EXCEÇÃO ORIGINAL: TypeError: this.http.request (...). Map (...). Catch não é uma função

mnv
fonte

Respostas:

213

Talvez você possa tentar adicionar isso em suas importações:

import 'rxjs/add/operator/catch';

Você também pode fazer:

return this.http.request(request)
  .map(res => res.json())
  .subscribe(
    data => console.log(data),
    err => console.log(err),
    () => console.log('yay')
  );

Por comentários:

EXCEÇÃO: TypeError: Observable_1.Observable.throw não é uma função

Da mesma forma, para isso, você pode usar:

import 'rxjs/add/observable/throw';
acdcjunior
fonte
2
Obrigado pela ajuda, funciona. Depois disso, eu tenho o mesmo problema com a throw()função. Eu adicionei esta linha import 'rxjs/Rx';em seu lugar. Agora todos os operadores funcionam corretamente.
mnv
Você simulou um erro para ver se .catchrealmente funciona? Isso .subscribe() funciona com certeza.
Acdcjunior
1
Sim, o segundo problema foi EXCEPTION: TypeError: Observable_1.Observable.throw is not a function. Ele pode ser corrigido com a resposta @MattScarpino ou de maneira mais manerosa a partir deste plunker, como eu disse acima: angular.io/resources/live-examples/server-communication/ts/…
mnv
16
Apenas importe o throw também: import 'rxjs/add/observable/throw';e não importe tudo, é muito grande.
dfsq
Ótima solução ,, muito útil, devo acrescentar o (err) é do tipo Response #
Mohammed Suez
77

Novo serviço atualizado para usar o HttpClientModule e o RxJS v5.5.x :

import { Injectable }                    from '@angular/core';
import { HttpClient, HttpErrorResponse } from '@angular/common/http';
import { Observable }                    from 'rxjs/Observable';
import { catchError, tap }               from 'rxjs/operators';
import { SomeClassOrInterface}           from './interfaces';
import 'rxjs/add/observable/throw';

@Injectable() 
export class MyService {
    url = 'http://my_url';
    constructor(private _http:HttpClient) {}
    private handleError(operation: String) {
        return (err: any) => {
            let errMsg = `error in ${operation}() retrieving ${this.url}`;
            console.log(`${errMsg}:`, err)
            if(err instanceof HttpErrorResponse) {
                // you could extract more info about the error if you want, e.g.:
                console.log(`status: ${err.status}, ${err.statusText}`);
                // errMsg = ...
            }
            return Observable.throw(errMsg);
        }
    }
    // public API
    public getData() : Observable<SomeClassOrInterface> {
        // HttpClient.get() returns the body of the response as an untyped JSON object.
        // We specify the type as SomeClassOrInterfaceto get a typed result.
        return this._http.get<SomeClassOrInterface>(this.url)
            .pipe(
                tap(data => console.log('server data:', data)), 
                catchError(this.handleError('getData'))
            );
    }

Serviço antigo, que usa o HttpModule descontinuado:

import {Injectable}              from 'angular2/core';
import {Http, Response, Request} from 'angular2/http';
import {Observable}              from 'rxjs/Observable';
import 'rxjs/add/observable/throw';
//import 'rxjs/Rx';  // use this line if you want to be lazy, otherwise:
import 'rxjs/add/operator/map';
import 'rxjs/add/operator/do';  // debug
import 'rxjs/add/operator/catch';

@Injectable()
export class MyService {
    constructor(private _http:Http) {}
    private _serverError(err: any) {
        console.log('sever error:', err);  // debug
        if(err instanceof Response) {
          return Observable.throw(err.json().error || 'backend server error');
          // if you're using lite-server, use the following line
          // instead of the line above:
          //return Observable.throw(err.text() || 'backend server error');
        }
        return Observable.throw(err || 'backend server error');
    }
    private _request = new Request({
        method: "GET",
        // change url to "./data/data.junk" to generate an error
        url: "./data/data.json"
    });
    // public API
    public getData() {
        return this._http.request(this._request)
          // modify file data.json to contain invalid JSON to have .json() raise an error
          .map(res => res.json())  // could raise an error if invalid JSON
          .do(data => console.log('server data:', data))  // debug
          .catch(this._serverError);
    }
}

eu uso .do() ( agora.tap() ) para depuração.

Quando há um erro no servidor, o bodydoResponse objeto que eu começar a partir do servidor que estou usando (lite-server) contém apenas texto, daí a razão de eu usar err.text()acima em vez de err.json().error. Pode ser necessário ajustar essa linha para o seu servidor.

Se res.json()gera um erro porque não pôde analisar os dados JSON,_serverError não obterá um Responseobjeto, daí o motivo da instanceofverificação.

Neste plunker, mudarurl para ./data/data.junkpara gerar um erro.


Os usuários de qualquer serviço devem ter um código que possa lidar com o erro:

@Component({
    selector: 'my-app',
    template: '<div>{{data}}</div> 
       <div>{{errorMsg}}</div>`
})
export class AppComponent {
    errorMsg: string;
    constructor(private _myService: MyService ) {}
    ngOnInit() {
        this._myService.getData()
            .subscribe(
                data => this.data = data,
                err  => this.errorMsg = <any>err
            );
    }
}
Mark Rajcok
fonte
4

Existem várias maneiras de fazer isso. Ambos são muito simples. Cada um dos exemplos funciona muito bem. Você pode copiá-lo em seu projeto e testá-lo.

O primeiro método é preferível, o segundo está um pouco desatualizado, mas até agora funciona também.

1) Solução 1

// File - app.module.ts
import { BrowserModule } from '@angular/platform-browser';
import { NgModule } from '@angular/core';
import { HttpClientModule } from '@angular/common/http';

import { AppComponent } from './app.component';
import { ProductService } from './product.service';
import { ProductModule } from './product.module';

@NgModule({
  declarations: [
    AppComponent
  ],
  imports: [
    BrowserModule,
    HttpClientModule
  ],
  providers: [ProductService, ProductModule],
  bootstrap: [AppComponent]
})
export class AppModule { }



// File - product.service.ts
import { Injectable } from '@angular/core';
import { HttpClient } from '@angular/common/http';

// Importing rxjs
import 'rxjs/Rx';
import { Observable } from 'rxjs/Rx';
import { catchError, tap } from 'rxjs/operators'; // Important! Be sure to connect operators

// There may be your any object. For example, we will have a product object
import { ProductModule } from './product.module';

@Injectable()
export class ProductService{
    // Initialize the properties.
    constructor(private http: HttpClient, private product: ProductModule){}

    // If there are no errors, then the object will be returned with the product data.
    // And if there are errors, we will get into catchError and catch them.
    getProducts(): Observable<ProductModule[]>{
        const url = 'YOUR URL HERE';
        return this.http.get<ProductModule[]>(url).pipe(
            tap((data: any) => {
                console.log(data);
            }),
            catchError((err) => {
                throw 'Error in source. Details: ' + err; // Use console.log(err) for detail
            })
        );
    }
}

2) Solução 2. É antigo, mas ainda funciona.

// File - app.module.ts
import { BrowserModule } from '@angular/platform-browser';
import { NgModule } from '@angular/core';
import { HttpModule } from '@angular/http';

import { AppComponent } from './app.component';
import { ProductService } from './product.service';
import { ProductModule } from './product.module';

@NgModule({
  declarations: [
    AppComponent
  ],
  imports: [
    BrowserModule,
    HttpModule
  ],
  providers: [ProductService, ProductModule],
  bootstrap: [AppComponent]
})
export class AppModule { }



// File - product.service.ts
import { Injectable } from '@angular/core';
import { Http, Response } from '@angular/http';

// Importing rxjs
import 'rxjs/Rx';
import { Observable } from 'rxjs/Rx';

@Injectable()
export class ProductService{
    // Initialize the properties.
    constructor(private http: Http){}

    // If there are no errors, then the object will be returned with the product data.
    // And if there are errors, we will to into catch section and catch error.
    getProducts(){
        const url = '';
        return this.http.get(url).map(
            (response: Response) => {
                const data = response.json();
                console.log(data);
                return data;
            }
        ).catch(
            (error: Response) => {
                console.log(error);
                return Observable.throw(error);
            }
        );
    }
}
Victor Isaikin
fonte
-1

As funções RxJS precisam ser importadas especificamente. Uma maneira fácil de fazer isso é importar todos os seus recursos comimport * as Rx from "rxjs/Rx"

Então, certifique-se de acessar a Observableclasse como Rx.Observable.

MatthewScarpino
fonte
15
Rxjs é um arquivo muito grande, se você importar todos os seus recursos, aumentará o seu tempo de carregamento
Soumya Gangamwar
Você não deve apenas importar tudo do Rxjs se precisar de apenas um ou dois operadores.
marcel-k
-4

na versão mais recente do angular4 use

import { Observable } from 'rxjs/Rx'

ele importará todas as coisas necessárias.

Munish Sharma
fonte
20
Não faça isso, ele importará todos os Rxjs.
marcel-k
E assim resultará no aumento do tamanho do pacote!
Tushar Walzade 15/0318