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.coding; 021 022import java.util.ArrayList; 023import java.util.Collections; 024import java.util.List; 025import java.util.Optional; 026 027import com.puppycrawl.tools.checkstyle.StatelessCheck; 028import com.puppycrawl.tools.checkstyle.api.AbstractCheck; 029import com.puppycrawl.tools.checkstyle.api.DetailAST; 030import com.puppycrawl.tools.checkstyle.api.TokenTypes; 031import com.puppycrawl.tools.checkstyle.utils.TokenUtil; 032 033/** 034 * <div> 035 * Checks that a given switch statement or expression that use a reference type in its selector 036 * expression has a {@code null} case label. 037 * </div> 038 * 039 * <p> 040 * Rationale: switch statements and expressions in Java throw a 041 * {@code NullPointerException} if the selector expression evaluates to {@code null}. 042 * As of Java 21, it is now possible to integrate a null check within the switch, 043 * eliminating the risk of {@code NullPointerException} and simplifies the code 044 * as there is no need for an external null check before entering the switch. 045 * </p> 046 * 047 * <p> 048 * See the <a href="https://docs.oracle.com/javase/specs/jls/se22/html/jls-15.html#jls-15.28"> 049 * Java Language Specification</a> for more information about switch statements and expressions. 050 * </p> 051 * 052 * <p> 053 * Specifically, this check validates switch statement or expression 054 * that use patterns or strings in their case labels. 055 * </p> 056 * 057 * <p> 058 * Due to Checkstyle not being type-aware, this check cannot validate other reference types, 059 * such as enums; syntactically, these are no different from other constants. 060 * </p> 061 * 062 * <p> 063 * <b>Attention</b>: this Check should be activated only on source code 064 * that is compiled by jdk21 or above. 065 * </p> 066 * 067 * @since 10.18.0 068 */ 069 070@StatelessCheck 071public class MissingNullCaseInSwitchCheck extends AbstractCheck { 072 073 /** 074 * A key is pointing to the warning message text in "messages.properties" file. 075 */ 076 public static final String MSG_KEY = "missing.switch.nullcase"; 077 078 /** 079 * Creates a new {@code MissingNullCaseInSwitchCheck} instance. 080 */ 081 public MissingNullCaseInSwitchCheck() { 082 // no code by default 083 } 084 085 @Override 086 public int[] getDefaultTokens() { 087 return getRequiredTokens(); 088 } 089 090 @Override 091 public int[] getAcceptableTokens() { 092 return getRequiredTokens(); 093 } 094 095 @Override 096 public int[] getRequiredTokens() { 097 return new int[] {TokenTypes.LITERAL_SWITCH}; 098 } 099 100 @Override 101 public void visitToken(DetailAST ast) { 102 final List<DetailAST> caseLabels = getAllCaseLabels(ast); 103 final boolean hasNullCaseLabel = caseLabels.stream() 104 .anyMatch(MissingNullCaseInSwitchCheck::hasLiteralNull); 105 if (!hasNullCaseLabel) { 106 final boolean hasPatternCaseLabel = caseLabels.stream() 107 .anyMatch(MissingNullCaseInSwitchCheck::hasPatternCaseLabel); 108 final boolean hasStringCaseLabel = caseLabels.stream() 109 .anyMatch(MissingNullCaseInSwitchCheck::hasStringCaseLabel); 110 if (hasPatternCaseLabel || hasStringCaseLabel) { 111 log(ast, MSG_KEY); 112 } 113 } 114 } 115 116 /** 117 * Gets all case labels in the given switch AST node. 118 * 119 * @param switchAST the AST node representing {@code LITERAL_SWITCH} 120 * @return a list of all case labels in the switch 121 */ 122 private static List<DetailAST> getAllCaseLabels(DetailAST switchAST) { 123 final List<DetailAST> caseLabels = new ArrayList<>(); 124 DetailAST ast = switchAST.getFirstChild(); 125 while (ast != null) { 126 // case group token may have several LITERAL_CASE tokens 127 TokenUtil.forEachChild(ast, TokenTypes.LITERAL_CASE, caseLabels::add); 128 ast = ast.getNextSibling(); 129 } 130 return Collections.unmodifiableList(caseLabels); 131 } 132 133 /** 134 * Checks if the given case AST node has a null label. 135 * 136 * @param caseAST the AST node representing {@code LITERAL_CASE} 137 * @return true if the case has {@code null} label, false otherwise 138 */ 139 private static boolean hasLiteralNull(DetailAST caseAST) { 140 return Optional.ofNullable(caseAST.findFirstToken(TokenTypes.EXPR)) 141 .map(exp -> exp.findFirstToken(TokenTypes.LITERAL_NULL)) 142 .isPresent(); 143 } 144 145 /** 146 * Checks if the given case AST node has a pattern variable declaration label 147 * or record pattern definition label. 148 * 149 * @param caseAST the AST node representing {@code LITERAL_CASE} 150 * @return true if case has a pattern in its label 151 */ 152 private static boolean hasPatternCaseLabel(DetailAST caseAST) { 153 return caseAST.findFirstToken(TokenTypes.RECORD_PATTERN_DEF) != null 154 || caseAST.findFirstToken(TokenTypes.PATTERN_VARIABLE_DEF) != null 155 || caseAST.findFirstToken(TokenTypes.PATTERN_DEF) != null; 156 } 157 158 /** 159 * Checks if the given case contains a string in its label. 160 * It may contain a single string literal or a string literal in a concatenated expression. 161 * 162 * @param caseAST the AST node representing {@code LITERAL_CASE} 163 * @return true if switch block contains a string case label 164 */ 165 private static boolean hasStringCaseLabel(DetailAST caseAST) { 166 DetailAST curNode = caseAST; 167 boolean hasStringCaseLabel = false; 168 boolean exitCaseLabelExpression = false; 169 while (!exitCaseLabelExpression) { 170 DetailAST toVisit = curNode.getFirstChild(); 171 if (curNode.getType() == TokenTypes.STRING_LITERAL) { 172 hasStringCaseLabel = true; 173 break; 174 } 175 while (toVisit == null) { 176 toVisit = curNode.getNextSibling(); 177 curNode = curNode.getParent(); 178 } 179 curNode = toVisit; 180 exitCaseLabelExpression = TokenUtil.isOfType(curNode, TokenTypes.COLON, 181 TokenTypes.LAMBDA); 182 } 183 return hasStringCaseLabel; 184 } 185 186}