创建一次使用的变量是好的做法吗?

我同意你对这个案子的看法。可读性是关键。我确信编译器在这两种情况下都会生成相同的可执行文件,编译器和今天一样智能。

但我也不会声称“总是使用一次使用的变量”。例子:

String name = "John";
person.setName(name);

没有必要,因为

person.setName("John");

读起来同样好——如果不是更好的话。但是,当然,并非所有案件都如此明确。毕竟,“可读性”是一个主观术语。

你所有的理由在我看来都是合理的。

为了避免使用中间变量,在需要单个表达式的情况下(例如Java/C#中的成员变量初始化),但在适用的情况下引入额外的变量以提高清晰度是绝对可以的。显然,不要这样做 论证到 每 方法,但适度会有很大帮助。

你的同事似乎不一致。

private void btnGeneral_Click(object sender, RoutedEventArgs e)
{
    UserControl userControl = ((UserControl)type.Assembly).CreateInstance(String.Format("{0}.{1}", this.GetType().Namespace, ((Button)e.OriginalSource).Name));
}

在这件事上,我完全同意你的观点。

public void OpenDocument(string filename, bool asReadonly, bool copyLocal, bool somethingElse)

bool asReadonly = true;
bool copyLocal = false;
bool somethingElse = true;

OpenDocument("somefile.txt", asReadonly, copyLocal, somethingElse);

OpenDocument("somefile.txt", true, false, true);

什么

虽然我同意你的想法,但添加额外的变量可以在方法中引入额外的概念,而这个概念可能并不总是与方法的总体目标相关。因此,过度添加变量也可能 增加 方法复杂性和易读性降低。注意以下用法 过度 在这里。

我想在某些情况下,它可能会对性能产生影响。特别是在这个例子中:

for (int i1 = 0; i1 < BIG_NR; i1++)
{
  for (int i2 = 0; i2 < BIG_NR; i2++)
  {
    for (int i3 = 0; i3 < BIG_NR; i3++)
    {
      for (int i4 = 0; i4 < BIG_NR; i4++)
      {
        int amount = a + b;
        someVar[i1][i2][i3][i4] = amount;
      }
    }
  }
}

我认为这是一个基于你希望你的代码有多整洁的判断。我也认为你和你的同事都是正确的。

在这种情况下,我会根据您提供的代码(出于性能原因)支持您的同事,但正如我之前所说,这确实取决于使用它的环境,我认为您的立场是完全可以接受的。

我想指出的是,为曾经使用过的参数创建变量可能毫无意义,除非它们是常量变量或需要在许多地方使用的东西。

创建一个新变量意味着读者还需要了解一个概念。(考虑极端情况:int b=a;c=b;)如果一个方法非常复杂——需要分解——那么额外的概念是值得付出的代价,那么你应该全力以赴,把它分成两个方法。通过这种方式,您可以获得有意义的名称和较小的尺寸。(对于你的原始方法来说,它更小:如果就像你说的那样,那么人们通常不需要阅读辅助方法。)

这是一种概括,特别是在一种有很多添加新方法的样板的语言中,但你不会经常不同意这种概括,以至于值得在你的风格指南中省略。

原则上我完全赞同你的同事,但在这种情况下我不同意。

如果变量被使用了多次。因此,您将立即让眼睛向下扫描代码,看看它是否在更多地方使用。函数越长,你就越需要查看是否能找到它。

我过去就是这么编码的。如今,我试图尽量减少中间变量。如果中间变量是不可变的,那么使用它是完全可以的。

List<string> someStrings = new List<string>();
for (int i = 0; i < 1000; i++)
{
    string localString = string.Format("prefix{0}", i);
    someStrings.Add(localString);
}

而不是:

List<string> someStrings = new List<string>();
string localString = string.Empty;
for (int i = 0; i < 1000; i++)
{
    localString = string.Format("prefix{0}", i);
    someStrings.Add(localString);
}

因此,在许多情况下,真的没有性能上的理由不使用它。

同意

“使代码更容易阅读,也就是说,你的代码中有更多“读起来像英语”

卡尔