C＃长开关语句

Question

我看过StackOverflow上有关c ++ long switch语句的文章，但是c＃在这方面有所不同。

具体来说，我想用c＃代码替换一个很长的switch语句，其中每个case语句执行相同的操作，只是字段名称发生变化。

代码如下：

 case Fields.TRANSMITTERCONTACTPHONEEXT:
                {
                    int lengthNeeded = 0;
                    int currentLength = TransmitterContactTelephoneExtension.Length;
                    int lengthRequired = TransmitterContactTelephoneExtensionLength;
                    if (currentLength < lengthRequired)
                    {
                        lengthNeeded = lengthRequired - currentLength;
                        for (int i = 0; i < lengthNeeded; i++)
                        {
                            TransmitterContactTelephoneExtension += " ";
                        }
                    }
                } break;
            case Fields.TRANSMITTERFEIN:
                {
                    int lengthNeeded = 0;
                    int currentLength = TransmitterFEIN.Length;
                    int lengthRequired = TransmitterFEINLength;
                    if (currentLength < lengthRequired)
                    {
                        lengthNeeded = lengthRequired - currentLength;
                        for (int i = 0; i < lengthNeeded; i++)
                        {
                            TransmitterFEIN += " ";
                        }
                    }
                } break;

我只想将其归结为一个函数，无需使用switch语句就可以弄清楚我的意思是哪个字段。 有没有办法传入包含字段名称的变量？

Answer 1

看起来就像一个字符串填充函数的代码重复了多次。 你可能只有

case Fields.TRANSMITTERCONTACTPHONEEXT:
     TransmitterContactTelephoneExtension = TransmitterContactTelephoneExtension.PadRight(TransmitterContactTelephoneExtensionLength, ' ');
     break;
    ...

每当您发现自己一遍又一遍地重复代码时，就可以将其分解为一个单独的函数（如果尚不存在），并使用正确的参数进行调用。

这也让我想知道您是否根本需要switch case语句，而不仅仅是一系列pad语句。 但这在您的代码中（在范围上）更进一步。 而且您的帖子没有给我们足够的信息。

最后，在某种程度上适用于您的问题，我的经验法则（最初不是我的，但我忘了我从哪里得到的）是，如果某个方法不仅仅是一个分页的（故意模糊的术语），那么我需要将其分解为其他方法单独的方法。 这使我无需滚动即可查看并理解方法。 这也迫使我将较长的过程分成较小的逻辑步骤。

Answer 2

将所有可变值放入数组中并对其进行索引。 确保枚举int值与目标数组中的数据相同。

var current = (int) Fields.TRANSMITTERCONTACTPHONEEXT;

int lengthNeeded = 0;
int currentLength = LengthData[ current ] ;
int lengthRequired = RequiredData[current ];

if (currentLength < lengthRequired)
{
      lengthNeeded = lengthRequired - currentLength;
      for (int i = 0; i < lengthNeeded; i++)
      {
         Extensions[ current ] = Extensions[ current ] + " ";
      }
}

---

这篇文章探讨了交换机中的操作模式 ，并没有解决单个操作的任何局部缺陷。 请查看每个单独的操作，并在需要的地方对其进行改进以实现最佳的操作效率。

Answer 3

预先声明变量，并仅将switch用于不同的分配：

int currentLength;
int lengthRequired;

switch (whatever) {
case Fields.TRANSMITTERCONTACTPHONEEXT:
    currentLength = TransmitterContactTelephoneExtension.Length;
    lengthRequired = TransmitterContactTelephoneExtensionLength;
    break;
case Fields.TRANSMITTERFEIN:
    currentLength = TransmitterFEIN.Length;
    lengthRequired = TransmitterFEINLength;
    break;
default:
    throw new Exception(); // Without this, the compiler will complain about uninitialized variables
}

int lengthNeeded = 0;
if (currentLength < lengthRequired)
{
    lengthNeeded = lengthRequired - currentLength;
    for (int i = 0; i < lengthNeeded; i++)
    {
        TransmitterFEIN += " ";
    }
}

switch (whatever) {
case Fields.TRANSMITTERCONTACTPHONEEXT:
    TransmitterContactTelephoneExtension += " ";
    break;
case Fields.TRANSMITTERFEIN:
    TransmitterFEIN += " ";
    break;
}

编辑：如果您可以选择用数组替换变量，则OmegaMan的解决方案会更好。

Answer 4

您确实应该将此分解为一种方法。 这样您就不会一遍又一遍地复制代码。 这样效率低下，并且可能在更新时导致潜在的错误。

同样，除了循环之外，还应该在字符串上使用PadRight（）方法。

我会这样做：

case Fields.TRANSMITTERCONTACTPHONEEXT:
     TransmitterContactTelephoneExtension = PadString(TransmitterContactTelephoneExtensionLength, TransmitterContactTelephoneExtension);
     break;
case Fields.TRANSMITTERFEIN:
     TransmitterFEIN = PadString(TransmitterFEINLength, TransmitterFEIN);
     break;


private string PadString(int requiredLen, string value)
{
     if (value == null) return String.Empty.PadRight(requiredLen, ' '); //Create an empty string when the value is null
     return value.PadRight(requiredLen, ' ');
}

Answer 5

在我看来，似乎看不到的代码需要一些重构，但是根据我们看到的内容，我建议执行以下操作：

// 1. Have a class to hold your data
class FieldData
{
    public string Value { get; set; }
    public int LengthRequired { get; set; }

    public string RightPaddedValue
    {
        get { return Value.PadRight(LengthRequired, ' '); }
    }
}

// 2. Fill your data into a dictionary somehow... for example:
Dictionary<Fields, FieldData> fields = new Dictionary<Fields, FieldData>
{
    { 
        Fields.TRANSMITTERCONTACTPHONEEXT, 
        new FieldData {
            Value = TransmitterContactTelephoneExtension,
            LengthRequired = TransmitterContactTelephoneExtensionLength
        }
    },
    { 
        Fields.TRANSMITTERFEIN,
        new FieldData {
            Value = TransmitterFEIN,
            LengthRequired = TransmitterFEINLength
        }
    }
};

// 3. Then use the data from that dictionary in your code:
FieldData data = fields[selectedField];
data.RightPaddedValue; // use RightPaddedValue