| Index: lib/src/rules/overriden_field.dart
|
| diff --git a/lib/src/rules/overriden_field.dart b/lib/src/rules/overriden_field.dart
|
| new file mode 100644
|
| index 0000000000000000000000000000000000000000..193d7fc2449ff14a3918d1bb9519119b3c9b60c8
|
| --- /dev/null
|
| +++ b/lib/src/rules/overriden_field.dart
|
| @@ -0,0 +1,102 @@
|
| +// Copyright (c) 2016, the Dart project authors. Please see the AUTHORS file
|
| +// for details. All rights reserved. Use of this source code is governed by a
|
| +// BSD-style license that can be found in the LICENSE file.
|
| +
|
| +library linter.src.rules.overriden_field;
|
| +
|
| +import 'package:analyzer/dart/ast/ast.dart';
|
| +import 'package:analyzer/dart/ast/visitor.dart';
|
| +import 'package:analyzer/dart/element/element.dart';
|
| +import 'package:analyzer/src/generated/resolver.dart';
|
| +import 'package:linter/src/linter.dart';
|
| +
|
| +const desc = r'Do not override fields.';
|
| +
|
| +const details = r'''
|
| +
|
| +**DO** Do not override fields.
|
| +
|
| +**BAD:**
|
| +```
|
| +class Base {
|
| + Object field = 'lorem';
|
| +
|
| + Object something = 'change';
|
| +}
|
| +
|
| +class Bad1 extends Base {
|
| + @override
|
| + final field = 'ipsum'; // LINT
|
| +}
|
| +
|
| +class Bad2 extends Base {
|
| + @override
|
| + Object something = 'done'; // LINT
|
| +}
|
| +```
|
| +
|
| +**GOOD:**
|
| +```
|
| +class Base {
|
| + Object field = 'lorem';
|
| +
|
| + Object something = 'change';
|
| +}
|
| +
|
| +class Ok extends Base {
|
| + Object newField; // OK
|
| +
|
| + final Object newFinal = 'ignore'; // OK
|
| +}
|
| +```
|
| +
|
| +''';
|
| +
|
| +class OverridenField extends LintRule {
|
| + OverridenField()
|
| + : super(
|
| + name: 'overriden_field',
|
| + description: desc,
|
| + details: details,
|
| + group: Group.style);
|
| +
|
| + @override
|
| + AstVisitor getVisitor() => new _Visitor(this);
|
| +}
|
| +
|
| +class _Visitor extends SimpleAstVisitor {
|
| + InheritanceManager _manager;
|
| +
|
| + final LintRule rule;
|
| + _Visitor(this.rule);
|
| +
|
| + @override
|
| + visitCompilationUnit(CompilationUnit node) {
|
| + LibraryElement library = node?.element?.library;
|
| + _manager = library == null ? null : new InheritanceManager(library);
|
| + }
|
| +
|
| + @override
|
| + visitFieldDeclaration(FieldDeclaration node) {
|
| + node.fields.variables.forEach((VariableDeclaration variable) {
|
| + ExecutableElement member = _getOverriddenMember(variable.element);
|
| + if (member is PropertyAccessorElement && member.isSynthetic) {
|
| + rule.reportLint(variable.name);
|
| + }
|
| + });
|
| + }
|
| +
|
| + ExecutableElement _getOverriddenMember(Element member) {
|
| + if (member == null || _manager == null) {
|
| + return null;
|
| + }
|
| +
|
| + ClassElement classElement =
|
| + member.getAncestor((element) => element is ClassElement);
|
| + if (classElement == null) {
|
| + return null;
|
| + }
|
| +
|
| + return _manager.lookupInheritance(classElement, member.name);
|
| + }
|
| +}
|
|
|