Список говнокодов пользователя invi

Всего: 7

C# / Говнокод #9445

+120

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20

//запись в журнал расчетов
try
{
    ompDataSet.OMP_History.AddOMP_HistoryRow(DateTime.Now, row["N"].ToString(), Convert.ToInt32(row["M"]), PS1I??0, PS1U??0, PS2I??0, PS2U??0, I1 / 1000, U1 / 1000, I2 / 1000, U2 / 1000, KZ, (Zone1.EditValue != null) ? Convert.ToDouble(Zone1.EditValue) : 0, (Zone2.EditValue != null) ? Convert.ToDouble(Zone2.EditValue) : 0, "", PS1_DeviceA.SelectedIndex, PS2_DeviceA.SelectedIndex, (Branch.Count > 0) ? true : false, "", "", "Время откл.: " + OffDT.DateTime.ToString("dd.MM.yyyy HH:mm") + "\n" + Lkz.ToolTip.Replace("\n", ";"));
    omp_HistoryTableAdapter.Update(ompDataSet);
    ompDataSet.OMP_History.AcceptChanges();
}
catch
{
    XtraMessageBox.Show("Не удалось сохранить данные в журнал расчётов!", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Warning);
}

try
{
    ompDataSet.Result.Clear();
    ompDataSet.Result.AddResultRow(((KeyValuePair<string, int>)NumberVL.EditValue).Key, ((KeyValuePair<string, int>)NumberVL.EditValue).Value, PS1I ?? 0, PS1U ?? 0, PS1I ?? 0, PS2U ?? 0, Convert.ToDouble(I0_A.EditValue), Convert.ToDouble(U0_A.EditValue), Convert.ToDouble(I0_B.EditValue), Convert.ToDouble(U0_B.EditValue), Convert.ToDouble(I0_A_Eval.EditValue), Convert.ToDouble(U0_A_Eval.EditValue), Convert.ToDouble(I0_B_Eval.EditValue), Convert.ToDouble(U0_B_Eval.EditValue), Convert.ToDouble(Lkz.EditValue), Convert.ToDouble(Zone1.EditValue), Convert.ToDouble(Zone2.EditValue), (PS1_DeviceA.SelectedIndex == 0) ? "ЛИФП" : "ИМФ", (PS2_DeviceA.SelectedIndex == 0) ? "ЛИФП" : "ИМФ", GetCurrentUserName(), DateTime.Now, Branch.Count, PS1.Text, PS2.Text, Convert.ToDouble(LengthVL.EditValue), UsingForEval[0], UsingForEval[1], UsingForEval[2], UsingForEval[3], OffDT.DateTime);
    ompDataSet.Result.AcceptChanges();
}
catch
{}

ZOMG! Что это!?

invi, 16 Февраля 2012

Комментарии (5)

C# / Говнокод #9444

+962

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49

for (int i = 0; i < 4; i++)
{
    if (!UseParam[i])
    {
        switch (i)
        {
            case 0:
                if (AutoCalc || XtraMessageBox.Show("Неверные показания амперметра " + PS1_DeviceName_A.Text + " " + PS1.Text + "\nИслючить из расчета?", "Проверка показаний", MessageBoxButtons.YesNo, MessageBoxIcon.Question, MessageBoxDefaultButton.Button1) == DialogResult.Yes)
                {
                    if (CorrectKm[i] == 0 || CorrectKm[i] == L)
                        I1 = (CorrectValue[i] ?? 0) * 1000;
                    else
                        I1 = GetCorrectElectricValue(CorrectKm[1] ?? 0, "I0_A", row["N"], row["M"]) * 1000;
                    I0_A.ErrorText = PS1_A.ErrorText = "Показание " + PS1_DeviceName_A.Text + " исключено из расчета";
                }
                break;
            case 1:
                if (AutoCalc || XtraMessageBox.Show("Неверные показания вольтметра" + PS1_DeviceName_B.Text + " " + PS1.Text + "\nИслючить из расчета?", "Проверка показаний", MessageBoxButtons.YesNo, MessageBoxIcon.Question, MessageBoxDefaultButton.Button1) == DialogResult.Yes)
                {
                    if (CorrectKm[i] == 0 || CorrectKm[i] == L)
                        U1 = (CorrectValue[i] ?? 0) * 1000;
                    else
                        U1 = GetCorrectElectricValue(CorrectKm[0] ?? 0, "U0_A", row["N"], row["M"]) * 1000;
                    U0_A.ErrorText = PS1_B.ErrorText = "Показание " + PS1_DeviceName_B.Text + " исключено из расчета";
                }
                break;
            case 2:
                if (AutoCalc || XtraMessageBox.Show("Неверные показания амперметра" + PS2_DeviceName_A.Text + " " + PS2.Text + "\nИслючить из расчета?", "Проверка показаний", MessageBoxButtons.YesNo, MessageBoxIcon.Question, MessageBoxDefaultButton.Button1) == DialogResult.Yes)
                {
                    if (CorrectKm[i] == 0 || CorrectKm[i] == L)
                        I2 = (CorrectValue[i] ?? 0) * 1000;
                    else
                        I2 = GetCorrectElectricValue(CorrectKm[3] ?? 0, "I0_B", row["N"], row["M"]) * 1000;
                    I0_B.ErrorText = PS2_A.ErrorText = "Показание " + PS2_DeviceName_A.Text + " исключено из расчета";
                }
                break;
            case 3:
                if (AutoCalc || XtraMessageBox.Show("Неверные показания вольтметра" + PS2_DeviceName_B.Text + " " + PS2.Text + "\nИслючить из расчета?", "Проверка показаний", MessageBoxButtons.YesNo, MessageBoxIcon.Question, MessageBoxDefaultButton.Button1) == DialogResult.Yes)
                {
                    if (CorrectKm[i] == 0 || CorrectKm[i] == L)
                        U2 = (CorrectValue[i] ?? 0) * 1000;
                    else
                        U2 = GetCorrectElectricValue(CorrectKm[2] ?? 0, "U0_B", row["N"], row["M"]) * 1000;
                    U0_B.ErrorText = PS2_B.ErrorText = "Показание " + PS2_DeviceName_B.Text + " исключено из расчета";
                }
                break;
        }
    }
}

Красота

invi, 16 Февраля 2012

Комментарии (13)

C# / Говнокод #9443

+960

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15

private static void WriteErrLog(string FileName, string errors)
{
    try
    {
        string name = Environment.CurrentDirectory + "\\log\\" + FileName.Replace("80020_", "");
        name = name.Replace(".xml", ".txt");
        if (File.Exists(name))
            name = name.Replace(".txt", "_" + DateTime.Now.ToString("yyyy-MM-dd hh_mm_ss") + ".txt");                
        using (StreamWriter f = new StreamWriter(name, true, Encoding.GetEncoding(1251)))
        {
            f.Write(errors);
        }
    }
    catch { }
}

Продолжаем...

invi, 16 Февраля 2012

Комментарии (16)

C# / Говнокод #9442

+113

01
02
03
04
05
06
07
08
09
10

private static string GetStringParam(ref string str, string param)
{
    string result = "";
    if (str.Contains(param))
    {
        result = str.Substring(str.IndexOf(param) + 1);
        str = str.Remove(str.IndexOf(param));
    }
    return result;
}

И еще

invi, 16 Февраля 2012

Комментарии (0)

C# / Говнокод #9441

+113

01
02
03
04
05
06
07
08
09
10
11
12
13
14

public static string GetShortName(string name, int length)
{
    StringBuilder shortname = new StringBuilder(name);
    string[] str = new string[] { "ПС", "ОАО" };
    for (int i = 0; i < str.Length; i++)
        if (shortname.ToString().Length > str[i].Length && shortname.ToString().Substring(0, str[i].Length).ToUpper() == str[i])
            shortname.Remove(0, str[i].Length);

    str = new string[] { "ТЭЦ", "ГЭС", "ГРЭС" };
    for (int i = 0; i < str.Length; i++)
        if (shortname.ToString().IndexOf(str[i], StringComparison.CurrentCultureIgnoreCase) > 0)
            shortname.Remove(1, shortname.ToString().IndexOf(str[i]) - 1);
    return (shortname.Length > length) ? shortname.ToString().Substring(0, length).Trim() : shortname.ToString().Trim();
}

Опять же, из того, что досталось в наследство)

invi, 16 Февраля 2012

Комментарии (0)

C++ / Говнокод #9423

+159

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78

void CXMLParser::Parse(std::map< std::pair<int, int>, int >& link)
{
  _di_IXMLNode areaNode = MessageNode->ChildNodes->FindNode( WideString("area") );
  if( areaNode )
  {
    Area.reset( new CAreaPart( areaNode ) );
    IXMLNodeList* pointList = areaNode->ChildNodes;
    if( pointList )
    {
      for(int pointIndex=0; pointIndex<pointList->GetCount(); pointIndex++)
      {
        _di_IXMLNode pointNode = pointList->Get( pointIndex );
        int typePoint = -1;
        if( pointNode->GetNodeName() == WideString("measuringpoint") )
          typePoint = MEASURING;
        if( pointNode->GetNodeName() == WideString("accountpoint") )
          typePoint = ACCOUNT;
        if( pointNode->GetNodeName() == WideString("deliverypoint") )
          typePoint = DELIVERY;
        switch( typePoint )
        {
        case MEASURING:
        case ACCOUNT:
        case DELIVERY:
          {
            const int pointCode    = pointNode->GetAttribute( WideString("code") );
            const String pointName = pointNode->GetAttribute( WideString("name") );
            IXMLNodeList* channelList = pointNode->ChildNodes;
            if( channelList )
            {
              for(int channelIndex=0; channelIndex<channelList->GetCount(); channelIndex++)
              {
                _di_IXMLNode channelNode = channelList->Get( channelIndex );
                if( channelNode->GetNodeName() == WideString("measuringchannel") )
                {
                  const int channelCode    = channelNode->GetAttribute( WideString("code") );
                  const String channelDesc = channelNode->GetAttribute( WideString("desc") );
                  IXMLNodeList* periodList = channelNode->ChildNodes;
                  if( periodList )
                  {
                    for(int periodIndex=0; periodIndex<periodList->GetCount(); periodIndex++)
                    {
                      _di_IXMLNode periodNode = periodList->Get( periodIndex );
                      if( periodNode )
                      {
                        String str = periodNode->GetAttribute( WideString("start") );
                        periodNode->GetAttribute( WideString("end") );
                        TDateTime time;
                        time = TDateTime( StrToInt( str.SubString( 1, 2 ) ), StrToInt( str.SubString( 3, 2 ) ), 0, 0 );
                        periodNode->GetAttribute( WideString("summer") );
                        time = IncMinute( time, 30 );
                        _di_IXMLNode valueNode = periodNode->ChildNodes->FindNode( WideString("value") );
                        if( valueNode )
                        {
                          const int status = valueNode->GetAttribute( WideString("status") );
                          const int value = StrToInt( valueNode->Text );
                          if( RTDBWrite )
                          {
                            map< pair<int, int>, int >::iterator i = link.find( make_pair( pointCode, channelCode ) );
                            if( i!=link.end() )
                            {
                              (*RTDBWrite)( Date->DateInterval.first + time, (*i).second, value, status );
                            }
                          }
                        }
                      }
                    }
                  }
                }
              }
            }
          }
        }
      }
    }
  }
  RTDBWrite->Flush();
}

Древний legacy-код на БорландЦэпэпэБыдлере, обнаруживаемый на текущей работе (почти госпредприятие), выглядит в основной своей массе примерно так. Новый код на C# чуть получше, но в целом тоже жуткое мракобесие.
Сил больше нет работать с подобным, увольняюсь и ищу норм работу.

invi, 15 Февраля 2012

Комментарии (9)

C++ / Говнокод #9421

+999

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29

#include <iostream>
#include <algorithm>
#include <stdlib.h>

const size_t MB = 1024*1024;
size_t MOD = 0;

unsigned char uniqueNumber () {
  static unsigned char number = 0;
  return ++number % MOD;
}

int main(int argc, char** argv) {
  if (argc < 3) {
    return 1;
  }

  size_t BLOCK_SIZE = atoi(argv[1]) * MB;
  MOD = atoi(argv[2]);

  unsigned char* garbage = (unsigned char *) malloc(BLOCK_SIZE);

  std::generate_n(garbage, BLOCK_SIZE, uniqueNumber);
  std::sort(garbage, garbage + BLOCK_SIZE);

  free(garbage);

  return 0;
}

http://habrahabr.ru/blogs/cpp/138132/

It makes me cry. Понятно, что это всего лишь демонстрационный пример. Но все таки это не оправдание. Итак, начнем по порядку с самого худшего:
1. Сишные malloc/free вперемешку с STL-алгоритмами. WTF? Зачем?
2. Глобальная переменная? Автор не осилил хотя бы bind? Который, к тому же, уже давно std::bind.
3. Uppercase для локальной переменной.
4. Отступ в джва пробела.

invi, 15 Февраля 2012

Комментарии (80)

Говнокод: по колено в коде.