Говнокод #20321 — C++ — Говнокод.ру

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

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

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

while (cycle) {
		#if DEBUG == 1
		mvprintw(0, 0, "cn: %i", cn);
		mvprintw(all_lines + 7, 0, "max_line: %i\nMaxX: %i\nMaxY: %i", max_line, maxX, maxY);
		#endif

		if (active_input == 1)
			attron(COLOR_PAIR(color_selected) | A_BOLD);

		if (active_input != 1)
			attron(COLOR_PAIR(dlgcfg.style) | A_BOLD);

		if (dlgcfg.keys > 1) {
			mvprintw(i + 3 + title_fix, left_border_x + 2, " %s ", dlgcfg.f_button.c_str()); // Первая кнопка
		} else {
			mvprintw(i + 3 + title_fix, maxX / 2 - (dlgcfg.f_button.length() / 2) - 1, " %s ", dlgcfg.f_button.c_str()); // Центровка первой кнопки (Если она одна)
		}

		if (active_input != 1)
			attroff(COLOR_PAIR(dlgcfg.style) | A_BOLD);		

		if (active_input == 1)
			attroff(COLOR_PAIR(color_selected) | A_BOLD);

		if (dlgcfg.keys  == 2) {
			if (active_input == 2)
				attron(COLOR_PAIR(color_selected) | A_BOLD);

			if (active_input != 2)
				attron(COLOR_PAIR(dlgcfg.style) | A_BOLD);

			mvprintw(i + 3 + title_fix, maxX / 2 + (max_line / 2) - dlgcfg.s_button.length() - 3 - fix, " %s ", dlgcfg.s_button.c_str()); // Вторая кнопка

			if (active_input != 2)
				attroff(COLOR_PAIR(dlgcfg.style) | A_BOLD);

			if (active_input == 2)
				attroff(COLOR_PAIR(color_selected) | A_BOLD);
		}

		if (dlgcfg.keys  == 3) {
			if (active_input == 2)
				attron(COLOR_PAIR(color_selected) | A_BOLD);

			if (active_input != 2)
				attron(COLOR_PAIR(dlgcfg.style) | A_BOLD);

			mvprintw(i + 3 + title_fix, s_key_pos - dlgcfg.s_button.length() / 2, " %s ", dlgcfg.s_button.c_str()); // Вторая кнопка

			if (active_input != 2)
				attroff(COLOR_PAIR(dlgcfg.style) | A_BOLD);

			if (active_input == 2)
				attroff(COLOR_PAIR(color_selected) | A_BOLD);

			if (active_input == 3)
				attron(COLOR_PAIR(color_selected) | A_BOLD);

			if (active_input != 3)
				attron(COLOR_PAIR(dlgcfg.style) | A_BOLD);

			mvprintw(i + 3 + title_fix, maxX / 2 + (max_line / 2) - dlgcfg.t_button.length() - 3 - fix, " %s ", dlgcfg.t_button.c_str()); // Вторая кнопка

			if (active_input != 3)
				attroff(COLOR_PAIR(dlgcfg.style) | A_BOLD);

			if (active_input == 3)
				attroff(COLOR_PAIR(color_selected) | A_BOLD);
		}
		cn	= getch();
		switch (cn) {
			case CtrlF1:	info_win();
							break;

			case KEY_LEFT:	if (active_input != 1)
								active_input--;
							break;

			case KEY_RIGHT:	if (active_input != dlgcfg.keys)
								active_input++;
							break;

			case TAB_KEY:	if (active_input != dlgcfg.keys) {
								active_input++;
							} else {
								active_input = 1;
							}
							break;

			case 27:		delete [] array;
							return 0;
							break;

			case '\n':		if (active_input != 0) {
								delete [] array;
								return active_input;
							}
							break;
		}
И ещё ~400 строк такой же дичи)

Даже если интерфейс кажется вам милым и опрятным, не распространяйте свои догадки на реализацию...

Запостил:

viktorokh96, 04 Июля 2016

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

cykablyad 04.07.2016 11:24 # 0

> return 0;
> break;

На случай если вдруг не выйдет?
Ответить
- dxd 04.07.2016 11:32 # +6
  
  На нашей летней школе в одной из мастерский дети писали конструкцию "return 0; break; exit(1);" для какого-то такого стрёмного случая. Мы даже родили название этого: антипаттерн "Борис Бритва".
  Ответить
  - Dummy00001 04.07.2016 11:37 # +2
    
    почти как в старой доброй шутке: "cli; 1: hlt; jmp 1b;"
    Ответить
    - inkanus-gray 04.07.2016 13:19 # 0
      
      Есть более продвинутые варианты. Во-первых, существуют немаскируемые прерывания, на которые CLI не действует. Их надо отключить, послав какую-то команду в контроллер прерываний. Во-вторых, особо бдительные куда-то впендюривали префикс LOCK.
      Ответить
      - Dummy00001 04.07.2016 13:26 # 0
        
        про "lock" забыл! а извращатся с nmi не нужно - для этого там и стоит "jmp 1b". (не то что бы на PC было так много источников NMI.)
        Ответить
        
        inkanus-gray 04.07.2016 13:29 # 0
        
        К сожалению, LOCK действует только на инструкции, разыменовывающие указатели в память. Поэтому найти для него место в таком коде очень трудно.
        Ответить
        
        Dummy00001 04.07.2016 13:31 # +5
        
        > К сожалению, LOCK действует только на инструкции, разыменовывающие указатели в память.
        
        у "lock"а есть еще одна функция: нубов пугать. потому что если в коде стоит "lock:" то значит что-то очень важное делается.
        Ответить
        
        inkanus-gray 04.07.2016 13:37 # +1
        
        А ещё в сишке можно писать InterlockedIncrement(i) (точное название функции зависит от компилятора) вместо i++, чтобы создавать впечатление серьёзности.
        Ответить
- Dummy00001 04.07.2016 11:37 # +5
  
  сам такое в прошлом писал: некоторые чекеры слишком тупы что бы распознать ретурн, и тупо жалаются на кэйсы без бряков.
  Ответить
  - Dummy00001 04.07.2016 13:42 # +6
    
    ... а потом, к слову, некоторые компилеры жалуются, на бряк пальцем тыкают, и говорят unreachable code. вобщем, хоть вешайся.
    Ответить
    - Steve_Brown 04.07.2016 13:50 # 0
      
      // Unreachable break #ifdef DUMB_COMPILER #define ubreak break #else #define ubreak #endif
      Ответить
      - roman-kashitsyn 04.07.2016 14:15 # 0
        
        > ubreak
        unsigned break?
        Ответить
        
        Dummy00001 04.07.2016 14:20 # +3
        
        unbreak. ну как в песне: unbreak my switch, case you goto again.
        Ответить
    - inkanus-gray 04.07.2016 13:56 # +3
      
      Компиляторы таким образом подталкивают к вступлению в секту одновозвратников.
      Ответить
- gost 04.07.2016 17:28 # +2
  
  Вообще, конечно, привычка писать break после каждого case, вне зависимости от его содержания (ну, кроме тех, где нужно именно "провалиться") - весьма полезна. Не придётся потом долго ломать голову, почему после мелкого изменения логики всё полетело к чертям.
  Ответить
roman-kashitsyn 04.07.2016 11:26 # +5

> attron

Исходники аттронного каллаидера?
Ответить
roman-kashitsyn 04.07.2016 12:06 # +11

> delete [] array;
Я нашёл C++!
Ответить
- Dummy00001 04.07.2016 13:12 # +2
  
  кресты и н-проклятия не далеки от друг друга.
  Ответить
Antervis 04.07.2016 15:12 # +2

"I'm unbreakable, unstoppable, I'm invincible..." © Bon Jovi, "Unbreakable"
Ответить

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

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

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

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