
org.sonar.java.checks.spring.ValueAnnotationShouldInjectPropertyOrSpELCheck Maven / Gradle / Ivy
/*
* SonarQube Java
* Copyright (C) 2012-2024 SonarSource SA
* mailto:info AT sonarsource DOT com
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the Sonar Source-Available License Version 1, as published by SonarSource SA.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
* See the Sonar Source-Available License for more details.
*
* You should have received a copy of the Sonar Source-Available License
* along with this program; if not, see https://sonarsource.com/license/ssal/
*/
package org.sonar.java.checks.spring;
import java.util.List;
import java.util.stream.Stream;
import org.sonar.check.Rule;
import org.sonar.java.checks.helpers.ExpressionsHelper;
import org.sonar.plugins.java.api.IssuableSubscriptionVisitor;
import org.sonar.plugins.java.api.tree.AnnotationTree;
import org.sonar.plugins.java.api.tree.AssignmentExpressionTree;
import org.sonar.plugins.java.api.tree.ClassTree;
import org.sonar.plugins.java.api.tree.ExpressionTree;
import org.sonar.plugins.java.api.tree.Tree;
import org.sonar.plugins.java.api.tree.VariableTree;
@Rule(key = "S6804")
public class ValueAnnotationShouldInjectPropertyOrSpELCheck extends IssuableSubscriptionVisitor {
private static final String SPRING_VALUE = "org.springframework.beans.factory.annotation.Value";
@Override
public List nodesToVisit() {
return List.of(Tree.Kind.CLASS, Tree.Kind.ANNOTATION_TYPE);
}
@Override
public void visitNode(Tree tree) {
ClassTree cls = (ClassTree) tree;
List fieldsAnnotations = cls.members()
.stream()
.filter(m -> m.is(Tree.Kind.VARIABLE))
.flatMap(field -> ((VariableTree) field).modifiers().annotations().stream())
.toList();
List interfaceAnnotations = cls.is(Tree.Kind.ANNOTATION_TYPE) ? cls.modifiers().annotations() : List.of();
Stream.concat(fieldsAnnotations.stream(), interfaceAnnotations.stream())
.filter(ValueAnnotationShouldInjectPropertyOrSpELCheck::isSimpleSpringValue)
.forEach(ann -> reportIssue(
ann,
"Either replace the \"@Value\" annotation with a standard field initialization," +
" use \"${propertyName}\" to inject a property " +
"or use \"#{expression}\" to evaluate a SpEL expression."));
}
private static boolean isSimpleSpringValue(AnnotationTree annotation) {
if (annotation.symbolType().is(SPRING_VALUE)) {
String value = extractArgumentValue(annotation.arguments().get(0));
return value != null && !isPropertyName(value) && !isSpEL(value) && !referenceResource(value);
}
return false;
}
private static String extractArgumentValue(ExpressionTree annotationArgument) {
if (annotationArgument.is(Tree.Kind.ASSIGNMENT)) {
ExpressionTree expression = ((AssignmentExpressionTree) annotationArgument).expression();
return ExpressionsHelper.getConstantValueAsString(expression).value();
}
return ExpressionsHelper.getConstantValueAsString(annotationArgument).value();
}
private static boolean isPropertyName(String value) {
return value.startsWith("${") && value.endsWith("}");
}
private static boolean isSpEL(String value) {
return value.startsWith("#{") && value.endsWith("}");
}
private static boolean referenceResource(String value) {
return value.startsWith("classpath:") || value.startsWith("file:") || value.startsWith("url:");
}
}
© 2015 - 2025 Weber Informatics LLC | Privacy Policy