diff --git a/agent/src/main/java/com/alibaba/testable/PreMain.java b/agent/src/main/java/com/alibaba/testable/PreMain.java
index 29ef73b..09020f7 100755
--- a/agent/src/main/java/com/alibaba/testable/PreMain.java
+++ b/agent/src/main/java/com/alibaba/testable/PreMain.java
@@ -4,6 +4,9 @@ import com.alibaba.testable.transformer.TestableFileTransformer;
 
 import java.lang.instrument.Instrumentation;
 
+/**
+ * @author flin
+ */
 public class PreMain {
 
     public static void premain(String agentArgs, Instrumentation inst) {
diff --git a/agent/src/main/java/com/alibaba/testable/model/MethodInfo.java b/agent/src/main/java/com/alibaba/testable/model/MethodInfo.java
index 3da61a8..0e9fabd 100644
--- a/agent/src/main/java/com/alibaba/testable/model/MethodInfo.java
+++ b/agent/src/main/java/com/alibaba/testable/model/MethodInfo.java
@@ -1,5 +1,8 @@
 package com.alibaba.testable.model;
 
+/**
+ * @author flin
+ */
 public class MethodInfo {
 
     private int access;
diff --git a/agent/src/main/java/com/alibaba/testable/transformer/TestableFileTransformer.java b/agent/src/main/java/com/alibaba/testable/transformer/TestableFileTransformer.java
index e8fe8d5..9fa54bf 100644
--- a/agent/src/main/java/com/alibaba/testable/transformer/TestableFileTransformer.java
+++ b/agent/src/main/java/com/alibaba/testable/transformer/TestableFileTransformer.java
@@ -11,6 +11,9 @@ import java.security.ProtectionDomain;
 import java.util.HashSet;
 import java.util.Set;
 
+/**
+ * @author flin
+ */
 public class TestableFileTransformer implements ClassFileTransformer {
 
     private static final String ENABLE_TESTABLE = "com.alibaba.testable.annotation.EnableTestable";
@@ -18,7 +21,7 @@ public class TestableFileTransformer implements ClassFileTransformer {
     private static final String SLASH = "/";
     private static final String TEST_POSTFIX = "Test";
 
-    private static Set<String> loadedClassNames = new HashSet<String>();
+    private static final Set<String> loadedClassNames = new HashSet<String>();
 
     public byte[] transform(ClassLoader loader, String className, Class<?> classBeingRedefined,
                             ProtectionDomain protectionDomain, byte[] classfileBuffer) {
diff --git a/agent/src/main/java/com/alibaba/testable/visitor/MethodRecordVisitor.java b/agent/src/main/java/com/alibaba/testable/visitor/MethodRecordVisitor.java
index c9c4acd..4eb1124 100755
--- a/agent/src/main/java/com/alibaba/testable/visitor/MethodRecordVisitor.java
+++ b/agent/src/main/java/com/alibaba/testable/visitor/MethodRecordVisitor.java
@@ -11,7 +11,7 @@ public class MethodRecordVisitor extends ClassVisitor {
     /**
      * Member methods
      */
-    private List<MethodInfo> methods = new ArrayList<MethodInfo>();
+    private final List<MethodInfo> methods = new ArrayList<MethodInfo>();
     private boolean needTransform;
 
     private static final String ENABLE_TESTABLE_INJECT = "Lcom/alibaba/testable/annotation/EnableTestableInject;";
diff --git a/agent/src/main/java/com/alibaba/testable/visitor/TestableVisitor.java b/agent/src/main/java/com/alibaba/testable/visitor/TestableVisitor.java
index 0768615..daeb252 100755
--- a/agent/src/main/java/com/alibaba/testable/visitor/TestableVisitor.java
+++ b/agent/src/main/java/com/alibaba/testable/visitor/TestableVisitor.java
@@ -8,6 +8,9 @@ import org.objectweb.asm.Opcodes;
 
 import java.util.List;
 
+/**
+ * @author flin
+ */
 public class TestableVisitor extends ClassVisitor {
 
     private List<MethodInfo> methods;