Warn the developer while changing the param name

Asked Jan 11 '21 at 06:07

Active Jul 13 '21 at 10:50

Viewed 86 times

I am having a requirement where I need to fetch a param the value and validate before a method is invoked. I use MethodInterceptor (org.aopalliance.intercept.MethodInvocation) for this.

My method has three String parameters. I need to validate only the second parameter or specifically the parameter with the name pname. Currently, I use the following logic to validate the value of this parameter. But I have to hard code the param name or hardcode the index.

    int i=0;
    for(Parameter param: invocation.getMethod().getParameters()) {
     
        if(param.getName().equalsIgnoreCase("pname")) {
            //do validation using invocation.getArguments()[i]
        }
     i++;
    }

Either approach will fail if somebody modifies the method. Do we have a better approach or in worst case warn the future developer while he modifies the method param.

edited Jul 13 '21 at 10:50

kaya3

47,440
4
68
97

asked Jan 11 '21 at 06:07

Renjith

1,122
1
19
45

1

A parameter **annotation** might solve this better, – Joop Eggen Jan 11 '21 at 06:15
@JoopEggen can you please elaborate? – Renjith Jan 11 '21 at 06:19
`void foo(int bar, @Baz String pname)` That is: not relying on the name. Quite traditional. – Joop Eggen Jan 11 '21 at 06:43
@JoopEggen. I created an annotation with '@Target(ElementType.PARAMETER)' and applied it on param. But params[i].getDeclaredAnnotation(MyParam.class) is returning null for all params! Where MyParam is the annotation class. – Renjith Jan 11 '21 at 07:52
Is it a runtime annotation? `@Retention(RetentionPolicy.RUNTIME)` – Joop Eggen Jan 11 '21 at 19:01
Yes, it is a Runtime annotation. I got it working. thanks. – Renjith Jan 12 '21 at 06:12

Warn the developer while changing the param name

0 Answers0