From 145040c2c665943bf096ce6253e250e2ddd47ac7 Mon Sep 17 00:00:00 2001 From: zaaarf Date: Tue, 23 Jan 2024 12:07:00 +0100 Subject: [PATCH] fix: optional get default --- src/main/java/foo/zaaarf/routecompass/RouteCompass.java | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/main/java/foo/zaaarf/routecompass/RouteCompass.java b/src/main/java/foo/zaaarf/routecompass/RouteCompass.java index b5a6b3f..7c34739 100644 --- a/src/main/java/foo/zaaarf/routecompass/RouteCompass.java +++ b/src/main/java/foo/zaaarf/routecompass/RouteCompass.java @@ -61,7 +61,6 @@ public class RouteCompass extends AbstractProcessor { * @return false, letting other processor process the annotations again */ @Override - @SuppressWarnings("OptionalGetWithoutIsPresent") public boolean process(Set annotations, RoundEnvironment env) { for(TypeElement annotationType : annotations) { env.getElementsAnnotatedWith(annotationType) @@ -80,7 +79,7 @@ public class RouteCompass extends AbstractProcessor { this.getDTO(this.processingEnv.getTypeUtils().asElement(elem.getReturnType())), this.getDTO(elem.getParameters().stream() .filter(e -> e.getAnnotation(RequestBody.class) != null) - .findFirst().get()), + .findFirst().orElse(null)), this.getQueryParams(elem.getParameters()) )); }); @@ -100,10 +99,10 @@ public class RouteCompass extends AbstractProcessor { out.print("\t- "); if(r.deprecated) out.print("[DEPRECATED] "); out.print(r.method + " " + r.path); - if(r.consumes != null) { + if(r.consumes != null && r.consumes.length > 0) out.print("(expects: " + Arrays.toString(r.consumes) + ")"); - } - if(r.produces != null) out.print("(returns: " + Arrays.toString(r.produces) + ")"); + if(r.produces != null && r.produces.length > 0) + out.print("(returns: " + Arrays.toString(r.produces) + ")"); out.println(); BiConsumer printParam = (name, params) -> {