提交代码有学问....

从上一篇上榜了....发文一来，LeetCode-Master开始有大量的代码合入请求。

先感谢每一位提交代码的录友，目前已经有34位录友提交了代码，这里有你么？快举手?

贡献者

这是目前代码提交量前六位，当然第一位是我，哈哈哈

这几天处理了大量的pull request（大家合入的代码），最让人绝望的是，处理完一批pull request之后 刷新一下页面，pull request数量更多了，[哭]，大家提交的速度可太快了。

所以我这几天都是早上处理一波，中午处理一波，晚上有处理一波，终于把大家的pull request处理完了。

别看这里有140个pull request，有的 pull request 是有多个commit的，例如

所以我这几天至少看了200道题目的代码了，而且是各种语言，我还要仔细审核。

哪有同学问了，合入代码有这么难么？点击一下 合入，不就完事了？

好了，重点来了，接下来，我来给说一说合入代码时候出现的一些问题。

• 如何提交代码
• 合入不规范
  • 提交信息不规范
  • Markdown 代码格式
  • pull request里的commit数量
  • 代码注释
  • 说明具体是哪种方法
  • 代码规范
  • 代码逻辑
  • 处理冲突

如何合入代码

首先来说一说如何合入代码，不少录友还不太会使用github，所以这里也做一下科普。

我特意申请一个新的Github账号，给大家做一个示范。

需要强调一下，一个commit就只更新一道题目，不要很多题目一起放在一个commit里，那样就很乱。

首先LeetCode-Master每天都有更新，如何保持你fork到自己的仓库是最新的版本呢。

点击这里Fetch upstream。

点击之后，这里就会显示最新的信息了

注意这时是你的远端仓库为最新版本，本地还不是最新的，本地要git pull一下。

基于最新的版本，大家在去提交代码。

如何提交代码呢，首先把自己的代码提交到自己的fork的远端仓库中，然后open pull request，如图：

点击 open pull request之后，就是如下画面，一个pull request有多个commit。

创建pullrequest

然后就是给pull request 添加备注，pull request是对本次commit的一个总结。如果一个pull request就一个commit，那么就和commit的备注保持一次。然后点击 create pull request 就可以了

此时你就提交成功了，我会在项目中的pull requests 处理列表里看到你的请求。

然后如果你发现自己的代码没有合入多半是有问题，如果有问题都有会在pull request里给出留言的，

合入不规范

提交信息不规范

大家提交代码的时候有两个地方需要写备注，一个是commit，一个是pull request，pull request包好多个commit。

commit 说清楚本文件多了哪些修改，而pull request则是对本次合入的所有commit做一个总结性描述。

commit备注，举例：添加Rust python3，那么commit备注就是：添加0001两数之和 Rust python3 版本

而pull request 如果只有一个commit，那么就也是：添加0001两数之和 Rust python3 版本

如果是多个commit ，则把本次commit都描述一遍。

Markdown 语法

关于 Markdown 代码格式，例如 添加C++代码，需要有代码块语法

```C++ C++代码 ```

例如这个commit，在添加java代码的时候，就直接添加代码

正确的格式应该是这样：

一般发现问题，我也会在代码中给出评论：

这样大家也可以学习一些 提交代码的规范方面的知识

有的录友 是添加的代码块语法，但没有标记是哪种语言，这样的话 代码就不会针对某种语言高亮显示了，也比较影响阅读，例如：

提交python代码的话，要注释好，是python2还是python3

例如这样：

当然python2的话，只这么写就行

```python python代码 ```

pull request里的commit数量

有的录友是一个pull request 里有很多commit （一个commit是一道题目的代码）。

有的录友是一个pull request 里有有一个commit。

其实如果大家是平时一天写了两三道题目的话，那么分三个commit，一个pull request提交上来就行。

一个pull request 一个commit也可以，这样大家就会麻烦一点。

但注意一个pull request也不要放太多的commit，一旦有一道题目代码不合格，我没有合入，就这个pull request里影响其他所有代码的合入了。

代码注释

提交的代码最好要有注释，这样也方便读者理解。

例如这位录友，在提交Java代码的时候，按照题解的意思对Java版本的代码进行的注释，这就很棒?

当然如果大家感觉 已有的代码 不符合以上要求的话，例如 代码思路不够清晰不够规范，注释不够友好，依然欢迎提交优化代码，要记得详细注释哦。

添加注释

说明具体是哪种方法

有的题解有两种甚至三四种解法，在添加代码的时候，注释上也清楚具体是哪一种方法的版本。

下面这位录友做的就很好

有的题解，是一起给出了多道题目的讲解，例如项目中0102.二叉树的层序遍历.md 中有八道题目，那么大家添加代码的时候 应该在代码注释上，或者 直接写上 是哪个题目的代码。

代码规范

大家提交代码要规范，当然代码可以在力扣上运行通过是最基本的。

同时还有有代码规范，例如昨天的文章看了这么多代码，谈一谈代码风格！，大家应该好好看一下，平时自己写代码的时候尽量标准一些。

虽然我主张没有绝对正确的代码风格，但既然是给LeetCode-Master提交代码，尽量遵循Google编程规范哈。

经常看我的代码的录友应该都知道，我的代码格严格按照 Google C++ 编程规范来的，这样看上去会比较整洁。

大家提交代码的时候遇到规范性问题，例如哪里应该由空格，哪里没有空格，可以参考我的代码来。

有一位录友在提交代码的时候会把之前的代码 做一下规范性的调整，这就很棒。

代码规范从你我做起！

代码逻辑

提交的代码要按照题解思路来写。

虽然大家自己发挥想象空间是好的，但是题解还是要一脉相承，读者看完题解，发现代码和题解不是一个思路的话，那和重新读代码有啥区别了。

所以和题解不是一个思路的代码，除非详细注释了自己的思路 或者 写一段自己代码的描述说明思路和优化的地方，否则我就不会通过合入了哈。

大家的代码 最好也将关键地方放上注释，这样有助于别人快速理解你的代码。

处理冲突

在合入的过程中还要处理冲突的代码， 理解大家代码的思路，解决冲突，然后在力扣提交一下，确保是没问题。

例如同一道题目， 一位录友提交了， 我还没处理如何，另一位录友也对这道题也提交了代码，这样就会发生冲突

大家提交代码的热情太高了，我有时候根本处理不过来，但我必须当天处理完，否则第二天代码冲突会越来越多。

一天晚上分别有两位录友提交了 30多道 java代码，全部冲突，解决冲突处理的我脖子疼[哭]

那么在处理冲突的时候 保留谁的代码，删点谁的代码呢？

我一定是看谁 代码逻辑和题解一致，代码风格好，注释友好，就保留谁的。

所以例如当你想提交Java代码的时候，即使发现该题解已经有Java版本了，只要你的代码写的好，一样可以提交，我评审合格一样可以合入代码库。

不要做额外修改

确保这种额外文件不要提交。

额外文件

还有添加不同方法的时候，直接用正文格式写，哪种方法就可以了，不要添加目录 ，例如这样，这样整篇文章目录结构就有影响了。

前面不要加 ## 前序遍历（迭代法），直接写前序遍历（迭代法）就可以了。

当然这里也没有给代码块标记上对应的语言，应该是

``` Go Go语言代码 ```

对代码保持敬畏

有的录友甚至提交的代码并不是本题的代码，虽然我是鼓励大家提交代码的，但是大家贡献代码的时候也要对 自己的代码有敬畏之心，自己的代码是要给很多读者看的。

• 代码运行无误
• 写的够不够简洁
• 注释清不清晰
• 备注规不规范

这也是培养大家以后协调工作的一种能力。

小感慨

最后大家会发现处理合入代码的工作量了吧，真的有点大！

但我还是会保证Leetcode—master的整体代码质量的。

最后还是要感谢每一位给Leetcode—master贡献过代码的录友，大家的头像会出现在项目简介中，也就是这里，https://github.com/youngyangyang04/leetcode-master/graphs/contributors

目前提交的代码中最多的就是Java，差不多leetcode-master中Java版本很快都要补全了。

目前 Python 和 go的相对少一些。同样也欢迎 Javascript，Rust 的录友们。

快去点亮你的头像吧。

最后项目地址：https://github.com/youngyangyang04/leetcode-master 。还没给star的录友们抓紧呀！