对Object.ToString()的冗余调用

问题描述 投票:10回答:4

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

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

str += privateCount + ...

c# .net resharper
4个回答
19
投票

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

string x = "123" + 45;

获取编译为:

String.Concat("123", 45);

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

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


8
投票

这不仅是不好的做法,而且性能也较差:整数必须被装箱,因为String.Concat预测了一个对象而int.ToString()不需要装箱。


2
投票

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

aString = aString + 23;

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

你写的时候会发生同样的事情:

aString += 23;

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


1
投票

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

myInt = myInt + 23;

它在这种形式下更具可读性和可理解性?

myInt = mInt + "23";

甚至:

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

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

© www.soinside.com 2019 - 2024. All rights reserved.