01: /**
02: * BSD-style license; for more info see http://pmd.sourceforge.net/license.html
03: */package net.sourceforge.pmd.typeresolution.rules;
04:
05: import net.sourceforge.pmd.AbstractJavaRule;
06: import net.sourceforge.pmd.ast.ASTClassOrInterfaceType;
07: import net.sourceforge.pmd.ast.ASTFieldDeclaration;
08: import net.sourceforge.pmd.ast.ASTFormalParameter;
09: import net.sourceforge.pmd.ast.ASTResultType;
10: import net.sourceforge.pmd.ast.Node;
11: import net.sourceforge.pmd.util.CollectionUtil;
12:
13: /**
14: * This is a separate rule, uses the type resolution facade
15: */
16: public class LooseCoupling extends AbstractJavaRule {
17:
18: public LooseCoupling() {
19: super ();
20: }
21:
22: public Object visit(ASTClassOrInterfaceType node, Object data) {
23: Node parent = node.getNthParent(3);
24: Class clazzType = node.getType();
25: boolean isType = CollectionUtil.isCollectionType(clazzType,
26: false);
27: if (isType
28: && (parent instanceof ASTFieldDeclaration
29: || parent instanceof ASTFormalParameter || parent instanceof ASTResultType)) {
30: addViolation(data, node, node.getImage());
31: }
32: return data;
33: }
34: }
|