Amap()创建一个数组，因此return所有代码路径（if/elses）都需要 a。

如果您不想要数组或返回数据，请forEach改用。

该警告表明您不会在每种情况下都在地图箭头函数的末尾返回某些内容。

更好的方法是首先使用 a .filter，然后使用 a .map，如下所示：

this.props.comments
  .filter(commentReply => commentReply.replyTo === comment.id)
  .map((commentReply, idx) => <CommentItem key={idx} className="SubComment"/>);

最简单的方法只有当您不需要返回某些东西时return null

问题似乎是，如果您的第一个if-case 为 false ，您将不会返回某些内容。

您收到的错误表明您的箭头函数(comment) => {没有 return 语句。虽然它在您的if-case 为真时起作用，但在它为假时它不会返回任何内容。

return this.props.comments.map((comment) => {
  if (comment.hasComments === true) {
    return (
      <div key={comment.id}>
        <CommentItem className="MainComment" />
        {this.props.comments.map(commentReply => {
          if (commentReply.replyTo === comment.id) { 
            return (
              <CommentItem className="SubComment"/>
            )
          }
        })
        }
      </div>
    )
  } else {
     //return something here.
  }
});

编辑您应该看看 Kris 的答案，以了解如何更好地实施您正在尝试做的事情。

来自 Kris Selbekk 的最高票答案是完全正确的。重要的是要强调，尽管它采用函数式方法，但您将循环遍历this.props.comments数组两次，第二次（循环）它很可能会跳过一些已过滤的元素，但如果没有comment被过滤，您将循环遍历整个阵列两次。如果您的项目不关心性能，那完全没问题。如果性能很重要，aguard clause会更合适，因为您只会循环数组一次：

return this.props.comments.map((comment) => {
  if (!comment.hasComments) return null; 

  return (
    <div key={comment.id}>         
      <CommentItem className="MainComment"/>
        {this.props.comments.map(commentReply => {             
          if (commentReply.replyTo !== comment.id) return null;

          return <CommentItem className="SubComment"/>
        })} 
    </div>          
  ) 
}

我指出这一点的主要原因是因为作为一名初级开发人员，我犯了很多这样的错误（比如多次循环同一个数组），所以我认为我在这里值得一提。

PS：我会重构你的react成分就更多了，因为我赞成重逻辑的我不是html part的JSX，但就是这个问题的话题了。