0

关于使用两个变量的最佳方法，我有一个非常简短的问题。基本上我有一个 enum 和一个 int，我想在几个 if 中得到它的值。我应该在 if 之外还是在内部声明它们 - 考虑以下示例：

例如：

public void test() {
    EnumName? value = null;
    int distance = 0;

     if(anotherValue == something) {
        distance = 10;
        value = getValue(distance);
     }
     else if(anotherValue == somethingElse) {
        distance = 20;
        value = getValue(distance);
     }


    if (value == theValueWeWant){
      //Do something
 }

或者

例2

public void test() {

     if(anotherValue == something) {
        int distance = 10;
        EnumType value = getValue(distance);
        if (value == theValueWeWant){
           //Do something
     }
     else if(anotherValue == somethingElse) {
        int distance = 20;
        EnumType value = getValue(distance);
        if (value == theValueWeWant){
            //Do something
     }

 }

我只是好奇哪个最好？或者是否有更好的方法？

4

4 回答 4

3

纯粹在维护方面，第一个代码块更好，因为它不会重复代码（假设“做某事”在两种情况下都是相同的）。

在性能方面，差异应该可以忽略不计。第二种情况确实会在编译的 IL 中生成两倍的本地变量，但 JIT 应该注意到它们的使用不会重叠并优化它们。第二种情况也会导致两次发射相同的代码（if (value == theValueWeWant) { ...），但这也不应该导致任何显着的性能损失。

（虽然第二个例子的两个方面都会导致编译的程序集稍微大一点，但更多的 IL 并不总是意味着更差的性能。）

于 2013-07-24T16:18:12.717 回答
1

这两个示例都做了两个不同的事情：

如果您获得所需的值，版本 1 将运行相同的代码，而版本 2 可能会运行不同的代码，即使您获得所需的值。

你可以做很多可能的（微）优化。

例如，如果距离仅在 getValue(distance) 中使用，您可以完全摆脱它：

 /*Waring, micro-optimization!*/
 public void test() {

      EnumType value = getValue((anotherValue == something) ? 10 : (anotherValue == somethingElse) ? 20 : 0);
      if (value == theValueWeWant){
  //Do something
      }
 }
于 2013-07-24T16:26:53.680 回答
0

如果您希望稍后使用它们，则 g 用于第二种方法。这些变量一旦超出范围就会丢失。

即使您以后不想使用它们，if也应该在 's 之前声明它们，以避免代码重复。

于 2013-07-24T16:17:17.090 回答
0

这个问题纯粹是风格问题，因此没有正确答案，只有意见

C# 最佳实践通常是在使用变量的范围内声明变量。这将指向第二个示例作为答案。尽管类型和名称相同，但它们代表不同的用途，应限制在创建它们的块中。

于 2013-07-24T16:17:29.020 回答