您在生产企业环境中见过的最恶劣的代码是什么？

Question

您在生产企业环境中见过的最恶劣的代码是什么？

您在公司的生产环境中见过的最邪恶或最危险的代码片段是什么？我从来没有遇到过我认为故意恶意和邪恶的生产代码,所以我很好奇看到其他人发现了什么.

我见过的最危险的代码是存储过程,两个链接服务器远离我们的核心生产数据库服务器.存储过程接受任何NVARCHAR(8000)参数,并通过双跳sp_executeSQL命令在目标生产服务器上执行该参数.也就是说,sp_executeSQL命令执行另一个sp_executeSQL命令以跳转两个链接的服务器.哦,链接的服务器帐户在目标生产服务器上具有sysadmin权限.

Answer 1

Jul*_*iet 331

警告:前面很长的可怕的帖子

我已经写过一篇关于我在这里和这里之前做过的一个应用程序.简而言之,我的公司从印度继承了130,000行垃圾.该应用程序是用C#编写的; 它是一个柜员应用程序,当你去银行时,同样的软件柜员在柜台后面使用.该应用程序每天崩溃40-50次,它根本无法重构为工作代码.我的公司必须在12个月内重写整个应用程序.

为什么这个应用程序邪恶？因为看到源代码足以让一个理智的男人疯了,一个疯子变得健全.用于编写此应用程序的扭曲逻辑只能受到Lovecraftian噩梦的启发.该应用程序的独特功能包括:

在130,000行代码中,整个应用程序包含5个类(不包括表单文件).所有这些都是公共静态类.一个类叫做Globals.cs,它包含1000s,1000s和1000s的公共静态变量,用于保存应用程序的整个状态.这五个类共包含20,000行代码,其余代码嵌入在表单中.
你不得不怀疑,程序员是如何在没有任何类的情况下编写这么大的应用程序的？他们用什么来表示他们的数据对象？事实证明,程序员通过组合ArrayLists,HashTables和DataTables,设法重新发明了我们所学到的关于OOP的一半概念.我们看到了很多这样的事情:
- ArrayList的哈希表
- 具有字符串键和DataRow值的哈希表
- DataTables的ArrayLists
- DataRows包含包含HashTables的ArrayLists
- DataRows的ArrayLists
- ArrayLists的ArrayLists
- 带有字符串键和HashTable值的HashTables
- HashTables的ArrayLists的ArrayLists
- 您可以想到的ArrayLists,HashTables,DataTables的其他组合.
请记住,上面没有任何数据结构是强类型的,因此您必须将从列表中获得的任何神秘对象转换为正确的类型.令人惊讶的是,您可以使用ArrayLists,HashTables和DataTables创建类似Rube Goldberg的复杂数据结构.
要分享如何使用上面详述的对象模型的示例,请考虑Accounts:原始程序员为帐户的每个可配置属性创建单独的HashTable:名为hstAcctExists的hashTable,hstAcctNeedsOverride,hstAcctFirstName.所有这些哈希表的键是一个"|"分隔的字符串.可以想到的键包括"123456 | DDA","24100 | SVG","100 | LNS"等.
由于可以从全局变量中轻松访问整个应用程序的状态,因此程序员发现不必将参数传递给方法.我会说90%的方法都采用了0个参数.在少数几个中,为了方便起见,所有参数都作为字符串传递,无论字符串表示什么.
没有副作用的功能.每个方法都修改了Globals类中的一个或多个变量.并非所有副作用都有意义; 例如,其中一种表单验证方法对于存储Globals.lngAcctNum的任何帐户的贷款计算超过和短期付款都会产生神秘的副作用.
虽然有很多表单,但有一种形式可以统治它们:frmMain.cs,其中包含高达20,000行的代码.frmMain做了什么？一切.它抬头看账号,打印收据,分发现金,它做了一切.

有时需要在frmMain上调用方法的其他形式.而不是将表单中的代码分解为单独的类,为什么不直接调用代码:
```
((frmMain)this.MDIParent).UpdateStatusBar(hstValues);
```
Run Code Online (Sandbox Code Playgroud)
为了查找帐户,程序员做了类似这样的事情:
```
bool blnAccountExists =
    new frmAccounts().GetAccountInfo().blnAccountExists
```
Run Code Online (Sandbox Code Playgroud)
虽然它已经创建了一个不可见的表单以执行业务逻辑,但您认为该表单如何知道要查找哪个帐户？这很简单:表单可以访问Globals.lngAcctNum和Globals.strAcctType.(谁不爱匈牙利乐谱？)
代码重用是ctrl-c,ctrl-v的同义词.我发现在20种表格中复制/粘贴了200行方法.
该应用程序有一个奇怪的线程模型,我喜欢称之为线程和计时器模型:产生线程的每个表单都有一个计时器.产生的每个线程都启动了一个200毫秒延迟的定时器; 一旦计时器启动,它将检查线程是否设置了一些魔术布尔值,然后它将中止线程.吞下了生成的ThreadAbortException.

你认为你只会看到一次这种模式,但我发现它至少在10个不同的地方.
说到线程,关键字"lock"从未出现在应用程序中.线程可以自由地操纵全局状态而不需要锁定.
应用程序中的每个方法都包含一个try/catch块.记录并吞下每个异常.
在打开字符串时,谁需要打开枚举就一样容易!

一些天才发现你可以将多个表单控件挂钩到同一个事件处理程序.程序员是如何处理这个的？

private void OperationButton_Click(object sender, EventArgs e)
{
    Button btn = (Button)sender;
    if (blnModeIsAddMc)
    {
        AddMcOperationKeyPress(btn);
    }
    else
    {
        string strToBeAppendedLater = string.Empty;
        if (btn.Name != "btnBS")
        {
            UpdateText();
        }
        if (txtEdit.Text.Trim() != "Error")
        {
            SaveFormState();
        }
        switch (btn.Name)
        {
            case "btnC":
                ResetValues();
                break;
            case "btnCE":
                txtEdit.Text = "0";
                break;
            case "btnBS":
                if (!blnStartedNew)
                {
                    string EditText = txtEdit.Text.Substring(0, txtEdit.Text.Length - 1);
                    DisplayValue((EditText == string.Empty) ? "0" : EditText);
                }
                break;
            case "btnPercent":
                blnAfterOp = true;
                if (GetValueDecimal(txtEdit.Text, out decCurrValue))
                {
                    AddToTape(GetValueString(decCurrValue), (string)btn.Text, true, false);
                    decCurrValue = decResultValue * decCurrValue / intFormatFactor;
                    DisplayValue(GetValueString(decCurrValue));
                    AddToTape(GetValueString(decCurrValue), string.Empty, true, false);
                    strToBeAppendedLater = GetValueString(decResultValue).PadLeft(20)
                                                + strOpPressed.PadRight(3);
                    if (arrLstTapeHist.Count == 0)
                    {
                        arrLstTapeHist.Add(strToBeAppendedLater);
                    }
                    blnEqualOccurred = false;
                    blnStartedNew = true;
                }
                break;
            case "btnAdd":
            case "btnSubtract":
            case "btnMultiply":
            case "btnDivide":
                blnAfterOp = true;
                if (txtEdit.Text.Trim() == "Error")
                {
                    btnC.PerformClick();
                    return;
                }
                if (blnNumPressed || blnEqualOccurred)
                {
                    if (GetValueDecimal(txtEdit.Text, out decCurrValue))
                    {
                        if (Operation())
                        {
                            AddToTape(GetValueString(decCurrValue), (string)btn.Text, true, true);
                            DisplayValue(GetValueString(decResultValue));
                        }
                        else
                        {
                            AddToTape(GetValueString(decCurrValue), (string)btn.Text, true, true);
                            DisplayValue("Error");
                        }
                        strOpPressed = btn.Text;
                        blnEqualOccurred = false;
                        blnNumPressed = false;
                    }
                }
                else
                {
                    strOpPressed = btn.Text;
                    AddToTape(GetValueString(0), (string)btn.Text, false, false);
                }
                if (txtEdit.Text.Trim() == "Error")
                {
                    AddToTape("Error", string.Empty, true, true);
                    btnC.PerformClick();
                    txtEdit.Text = "Error";
                }
                break;
            case "btnEqual":
                blnAfterOp = false;
                if (strOpPressed != string.Empty || strPrevOp != string.Empty)
                {
                    if (GetValueDecimal(txtEdit.Text, out decCurrValue))
                    {
                        if (OperationEqual())
                        {
                            DisplayValue(GetValueString(decResultValue));
                        }
                        else
                        {
                            DisplayValue("Error");
                        }
                        if (!blnEqualOccurred)
                        {
                            strPrevOp = strOpPressed;
                            decHistValue = decCurrValue;
                            blnNumPressed = false;
                            blnEqualOccurred = true;
                        }
                        strOpPressed = string.Empty;
                    }
                }
                break;
            case "btnSign":
                GetValueDecimal(txtEdit.Text, out decCurrValue);
                DisplayValue(GetValueString(-1 * decCurrValue));
                break;
        }
    }
}

归档时间：	17 年前
查看次数：	53177 次
最近记录：	10 年，3 月前