http://www.cocoachina.com/gamedev/misc/2013/0227/5728.html
这篇文章的内容由ThoughtBot在github上官方主页提供,指导你如何在github上进行代码审查和如果让别人审查自己的代码。
针对所有人的审查
•接受这样的事实:很多编程上的主张都是一种个人观点。应该讨论它们的利与弊,提出你的倾向观点,迅速的达成一种解决方案。
•提问,而不是命令。(“把这个变量命名成:user_id你觉得怎样?”)
•请求说明。(“我不明白。你能解释一下吗?”)
•避免代码的归属之争。(“我的”,“不是我的”,“你的”)
•避免使用一些会被认为是有关人身特征的词语。(“笨蛋”,“愚蠢”)要把所有人都看作是有魅力的、聪明的、善意的。
•要明确。要记着并不是每个人都能理解你的意图。
•要谦虚。(“我不能确定——我们来分析一下。”)
•不要用夸张修辞语。(“总是”,“从不”,“永远”,“毫无…”)
•不要讽刺。
•展现真实的你。如果你不是幽默型的人,不喜欢使用一些表情符号或动画gif图,不要勉强。如果你是这种人,请自信的发挥。
•如果有太多的“我不理解”或“另一种方案:”的评论,请专门针对这个人进行交流。可以把你们线下的交流总结成一个帖子附在后面。
让别人审查你的代码
•对审查者的建议表示感激。(“谢谢提醒。我会把它改正。”)
•理解审查是对事不对人。审查的是你的代码,而不是你。
•解释为什么代码写成这样。(“因为xxx原因我才写成这样。如果我把这个类/文件/方法/变量改个名会更清晰些吗?”)
•整理所作的改动,在以后的迭代中重构它们。
•在做修改的版本上注明代码审查的链接。(“Ready for review: http://github.com/organization/project/pull/1″)
•push提交要基于最早的一轮反馈,并形成一个独立的分支。等这个分支上的任务完全完成了再合并。这让审查者能够根据早先的反馈找到你的单独的更新。
•努力站在审查者的立场上理解。
•争取回复每个评论。
•直到最后一个人退出登录后再合并分支。
•直到持续集成测试(TDDium, TravisCI,等)告诉你这个分支的测试套件通过后再合并分支。
代码审查的过程
先要清楚你提交的代码的必要性(是修补bug,提升用户体验,重构…),然后:
•针对你感觉非常好的地方以及不是很好的地方与作者交流。
•找出既能解决问题又能简化代码的方法。
•如果讨论变得过于哲学或理论,把讨论转到线下,做成一个有规律的每周五下午的讨论会。同时,是否采用你提出的实现方案,让作者自己做决定。
•提出你的实现方案,但要表现出作者也在考虑这种方案。(“你觉得这里用一个自定义校验如何?”)
•努力理解作者的立场。
•pull请求登出时,加一个或“可以合并了”的注释。
关于程序风格样式的评论注释
审查者应该对那些不符合样式指导的地方进行注释。例如这样注释:
如果你不同意某个指导原则,请在指导repo里创建一个问题,而不要再代码审查中争论它。同时,请运用这个指导原则。
相关推荐
6、上传IPA到AppStore市场 使用AppUploader等第三方工具上传ipa文件到App Store。上传过程中,工具会自动处理上传事宜,并显示上传进度。上传成功后,用户将看到进度条完成以及成功提示信息。 7、上传好IPA回到...
开发的 AppStore 产品的技术方案,旨在提供一个完整的 AppStore 解决方案,涵盖系统架构、接口设计、开发者社区、个人主页、存储平台、用户信息中心、业务核心数据、DRM 推荐平台、搜索引擎、公共业务组件、IMU ...
App_store发布流程详解 本文将详细介绍如何将APP发布到App Store的步骤,涵盖证书的导出、App ID的申请、发布证书的申请等重要知识点。 一、证书的导出 要将APP发布到App Store,首先需要导出证书。在苹果的...
随着移动互联网的发展,应用程序(App)已成为企业和个人接触用户的重要渠道之一。对于iOS平台上的开发者来说,如何有效地推广自己的应用显得尤为重要。苹果公司为此制定了《App Store:开发者营销广告指南》,这份...
App Store模式,自2008年由苹果公司推出以来,已成为全球移动应用分发的重要典范,引领了移动互联网行业的创新和发展。本报告将深入探讨App Store的各个方面,包括其定义、业务模式、应用内容的分析以及背后的商业...
《App Store审核指南》是苹果公司为开发者提供的详细文档,旨在指导开发者如何创建符合App Store标准的应用程序。这份文档详尽地涵盖了各种开发和提交应用时需要注意的规则和标准,包括应用的内容、功能、设计、性能...
本书旨在帮助开发者在Apple的AppStore中找到成功的路径,通过掌握iPhone SDK的关键技术和最佳实践,实现创新与盈利。 首先,iPhone SDK是苹果公司为开发者提供的一个强大的工具集,它包含了开发iOS应用所需的一切,...
在iOS应用开发中,创建与iOS 11 App Store类似的列表效果是一项常见的需求,它可以提升用户的体验感,使应用更具吸引力。本项目标题为“高仿iOS11 App Store列表效果”,其主要目的是通过编程技术实现类似App Store...
"appstore开发用火狐打开"这个标题可能是指使用Firefox浏览器来访问和处理与AppStore相关的开发任务,这通常涉及到Web开发工具的使用,尤其是对于基于Sencha Touch框架的应用程序。 Sencha Touch是一个强大的...
可以选择是否参与App Store的自动价格调整和促销活动。 9. **发布**: 当审核通过,一切准备就绪后,可以选择立即发布或设置预定发布日期。发布后,应用将在App Store上可供用户搜索和下载。 10. **后续更新与...
在iOS 11中,苹果引入了一系列新的设计元素和交互方式,其中App Store的转场动画就是一个引人注目的亮点。这个动画不仅提升了用户体验,也展示了Swift在实现动态界面效果方面的强大能力。下面我们将详细探讨如何使用...
标题中的“swift-react-native-app-store-review”指的是一个项目或库,它是用React Native框架实现的,目的是在iOS应用中提供类似Apple App Store的用户评分功能。React Native是一种由Facebook开发的开源...
标题中的"envi_app_store.zip"表明这可能是一个与ENVI(Environment for Visualizing Images)相关的软件应用商店或扩展库的压缩文件。ENVI是一款强大的遥感图像处理和分析软件,广泛应用于地球科学、环境研究、农业...
#### 一、IPv6与App Store的关系 App Store对应用的要求非常严格,其中包括必须支持IPv6。这是因为苹果认为IPv6是未来互联网的重要组成部分,所有在其平台上发布的应用都应具备这一能力。对于开发者来说,这是一项...
2016年10月5日,苹果在美国地区推出了App Store搜索竞价排名广告系统(Search Ads),为开发者提供了一个全新的ASO(应用商店优化)排名优化手段。相比于传统的推广方式,如刷分和推广,这种官方渠道不仅更加安全可靠,...
《App Store 审核指南》是苹果公司针对iOS开发者发布的一款重要文档,旨在规范和指导开发者如何设计、开发和提交应用程序到App Store的过程。这份2013年1月29日的中文版审核指南,详细阐述了苹果对应用程序的各种...
在移动互联网时代,二手市场APP已成为许多人处理闲置物品、寻找实惠商品的重要平台。"基于UNI-APP的仿转转二手市场APP页面模板"是一个开发项目,旨在为开发者提供一个快速构建类似转转APP的用户界面的工具。这个模板...