blob: 99bab04189b228eb398bc43871eea76f86b216a1 [file] [log] [blame]
// 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.
import '../analyzer.dart';
import '../util/dart_type_utilities.dart';
import '../util/unrelated_types_visitor.dart';
const _desc = r'Invocation of Iterable<E>.contains with references of unrelated'
r' types.';
const _details = r'''
**DON'T** invoke `contains` on `Iterable` with an instance of different type
than the parameter type.
Doing this will invoke `==` on its elements and most likely will return `false`.
**BAD:**
```
void someFunction() {
var list = <int>[];
if (list.contains('1')) print('someFunction'); // LINT
}
```
**BAD:**
```
void someFunction3() {
List<int> list = <int>[];
if (list.contains('1')) print('someFunction3'); // LINT
}
```
**BAD:**
```
void someFunction8() {
List<DerivedClass2> list = <DerivedClass2>[];
DerivedClass3 instance;
if (list.contains(instance)) print('someFunction8'); // LINT
}
```
**BAD:**
```
abstract class SomeIterable<E> implements Iterable<E> {}
abstract class MyClass implements SomeIterable<int> {
bool badMethod(String thing) => this.contains(thing); // LINT
}
```
**GOOD:**
```
void someFunction10() {
var list = [];
if (list.contains(1)) print('someFunction10'); // OK
}
```
**GOOD:**
```
void someFunction1() {
var list = <int>[];
if (list.contains(1)) print('someFunction1'); // OK
}
```
**GOOD:**
```
void someFunction4() {
List<int> list = <int>[];
if (list.contains(1)) print('someFunction4'); // OK
}
```
**GOOD:**
```
void someFunction5() {
List<ClassBase> list = <ClassBase>[];
DerivedClass1 instance;
if (list.contains(instance)) print('someFunction5'); // OK
}
abstract class ClassBase {}
class DerivedClass1 extends ClassBase {}
```
**GOOD:**
```
void someFunction6() {
List<Mixin> list = <Mixin>[];
DerivedClass2 instance;
if (list.contains(instance)) print('someFunction6'); // OK
}
abstract class ClassBase {}
abstract class Mixin {}
class DerivedClass2 extends ClassBase with Mixin {}
```
**GOOD:**
```
void someFunction7() {
List<Mixin> list = <Mixin>[];
DerivedClass3 instance;
if (list.contains(instance)) print('someFunction7'); // OK
}
abstract class ClassBase {}
abstract class Mixin {}
class DerivedClass3 extends ClassBase implements Mixin {}
```
''';
class IterableContainsUnrelatedType extends LintRule implements NodeLintRule {
IterableContainsUnrelatedType()
: super(
name: 'iterable_contains_unrelated_type',
description: _desc,
details: _details,
group: Group.errors);
@override
void registerNodeProcessors(
NodeLintRegistry registry, LinterContext context) {
final visitor = _Visitor(this, context.typeSystem);
registry.addMethodInvocation(this, visitor);
}
}
class _Visitor extends UnrelatedTypesProcessors {
static final _DEFINITION = InterfaceTypeDefinition('Iterable', 'dart.core');
_Visitor(LintRule rule, TypeSystem typeSystem) : super(rule, typeSystem);
@override
InterfaceTypeDefinition get definition => _DEFINITION;
@override
String get methodName => 'contains';
}