exercism / exercism/java-analyzer

Analyzer feedback for reformat method in LogLevels flags .formatted() specifically as String.format

Offen
#355 3 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @Thakurji890 Auf GitHub ansehen
Vorherrschende Sprache
Java
Sterne
13
Forks
27
Ø Merge
4 T. 9 Std.
Gemergte PRs (30 T.)
6

Beschreibung

In the LogLevels exercise (Java track), I used the Java 15+ instance method String.formatted() to concatenate the final log message in the reformat method:

`return "%s (%s)".formatted(textMsg, textLevel);`

The automated Java Analyzer flagged this and suggested using the + operator for performance reasons. However, the feedback message explicitly references the older static method String.format instead of the one I actually used (.formatted()).

While I understand the performance suggestion to use standard string concatenation (+) for simple cases, the analyzer's feedback text should ideally recognize the .formatted() method or be reworded to encompass both formatting approaches to avoid confusion.

Just a minor tweak for the analyzer's message accuracy! Thanks for maintaining this track.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.