blob: 5642fae4407bbf428bf4d9e25e2aa8cae43a2bf4 [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 'package:analyzer/dart/ast/ast.dart';
import 'package:analyzer/dart/ast/visitor.dart';
import 'package:analyzer/dart/element/element2.dart';
import 'package:analyzer/dart/element/type.dart';
import 'package:analyzer/src/lint/constants.dart'; // ignore: implementation_imports
import '../analyzer.dart';
import '../extensions.dart';
const _desc = r'Prefer `const` with constant constructors.';
class PreferConstConstructors extends LintRule {
PreferConstConstructors()
: super(
name: LintNames.prefer_const_constructors,
description: _desc,
);
@override
LintCode get lintCode => LinterLintCode.prefer_const_constructors;
@override
void registerNodeProcessors(
NodeLintRegistry registry, LinterContext context) {
var visitor = _Visitor(this);
registry.addInstanceCreationExpression(this, visitor);
}
}
class _Visitor extends SimpleAstVisitor<void> {
final LintRule rule;
_Visitor(this.rule);
@override
void visitInstanceCreationExpression(InstanceCreationExpression node) {
if (node.isConst) return;
if (node.constructorName.type.isDeferred) return;
var element = node.constructorName.element;
if (element == null) return;
if (!element.isConst) return;
// Handled by an analyzer warning.
if (element.metadata2.hasLiteral) return;
var enclosingElement = element.enclosingElement2;
if (enclosingElement is ClassElement2 &&
enclosingElement.isDartCoreObject) {
// Skip lint for `new Object()`, because it can be used for ID creation.
return;
}
if (enclosingElement.typeParameters2.isNotEmpty &&
node.constructorName.type.typeArguments == null) {
var approximateContextType = node.approximateContextType;
var contextTypeAsInstanceOfEnclosing =
approximateContextType?.asInstanceOf2(enclosingElement);
if (contextTypeAsInstanceOfEnclosing != null) {
if (contextTypeAsInstanceOfEnclosing.typeArguments
.any((e) => e is TypeParameterType)) {
// The context type has type parameters, which may be substituted via
// upward inference from the static type of `node`. Changing `node`
// from non-const to const will affect inference to change its own
// type arguments to `Never`, which affects that upward inference.
// See https://github.com/dart-lang/linter/issues/4531.
return;
}
}
}
if (node.canBeConst) {
rule.reportLint(node);
}
}
}