OLD | NEW |
(Empty) | |
| 1 // Copyright (c) 2016, the Dart project authors. Please see the AUTHORS file |
| 2 // for details. All rights reserved. Use of this source code is governed by a |
| 3 // BSD-style license that can be found in the LICENSE file. |
| 4 |
| 5 library linter.src.rules.overriden_field; |
| 6 |
| 7 import 'package:analyzer/dart/ast/ast.dart'; |
| 8 import 'package:analyzer/dart/ast/visitor.dart'; |
| 9 import 'package:analyzer/dart/element/element.dart'; |
| 10 import 'package:analyzer/src/generated/resolver.dart'; |
| 11 import 'package:linter/src/linter.dart'; |
| 12 |
| 13 const desc = r'Do not override fields.'; |
| 14 |
| 15 const details = r''' |
| 16 |
| 17 **DO** Do not override fields. |
| 18 |
| 19 **BAD:** |
| 20 ``` |
| 21 class Base { |
| 22 Object field = 'lorem'; |
| 23 |
| 24 Object something = 'change'; |
| 25 } |
| 26 |
| 27 class Bad1 extends Base { |
| 28 @override |
| 29 final field = 'ipsum'; // LINT |
| 30 } |
| 31 |
| 32 class Bad2 extends Base { |
| 33 @override |
| 34 Object something = 'done'; // LINT |
| 35 } |
| 36 ``` |
| 37 |
| 38 **GOOD:** |
| 39 ``` |
| 40 class Base { |
| 41 Object field = 'lorem'; |
| 42 |
| 43 Object something = 'change'; |
| 44 } |
| 45 |
| 46 class Ok extends Base { |
| 47 Object newField; // OK |
| 48 |
| 49 final Object newFinal = 'ignore'; // OK |
| 50 } |
| 51 ``` |
| 52 |
| 53 '''; |
| 54 |
| 55 class OverridenField extends LintRule { |
| 56 OverridenField() |
| 57 : super( |
| 58 name: 'overriden_field', |
| 59 description: desc, |
| 60 details: details, |
| 61 group: Group.style); |
| 62 |
| 63 @override |
| 64 AstVisitor getVisitor() => new _Visitor(this); |
| 65 } |
| 66 |
| 67 class _Visitor extends SimpleAstVisitor { |
| 68 InheritanceManager _manager; |
| 69 |
| 70 final LintRule rule; |
| 71 _Visitor(this.rule); |
| 72 |
| 73 @override |
| 74 visitCompilationUnit(CompilationUnit node) { |
| 75 LibraryElement library = node?.element?.library; |
| 76 _manager = library == null ? null : new InheritanceManager(library); |
| 77 } |
| 78 |
| 79 @override |
| 80 visitFieldDeclaration(FieldDeclaration node) { |
| 81 node.fields.variables.forEach((VariableDeclaration variable) { |
| 82 ExecutableElement member = _getOverriddenMember(variable.element); |
| 83 if (member is PropertyAccessorElement && member.isSynthetic) { |
| 84 rule.reportLint(variable.name); |
| 85 } |
| 86 }); |
| 87 } |
| 88 |
| 89 ExecutableElement _getOverriddenMember(Element member) { |
| 90 if (member == null || _manager == null) { |
| 91 return null; |
| 92 } |
| 93 |
| 94 ClassElement classElement = |
| 95 member.getAncestor((element) => element is ClassElement); |
| 96 if (classElement == null) { |
| 97 return null; |
| 98 } |
| 99 |
| 100 return _manager.lookupInheritance(classElement, member.name); |
| 101 } |
| 102 } |
OLD | NEW |