对Object.ToString()的冗余调用

时间:2008-09-16 16:52:37

标签: c# .net resharper

我有一个函数,其中包含一个声明为 int privateCount 的参数。当我想在这个参数上调用ToString()时,ReSharper将其灰显并将其标记为冗余调用。所以,好像我一样,我删除ToString(),代码仍然构建!

C#编译器如何允许这样做,其中 str 是一个字符串?

str += privateCount + ...

4 个答案:

答案 0 :(得分:19)

字符串的+运算符被重载以调用在表达式的左侧和右侧传递的String.Concat。因此:

string x = "123" + 45;

获取编译为:

String.Concat("123", 45);

由于String.Concat接收两个对象,因此右侧(45)被装箱,然后在其上调用ToString()。

请注意,这种“重载”不是通过语言中的运算符重载(也就是说它不是名为op_Addition的方法),而是由编译器处理。

答案 1 :(得分:8)

这不仅是不好的做法,而且性能也较差:整数必须装箱,因为String.Concat在int.ToString()不需要装箱的情况下会取消一个对象。

答案 2 :(得分:2)

C#会自动将对象转换为字符串。考虑这行代码:

aString = aString + 23;

这是有效的C#;它编译为String.Concat()的调用,它接受两个对象作为参数。这些对象会自动转换为字符串对象。

写作时会发生同样的事情:

aString += 23;

同样,这将编译为对String.Concat()的相同调用;它的写法不同。

答案 3 :(得分:1)

这是一种有效的不良做法,因为如果变量是字符串或int,您必须三思而后行。如果变量被称为myInt会怎么样?

myInt = myInt + 23;

这种形式更具可读性和可理解性吗?

myInt = mInt + "23";

甚至:

myInt = string.Format("{0}{1}", myInt, 23);

我们从代码中知道它是一个字符串而不是整数。