JS de reação - TypeError não capturado: this.props.data.map não é uma função

110

Estou trabalhando com reactjs e não consigo evitar este erro ao tentar exibir dados JSON (de arquivo ou servidor):

Uncaught TypeError: this.props.data.map is not a function

Eu olhei para:

Código de reação lançando “TypeError: this.props.data.map não é uma função”

React.js this.props.data.map () não é uma função

Nenhum deles me ajudou a resolver o problema. Depois que minha página carregar, posso verificar se this.data.props não é indefinido (e tem um valor equivalente ao objeto JSON - pode chamar com window.foo), então parece que não está carregando a tempo quando é chamado por ConversationList. Como posso ter certeza de que o mapmétodo está funcionando nos dados JSON e não em uma undefinedvariável?

var converter = new Showdown.converter();

var Conversation = React.createClass({
  render: function() {
    var rawMarkup = converter.makeHtml(this.props.children.toString());
    return (
      <div className="conversation panel panel-default">
        <div className="panel-heading">
          <h3 className="panel-title">
            {this.props.id}
            {this.props.last_message_snippet}
            {this.props.other_user_id}
          </h3>
        </div>
        <div className="panel-body">
          <span dangerouslySetInnerHTML={{__html: rawMarkup}} />
        </div>
      </div>
    );
  }
});

var ConversationList = React.createClass({
  render: function() {

    window.foo            = this.props.data;
    var conversationNodes = this.props.data.map(function(conversation, index) {

      return (
        <Conversation id={conversation.id} key={index}>
          last_message_snippet={conversation.last_message_snippet}
          other_user_id={conversation.other_user_id}
        </Conversation>
      );
    });

    return (
      <div className="conversationList">
        {conversationNodes}
      </div>
    );
  }
});

var ConversationBox = React.createClass({
  loadConversationsFromServer: function() {
    return $.ajax({
      url: this.props.url,
      dataType: 'json',
      success: function(data) {
        this.setState({data: data});
      }.bind(this),
      error: function(xhr, status, err) {
        console.error(this.props.url, status, err.toString());
      }.bind(this)
    });
  },
  getInitialState: function() {
    return {data: []};
  },
  componentDidMount: function() {
    this.loadConversationsFromServer();
    setInterval(this.loadConversationsFromServer, this.props.pollInterval);
  },
  render: function() {
    return (
      <div className="conversationBox">
        <h1>Conversations</h1>
        <ConversationList data={this.state.data} />
      </div>
    );
  }
});

$(document).on("page:change", function() {
  var $content = $("#content");
  if ($content.length > 0) {
    React.render(
      <ConversationBox url="/conversations.json" pollInterval={20000} />,
      document.getElementById('content')
    );
  }
})

EDITAR: adicionando conversations.json de amostra

Observação - chamar this.props.data.conversationstambém retorna um erro:

var conversationNodes = this.props.data.conversations.map...

retorna o seguinte erro:

TypeError não capturado: Não é possível ler a propriedade 'map' de indefinido

Aqui está conversations.json:

{"user_has_unread_messages":false,"unread_messages_count":0,"conversations":[{"id":18768,"last_message_snippet":"Lorem ipsum","other_user_id":10193}]}
pulsações da tecla
fonte
Eu atualizei a resposta. Além disso, algumas recomendações: adicione propTypes: {data: React.PropTypes.array} a ConversationList para verificar o tipo de dados e adicione um componentWillUnmount: fn () que "clearInterval" o intervalo em ConversationBox.
user120242

Respostas:

134

A .mapfunção está disponível apenas na matriz.
Parece que datanão está no formato que você espera (está {}, mas você está esperando []).

this.setState({data: data});

deveria estar

this.setState({data: data.conversations});

Verifique o tipo de "dados" que está sendo definido e certifique-se de que seja uma matriz.

Código modificado com algumas recomendações (validação de propType e clearInterval):

var converter = new Showdown.converter();

var Conversation = React.createClass({
  render: function() {
    var rawMarkup = converter.makeHtml(this.props.children.toString());
    return (
      <div className="conversation panel panel-default">
        <div className="panel-heading">
          <h3 className="panel-title">
            {this.props.id}
            {this.props.last_message_snippet}
            {this.props.other_user_id}
          </h3>
        </div>
        <div className="panel-body">
          <span dangerouslySetInnerHTML={{__html: rawMarkup}} />
        </div>
      </div>
    );
  }
});

var ConversationList = React.createClass({
 // Make sure this.props.data is an array
  propTypes: {
    data: React.PropTypes.array.isRequired
  },
  render: function() {

    window.foo            = this.props.data;
    var conversationNodes = this.props.data.map(function(conversation, index) {

      return (
        <Conversation id={conversation.id} key={index}>
          last_message_snippet={conversation.last_message_snippet}
          other_user_id={conversation.other_user_id}
        </Conversation>
      );
    });

    return (
      <div className="conversationList">
        {conversationNodes}
      </div>
    );
  }
});

var ConversationBox = React.createClass({
  loadConversationsFromServer: function() {
    return $.ajax({
      url: this.props.url,
      dataType: 'json',
      success: function(data) {
        this.setState({data: data.conversations});
      }.bind(this),
      error: function(xhr, status, err) {
        console.error(this.props.url, status, err.toString());
      }.bind(this)
    });
  },
  getInitialState: function() {
    return {data: []};
  },

 /* Taken from 
    https://facebook.github.io/react/docs/reusable-components.html#mixins
    clears all intervals after component is unmounted
  */
  componentWillMount: function() {
    this.intervals = [];
  },
  setInterval: function() {
    this.intervals.push(setInterval.apply(null, arguments));
  },
  componentWillUnmount: function() {
    this.intervals.map(clearInterval);
  },

  componentDidMount: function() {
    this.loadConversationsFromServer();
    this.setInterval(this.loadConversationsFromServer, this.props.pollInterval);
  },
  render: function() {
    return (
      <div className="conversationBox">
        <h1>Conversations</h1>
        <ConversationList data={this.state.data} />
      </div>
    );
  }
});

$(document).on("page:change", function() {
  var $content = $("#content");
  if ($content.length > 0) {
    React.render(
      <ConversationBox url="/conversations.json" pollInterval={20000} />,
      document.getElementById('content')
    );
  }
})
user120242
fonte
32

o que funcionou para mim foi converter o props.data em um array usando data = Array.from(props.data); então eu poderia usar a data.map()função

Vishal Seshagiri
fonte
1
Meu tipo já era um array, por exemplo, typeOf estava mostrando a coisa certa e o comprimento estava correto, mas ainda assim, o erro estava aparecendo. Isso consertou para mim. Obrigado!
user1829319
10

De forma mais geral, você também pode converter os novos dados em uma matriz e usar algo como concat:

var newData = this.state.data.concat([data]);  
this.setState({data: newData})

Na verdade, esse padrão é usado no aplicativo de demonstração ToDo do Facebook (consulte a seção "Um aplicativo") em https://facebook.github.io/react/ .

Bresson
fonte
1
Essa é uma dica / truque bacana, mas quero observar que isso mascararia o problema real e não resolveria o OP. No caso do OP isso não ajudaria, pois os dados ainda estariam malformados (não o que se pretendia). A conversa provavelmente terminaria vazia, pois todos os adereços seriam nulos. Eu recomendaria não usar essa dica, a menos que você tenha certeza de que seus dados estão no formato desejado, para não ter um comportamento inesperado quando os dados retornados não forem o que você pensou que deveriam ser.
user120242
1

Você não precisa de um array para fazer isso.

var ItemNode = this.state.data.map(function(itemData) {
return (
   <ComponentName title={itemData.title} key={itemData.id} number={itemData.id}/>
 );
});
CodeGuru
fonte
Como é diferente mano?
Amar Pathak
1

Isso acontece porque o componente é renderizado antes da chegada dos dados assíncronos, você deve controlar antes de renderizar.

Resolvi desta forma:

render() {
    let partners = this.props && this.props.partners.length > 0 ?
        this.props.partners.map(p=>
            <li className = "partners" key={p.id}>
                <img src={p.img} alt={p.name}/> {p.name} </li>
        ) : <span></span>;

    return (
        <div>
            <ul>{partners}</ul>
        </div>
    );
}
  • O mapa não pode ser resolvido quando a propriedade é nula / indefinida, então fiz um controle primeiro

this.props && this.props.partners.length > 0 ?

JC
fonte
0

Você precisa converter o objeto em uma matriz para usar a mapfunção:

const mad = Object.values(this.props.location.state);

onde this.props.location.stateé o objeto passado para outro componente.

ISRAEL ODUGUWA
fonte
-2

experimente o componentDidMount()ciclo de vida ao buscar dados

Arif Ramadhani
fonte
5
Além disso, forneça algumas explicações, por que isso funciona e como?
Mittal Patel