Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(276)

Unified Diff: lib/src/rules/overriden_field.dart

Issue 1870423002: Lint rule: Do not override fields. Base URL: https://github.com/dart-lang/linter.git@master
Patch Set: Hint if a field overrides or hides other field. Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « lib/src/rules.dart ('k') | test/rules/overriden_field.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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);
+ }
+}
« no previous file with comments | « lib/src/rules.dart ('k') | test/rules/overriden_field.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698