Come catturare correttamente l'eccezione da http.request ()?


132

Parte del mio codice:

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() produce un'eccezione nella console del browser:

ECCEZIONE ORIGINALE: TypeError: this.http.request (...). Map (...). Catch non è una funzione

Risposte:


213

Forse puoi provare ad aggiungere questo nelle tue importazioni:

import 'rxjs/add/operator/catch';

Puoi anche fare:

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

Per commenti:

ECCEZIONE: TypeError: Observable_1.Observable.throw non è una funzione

Allo stesso modo, per quello, puoi usare:

import 'rxjs/add/observable/throw';

2
Grazie per l'aiuto, funziona. Dopodiché ho lo stesso problema con la throw()funzione. Ho aggiunto import 'rxjs/Rx';invece questa riga . Ora tutti gli operatori funzionano correttamente.
mnv,

Hai simulato un errore per vedere se .catchfunziona davvero? Che .subscribe() opere di sicuro.
acdcjunior,

1
Sì, il secondo problema era EXCEPTION: TypeError: Observable_1.Observable.throw is not a function. Può essere risolto con la risposta di @MattScarpino o in modo diretto da questo plunker come ho detto sopra: angular.io/resources/live-examples/server-communication/ts/…
mnv,

16
Importa anche buttare: import 'rxjs/add/observable/throw';e non importare tutto, è troppo grande.
dfsq,

Ottima soluzione ,, molto utile, potrei aggiungere il (err) è di tipo Response
Mohammed Suez,

77

Nuovo servizio aggiornato per utilizzare HttpClientModule e 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'))
            );
    }

Vecchio servizio, che utilizza HttpModule obsoleto:

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);
    }
}

Uso .do()( ora.tap() ) per il debug.

Quando c'è un errore del server, l' bodydel Responseoggetto ricevo dal server che sto utilizzando (lite-server) contiene solo testo, quindi il motivo che uso err.text()sopra piuttosto che err.json().error. Potrebbe essere necessario regolare quella linea per il tuo server.

Se res.json()genera un errore perché non è stato in grado di analizzare i dati JSON, _serverErrornon otterrà un Responseoggetto, quindi il motivo del instanceofcontrollo.

In questo plunker, cambia urlin ./data/data.junkper generare un errore.


Gli utenti di entrambi i servizi dovrebbero avere un codice in grado di gestire l'errore:

@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
            );
    }
}

4

Esistono diversi modi per farlo. Entrambi sono molto semplici. Ciascuno degli esempi funziona alla grande. Puoi copiarlo nel tuo progetto e testarlo.

Il primo metodo è preferibile, il secondo è un po 'obsoleto, ma finora funziona anche.

1) Soluzione 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) Soluzione 2. È vecchio ma funziona ancora.

// 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);
            }
        );
    }
}

-1

Le funzioni RxJS devono essere importate in modo specifico. Un modo semplice per farlo è importare tutte le sue funzionalità conimport * as Rx from "rxjs/Rx"

Quindi assicurati di accedere alla Observableclasse come Rx.Observable.


15
Rxjs è un file molto grande, se importi tutte le sue funzionalità aumenterà il tuo tempo di caricamento
Soumya Gangamwar

Non dovresti semplicemente importare tutto da Rxjs se hai bisogno solo di uno o due operatori.
marcel-k,

-4

nell'ultima versione di angular4 use

import { Observable } from 'rxjs/Rx'

importerà tutte le cose richieste.


20
Non farlo, importerà tutti gli Rxjs.
marcel-k,

E così si otterrà un aumento delle dimensioni del pacchetto!
Tushar Walzade,
Utilizzando il nostro sito, riconosci di aver letto e compreso le nostre Informativa sui cookie e Informativa sulla privacy.
Licensed under cc by-sa 3.0 with attribution required.