开发者问题收集

获取 API、React 后未定义值

2018-01-27
11346

我正在使用 themoviedb API 构建一个简单的电影目录,但是我遇到了一个无法解决的问题。 问题是获取后的结果始终未定义。

我尝试使用 componentWillMount 方法获取数据并在此方法内设置状态,但它不起作用。 我尝试在构造函数内获取,没有结果。

这是我目前的代码

import React, { Component } from 'react';
import Header from './components/Header';
import MovieList from './components/MovieList';
import Footer from './components/Footer';

const MOVIE_API = "http://api.themoviedb.org/3/discover/movie?api_key=72049b7019c79f226fad8eec6e1ee889&language=en-US&sort_by=release_date.desc&include_adult=true&include_video=false&page=2&primary_release_year=2018";

//class
class App extends Component {
  constructor(props){
    super(props);

    this.state = {
      movies: [],
      movieName: ''
    }
  }


  componentWillMount(){
    this.fetchMovie();
  }


//fetching movie
  fetchMovie = () =>{
    const req = new Request(MOVIE_API, {
      method: 'GET',
      cache: 'default'
    });

    fetch(req).then(response =>{
      return response.json();
    }).then(data =>{
      console.log(data); //REF 1;
      this.setState({
        movies: data
      });
    }).catch(err => {
      console.log("ERROR: " + err);
    })
  }

  render() {
    return (
      <div className="root">
        <Header />
        <MovieList moviesRes={this.state.movies}/>
        <Footer />

      </div>
    );
  }
}

export default App;

如您所见,我调用了 componentWillMount 方法获取数据,但它不起作用。 还值得注意的是,如果我记录数据(REF 1),我可以看到结果(json)。

============================

编辑

这是 MovieList 的代码

/*import React, { Component } from 'react';

export default class MovieList extends Component{
    constructor(props){
        super(props);
        this.state = {
            movies: this.props.movieRes
        }
    }
    render(){

        //if result is undefined 
        if(this.state.movieRes === undefined){
            return(
                <h1>Loading...</h1>
            );
        }else{
            return(
                <ul>
                {this.state.movieRes.map((movie, index)=>{
                    return (
                        <li key={index}>{movie.title}</li>
                    );
                })}
                </ul>
            );
        }
    }
}*/

==================

更新子代码

import React, { Component } from 'react';

export default class MovieList extends Component{

    render(){
        const { movieRes = [] } = this.props; // we are assigning a default prop here of an empty array.
        return(
            <ul>
                {
                    //return movie from array
                    movieRes.map((movie, index)=>{
                        return (
                            <li key={index}>
                                {movie.id}
                            </li>
                        );
                    })
                }
            </ul>
        );
    }
}

通过这种方式,我可以抑制错误,但仍然不起作用。 据我所知,React 应该在检测到变化后立即进行渲染,但由于某种原因,事实并非如此。

图片

从图片中可以看到,当我将数组从父组件传递到子组件时,数组长度为 20,但在子组件中,数组长度似乎为 0

====================

解决方案

我将组件从类更改为 const,并将数组传递给它,一切都很顺利。这是最终代码:

import React from 'react';

const MovieList = ({movies}) =>{
    if(!movies){
        return <h1>Loading...</h1>
    }
    return (
        <ul>
            {
                movies.map((movie, index) => {
                    return (
                        <li key={index}>
                            <p>{movie.title}</p>
                        </li>
                    )
                })
            }
        </ul>
    );

}

export default MovieList;
1个回答

最初我误解了您的问题,但重新阅读后,我注意到您在构造函数中将电影定义为数组。

如果没有实际的错误消息,我将假设 MovieList 需要一个数组作为其 prop movieRes,然后您可能会尝试执行 .map 之类的操作或循环来呈现电影。

但是,您使用的 API 不会返回数组。它返回一个带有数组键的对象,该键位于 results 下。因此,我将其更改为在执行 setState 时访问 data.results

//fetching movie
  fetchMovie = () =>{
    const req = new Request(MOVIE_API, {
      method: 'GET',
      cache: 'default'
    });

    fetch(req).then(response =>{
      return response.json();
    }).then(data =>{
      console.log(data);
      this.setState({
        movies: data.results // <-- change made here.
      });
    }).catch(err => {
      console.log("ERROR: " + err);
    })
  }

这是一个有效的 JSFiddle:

https://jsfiddle.net/patrickgordon/69z2wepo/99513/

编辑: 在子组件中,不要将 props 分配给 state,只需使用 props 和默认 props。

import React, { Component } from 'react';

export default class MovieList extends Component{
    render(){
        const { movieRes = [] } = this.props; // we are assigning a default prop here of an empty array.
        return(
            <ul>
                {movieRes.map((movie, index)=>{
                    return (
                        <li key={index}>{movie.title}</li>
                    );
                })}
            </ul>
        );
    }
}
patrick
2018-01-27