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

Side by Side 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 unified diff | Download patch
« no previous file with comments | « lib/src/rules.dart ('k') | test/rules/overriden_field.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(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 }
OLDNEW
« 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