Angular：返回 Observable 的正确方法是什么？

Question

我有以下方法不正确：

  getProducts(): Observable<Product[]> {
      let PRODUCTS: Product[];
      this.http.get(this.base_url + "api/products")
      .subscribe(
        (data) => {
            for(var i in data) {
                PRODUCTS.push(new Product(data[i].id, data[i].name, data[i].category, data[i].description, data[i].price, data[i].amount));
            }
        },
        (error) => {
            console.log(error);
      });
      return of(PRODUCTS);
  }

我得到的错误是：

TypeError: Cannot read property 'push' of undefined

现在，我知道 PRODUCT 数组无法从 subscribe 函数中访问，但我无法获得正确的解决方案。

谁能帮帮我。我想返回一个Observable<Product[]>。

提前谢谢你！

Answer 1

编辑：更新以说明 API 似乎返回一个类似数组的对象而不是真正的数组。

你想使用map:

getProducts(): Observable<Product[]> {
  return this.http.get(this.base_url + "api/products")
    .map(data => {
      let products = [];
      for (let i in data) {
        products.push(new Product(data[i].id, data[i].name, data[i].category, data[i].description, data[i].price, data[i].amount));
      }

      return products;
    })
    .do(null, console.log);
}

---

由于@pixelbit 的评论尽管有误，但仍不断获得支持，这里有一个示例说明为什么它是错误的：

// Fakes a HTTP call which takes half a second to return
const api$ = Rx.Observable.of([1, 2, 3]).delay(500);

function getProducts() {
  let products = [];
  api$.subscribe(data => {
    for (let i in data) {
      products.push(data[i]);
    }
  });

  return Rx.Observable.of(products);
}

// Logs '[]' instead of '[1, 2, 3]'
getProducts().subscribe(console.log);