diff --git a/python-checks/src/main/resources/org/sonar/l10n/py/rules/python/S5300.html b/python-checks/src/main/resources/org/sonar/l10n/py/rules/python/S5300.html index 963221322a..e2907ee82d 100644 --- a/python-checks/src/main/resources/org/sonar/l10n/py/rules/python/S5300.html +++ b/python-checks/src/main/resources/org/sonar/l10n/py/rules/python/S5300.html @@ -18,7 +18,7 @@
You are at risk if you answered yes to any of those questions.
The complexity of a regular expression is determined as follows:
Each of the following operators increases the complexity by an amount equal to the current nesting level and also increases the current nesting level by one for its arguments:
-|
- when multiple |
operators are used together, the subsequent ones only increase the complexity by 1 &&
(inside character classes) - when multiple &&
operators are used together, the subsequent ones
- only increase the complexity by 1 *
, +
, ?
, {n,m}
, {n,}
or {n}
) (?i:some_pattern)
or (?i)some_pattern
) Additionally, each use of the following features increase the complexity by 1 regardless of nesting:
-If a regular expression is split among multiple variables, the complexity is calculated for each variable individually, not for the whole regular -expression. If a regular expression is split over multiple lines, each line is treated individually if it is accompanied by a comment (either a Java -comment or a comment within the regular expression), otherwise the regular expression is analyzed as a whole.
--if (dateString.matches("^(?:(?:31(\\/|-|\\.)(?:0?[13578]|1[02]))\\1|(?:(?:29|30)(\\/|-|\\.)(?:0?[13-9]|1[0-2])\\2))(?:(?:1[6-9]|[2-9]\\d)?\\d{2})$|^(?:29(\\/|-|\\.)0?2\\3(?:(?:(?:1[6-9]|[2-9]\\d)?(?:0[48]|[2468][048]|[13579][26])|(?:(?:16|[2468][048]|[3579][26])00))))$|^(?:0?[1-9]|1\\d|2[0-8])(\\/|-|\\.)(?:(?:0?[1-9])|(?:1[0-2]))\\4(?:(?:1[6-9]|[2-9]\\d)?\\d{2})$")) { - handleDate(dateString); -} --
-if (dateString.matches("^\\d{1,2}([-/.])\\d{1,2}\\1\\d{1,4}$")) { - String dateParts[] = dateString.split("[-/.]"); - int day = Integer.parseInt(dateParts[0]); - int month = Integer.parseInt(dateParts[1]); - int year = Integer.parseInt(dateParts[2]); - // Put logic to validate and process the date based on its integer parts here -} --
Regular expressions are only analyzed if all parts of the regular expression are either string literals, effectively final local variables or
-static final
fields, all of which can be combined using the '+
' operator.
When a regular expression is split among multiple variables or commented lines, each part is only analyzed if it is syntactically valid by -itself.
|
- when multiple |
operators are used together, the subsequent ones only increase the complexity by 1 *
, +
, ?
, {n,m}
, {n,}
or {n}
)