I am certain that string.Empty is more efficient than using "" in .Net languages. What I would like to know is WHY is it more efficient?
-
8what does make you think it's more efficient?Gregory Pakosz– Gregory Pakosz2009-12-11 11:31:16 +00:00Commented Dec 11, 2009 at 11:31
-
@Gregory - StyleCop recommends making the change from "" to string.Empty. I like to think there is some reasoning behind this...JL.– JL.2009-12-11 11:34:09 +00:00Commented Dec 11, 2009 at 11:34
-
3But what makes you think the reasoning has to do with efficiency?Stack Overflow is garbage– Stack Overflow is garbage2009-12-11 11:39:11 +00:00Commented Dec 11, 2009 at 11:39
-
6And what makes you think that StyleCop is always right, and always up-to-date in its recommendations?Jon Skeet– Jon Skeet2009-12-11 11:48:17 +00:00Commented Dec 11, 2009 at 11:48
-
It is not. See youtu.be/qWBi32-Njm8TimTIM Wong– TimTIM Wong2022-11-06 00:15:46 +00:00Commented Nov 6, 2022 at 0:15
5 Answers
I think in most cases there is no difference. In normal cases, when you use "" in your code this string will be interned and the same string instance will be reused. So there will not be more string instances around when using "" as compared to String.Empty.
If you want proof:
Dim a As String
Dim b As String
a = ""
b = ""
Console.WriteLine(Object.ReferenceEquals(a, b)) ' Prints True '
The above code prints True also if you replace one of them with String.Empty, so you can even mix the approaches without creating extra string instances.
Bottom line: the difference is personal taste.
3 Comments
First of all, I'm not sure that it is more efficient. A reason it is a static is, it is basically a constant: there is only one empty string.
A more important issue is that there are some inconsistencies with string interning in .NET, which can result in comparisons with String.empty not always returning the expected result, depending on what version of the runtime you're using. Eric Lippert has written a fascinating blog post about it here.
(Example taken from Eric's post)
object obj = "";
string str1 = "";
string str2 = String.Empty;
Console.WriteLine(obj == str1); // true
Console.WriteLine(str1 == str2); // true
Console.WriteLine(obj == str2); // sometimes true, sometimes false?!
1 Comment
I don't think there is any performance gain in using string.Empty over "".
Its just a matter of personal preference.
3 Comments
string.Empty is definitely more explicit than ""string.Empty is a singleton static const string that has been constructed, but "" will create a new string that is empty.
6 Comments
new string("") however, you'll use more memory...new string("") you'll get a compile-time error - there isn't a string constructor taking just a string.