替换pom.xml中的变量, bug fix

[MoneZhao]

17

我确实没考虑到一个节点多个变量的问题，老哥考虑周全 不过还是有些bug 我在你的基础上改了

            Element itemElement = element.element(nameItem.substring(
                    nameItem.indexOf("{") + 1,
                    nameItem.indexOf("}")
            )); // 这里相当于element.element("project.build.finalName"), 节点报NPE

[MoneZhao]

类似于 ${project.build.finalName}-${project.version}这种变量在我的项目里面测试没问题了

有个问题想问一下, core.view.icon.impl.CuteIcon这个类18行有个import javax.annotation.Nullable; , 我这儿一直报找不到类，是有什么包我没导入吗?

.gitignore 里面能不能把.idea 整个目录加上，还有*.iml文件

[ZhangYuanSheng1217]

这是jetbrains里面的，我导错包了，已经改了

[MoneZhao]

@ZhangYuanSheng1217 老哥，review一下我这个pr呗

[ZhangYuanSheng1217]

……我不会

[MoneZhao]

@ZhangYuanSheng1217 https://boxueio.com/series/git-essential/ebook/459 可以参考一下这片blog

[MoneZhao]

https://image.boxueio.com/your-first-pr-10@2x.jpg @ZhangYuanSheng1217 如果你觉得我改的没问题的话就merge我的pr吧😄

[ZhangYuanSheng1217]

而且你为啥要用.分割获取啊

[MoneZhao]

17

我确实没考虑到一个节点多个变量的问题，老哥考虑周全 不过还是有些bug 我在你的基础上改了

            Element itemElement = element.element(nameItem.substring(
                    nameItem.indexOf("{") + 1,
                    nameItem.indexOf("}")
            )); // 这里相当于element.element("project.build.finalName"), 节点报NPE

我说了啊，不分隔的话获取pom节点会报错element.element("project.build.finalName"), 节点报NPE, 我截个图你看看

[MoneZhao]

而且你为啥要用.分割获取啊

review 用选

[ZhangYuanSheng1217]

你咋定义的，我这边不会报错啊，而且$(project.build.finalName)是定义在maven包里面的pom.xml中，是获取不到这个文件的，现在写的是只能获取当前pom中已经定义的属性才能拿到

[MoneZhao]

git clone --depth=1 https://github.com/MoneZhao/account-balance.git 这是我的项目，你试试吧，balance-manager这个module $(project.build.finalName)是可以获取到的

[ZhangYuanSheng1217]

老哥，我修改了下，你那种还是只能获取到pom已经定义出的elem，否则全都获取不到，前段时间有点忙，没来得及改

[MoneZhao]

ok, 我看到了，已经没问题了, 感谢

src/core/service/topic/RestDetailTopic.java 这个类你又引入了javax.annotation.Nullable