反应：设置审查[关闭]

Question

需要对我在以下要求上的工作进行代码审查。您的 cmets 帮助我提高。

我正在为 api 使用 https://developer.github.com/v3 文档。

应用程序应该有两个视图来显示用户数据：

1. 最近的 Github 用户列表。确保您至少为用户列表显示以下信息
   • 个人资料头像
   • 句柄（用户名）
   • 单击任何用户应显示有关该用户的详细视图
2. 详细视图。确保您至少为用户列表显示以下信息
   • 姓名
   • 个人资料头像的大版本
   • 手柄
   • 位置
   • 生物
   • 关注次数
   • 关注人数

代码沙盒：https://codesandbox.io/s/elastic-goldwaser-nn047

Answer 1

总体来说做得很好，这里有几点小建议

1. 创建目录views 或components 并将users.jsx 和userDetails.jsx 放入其中
2. 您在 index.js 和 app.js 中包装了两次 <BrowserRouter>。您可以在 app.js 文件中使用 <Switch> 语句来包装您的路线。
3. 您在 users.js 中有一个 getUsersDetail 方法，但它没有被调用或作为道具传递。

最后，您似乎在进行 API 调用，但 json 文件中还有虚拟数据，不清楚它们在 http.js 文件中是如何交互的。