重定向修改jar by 247321453 · Pull Request #301 · Qihoo360/RePlugin

247321453 · 2017-08-15T06:24:33Z

引用的jar，重定向到build下面修改过的jar，版本信息存在version.json

wangfuda · 2017-08-17T11:47:15Z

replugin-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/Util.groovy

+                            /* 将 jar 包解压，并将解压后的目录加入 classpath */
+                            // println ">>> 解压Jar${jarPath}"
+                            String jarZipDir = reJar.getParent() + File.separatorChar + reJar.getName().replace('.jar', '')
+                            if (unzip(jarPath, jarZipDir)) {


在原逻辑中的unzip(...)之前所新增的以上全部代码，都可以封装到一个isNeedInject(...)方法中，这样这块核心逻辑分支会整洁很多。

wangfuda · 2017-08-17T11:58:43Z

replugin-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/Util.groovy

+                        }
+                        //设置重定向jar
+                        setJarInput(jarInput, reJar)
+                        if (needInject) {


目前看，只在一种buildType下会去动态编译，后续依次编译其他buildType时，就不会去动态编译了。这样是存在潜在问题的，比如用户指定了不同buildType下的manifest中包含有不同acitivity，那么就可能存在这样一种情况：

场景
debug下manifest中注册了Activity A,其中A在test.jar中包含。
release下manifest中注册了Activity B,其中B在test.jar中包含。

结果
编译debug时，对test.jar中 A的顶级父类做了替换。
编译release时，不再对test.jar进行动态编译，test.jar中的Activity B的顶级父类并未做替换。

这个的确

还有一种case，就是一次完整的buildtype遍历编译完成后，用户在manifest中新增注册了jar中的某个Activity，那也会出现不替换jar中的对应顶级父类的情况

wangfuda · 2017-08-17T12:00:53Z

replugin-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/Util.groovy

+                                needInject = true
+                            }
+                        } else {
+                            FileUtils.copyFile(jar, reJar)


拷贝jar是不必要的。可以参考源码中对非项目目录下的jar的解压方式。

wangfuda · 2017-08-17T12:01:35Z

replugin-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/Util.groovy

+                            needInject = true;
+                        }
+                        //设置重定向jar
+                        setJarInput(jarInput, reJar)


通过反射方式，修改gradle的transform.JarInput字段的重定向jar是不必要的。（实际上可以在不改动默认编译流程的前提下，实现功能。如有疑问，可以私信交流哈）

wangfuda · 2017-08-17T12:02:16Z

replugin-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/Util.groovy

+                        setJarInput(jarInput, reJar)
+                        if (needInject) {
+                            includeJars << reJarPath
+                            map.put(reJarPath, reJarPath)


这里还请与其他jar处理逻辑分支保持一致，保存的是orginal 'jarPath'

wangfuda · 2017-08-17T12:17:08Z

replugin-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/Util.groovy

+                        boolean needInject = false
+                        if (reJar.exists()) {
+                            //检查修改插件版本
+                            InjectorVersion injectorVersion = injectorMap.get(jar.getAbsolutePath());


这里方法和变量的命名，还是建议更精准一些，比如InjectorVersion ，里面包含了插件版本信息，和jar的md5信息，类名建议叫JarInfo（或者有更确切的名字也可以），包括injectorMap也建议修改为jarInfoMap，这部分相关的代码和类名尽量不带‘injector’字样，毕竟‘injector’的本意是注入器，这只应该injector模块及相关部分中使用。
另：json文件也建议与此保持一致，这样代码整体一致性更强些。

或者类名叫BuildCacheInfo，更确切些，供参考

wangfuda · 2017-08-27T05:49:14Z

...-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/JarPatchInfo.groovy

    def String pluginVersion

-    InjectorVersion(){
+    JarPatchInfo(){


无用的import和方法，变量声明等，请移除

wangfuda · 2017-08-27T05:49:35Z

...-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/JarPatchInfo.groovy

    }

-    InjectorVersion(String jarMd5, String pluginVersion) {
+    JarPatchInfo(String jarMd5, String pluginVersion) {


无用的import和方法，变量声明等，请移除

wangfuda · 2017-08-27T05:49:45Z

...-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/JarPatchInfo.groovy

    }

-    InjectorVersion(File jar){
+    JarPatchInfo(File jar){


无用的import和方法，变量声明等，请移除

wangfuda · 2017-08-27T05:50:43Z

...gin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/ReClassTransform.groovy

     * 初始化 ClassPool
     */
-    def initClassPool(Collection<TransformInput> inputs) {
+    def initClassPool(Collection<TransformInput> inputs,String buildType) {


此行代码格式化一下

wangfuda · 2017-08-27T05:51:12Z

replugin-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/Util.groovy

    /** 生成 ClassPool 使用的 ClassPath 集合，同时将要处理的 jar 写入 includeJars */
    def
-    static getClassPaths(Project project, GlobalScope globalScope, Collection<TransformInput> inputs, Set<String> includeJars, Map<String, String> map) {
+    static getClassPaths(Project project, String buildType,GlobalScope globalScope, Collection<TransformInput> inputs, Set<String> includeJars, Map<String, String> map) {


代码格式化

wangfuda · 2017-08-27T06:43:50Z

replugin-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/Util.groovy

-                                    //版本变化了
+                    String md5 = md5File(jar);
+                    File reJar = new File(injectDir + File.separator + md5 + ".jar");
+                    jarPath = reJar.getAbsolutePath()


这里不要复用之前的jarPath 变量，重新定义个reJarPath

wangfuda · 2017-08-27T06:46:29Z

replugin-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/Util.groovy


                    //设置重定向jar
-                    setJarInput(jarInput, reJar)
+//                    setJarInput(jarInput, reJar)


无用的import和方法，变量声明等，请移除

wangfuda · 2017-08-27T06:47:22Z

replugin-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/Util.groovy

                        if (unzip(jar.getAbsolutePath(), jarZipDir)) {

-                            includeJars << jarPath
+                            includeJars << jar.getAbsolutePath()


保持原代码不变，jarPath已经获取了全路径。相关代码需要调整下

wangfuda · 2017-08-27T07:03:39Z

replugin-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/Util.groovy

+        }
+        return needInject;
+    }
+


代码逻辑可以精简为：

def static checkNeedInjector(Map<String, JarPatchInfo> infoMap, File jar, File reJar, String activityMd5, String md5) { boolean needInject = true if (reJar.exists()) { JarPatchInfo info = infoMap.get(jar.getAbsolutePath()); if (info != null) { if (activityMd5.equals(info.manifestActivitiesMd5) && AppConstant.VER.equals(info.pluginVersion) && md5.equals(info.jarMd5)) { needInject = false } } } return needInject; }

wangfuda · 2017-08-27T07:10:59Z

replugin-plugin-gradle/src/main/groovy/com/qihoo360/replugin/gradle/plugin/inner/Util.groovy

-                    }
-                } else {
+                }else{
                    //重定向jar


这里严格说，不是重定向文件，只是将文件解压到临时目录。请移除该注释。

wangfuda · 2017-09-16T05:49:00Z

review及test case均已pass，请merge

jiongxuan · 2017-09-20T02:52:38Z

非常好。其实 @wangfuda 你也可以直接点Merge的 :) （我一开始以为你还在Review，哈哈）

那我就Merge进来了

keyongyu added 3 commits August 15, 2017 11:37

libs的jar重定向修改

cce6904

改为json保存修改信息

58e5a0d

不需要保存jar路径

82142dd

wangfuda reviewed Aug 17, 2017

View reviewed changes

keyongyu added 4 commits August 21, 2017 15:01

分buildType

e27e49d

Activities变化，也需要更新jar

c03c6aa

aar的jar都一样处理

a4c067f

不需要反射重定向

d4853cb

wangfuda reviewed Aug 27, 2017

View reviewed changes

247321453 added 3 commits August 27, 2017 16:50

代码调整

c2ca290

fix:repackage

0ca937c

code:reJarPath

a49598f

jiongxuan added the enhancement label Sep 20, 2017

Conversation

247321453 commented Aug 15, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

wangfuda Aug 17, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

wangfuda Aug 17, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

wangfuda commented Sep 16, 2017

Uh oh!

jiongxuan commented Sep 20, 2017

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

wangfuda Aug 17, 2017 •

edited

Loading

wangfuda Aug 17, 2017 •

edited

Loading