001/////////////////////////////////////////////////////////////////////////////////////////////// 002// checkstyle: Checks Java source code and other text files for adherence to a set of rules. 003// Copyright (C) 2001-2026 the original author or authors. 004// 005// This library is free software; you can redistribute it and/or 006// modify it under the terms of the GNU Lesser General Public 007// License as published by the Free Software Foundation; either 008// version 2.1 of the License, or (at your option) any later version. 009// 010// This library is distributed in the hope that it will be useful, 011// but WITHOUT ANY WARRANTY; without even the implied warranty of 012// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU 013// Lesser General Public License for more details. 014// 015// You should have received a copy of the GNU Lesser General Public 016// License along with this library; if not, write to the Free Software 017// Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA 018/////////////////////////////////////////////////////////////////////////////////////////////// 019 020package com.puppycrawl.tools.checkstyle.checks.sizes; 021 022import com.puppycrawl.tools.checkstyle.StatelessCheck; 023import com.puppycrawl.tools.checkstyle.api.AbstractCheck; 024import com.puppycrawl.tools.checkstyle.api.DetailAST; 025import com.puppycrawl.tools.checkstyle.api.TokenTypes; 026 027/** 028 * <div> 029 * Checks for long anonymous inner classes. 030 * </div> 031 * 032 * <p> 033 * Rationale: If an anonymous inner class becomes very long 034 * it is hard to understand and to see the flow of the method 035 * where the class is defined. Therefore, long anonymous inner 036 * classes should usually be refactored into a named inner class. 037 * See also Bloch, Effective Java, p. 93. 038 * </p> 039 * 040 * @since 3.2 041 */ 042@StatelessCheck 043public class AnonInnerLengthCheck extends AbstractCheck { 044 045 /** 046 * A key is pointing to the warning message text in "messages.properties" 047 * file. 048 */ 049 public static final String MSG_KEY = "maxLen.anonInner"; 050 051 /** Default maximum number of lines. */ 052 private static final int DEFAULT_MAX = 20; 053 054 /** Specify the maximum number of lines allowed. */ 055 private int max = DEFAULT_MAX; 056 057 /** 058 * Creates a new {@code AnonInnerLengthCheck} instance. 059 */ 060 public AnonInnerLengthCheck() { 061 // no code by default 062 } 063 064 @Override 065 public int[] getDefaultTokens() { 066 return getRequiredTokens(); 067 } 068 069 @Override 070 public int[] getAcceptableTokens() { 071 return getRequiredTokens(); 072 } 073 074 @Override 075 public int[] getRequiredTokens() { 076 return new int[] {TokenTypes.LITERAL_NEW}; 077 } 078 079 @Override 080 public void visitToken(DetailAST ast) { 081 final DetailAST openingBrace = ast.findFirstToken(TokenTypes.OBJBLOCK); 082 if (openingBrace != null) { 083 final DetailAST closingBrace = 084 openingBrace.findFirstToken(TokenTypes.RCURLY); 085 final int length = 086 closingBrace.getLineNo() - openingBrace.getLineNo() + 1; 087 if (length > max) { 088 log(ast, MSG_KEY, length, max); 089 } 090 } 091 } 092 093 /** 094 * Setter to specify the maximum number of lines allowed. 095 * 096 * @param length the maximum length of an anonymous inner class. 097 * @since 3.2 098 */ 099 public void setMax(int length) { 100 max = length; 101 } 102 103}