1

注意：使用 .Net 4.0

考虑以下代码。

String ad = "FE23658978541236";
String ad2 = "00FABE002563447E".ToLower();
try
{
    PhysicalAddress.Parse(ad);
}
catch (Exception)
{
    //We dont get here, all went well
}
try
{
    PhysicalAddress.Parse(ad2);
}
catch (Exception)
{
    //we arrive here for what reason?
}
try
{
    //Ok, I do it myself then.
    ulong dad2 = ulong.Parse(ad2, System.Globalization.NumberStyles.HexNumber);
    byte[] bad2 = BitConverter.GetBytes(dad2);
    if (BitConverter.IsLittleEndian)
    {
        bad2 = bad2.Reverse().ToArray<byte>();
    }
    PhysicalAddress pa = new PhysicalAddress(bad2);
}
catch (Exception ex)
{
    //We don't get here as all went well
}

因此，在尝试解析小写地址时， Phy​​sicalAddress.Parse 方法会引发异常。当我查看 .Net 的源代码时，我完全清楚为什么。这是因为下面的一段代码。

    if (value >= 0x30 && value <=0x39){ 
        value -= 0x30;
    } 
    else if (value >= 0x41 && value <= 0x46) {
        value -= 0x37;
    }

这是在Parse方法中找到的。

    public static PhysicalAddress Parse(string address) {
    int validCount = 0; 
    bool hasDashes = false; 
    byte[] buffer = null;

    if(address == null)
    {
        return PhysicalAddress.None;
    } 

    //has dashes? 
    if (address.IndexOf('-') >= 0 ){ 
        hasDashes = true;
        buffer = new byte[(address.Length+1)/3];    
    }
    else{

        if(address.Length % 2 > 0){  //should be even 
            throw new FormatException(SR.GetString(SR.net_bad_mac_address));
        } 

        buffer = new byte[address.Length/2];
    } 

    int j = 0;
    for (int i = 0; i < address.Length; i++ ) {

        int value = (int)address[i];

        if (value >= 0x30 && value <=0x39){ 
            value -= 0x30;
        } 
        else if (value >= 0x41 && value <= 0x46) {
            value -= 0x37;
        }
        else if (value == (int)'-'){ 
            if (validCount == 2) {
                validCount = 0; 
                continue; 
            }
            else{ 
                throw new FormatException(SR.GetString(SR.net_bad_mac_address));
            }
        }
        else{ 
            throw new FormatException(SR.GetString(SR.net_bad_mac_address));
        } 

        //we had too many characters after the last dash
        if(hasDashes && validCount >= 2){ 
            throw new FormatException(SR.GetString(SR.net_bad_mac_address));
        }

        if (validCount%2 == 0) { 
            buffer[j] = (byte) (value << 4);
        } 
        else{ 
            buffer[j++] |= (byte) value;
        } 

        validCount++;
    }

    //we too few characters after the last dash
    if(validCount < 2){ 
        throw new FormatException(SR.GetString(SR.net_bad_mac_address)); 
    }

    return new PhysicalAddress(buffer);
}

这可以被认为是一个错误吗？或者在字符串中使用小写十六进制值是否非常错误？还是有一些我不知道的约定。就个人而言，我认为这个程序员不友好。

4

2 回答 2

6

来自MSDN：

address 参数必须包含一个字符串，该字符串只能由数字和大写字母组成的十六进制数字。一些可接受的字符串格式示例如下 .... 请注意，包含 f0-e1-d2-c3-b4-a5 的地址将无法解析并引发异常。

所以你可以简单地做：PhysicalAddress.Parse(ad.ToUpper());

于 2013-08-29T08:56:31.843 回答
4

不，它只是一个错误，如果它没有做文档声明它做的事情，或者它做了文档声明它没有做的事情。它的行为与您期望的不一样这一事实并不能使它成为一个错误。你当然可以认为这是一个糟糕的设计决定（或者，正如你所说的那样，对程序员不友好），但这不是一回事。

我倾向于同意你的观点，因为我喜欢遵循“对你的期望保持自由，在你交付的东西上保持一致”的理念，并且代码可能很容易通过以下方式修复：

if (value >= 0x30 && value <=0x39) { 
    value -= 0x30;
}
else if (value >= 0x41 && value <= 0x46) {
    value -= 0x37;
}
else if (value >= 0x61 && value <= 0x66) {  // added
    value -= 0x57;                          // added
}                                           // added
else if ...

但是，当然，您还必须更改 doco，并运行大量测试以确保您没有把事情搞砸。

关于 doco，可以在这里找到，重要的部分在下面重复（用我的粗体字）：

address 参数必须包含一个字符串，该字符串只能由数字和大写字母组成的十六进制数字。一些可接受的字符串格式示例如下：

001122334455
00-11-22-33-44-55
F0-E1-D2-C3-B4-A5

请注意，包含的地址f0-e1-d2-c3-b4-a5 将无法解析并引发异常。

于 2013-08-29T08:58:22.693 回答