在更改以下 VB.net 代码时,我看到一些奇怪的行为。这是原始源代码:
Private Function ValidateSelectedId(ByVal purposeId As String) As Boolean
Dim possiblePurposes As New InfoCollector.Purpose
Dim isPurposeValid As Boolean = False
'Any of the following purposes (but only these)
'should be considered valid
Select Case UCase(purposeId)
Case UCase(possiblePurposes.FirstPurpose), _
UCase(possiblePurposes.SecondPurpose), _
UCase(possiblePurposes.ThirdPurpose), _
UCase(possiblePurposes.FourthPurpose)
isPurposeValid = True
Case Else
isPurposeValid = False
End Select
Return isPurposeValid
End Function
这是新版本。唯一的变化是添加了第五个有效目的:
Private Function ValidateSelectedId(ByVal purposeId As String) As Boolean
Dim possiblePurposes As New InfoCollector.Purpose
Dim isPurposeValid As Boolean = False
Select Case UCase(purposeId)
Case UCase(possiblePurposes.FirstPurpose), _
UCase(possiblePurposes.SecondPurpose), _
UCase(possiblePurposes.ThirdPurpose), _
UCase(possiblePurposes.FourthPurpose), _
UCase(possiblePurposes.FifthPurpose)
isPurposeValid = True
Case Else
isPurposeValid = False
End Select
Return isPurposeValid
End Function
我在本地PC上编译了这个新版本并测试了功能,效果很好。然而,在将其签入我们的中央代码存储库并在服务器上构建它之后,使用失败了(它似乎忽略了新用途)。在试图找出缺少的内容时,我反编译了在服务器上找到的 .DLL,这就是它给我的内容(注意:我更改了变量名称并稍微重新格式化):
Private Function ValidateSelectedId(ByVal purposeId As String) As Boolean
Dim ValidateSelectedId As Boolean
Dim possiblePurposes As Purpose = New Purpose()
Dim isPurposeValid As Boolean = False
Dim str As String = Strings.UCase(purposeId)
If (Operators.CompareString(str, Strings.UCase(possiblePurposes.FirstPurpose), False) <> 0) Then
If (Operators.CompareString(str, Strings.UCase(possiblePurposes.SecondPurpose), False) <> 0
AndAlso (Operators.CompareString(str, Strings.UCase(possiblePurposes.ThirdPurpose), False) = 0
OrElse Operators.CompareString(str, Strings.UCase(possiblePurposes.FourthPurpose), False) <> 0)) Then
If (Operators.CompareString(str, Strings.UCase(possiblePurposes.FifthPurpose), False) = 0) Then
Return True
End If
isPurposeValid = False
End If
End If
Return isPurposeValid
End Function
我还尝试将其反编译为 C#,这对于我们中的一些人来说可能更容易阅读:
private bool ValidateSelectedId(string purposeId)
{
bool ValidateSelectedId;
Purpose possiblePurposes = new Purpose();
bool isPurposeValid = false;
string str = Strings.UCase(purposeId);
if (Operators.CompareString(str, Strings.UCase(possiblePurposes.FirstPurpose), false) != 0)
{
if (Operators.CompareString(str, Strings.UCase(possiblePurposes.SecondPurpose), false) != 0
&& (Operators.CompareString(str, Strings.UCase(possiblePurposes.ThirdPurpose), false) == 0
|| Operators.CompareString(str, Strings.UCase(possiblePurposes.FourthPurpose), false) != 0))
{
if (Operators.CompareString(str, Strings.UCase(possiblePurposes.FifthPurpose), false) == 0)
{
return true;
}
isPurposeValid = false;
}
}
return isPurposeValid;
}
然而,这似乎做了与我想要的相反的事情,根本不是原始源代码所说的!
我看不出原始源代码是如何编译成这样的。我的反编译器是否有问题(我使用的是 Telerik 的 Just Decompile)?如果是这样,为什么逻辑在我的本地 PC 上有效,但在服务器上却不起作用?
编译过程中是否真的存在某种错误?或者我是一个彻底的Id**t,只是误解了反编译代码中的逻辑?
任何解释这一点的相关理论将不胜感激。
我真傻。
我最终发现了这在服务器上不起作用的原因:还有另一个相关的错误已在本地修复,但在服务器上没有修复。
愚蠢的反编译器。
我对反编译器的结果感到困惑,这显然是一团糟。我以前曾多次使用JustDecompile,但从未遇到过这样的问题。显然,它无法以任何合理的方式反编译上述方法的代码。
我的假设是在编译过程中进行了某种形式的优化,JustDecompile 难以理解和正确显示。
我只是想验证我的更改是否已发布到服务器。相反,我被派去徒劳地寻找一只幽灵虫子。 经验教训:需要时使用反编译器,但不要自动相信它告诉你的一切。