怒肝 8 个月源码，我成为了 Spring 开源贡献者
The following article is from 程序员cxuan Author cxuan
（给ImportNew加星标，提高Java技能）
作者：cxuan
// 本文不探讨技术细节，只是为了简单说明这个问题，所以省略无关代码 private int resolveConstructorArguments(String beanName, RootBeanDefinition mbd, BeanWrapper bw, ConstructorArgumentValues cargs, ConstructorArgumentValues resolvedValues) {
// .... for (Map.Entry<Integer, ConstructorArgumentValues.ValueHolder> entry : cargs.getIndexedArgumentValues().entrySet()) { int index = entry.getKey(); if (index < 0) { throw new BeanCreationException(mbd.getResourceDescription(), beanName, "Invalid constructor argument index: " + index); } // 问题就出在这里 if (index > minNrOfArgs) { minNrOfArgs = index + 1; } // .....上述代码中，minNrOfArgs这个变量就是保存方法需要的最小参数个数，但是index是下标索引，索引是从0开始的，如果有下标为n的元素，那么最小的参数个数应该是n+1嘛，所以if中的逻辑是没有问题的，但是if这个判断是有问题的，正确的做法应该是：
if (index+1 > minNrOfArgs) { minNrOfArgs = index + 1;}当发现这个问题的时候，第一反应就是肯定是我的姿势不对，错的怎么可能是代码，肯定是我！
接下来，我就对这段代码进行了惨无人道的调试，在无数次debug后，我发现，这个地方确实有问题！
在确认了这个问题之后，我要思考的就是怎么把自己的想法反馈给Spring，换而言之，怎么为伟大的开源来做贡献呢？正常来要达到这个目的有两个方式
提交issue 直接在GitHub上提交PR（pull request）
对应的就是在GitHub上点击下图红框选中的两个位置
如果是使用提交 issue 的方式，相当于给官方团队提交了一个议题，这个议题可能是你发现代码中的某个 bug，也可能是你觉得官方的做法不够好，你有更好的想法等等。感兴趣的话，大家可以去看看 Spring 中现在有哪些还未关闭的 issue，说不定其中一个你就能解决呢~！
如果要采用提交 PR 的方式的话，首先你得将代码 fork 到自己的 GitHub 中，然后在从自己的 GitHub 检出到本地，在本地做完修改后，提交到 GitHub 仓库中，最后从自己的 GitHub 向 Spring 官方仓库发起一个 PR。
像我的话很早就已经将代码 fork 到了自己 GitHub
上图中的第一个红框，说明我这个仓库是从 Spring 官方 fork 过来的，第二个红框就是可以从这里向 Spring 官方提交一个 PR。关于详细的如何提交 PR，大家可以自行百度，这里不做详细的介绍了。
另外，说了这么多，先给大家看下我提交的 issue 吧。
issue链接：https://github.com/spring-projects/spring-framework/issues/25130
因为内容也不长，所以我这里把原文就直接放到下面了
In ConstructorResolver:
private int resolveConstructorArguments(String beanName, RootBeanDefinition mbd, BeanWrapper bw, ConstructorArgumentValues cargs, ConstructorArgumentValues resolvedValues) { TypeConverter customConverter = this.beanFactory.getCustomTypeConverter(); // ...
for (Map.Entry<Integer, ConstructorArgumentValues.ValueHolder> entry : cargs.getIndexedArgumentValues().entrySet()) { int index = entry.getKey(); if (index < 0) { throw new BeanCreationException(mbd.getResourceDescription(), beanName, "Invalid constructor argument index: " + index); } if (index > minNrOfArgs) { minNrOfArgs = index + 1; } // .... }// .... return minNrOfArgs;}I assume that method resolveConstructorArguments is to resolve contructor arguments in the XML file and return the minimum number of parameters required by contructor 。but if the first parameter is autowired , the second parameter is config by XML file，the method will not work well。
example:
public class FactoryObject { public DmzService getDmz(String name, int age, Date birthDay, OrderService orderService) {
public DmzService getDmz(OrderService orderService,String name) { return new DmzService(orderService,name); }
}<?xml version="1.0" encoding="UTF-8"?><beans xmlns="http://www.springframework.org/schema/beans" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.springframework.org/schema/beans http://www.springframework.org/schema/beans/spring-beans.xsd" default-autowire="constructor"> <bean id="factoryObject" class="com.dmz.spring.first.instantiation.service.FactoryObject"/>
<bean class="com.dmz.spring.first.instantiation.service.OrderService" id="orderService"/>
<bean id="dmzService" factory-bean="factoryObject" factory-method="getDmz"> <constructor-arg index="1" value="dmz"/> </bean>
</beans>the resolveConstructorArguments method will return 1,but correct answer is 2。
I think the problem arises because of this judgment:
if (index > minNrOfArgs) { minNrOfArgs = index + 1;}It might be better to change it to look like this
if (index + 1 > minNrOfArgs) { minNrOfArgs = index + 1;}s我在提交 issue 时主要是按照这种思路
首先摆出有问题的代码 描述具体的问题，我是直接通过一个例子来描述的 说出自己的建议
这几天我又多看了看别人提交的issue，对比起来，我觉得至少应该还要添加一点
应该要明确的指出具体哪个版本上出现的问题
碰到的问题
担心闹乌龙
虽然在之前我已经调试过了无数次代码，但是心里还是没谱啊。毕竟我这么谨（cai）慎（ji）的一个人，万一被人喷了怎么办？不知道你会不会这么想，反正我当时就是这么想的，如果你是这么想的，建议你去看看别人提交的 issue。搜索条件如下
is:closed label:"status: invalid"我觉得你看几个，自然就有信心了！
不知道要怎么提交
每个开源的项目，只要作者希望这个项目越来越好的话，都会详细的说明如何给这个项目做开源贡献，Spring肯定也不例外，这里还是以提交 issue 为例，当你点击 New issue 的时候会出现下面这张图
在上图左边的框里很明确的告诉了你提交issue应该要注意什么
首先，你应该要去 Stack Overflow 提问 如果是 bug，你应该要指明版本以及你想要做什么 如果是一个增强的话，要提供上下文并且描述清楚问题 同一个问题，issue 跟 PR 最好只提交一个，因为 GitHub 认为它们是一样的，如果你还不能确定的话，先提交一个 issue
而右上角还有更加详细的文档可供参考。
英文
大家应该看到了，整个 issue 都是用英文写的，那么英文不好怎么办呢？这个时候就要掏出我们的神器了
嗯，就是词典，笔者习惯是使用有道词典。我建议英文不好的同学可以这样，先将整个 issue 用中文写好，如果你真的英文一窍不通的话，可以直接通过翻译软件逐句翻译，然后粘贴到 GitHub 上。但是千万千万不要使用中文，就像下面这个哥们
issue链接：https://github.com/spring-projects/spring-framework/pull/25127
像这种 issue 是会被直接打上 invalid（不合格）标签的，你就想想吧，你学不会英文，你指望我们的外国朋友能看懂中文嘛？是我中华上线五千年的文化不够博大精深吗？
担心问题描述的不清楚
其实这个问题就是因为英文不好衍生出来的。因为英文不好，自然就会担心我写的东西他能不能看懂呢？我的建议就是，结合你测试的代码去描述问题。你不用去担心别人看不懂你写的代码，就以我那个 issue 的处理流程为例吧。
在你刚刚提交issue时，有专门的issuemaster（issue管理员）会给你提交的 issue 打上
一个wait-for-triage 的标签，标志这个 issue 是待处理的。
随后我提交的这个 issue，就被指派给了jhoeller。你要担心他看不懂代码吗？给你看两个东西吧
你知道那个红框是啥意思吗？就是说我发现的那个有问题代码的类的作者就是他。
再看一张
就是说，jhoeller从 2003 年开始就已经是 Spring 这个项目的管理者以及发布经理了。2003 年，我还是一个小学生........
所以啊，只要你稍微正常点，基本上人家都能 get 到你的点。
其实笔者从发现这个问题到最终提交 issue 大概经过了一周时间，期间一直在犹豫要不要提交 issue，就是因为上面提到的几个问题，一直踌躇不前。但是等我下定决心要去做这件事的时候总共就花了几个小时的时间。包括研究issue 提交的规则以及写一篇英文版的 issue。并且我提交 issue 的第二天就马上被处理了，并且jhoeller在 f9aae8d 这个 commit 中已经接受我的建议。
所以我要说的就是，真正动手的话，不管什么问题总能找到解决方案
而只是停留在空想，在踌躇，你永远有一堆问题
临渊羡鱼，不如退而结网
以此文与君共勉！
看完本文有收获？请转发分享给更多人
关注「ImportNew」，提升Java技能
好文章，我在看❤️