Format strings should use
%n instead of \\n JAVA-W0379
Anti-pattern
Minor
2 years ago
—
2 years old
\n should be replaced with %n
20
21 @Override
22 public String format_code_block(String source) {
23 return String.format("<code>\n%s\n</code>", source);24 }
25}
\n should be replaced with %n
20
21 @Override
22 public String format_code_block(String source) {
23 return String.format("```\n%s\n```", source);24 }
25}
\n should be replaced with %n
13 StackTraceElement[] stackTraceElements = Thread.currentThread().getStackTrace();
14 String new_message = "";
15 for (String line : message.split("\n")) {
16 new_message += String.format("[%s::%s at %s:%s] %s\n", stackTraceElements[2].getClassName(), stackTraceElements[2].getMethodName(), stackTraceElements[2].getFileName(), stackTraceElements[2].getLineNumber(), line);17 }
18
19 new_message = new_message.strip();\n should be replaced with %n
43 while (true) {
44 untis_message.editMessage("Fetching data...").queue();
45 try {
46 untis_message.editMessage(toString() + String.format("\nLast update: %s", new Date())).queue();47 } catch (RuntimeException e) {
48 untis_message.editMessage("Failed to fetch untis info: " + e.getMessage()).queue();
49 }\n should be replaced with %n
47 rate_limit.put(event.get_sender_id(), System.currentTimeMillis());
48 }
49
50 String ret = String.format("Uploaded %s files to:\n", event.get_num_files());51
52 Json _json = Json.json();
53
Description
This format string includes a newline character (\\n). This may cause issues on platforms like Windows that do not use Unix line separators.
In format strings, it is generally preferable to use %n, which will produce the platform-specific line separator.
Bad practice
String.format("%s\\n%d", "number", 3);
Recommended
String.format("%s%n%d", "number", 3);
References
- SpotBugs - VA_FORMAT_STRING_USES_NEWLINE