fix capacity for option names, deprecate parser constructor

jbock-java · Mar 5, 2024 · cee798f · cee798f
1 parent 476368d
commit cee798f
Show file tree

Hide file tree

Showing 46 changed files with 91 additions and 159 deletions.
diff --git a/compiler/src/main/java/net/jbock/writing/CommandRepresentation.java b/compiler/src/main/java/net/jbock/writing/CommandRepresentation.java
@@ -1,7 +1,6 @@
 package net.jbock.writing;
 
 import io.jbock.javapoet.ClassName;
-import io.jbock.javapoet.FieldSpec;
 import net.jbock.annotated.Option;
 import net.jbock.annotated.Parameter;
 import net.jbock.annotated.VarargsParameter;
@@ -13,18 +12,10 @@
 import java.util.Optional;
 import java.util.function.Supplier;
 
-import static javax.lang.model.element.Modifier.FINAL;
-import static javax.lang.model.element.Modifier.PRIVATE;
-import static net.jbock.common.Constants.STRING;
-import static net.jbock.common.Constants.mapOf;
 import static net.jbock.common.Suppliers.memoize;
 
 public final class CommandRepresentation {
 
-    private final Supplier<FieldSpec> optionNames = memoize(() -> FieldSpec.builder(
-                    mapOf(STRING, optType()), "optionNames")
-            .addModifiers(PRIVATE, FINAL).build());
-
     private final Supplier<ClassName> optType = memoize(() -> namedOptions().isEmpty() ?
             ClassName.get(Void.class) : // javapoet #739
             sourceElement().optionEnumType());
@@ -70,10 +61,6 @@ List<Mapping<Option>> namedOptions() {
         return namedOptions;
     }
 
-    FieldSpec optionNames() {
-        return optionNames.get();
-    }
-
     /** Returns the type of the option enum. */
     ClassName optType() {
         return optType.get();

diff --git a/compiler/src/main/java/net/jbock/writing/CreateModelMethod.java b/compiler/src/main/java/net/jbock/writing/CreateModelMethod.java
@@ -9,6 +9,7 @@
 import net.jbock.model.Multiplicity;
 import net.jbock.model.Parameter;
 
+import javax.lang.model.element.Modifier;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.function.Supplier;
@@ -48,6 +49,7 @@ final class CreateModelMethod extends HasCommandRepresentation {
                 .addStatement(joinByNewline(code))
                 .returns(CommandModel.class)
                 .addModifiers(sourceElement().accessModifiers())
+                .addModifiers(Modifier.STATIC)
                 .build();
     });
 

diff --git a/compiler/src/main/java/net/jbock/writing/HasCommandRepresentation.java b/compiler/src/main/java/net/jbock/writing/HasCommandRepresentation.java
@@ -1,7 +1,6 @@
 package net.jbock.writing;
 
 import io.jbock.javapoet.ClassName;
-import io.jbock.javapoet.FieldSpec;
 import net.jbock.annotated.Option;
 import net.jbock.annotated.Parameter;
 import net.jbock.annotated.VarargsParameter;
@@ -31,10 +30,6 @@ final ClassName optType() {
         return commandRepresentation.optType();
     }
 
-    final FieldSpec optionNames() {
-        return commandRepresentation.optionNames();
-    }
-
     final List<Mapping<Parameter>> positionalParameters() {
         return commandRepresentation.positionalParameters();
     }

diff --git a/compiler/src/main/java/net/jbock/writing/OptionNamesMethod.java b/compiler/src/main/java/net/jbock/writing/OptionNamesMethod.java
@@ -13,6 +13,8 @@
 
 import static javax.lang.model.element.Modifier.PRIVATE;
 import static javax.lang.model.element.Modifier.STATIC;
+import static net.jbock.common.Constants.STRING;
+import static net.jbock.common.Constants.mapOf;
 import static net.jbock.common.Suppliers.memoize;
 
 final class OptionNamesMethod extends HasCommandRepresentation {
@@ -25,15 +27,16 @@ final class OptionNamesMethod extends HasCommandRepresentation {
 
     private final Supplier<MethodSpec> define = memoize(() -> {
         ParameterSpec result = ParameterSpec.builder(
-                optionNames().type, "result").build();
+                mapOf(STRING, optType()), "result").build();
         long mapSize = namedOptions().stream()
                 .map(Mapping::item)
                 .map(Option::names)
                 .map(List::size)
                 .mapToLong(i -> i)
                 .sum();
         CodeBlock.Builder code = CodeBlock.builder();
-        code.addStatement("$T $N = new $T<>($L)", result.type, result, HashMap.class, mapSize);
+        int capacity = (int) (1 + Math.max(mapSize * 1.35, 15));
+        code.addStatement("$T $N = new $T<>($L)", result.type, result, HashMap.class, capacity);
         for (Mapping<Option> namedOption : namedOptions()) {
             for (String dashedName : namedOption.item().names()) {
                 code.addStatement("$N.put($S, $T.$L)",

diff --git a/compiler/src/main/java/net/jbock/writing/OptionStatesMethod.java b/compiler/src/main/java/net/jbock/writing/OptionStatesMethod.java
@@ -17,6 +17,7 @@
 import java.util.function.Supplier;
 
 import static javax.lang.model.element.Modifier.PRIVATE;
+import static javax.lang.model.element.Modifier.STATIC;
 import static net.jbock.common.Constants.mapOf;
 import static net.jbock.common.Suppliers.memoize;
 
@@ -45,7 +46,7 @@ result, sourceElement().optionEnumType(),
         return MethodSpec.methodBuilder("optionStates")
                 .addCode(code.build())
                 .returns(result.type)
-                .addModifiers(PRIVATE)
+                .addModifiers(PRIVATE, STATIC)
                 .build();
     });
 

diff --git a/compiler/src/main/java/net/jbock/writing/ParseMethod.java b/compiler/src/main/java/net/jbock/writing/ParseMethod.java
@@ -6,6 +6,7 @@
 import io.jbock.simple.Inject;
 import net.jbock.util.ExFailure;
 
+import javax.lang.model.element.Modifier;
 import java.util.function.Supplier;
 
 import static io.jbock.javapoet.ParameterSpec.builder;
@@ -57,6 +58,7 @@ EITHER, e, createModelMethod().get())
                 .returns(generatedTypes().parseResultType())
                 .addCode(code.build())
                 .addModifiers(sourceElement().accessModifiers())
+                .addModifiers(Modifier.STATIC)
                 .build();
     });
 

diff --git a/compiler/src/main/java/net/jbock/writing/ParseOrExitMethod.java b/compiler/src/main/java/net/jbock/writing/ParseOrExitMethod.java
@@ -9,6 +9,7 @@
 import net.jbock.util.AtFileError;
 import net.jbock.util.ParseRequest;
 
+import javax.lang.model.element.Modifier;
 import java.util.List;
 
 import static io.jbock.javapoet.MethodSpec.methodBuilder;
@@ -77,6 +78,7 @@ MethodSpec define() {
         return methodBuilder("parseOrExit").addParameter(args)
                 .addModifiers(sourceElement().accessModifiers())
                 .returns(generatedTypes.sourceElement().typeName())
+                .addModifiers(Modifier.STATIC)
                 .addCode(code.build())
                 .build();
     }

diff --git a/compiler/src/main/java/net/jbock/writing/ParserClass.java b/compiler/src/main/java/net/jbock/writing/ParserClass.java
@@ -1,5 +1,7 @@
 package net.jbock.writing;
 
+import io.jbock.javapoet.AnnotationSpec;
+import io.jbock.javapoet.MethodSpec;
 import io.jbock.javapoet.TypeSpec;
 import io.jbock.simple.Inject;
 
@@ -53,17 +55,20 @@ TypeSpec define() {
             spec.addMethod(parseOrExitMethod.define());
         }
         if (!namedOptions().isEmpty()) {
-            spec.addField(optionNames().toBuilder()
-                    .initializer("$N()", optionNamesMethod.get()).build());
             spec.addMethod(optionNamesMethod.get());
             spec.addMethod(optionStatesMethod.get());
             spec.addType(optionEnum.define());
         }
 
         spec.addMethod(createModelMethod.get());
+        Modifier[] modifiers = sourceElement().accessModifiers().toArray(new Modifier[0]);
+        spec.addMethod(MethodSpec.constructorBuilder().addModifiers(modifiers)
+                .addJavadoc("Constructor is deprecated, use the static methods instead.")
+                .addAnnotation(AnnotationSpec.builder(Deprecated.class).addMember("forRemoval", "true").build())
+                .build());
 
         return spec.addOriginatingElement(sourceElement().element())
-                .addModifiers(sourceElement().accessModifiers().toArray(new Modifier[0]))
+                .addModifiers(modifiers)
                 .addModifiers(Modifier.FINAL)
                 .addType(implClass.define())
                 .addAnnotation(generatedAnnotation.define()).build();

diff --git a/compiler/src/main/java/net/jbock/writing/ParserTypeFactory.java b/compiler/src/main/java/net/jbock/writing/ParserTypeFactory.java
@@ -16,19 +16,22 @@
 final class ParserTypeFactory extends HasCommandRepresentation {
 
     private final OptionStatesMethod optionStatesMethod;
+    private final OptionNamesMethod optionNamesMethod;
 
     @Inject
     ParserTypeFactory(
             CommandRepresentation commandRepresentation,
-            OptionStatesMethod optionStatesMethod) {
+            OptionStatesMethod optionStatesMethod,
+            OptionNamesMethod optionNamesMethod) {
         super(commandRepresentation);
         this.optionStatesMethod = optionStatesMethod;
+        this.optionNamesMethod = optionNamesMethod;
     }
 
     private final Supplier<ParserType> parserType = memoize(() -> {
         CodeBlock optionNames = namedOptions().isEmpty() ?
                 CodeBlock.of("$T.of()", Map.class) :
-                CodeBlock.of("$N", optionNames());
+                CodeBlock.of("$N()", optionNamesMethod().get());
         CodeBlock optionStates = namedOptions().isEmpty() ?
                 CodeBlock.of("$T.of()", Map.class) :
                 CodeBlock.of("$N()", optionStatesMethod().get());
@@ -61,4 +64,8 @@ ParserType get() {
     private OptionStatesMethod optionStatesMethod() {
         return optionStatesMethod;
     }
+
+    private OptionNamesMethod optionNamesMethod() {
+        return optionNamesMethod;
+    }
 }
diff --git a/examples/src/main/java/examples/dustin/commandline/jbock/Main.java b/examples/src/main/java/examples/dustin/commandline/jbock/Main.java
@@ -28,7 +28,7 @@ abstract static class Arguments {
     }
 
     public static void main(String[] arguments) {
-        Arguments args = new Main_ArgumentsParser().parseOrExit(arguments);
+        Arguments args = Main_ArgumentsParser.parseOrExit(arguments);
         System.out.println("The file '" + args.file() + "' was provided and verbosity is set to '" + args.verbose() + "'.");
     }
 }
diff --git a/examples/src/test/java/examples/dustin/commandline/jbock/MainTest.java b/examples/src/test/java/examples/dustin/commandline/jbock/MainTest.java
@@ -7,10 +7,8 @@
 
 class MainTest {
 
-    private final Main_ArgumentsParser parser = new Main_ArgumentsParser();
-
     private final ParserTestFixture<Main.Arguments> f =
-            ParserTestFixture.create(parser::parse);
+            ParserTestFixture.create(Main_ArgumentsParser::parse);
 
     @Test
     void testMain() {

diff --git a/examples/src/test/java/net/jbock/examples/AdditionArgumentsTest.java b/examples/src/test/java/net/jbock/examples/AdditionArgumentsTest.java
@@ -7,10 +7,8 @@
 
 class AdditionArgumentsTest {
 
-    private final AdditionArgumentsParser parser = new AdditionArgumentsParser();
-
     private final ParserTestFixture<AdditionArguments> f =
-            ParserTestFixture.create(parser::parse);
+            ParserTestFixture.create(AdditionArgumentsParser::parse);
 
     @Test
     void optionalAbsent() {
@@ -48,7 +46,7 @@ void dashesIgnored() {
     @Test
     void testPrint() {
         f.assertPrintsHelp(
-                parser.createModel(),
+                AdditionArgumentsParser.createModel(),
                 "\u001B[1mUSAGE\u001B[m",
                 "  addition-arguments A B [C]",
                 "",

diff --git a/examples/src/test/java/net/jbock/examples/AllCharactersArgumentsTest.java b/examples/src/test/java/net/jbock/examples/AllCharactersArgumentsTest.java
@@ -8,10 +8,8 @@
 
 class AllCharactersArgumentsTest {
 
-    private final AllCharactersArgumentsParser parser = new AllCharactersArgumentsParser();
-
     private final ParserTestFixture<AllCharactersArguments> f =
-            ParserTestFixture.create(parser::parse);
+            ParserTestFixture.create(AllCharactersArgumentsParser::parse);
 
     @Test
     void tests() {

diff --git a/examples/src/test/java/net/jbock/examples/AllDoublesArgumentsTest.java b/examples/src/test/java/net/jbock/examples/AllDoublesArgumentsTest.java
@@ -8,10 +8,8 @@
 
 class AllDoublesArgumentsTest {
 
-    private final AllDoublesArgumentsParser parser = new AllDoublesArgumentsParser();
-
     private final ParserTestFixture<AllDoublesArguments> f =
-            ParserTestFixture.create(parser::parse);
+            ParserTestFixture.create(AllDoublesArgumentsParser::parse);
 
     @Test
     void listOfInteger() {

diff --git a/examples/src/test/java/net/jbock/examples/AllFlagsArgumentsTest.java b/examples/src/test/java/net/jbock/examples/AllFlagsArgumentsTest.java
@@ -5,10 +5,8 @@
 
 class AllFlagsArgumentsTest {
 
-    private final AllFlagsArgumentsParser parser = new AllFlagsArgumentsParser();
-
     private final ParserTestFixture<AllFlagsArguments> f =
-            ParserTestFixture.create(parser::parse);
+            ParserTestFixture.create(AllFlagsArgumentsParser::parse);
 
     @Test
     void tests() {

diff --git a/examples/src/test/java/net/jbock/examples/AllFloatsArgumentsTest.java b/examples/src/test/java/net/jbock/examples/AllFloatsArgumentsTest.java
@@ -8,10 +8,8 @@
 
 class AllFloatsArgumentsTest {
 
-    private final AllFloatsArgumentsParser parser = new AllFloatsArgumentsParser();
-
     private final ParserTestFixture<AllFloatsArguments> f =
-            ParserTestFixture.create(parser::parse);
+            ParserTestFixture.create(AllFloatsArgumentsParser::parse);
 
     @Test
     void listOfInteger() {

diff --git a/examples/src/test/java/net/jbock/examples/AllIntegersArgumentsTest.java b/examples/src/test/java/net/jbock/examples/AllIntegersArgumentsTest.java
@@ -9,10 +9,8 @@
 
 class AllIntegersArgumentsTest {
 
-    private final AllIntegersArgumentsParser parser = new AllIntegersArgumentsParser();
-
     private final ParserTestFixture<AllIntegersArguments> f =
-            ParserTestFixture.create(parser::parse);
+            ParserTestFixture.create(AllIntegersArgumentsParser::parse);
 
     @Test
     void listOfInteger() {

diff --git a/examples/src/test/java/net/jbock/examples/AllLongsArgumentsTest.java b/examples/src/test/java/net/jbock/examples/AllLongsArgumentsTest.java
@@ -8,10 +8,8 @@
 
 class AllLongsArgumentsTest {
 
-    private final AllLongsArgumentsParser parser = new AllLongsArgumentsParser();
-
     private final ParserTestFixture<AllLongsArguments> f =
-            ParserTestFixture.create(parser::parse);
+            ParserTestFixture.create(AllLongsArgumentsParser::parse);
 
     @Test
     void listOfInteger() {

diff --git a/examples/src/test/java/net/jbock/examples/ClusteredShortOptionsTest.java b/examples/src/test/java/net/jbock/examples/ClusteredShortOptionsTest.java
@@ -5,9 +5,7 @@
 
 class ClusteredShortOptionsTest {
 
-    private final ClusteredShortOptionsParser parser = new ClusteredShortOptionsParser();
-
-    private final ParserTestFixture<ClusteredShortOptions> f = ParserTestFixture.create(parser::parse);
+    private final ParserTestFixture<ClusteredShortOptions> f = ParserTestFixture.create(ClusteredShortOptionsParser::parse);
 
     @Test
     void testAttached() {

diff --git a/examples/src/test/java/net/jbock/examples/ComplicatedMapperArgumentsTest.java b/examples/src/test/java/net/jbock/examples/ComplicatedMapperArgumentsTest.java
@@ -15,10 +15,8 @@
 
 class ComplicatedMapperArgumentsTest {
 
-    private final ComplicatedMapperArgumentsParser parser = new ComplicatedMapperArgumentsParser();
-
     private final ParserTestFixture<ComplicatedMapperArguments> f =
-            ParserTestFixture.create(parser::parse);
+            ParserTestFixture.create(ComplicatedMapperArgumentsParser::parse);
 
     @Test
     void lazyNumber() {
@@ -34,7 +32,7 @@ void lazyNumber() {
 
     @Test
     void nullConverter() {
-        Either<ParsingFailed, ComplicatedMapperArguments> parsed = parser.parse(List.of(
+        Either<ParsingFailed, ComplicatedMapperArguments> parsed = ComplicatedMapperArgumentsParser.parse(List.of(
                 "-N", "12",
                 "--date", "2020-01-10"));
         assertTrue(parsed.getLeft().isPresent());

diff --git a/examples/src/test/java/net/jbock/examples/CpArgumentsTest.java b/examples/src/test/java/net/jbock/examples/CpArgumentsTest.java
@@ -8,10 +8,8 @@
 
 class CpArgumentsTest {
 
-    private final CpArgumentsParser parser = new CpArgumentsParser();
-
     private final ParserTestFixture<CpArguments> f =
-            ParserTestFixture.create(parser::parse);
+            ParserTestFixture.create(CpArgumentsParser::parse);
 
     @Test
     void errorMissingSource() {
@@ -102,7 +100,7 @@ void testEnum() {
     @Test
     void testPrint() {
         f.assertPrintsHelp(
-                parser.createModel(),
+                CpArgumentsParser.createModel(),
                 "\u001B[1mUSAGE\u001B[m",
                 "  cp-arguments [OPTIONS] SOURCE DEST",
                 "",