Говнокод #2171 — Си — Говнокод.ру

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

+106.7
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
21. 21
22. 22
23. 23
24. 24
25. 25
26. 26
27. 27
28. 28
29. 29
30. 30
31. 31
32. 32
33. 33
34. 34
35. 35
36. 36
37. 37
38. 38
39. 39
40. 40
41. 41
42. 42
43. 43
```
char *getUI_ID_AsString(MENU_ID menuId)
{
    char *IdStr;
    	
	if((menuId == VOLUME_ID) || (menuId == POPUPWIN_ID) || (menuId == YESNOWIN_ID) || (menuId == MAINMENU_ID) || (menuId == 
		MODEMENU_ID) || (menuId == USBMODE_ID) || (menuId == SETTINGSMENU_ID) || (menuId == MENUPLAYMODE_ID) || (menuId == MENUEQ_ID) 
		|| (menuId == MENUSETTING_ID) || (menuId == MENULOCK_ID) || (menuId == MENUPLAYSPEED_ID) || (menuId == MENUCONTRAST_ID) || (menuId == 
		MENUBACKLIGHT_ID) || (menuId == MENULANGUAGE_ID) || (menuId == MENUSHUTDOWN_ID) || (menuId == MENUINFO_ID) || (menuId == 
		MENUBEEP_ID) || (menuId == MENUFACTORYSET_ID) || (MENUUSEREQ_ID) || (menuId == SET_PREFERENCES_ID) || (menuId == 
		MENUCONFIRM_RESET_ID) || (menuId == MENUCONFIRM_FORMAT_ID) || (menuId == MAINRECMENU_ID) || (menuId == RECSETMODE_ID) || 
		(menuId == RECSETINPUT_ID) || (menuId == RECSETVAD_ID) || (menuId == RECSETAUTOSYNC_ID) || (menuId == NAVIGATION_ID) ||
		(menuId == PROMPT_VOLUME_ID) || (menuId == MENUGPSAPPL_MAIN_ID) || (menuId == POWERON_CONNECTING_ID) || (menuId == 
		HOMESCREEN_ID) || (menuId == RC_DOWNLOADING_ID) || (menuId == SCREEN_BRIGHTNESS_ID) || (menuId == ABOUT_ID) || (menuId == 
		INSTMENU_ID) || (menuId == INST_ABOUT_ID) || (menuId == DISPL_ORIENT_ID) || (menuId == LINEOUT_ID) || (menuId == 
		PRIVACY_MUTE_ID) || (menuId == TEST_INSTALL_ID) || (menuId == FACTORY_RESET_ID) || (menuId == CALL_DTMF_ID) ||	(menuId == 
		NAVI_FILEMENU_ID) || (menuId == NAVI_CONFIRM_DEL_ID) || (menuId == NAVI_SELECTFOLDER_ID) || (menuId == LANG_SELECT_ID) ||
		(menuId == DISPLAY_JPG_ID) || (menuId == DISPLAY_TEXT_ID) || (menuId == SET_DISPLAY_ID))
	{
	    IdStr = (char *) malloc(4);
    	strcpy(IdStr, " MENU");
	}
	else if( <тут примерно еще столько же условий> )
		{
		       IdStr = (char *) malloc(5);
			   strcpy(IdStr, " PHONE");
		}
	else if( <тут примерно еще столько же условий> )
		{
		    strcpy(IdStr, " PBOOK");
		}
	else if( <тут примерно еще столько же условий> )
		{
		    IdStr = (char *) malloc(5);
		    strcpy(IdStr, " MUSIC");
		}
	else if( <тут примерно еще столько же условий> )
		{
		    IdStr = (char *) malloc(3);
		    strcpy(IdStr, " MSG");
		}
		
		return IdStr;
}
```
Шедевр индусского говнокодирования. А потом с нас спрашивают, почему это у нас система нестабильно работает.
switch, по ходу, они еще в школе не прошли, да и с математикой у них плоховато. А где-то память вообще выделять не надо - зачем?

Запостил: Scrambler, 24 Ноября 2009

Tweet
Комментарии (31) RSS
- paranoid 24.11.2009 15:54 # −0.2
  
  нисмешно
  на говнокоде пора раздел сделать быдлокод
  Ответить
  - nil 24.11.2009 16:26 # 0
    
    По моему разумению, говнокод (ваш код - говно!) - это быдлокод. А тот, "на который без улыбки не взглянешь", это либо классная находка, изящество, либо изврат.
    Изврат получается в тех случаях, когда средства разработки ограничивают полет мысли:) Когда не получается втиснуть идею в узкие рамки.
    В любом случае, большинство представленных кусков кода под определение говнокода не попадают...
    Ответить
    - paranoid 24.11.2009 16:55 # 0
      
      не улыбнулся - низачот. а смешной код может получиться даже при полной свободе полёта (и обычно он там и получается).
      
      p.s.
      мне кажется, что слоган говнокода исключает истинность твоего высказывания
      Ответить
      
      nil 25.11.2009 00:09 # 0
      
      Мне просто претит называть смешной код говнокодом.
      Funnycode, cunningcode — запросто, но, похоже, здесь имеется в виду то, что я называю извратами, когда можно сделать проще, но сделано через жопу. А вот когда сделано в лоб, тогда быдлокод, да?
      Ответить
      
      paranoid 25.11.2009 11:20 # 0
      
      когда делано непрошареными людьми и абакак - быдлокод,
      когда смешной лаг - говно
      Ответить
      
      guest 06.01.2010 19:10 # 0
      
      У каждого свои понятия и вкусы...
      Ответить
- guest 24.11.2009 17:14 # −1
  
  я сомневаюсь, что такоевообще наплодить можно, а тем более - в таком виде. автор поста сам его придумал? :)
  Ответить
  - Scrambler 24.11.2009 18:04 # 0
    
    Ты явно недооцениваешь индусов... это ужасные люди. Они легко могут наплодить еще и не такое... эх...
    Ответить
    - guest 24.11.2009 18:50 # 0
      
      к сожалению, я работал с настоящими индусами равшанами и тд, не раз, но такого "юмора" у них не встречал
      слава богу, меня относительно недавно уберегло от этого "сотрудничества" в будущем, чему я теперь искренне рад
      и с нашими "индусами" я работал и работаю, но наши ребята куда более славные, это факт
      но всё-равно, смотря на это, как станиславский, сомневаюсь и говорю "не верю!"
      ну не бывает таких полоумных, не бы ва ет
      Ответить
      
      guest 24.11.2009 19:42 # 0
      
      Ёпт. Бывает. Сам года 3 назад такое делал. Как сейчас помню. o_0 Разве условия были записаны в одну строчку, но их было именно не меньше.
      Ответить
      
      guest 24.11.2009 19:46 # +1
      
      так это ты "индус" и двоечник, который ещё тогда не работал в сфере ит :D
      Ответить
      
      guest 24.11.2009 22:16 # 0
      
      Я - Русский! И ещё я исправляюсь. :)
      Ответить
      
      paranoid 25.11.2009 11:21 # 0
      
      нам то не заливай! за версту видно, что ты людоедом был.
      Ответить
      
      guest 25.11.2009 17:51 # 0
      
      С чего это я людоед?
      Ответить
      
      guest 26.11.2009 11:09 # 0
      
      Жмурки посмотри ))))
      Ответить
- vyalcevanet 25.11.2009 00:12 # 0
  
  Тут ещё памяти во всех местах выделено мало, а в одном месте вообще забыли выделить. Второе, скорее всего, приведёт к падению программы сразу, а от первого она ещё помучается :)
  Ответить
  - guest 25.11.2009 15:05 # 0
    
    Программа, как ни странно падает довольно редко (система довольно большая) даже с ТАКИМ ужасом внутри
    Ответить
    - guest 25.11.2009 15:55 # 0
      
      так значит, этот код вполне нормально работает, да? :)
      Ответить
      
      guest 26.11.2009 16:28 # +1
      
      Этот кусок кода вертится на некой железке... И она действительно работает и падает довольно редко.
      
      Там есть ещё такое:
      
      static char musicInfo[70] = {0}; // ~~skip~~ for(count =0; count < 70; count++) { musicInfo[70] = '\0'; }
      
      ... тоже Made in India (tm)
      Ответить
    - guest 27.11.2009 05:13 # +1
      
      Ну так, индусам везёт, что оно вообще работает: malloc выравнивает выделенную память до некой константы, кратной двум (минимум восьмёрка...)
      
      В итоге "IdStr = (char *) malloc(5); strcpy(IdStr, " PHONE");" компьютер кушает и непоперхивается. Но боже упаси подобный код запустить на другой машине/ОС... Там уже как ляжет :)
      Ответить
  - guest 25.11.2009 15:58 # 0
    
    кстати, реально, никто не обратил внимания сначала, что в некоторых ситуациях память под IdStr не выделяется и там, по ходу, мусор, а strcpy() всегда переполняет буффер в куче ) мне кажется, здесь баг куда позадорней, что-ли, чем ужасные ифы. суть не в том. мне интересно, как часто эта аппликуха падает, когда она использует технику mem-allo(size) - mem-copy(size + N) )) попробуй, блин, отлови такой баг )
    Ответить
    - guest 25.11.2009 17:54 # 0
      
      >попробуй, блин, отлови такой баг
      Легко! Просто переписать говнокод. :)
      Ответить
      
      guest 25.11.2009 20:11 # 0
      
      тю ) я сказал "отловить", а не "пофиксить" )))
      Ответить
- guest 25.11.2009 15:58 # 0
  
  Ещё и не освобождают же...
  Ответить
  - guest 25.11.2009 16:27 # 0
    
    очевидно, этой функции освобождать память и не нужно
    память освобождает код, вызвавший getUI_ID_AsString()
    Ответить
    - guest 25.11.2009 17:52 # 0
      
      Верный признак говнокода.
      Ответить
      
      guest 25.11.2009 20:10 # 0
      
      разве что в С с его убогими "строками"
      Ответить
      
      guest 27.11.2009 05:09 # +1
      
      в си нормальные строки, это ты убогий
      Ответить
    - nil 25.11.2009 18:41 # 0
      
      Память освобождается автоматически по 11 сигналу:)
      Ответить
      
      guest 25.11.2009 19:53 # 0
      
      Аппаратчик?
      Ответить
      
      nil 26.11.2009 00:54 # 0
      
      Ни в коем разе! Наоборот!
      Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только в понедельник, среду, четверг или воскресение
Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

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

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

Комментарии (31) RSS

Добавить комментарий