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

Всего: 94

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

+998

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

#define TRACE_VAR	ATraceClass *mTrace
#define TRACE_VAR_INIT	mTrace = ATraceClass::getInstance()

class ATraceClass {
public:
	void log(int severity, const char *format, ...);
	/* ... */
}

class ATraceUser {
public:
	ATraceUser::ATraceUser() {
		TRACE_VAR_INIT;
	}
        int SomeMethod1();
	/* ... */
	TRACE_VAR;
}

int ATraceUser::SomeMethod1()
{
  /* ... */
  if (errorCondition)
  {
    return mTrace->log( ERR, "Something went terribly"\
                        "wrong when doing the operation %s\n",\
                        mOperation),-1;
  }
  /* ... */
}

в догонку к #3355 - это уже как бы пример стиля использования логгера егоным же автором.

пришел ко мне как-то народ, показал нечто что я тут иллюстрирую в SomeMethod1() и спросил:
(1) а на кой эти слеши? может нам и весь остальной код нужно ими разукрасить??
(2) а как это void можно из int функции возвращать? и как это вообще работает? вроде в С/С++ такое нельзя....

у меня до сих пор в голове не укладывается каким .... надо быть что бы такой код писать.

Dummy00001, 31 Мая 2010

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

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

+1003

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

// укорочено для краткости.
void ATraceClass::LogFunction(const char *format, va_list ap)
{
  /* ... declarations ... */

  for (p=format; (*p)&&(!isTruncated); p++)
  {
    if ( * p == '%' )
      switch( * ++p )
      {
        case 'c':
        { /* ... */ }
        break;

        case 'd':
        case 'i':
        { /* ... */ }
        break;

        case 'l':   // это %ld
        { /* ... */ }
        break;

        case 'L':    // это %Ld
        { /* ... */ }
        break;
  
        case 'f':   // a это %g
        { /* ... */ }
        break;

        case 'F':  // a это %Lg
        { /* ... */ }
        break;

        case 's': 
        { /* ... */ }
        break;

        default: // this is an ERROR case, but we will not perform coding at this point ... maybe later
          break;
      }
      else {
        /* ... */
      }
  }
}

наболело. одно чудило (с большой букмы Му) наговнокодило это дело под эгидой "stdio это С, мы в С++ можем круче!!!" ну с тех пор и переизобретают велосипед - с квадратными колёсами. даже %% не догадались сделать.

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

самое гадкое что народ пишет код как обычно ожидая стандартные stdio шорткаты, и потом долго гадает куда блин сообщение подевалось.

Dummy00001, 31 Мая 2010

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

Си / Говнокод #3154

+143

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

// народ на протяжении 5+ лет жаловался что постоянно что-то в этой библиотеке Х
// криво потому что часто (с их слов) попытки вызова ее функций просто обламываются:

int module_main_function()
{
  int rc = 0;
  // ...

  rc = libX_do_work( ... )
  if (rc != OK) {
    // error!!!
  }

  // ...
}

// теперь заглянем в инициализацию этого модуля:

int module_init()
{
   int rc; 
   // ... кучи дежурного копи-паста для инициализации скипнуты ...
   rc = libZ_init();
   if (rc != OK)
   {
     /* error handling */
   }

   rc = libY_init();
   if (rc != OK)
   {
     /* error handling */
   }

   libX_init();   /* лопата */
   if (rc != OK)
   {
     /* error handling */
   }

   rc = libA_init();
   if (rc != OK)
   {
     /* error handling */
   }

   // ... и т.д. и т.п.
}

меня жалобы одного отдела начали потиху доставать что наша библиотека (назовем ее libX) у них в модуле постоянно не работает и шо ваабще за херня. как встроили ее - так и не работает. иногда. в 80+ модулях работает как часы - ни один другой отдел не жалуется - а вот в ихнем очень специальном модуле нет.

ну я к ним подвалил. они меня сразу в рабочую функцию тыкать стали - вот тут дескать не работает. я им методично: покаж где инициализация. я когда в инициализацию заглянул, почти сразу начал роготять (давно так не смеялся, наболело за годы).
им минут пять понадобилось что бы в своем копи-паст параде рассмотреть эту глюку.

ЗЫ ответ: на строке с лопатой, не хватает "rc = ". у библиотеки есть внешняя конфигурация на отсутствии коей она у них обламывалась постоянно. и исправить скрипты свои что бы чужая конфигурация не убивалясь им похоже религия не позволяет.

Dummy00001, 04 Мая 2010

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

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

+924

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

// придерживаюсь оригинала как могу
// код как был:

static char message_buf[64000];  // must be actually 64K == 64<<10 == 65536

class MessageHandler {
   // [...]
   char *header;         // point into the message_buf
   char *content;
   std::list<char *> tlvs; // pointers to the elements of the message
};

// я пожаловался - пообещали исправить в новой версии по ходу реализации новой фичи.

// новый "усовершенствованый" код, с "поддержкой" многопоточности.

static char message_buf[100][65000];  // 100 == max threads, buffer size now is much closer to the standard

class MessageHandler {
   // [...]
   int threadId;             // index into the new message_buf[]
   char *header;          // point into the message_buf[threadId]
   char *content;
   std::vector<char *> tlvs; // same, but in std::vector because it is "faster"
};

вот так кодируют и совершенствуют код матерые шведские программистки.
самое обидное что мой коллега почти все это пофиксил в оригинальной версии, но из-за важной фичи отказались мержить с новой версией и выкинули почти все его изменения.

Dummy00001, 26 Апреля 2010

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

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