Robin Rosenberg wrote:
>> + super(String.format("No repository found for file '%s'",
>> + fileWeSearchedRepositoryFor));
>
> This is any odd patterrn. Usually we use + to concatenate strimgs.
Hi Robin,
I know two good reason why you should use formating functions over
string concatenation via the plus sign:
1.) formated strings are better translatable. e.g.
"The string '%s' is more then %s characters long"
is easier to translate then the three strings:
"The string '"
"' is more then "
" characters long"
2.) Whitespace mistakes are easier to see. e.g.
(i + "coins") and ("%scoins", i)
Sure you can make the mistake to provide not enough arguments, but
that's not so worse:
a.) Some IDEs, like Intellij IDEA warn you if you call a formating
function with to few arguments.
b.) A exception will be thrown at the time the function is called, if
there are not enough arguments. Thus you will notice your mistake very soon.
Best regards,
Florian
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
