Why is this an issue?
A printf--style format string is a string that contains placeholders, usually represented by special characters such as "%s" or "{}",
depending on the technology in use. These placeholders are replaced by values when the string is printed or logged.
Because printf-style format strings are interpreted at runtime, rather than validated by the compiler, they can contain errors that
result in the wrong strings being created.
This rule checks whether every format string specifier can be correctly matched with one of the additional arguments when calling the following
methods:
How to fix it
A printf--style format string is a string that contains placeholders, which are replaced by values when the string is printed or
logged. Mismatch in the format specifiers and the arguments provided can lead to incorrect strings being created.
To avoid issues, a developer should ensure that the provided arguments match format specifiers.
Code examples
Noncompliant code example
String.format("Too many arguments %d and %d", 1, 2, 3); // Noncompliant; the third argument '3' is unused
String.format("First {0} and then {1}", "foo", "bar"); //Noncompliant. It appears there is confusion with the use of "java.text.MessageFormat"; parameters "foo" and "bar" will be ignored here
org.slf4j.Logger slf4jLog;
slf4jLog.debug("The number: ", 1); // Noncompliant - String contains no format specifiers.
Compliant solution
String.format("Too many arguments %d and %d", 1, 2);
String.format("First %s and then %s", "foo", "bar");
org.slf4j.Logger slf4jLog;
slf4jLog.debug("The number: {}", 1);
Resources
