Refactor ProcessingContext initialization with factory method

Author: nakamura-to

doma-processor/src/main/java/org/seasar/doma/internal/apt/DomaProcessor.java

@@ -99,8 +99,7 @@ public class DomaProcessor extends AbstractProcessor {
   @Override
   public synchronized void init(ProcessingEnvironment processingEnv) {
     super.init(processingEnv);
-    processingContext = new ProcessingContext(processingEnv);
-    processingContext.init();
+    processingContext = ProcessingContext.of(processingEnv);
   }
 
   @Override

doma-processor/src/main/java/org/seasar/doma/internal/apt/ProcessingContext.java

@@ -30,11 +30,11 @@ public class ProcessingContext {
   private Reporter reporter;
   private Resources resources;
 
-  public ProcessingContext(ProcessingEnvironment env) {
+  private ProcessingContext(ProcessingEnvironment env) {
     this.env = Objects.requireNonNull(env);
   }
 
-  public void init() {
+  private void init() {
     if (initialized) {
       throw new AptIllegalStateException("already initialized");
     }
@@ -103,4 +103,11 @@ private void assertInitialized() {
       throw new AptIllegalStateException("not yet initialized");
     }
   }
+
+  public static ProcessingContext of(ProcessingEnvironment env) {
+    Objects.requireNonNull(env);
+    var processingContext = new ProcessingContext(env);
+    processingContext.init();
+    return processingContext;
+  }
 }

doma-processor/src/test/java/org/seasar/doma/internal/apt/TestProcessor.java

@@ -41,8 +41,7 @@ protected TestProcessor() {}
   @Override
   public synchronized void init(ProcessingEnvironment env) {
     super.init(env);
-    ctx = new ProcessingContext(env);
-    ctx.init();
+    ctx = ProcessingContext.of(env);
   }
 
   @Override

doma-processor/src/test/java/org/seasar/doma/internal/apt/processor/MyAnnotationProcessor.java

@@ -38,8 +38,7 @@ public MyAnnotationProcessor(Function<TypeElement, NullElementMeta> handler) {
   @Override
   public synchronized void init(ProcessingEnvironment processingEnv) {
     super.init(processingEnv);
-    this.ctx = new ProcessingContext(processingEnv);
-    ctx.init();
+    this.ctx = ProcessingContext.of(processingEnv);
   }
 
   @Override