1. Список говнокодов пользователя dm-ua

    Всего: 9

  2. Куча / Говнокод #21318

    −22

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    void TDataModule1::GetUniqueValues(TStringList* buffer, AnsiString columnName) {
  AnsiString icn = columnName.Trim();
  AnsiString sqlQuery = "SELECT ["+icn+"] FROM [index] ORDER BY ["+icn+"];";
  adq->Close(); 
  adq->SQL->Clear();
  adq->SQL->Add(sqlQuery);
  adq->Open();

  AnsiString lastValue = "";
  AnsiString nextValue = "";
  while(!adq->Eof) {
    nextValue = adq->FieldByName(icn)->AsString.Trim();
    if(nextValue!=lastValue) {
      buffer->Add(nextValue);
      lastValue = nextValue;
    }
    adq->Next();
  }
  adq->Close(); 
}

    SELECT DISTINCT? не, не слышали...

    dm-ua, 03 Октября 2016

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

  3. C++ / Говнокод #21156

    −32

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    try
        {
		//	...
                 Application->Run();
        }
        catch (...)
        {
                 try
                 {
                         throw Exception("");
                 }
                 catch (Exception &exception)
                 {
                         Application->ShowException(&exception);
                 }
        }

    Сам в шоке

    dm-ua, 09 Сентября 2016

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

  4. Си / Говнокод #15872

    +154

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    if(sel.ToInt()==1){
  Form1->NumberOfIndexes = 1;
}else if(sel.ToInt()==2){
  ...
  //еще 11 таких же кучек
  ...
}else if(sel.ToInt()==13){
  Form1->NumberOfIndexes = 13;
}else{
  Form1->NumberOfIndexes = 1;
}

    dm-ua, 30 Апреля 2014

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

  5. Си / Говнокод #15871

    +134

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    try{
    tempPage1Int = tempPage1.ToInt();
  }catch(Exception &E){
    tempPage1Int = 0;
    goto NEXTFUCKER;
  }
  NEXTFUCKER:

    Дописываю из-под стола.

    dm-ua, 30 Апреля 2014

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

  6. Си / Говнокод #15870

    +132

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    if(fileCounter < 1000){
      subDir =  "SUBDIREC.001";
      //много кода
    }else if (fileCounter < 2000 && fileCounter > 999){
      subDir =  "SUBDIREC.002";
      //много кода, плюс еще 17 таких же веток
    }else if (fileCounter < 20000 && fileCounter > 18999){
      subDir =  "SUBDIREC.020";
      //много кода
    }

    dm-ua, 30 Апреля 2014

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

  7. Си / Говнокод #15867

    +134

    1. 1
    2. 2
    3. 3
    while(SLOBirthsISEX->Strings[p].Length() < 1){
    SLOBirthsISEX->Strings[p] = " " + SLOBirthsISEX->Strings[p];
  }

    нет слов

    dm-ua, 30 Апреля 2014

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

  8. Си / Говнокод #15540

    +142

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    for(int i=0; i<10; i++) {
  if(i==5) {
    //do something
    break;
  }
}

    Без слов... Взято из реального проекта, с некоторыми упрощениями

    dm-ua, 20 Марта 2014

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

  9. C++ / Говнокод #9385

    +997

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    for (int z=NewArray[y-1];z<(NewArray[y]-1);z++)
                {
                  AnsiString ts = IntToStr(z+1);
                  int inum = -1;
                  try{
                    inum = StrToInt(ts);
                  } catch(...) {}
..... //дальше inum не используется
                }

    Завис надолго. Так и не понял, зачем автор преобразовывает ts обратно

    dm-ua, 10 Февраля 2012

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

  10. C++ / Говнокод #8873

    +1000

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    if(SpokaneFileList->Strings[i].SubString(1,8).ToInt() < 1000){
      imageFolder="00000\\";
    }else if (SpokaneFileList->Strings[i].SubString(1,8).ToInt() < 2000){
      imageFolder="00001\\";
    }else if (SpokaneFileList->Strings[i].SubString(1,8).ToInt() < 3000){
      imageFolder="00002\\";
    }else if (SpokaneFileList->Strings[i].SubString(1,8).ToInt() < 4000){
....
      imageFolder="00034\\";
    }else if (SpokaneFileList->Strings[i].SubString(1,8).ToInt() < 36000){
      imageFolder="00035\\";
    }else if (SpokaneFileList->Strings[i].SubString(1,8).ToInt() < 37000){
      imageFolder="00036\\";
    }

    Сегодня чувствую себя сказочным героем (Гераклом) в весьма не сказочно обкаканном месте (авгиевых конюшнях),
    которому надо это место отмыть до блеска...

    меняю куски наподобие такого, как выше, каждый длиной по 75 строк - на вот такую конструкцию собственного сочинения -

    int sfFactor = SpokaneFileList->Strings[i].SubString(1,8).ToInt();
    int imageFolder = StringHelper::NormalizeString(IntToStr(s fFactor/1000), 5, "0", true, true) + "\\";

    - FYI функция StringHelper::NormalizeString() занимает 19 строк.

    На добивание:
    такие куски встречаются в программе около 20 раз (!!!)

    Индийский код - не признак национальности, а способ мышления...

    dm-ua, 19 Декабря 2011

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