/plugins/InspectionGadgets/src/com/siyeh/ig/visibility/MethodOverridesPrivateMethodInspection.java
https://bitbucket.org/nbargnesi/idea · Java · 93 lines · 67 code · 11 blank · 15 comment · 10 complexity · 0a56e10fb86471675fc52c18e75bc4d7 MD5 · raw file
- /*
- * Copyright 2003-2007 Dave Griffith, Bas Leijdekkers
- *
- * 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.siyeh.ig.visibility;
- import com.intellij.psi.PsiClass;
- import com.intellij.psi.PsiMethod;
- import com.intellij.psi.PsiModifier;
- import com.siyeh.InspectionGadgetsBundle;
- import com.siyeh.ig.BaseInspection;
- import com.siyeh.ig.BaseInspectionVisitor;
- import com.siyeh.ig.InspectionGadgetsFix;
- import com.siyeh.ig.fixes.RenameFix;
- import org.jetbrains.annotations.NotNull;
- import java.util.HashSet;
- import java.util.Set;
- public class MethodOverridesPrivateMethodInspection extends BaseInspection {
- @NotNull
- public String getID() {
- return "MethodOverridesPrivateMethodOfSuperclass";
- }
- @NotNull
- public String getDisplayName() {
- return InspectionGadgetsBundle.message(
- "method.overrides.private.display.name");
- }
- @NotNull
- public String buildErrorString(Object... infos) {
- return InspectionGadgetsBundle.message(
- "method.overrides.private.display.name.problem.descriptor");
- }
- protected InspectionGadgetsFix buildFix(Object... infos) {
- return new RenameFix();
- }
- protected boolean buildQuickFixesOnlyForOnTheFlyErrors() {
- return true;
- }
- public BaseInspectionVisitor buildVisitor() {
- return new MethodOverridesPrivateMethodVisitor();
- }
- private static class MethodOverridesPrivateMethodVisitor
- extends BaseInspectionVisitor {
- @Override
- public void visitMethod(@NotNull PsiMethod method) {
- final PsiClass aClass = method.getContainingClass();
- if (aClass == null) {
- return;
- }
- if (method.getNameIdentifier() == null) {
- return;
- }
- PsiClass ancestorClass = aClass.getSuperClass();
- final Set<PsiClass> visitedClasses = new HashSet<PsiClass>();
- while (ancestorClass != null) {
- if (!visitedClasses.add(ancestorClass)) {
- return;
- }
- final PsiMethod overridingMethod =
- ancestorClass.findMethodBySignature(method, false);
- if (overridingMethod != null) {
- if (overridingMethod.hasModifierProperty(
- PsiModifier.PRIVATE)) {
- registerMethodError(method);
- return;
- }
- }
- ancestorClass = ancestorClass.getSuperClass();
- }
- }
- }
- }