TE
科技回声
首页24小时热榜最新最佳问答展示工作
GitHubTwitter
首页

科技回声

基于 Next.js 构建的科技新闻平台,提供全球科技新闻和讨论内容。

GitHubTwitter

首页

首页最新最佳问答展示工作

资源链接

HackerNews API原版 HackerNewsNext.js

© 2025 科技回声. 版权所有。

Ask HN: How do you perform code review?

2 点作者 dome82大约 10 年前
Hi.<p>As a developer, I know the importance of code reviews and I would like to be better at it. I have been working in different companies and I could see that there was not a consistent approach with it.<p>I could see different behaviors that code reviewers do, such as: - Leaving comments in a commit or a pull request - Leaving comments in their task tool (e.g. Jira, Trello) - Making the code review in isolation - Making the code review with the developer that made the changes - Making the code review with another developer that did not touch that code - Gathering multiple people in a room and letting the developer to walk through their changes. - Printing the code on paper (yes, I have also seen that..) - Strict code review - Soft code review, always accepting it<p>I know that there is not a unique way to do it,and you should use common sense in different cases but I was curious to know how do people in HN perform code reviews, what are your best practices(and why), what is the worse code review that you have received and what is the best one.

1 comment

jhildings大约 10 年前
The best method for me is to go through &quot;hard&quot; or &quot;buggy&quot; code together with 1-2 other devs, and then we can discuss why something was hard to solve or some bug appeared. Then maybe make some change to internal code or document in for future use