| /* |
| * Copyright 2000-2013 JetBrains s.r.o. |
| * |
| * Licensed under the Apache License, Version 2.0 (the "License"); |
| * you may not use this file except in compliance with the License. |
| * You may obtain a copy of the License at |
| * |
| * http://www.apache.org/licenses/LICENSE-2.0 |
| * |
| * Unless required by applicable law or agreed to in writing, software |
| * distributed under the License is distributed on an "AS IS" BASIS, |
| * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| * See the License for the specific language governing permissions and |
| * limitations under the License. |
| */ |
| package com.jetbrains.python.inspections; |
| |
| import com.intellij.codeInspection.LocalInspectionToolSession; |
| import com.intellij.codeInspection.ProblemsHolder; |
| import com.intellij.psi.PsiElementVisitor; |
| import com.intellij.util.containers.hash.HashMap; |
| import com.jetbrains.python.PyBundle; |
| import com.jetbrains.python.inspections.quickfix.PyRemoveAssignmentQuickFix; |
| import com.jetbrains.python.psi.*; |
| import com.jetbrains.python.psi.search.PyOverridingMethodsSearch; |
| import com.jetbrains.python.psi.types.PyNoneType; |
| import com.jetbrains.python.psi.types.PyType; |
| import com.jetbrains.python.psi.types.PyTypeChecker; |
| import com.jetbrains.python.sdk.PySdkUtil; |
| import org.jetbrains.annotations.Nls; |
| import org.jetbrains.annotations.NotNull; |
| import org.jetbrains.annotations.Nullable; |
| |
| import java.util.Map; |
| |
| /** |
| * User: ktisha |
| * |
| * pylint E1111 |
| * |
| * Used when an assignment is done on a function call but the inferred function doesn't return anything. |
| */ |
| public class PyNoneFunctionAssignmentInspection extends PyInspection { |
| @Nls |
| @NotNull |
| @Override |
| public String getDisplayName() { |
| return PyBundle.message("INSP.NAME.none.function.assignment"); |
| } |
| |
| @NotNull |
| @Override |
| public PsiElementVisitor buildVisitor(@NotNull ProblemsHolder holder, |
| boolean isOnTheFly, |
| @NotNull LocalInspectionToolSession session) { |
| return new Visitor(holder, session); |
| } |
| |
| |
| private static class Visitor extends PyInspectionVisitor { |
| private final Map<PyFunction, Boolean> myHasInheritors = new HashMap<PyFunction, Boolean>(); |
| |
| public Visitor(@Nullable ProblemsHolder holder, @NotNull LocalInspectionToolSession session) { |
| super(holder, session); |
| } |
| |
| @Override |
| public void visitPyAssignmentStatement(PyAssignmentStatement node) { |
| final PyExpression value = node.getAssignedValue(); |
| if (value instanceof PyCallExpression) { |
| final PyType type = myTypeEvalContext.getType(value); |
| final PyExpression callee = ((PyCallExpression)value).getCallee(); |
| |
| if (type instanceof PyNoneType && callee != null) { |
| final PyTypeChecker.AnalyzeCallResults analyzeCallResults = PyTypeChecker.analyzeCall(((PyCallExpression)value), myTypeEvalContext); |
| if (analyzeCallResults != null) { |
| final Callable callable = analyzeCallResults.getCallable(); |
| if (PySdkUtil.isElementInSkeletons(callable)) { |
| return; |
| } |
| if (callable instanceof PyFunction) { |
| final PyFunction function = (PyFunction)callable; |
| // Currently we don't infer types returned by decorators |
| if (hasInheritors(function) || PyUtil.hasCustomDecorators(function)) { |
| return; |
| } |
| } |
| registerProblem(node, PyBundle.message("INSP.none.function.assignment", callee.getName()), new PyRemoveAssignmentQuickFix()); |
| } |
| } |
| } |
| } |
| |
| private boolean hasInheritors(@NotNull PyFunction function) { |
| final Boolean cached = myHasInheritors.get(function); |
| if (cached != null) { |
| return cached; |
| } |
| final boolean result = PyOverridingMethodsSearch.search(function, true).findFirst() != null; |
| myHasInheritors.put(function, result); |
| return result; |
| } |
| } |
| } |