Rule: SA1122 UseStringEmptyForEmptyStrings about atc-coding-rules HOT 3 CLOSED

I almost assume the compiler is smart enough to simply replace all "" with string.Empty at compile time.
A quick test in sharplab seems to indicate different LI is generated by the compiler, but the JIT produced assembler code is indeed the same. So it might be that there is a very so slight overhead during runtime for the JIT to do it's work.

That said, I do agree that this is not a readability concern, and more often than not I actually write "" in tests because it makes it easier to understand what is going on.

from atc-coding-rules.

Pr. meeting 05/02-2020, we agreed upon that we want this rule enabled for src/props and set to none for test/props.

Arguments:

• It explicitly clarifies our intent
• Its easier to spot string.Empty than ""
• Its easier to search for string.Empty than a literal
• Eliminates zero width space issues LINK
• Minor performance gain

from atc-coding-rules.

https://github.com/atc-net/atc-coding-rules/blob/main/documentation/CodeAnalyzersRules/StyleCop/SA1122.md

from atc-coding-rules.