Как реализовать переключатель?

Question

[CodeInside]

Делаю так, чтобы пользователь нажимая стрелки вверх/вниз выбирал "1 player"/"2 players". Выбранный вариант должен подсвечиваться белым фоном. Вроде реализовал, но работает неправильно. Сложно понять как задавать цвета в этом С. Вот что у меня получилось:

#include <iostream>
#include <windows.h>
#include <conio.h>
using namespace std;

void Select1Player();
void Select2Players();

void main()
{
	bool player2 = false;
	Select1Player();
	
	char c = 0;
	while(c != 13)
	{
		c = _getch();
		if((c == 72 || c == 80) && player2 == false)//38 и 40 - стрелки вверх и вниз у таблице ASCII
		{
			player2 = true;
			Select2Players();
		}
		else if(c == 72 || c == 80)
		{
			player2 = false;
			Select1Player();
		}
	}
}

//выводит выбор параметров и выделяет "1 player"
void Select1Player()
{
	system("cls");
	COORD coord = { 50, 10 };
	static HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
	SetConsoleCursorPosition(hStdOut, coord);
	SetConsoleTextAttribute(hStdOut,
		BACKGROUND_GREEN | BACKGROUND_BLUE | BACKGROUND_INTENSITY | BACKGROUND_RED);
	cout << "1 PLAYER";
	SetConsoleTextAttribute(hStdOut,
		FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
	coord = { 50, 11 };
	SetConsoleCursorPosition(hStdOut, coord);
	cout << "2 PLAYERS";
}

//выводит выбор параметров и выделяет "2 players"
void Select2Players()
{
	system("cls");
	 COORD coord = { 50, 10 };
	static HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
	SetConsoleCursorPosition(hStdOut, coord);
	SetConsoleTextAttribute(hStdOut,
		FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
	cout << "1 PLAYER";
	SetConsoleTextAttribute(hStdOut,
		BACKGROUND_GREEN | BACKGROUND_BLUE | BACKGROUND_INTENSITY | BACKGROUND_RED);
	coord = { 50, 11 };
	SetConsoleCursorPosition(hStdOut, coord);
	cout << "2 PLAYERS";
}

Я понимаю где у меня ошибка, но не знаю как её решить. Кто может помочь?

Comments

[Анатолий Медведев]

Я прям заностальгировал как я подобной фигней маялся только на паскале)

Answer 1

[Анатолий Медведев]

#include <iostream>
#include <windows.h>
#include <conio.h>
using namespace std;

void Select1Player();
void Select2Players();

int main()
{
  bool player2 = false;
  Select1Player();

  char c = 0;
  while(c != 13)
  {
    c = _getch();
    if((c == 72 || c == 80) && player2 == false)//38 и 40 - стрелки вверх и вниз у таблице ASCII
    {
      player2 = true;
      Select2Players();
    }
    else if(c == 72 || c == 80)
    {
      player2 = false;
      Select1Player();
    }
  }
  return 0;
}

//выводит выбор параметров и выделяет "1 player"
void Select1Player()
{
  COORD coord = { 50, 10 };
  static HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
  SetConsoleTextAttribute(hStdOut,
    FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
  system("cls");

  SetConsoleCursorPosition(hStdOut, coord);
  SetConsoleTextAttribute(hStdOut,
    BACKGROUND_GREEN | BACKGROUND_BLUE | BACKGROUND_INTENSITY | BACKGROUND_RED);
  cout << "1 PLAYER";
  SetConsoleTextAttribute(hStdOut,
    FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
  coord = { 50, 11 };
  SetConsoleCursorPosition(hStdOut, coord);
  cout << "2 PLAYERS";
}

//выводит выбор параметров и выделяет "2 players"
void Select2Players()
{
   COORD coord = { 50, 10 };
  static HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
  SetConsoleTextAttribute(hStdOut,
    FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
  system("cls");

  SetConsoleCursorPosition(hStdOut, coord);
  SetConsoleTextAttribute(hStdOut,
    FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
  cout << "1 PLAYER";
  SetConsoleTextAttribute(hStdOut,
    BACKGROUND_GREEN | BACKGROUND_BLUE | BACKGROUND_INTENSITY | BACKGROUND_RED);
  coord = { 50, 11 };
  SetConsoleCursorPosition(hStdOut, coord);
  cout << "2 PLAYERS";
}

Comments

[Анатолий Медведев]

Но лучше замени свой код на это:

#include <iostream>
#include <windows.h>
#include <conio.h>
using namespace std;

void selectPlayer(bool palayer);

int main()
{
  bool player2 = false;
  selectPlayer(player2);

  char c = 0;
  while(c != 13)
  {
    c = _getch();
    if((c == 72 || c == 80) && player2 == false)//38 и 40 - стрелки вверх и вниз у таблице ASCII
    {
      player2 = true;
      selectPlayer(player2);
    }
    else if(c == 72 || c == 80)
    {
      player2 = false;
      selectPlayer(player2);
    }
  }
  return 0;
}

void clearScreen() {
    static HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
    SetConsoleTextAttribute(hStdOut,
      FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
    system("cls");
}

void selectPlayer(bool palayer) {
    clearScreen();
    COORD coord = { 50, 10 };
    static HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
    SetConsoleCursorPosition(hStdOut, coord);
    if(palayer) {
        SetConsoleTextAttribute(hStdOut,
          FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
        cout << "1 PLAYER";
        SetConsoleTextAttribute(hStdOut,
          BACKGROUND_GREEN | BACKGROUND_BLUE | BACKGROUND_INTENSITY | BACKGROUND_RED);
        coord = { 50, 11 };
        SetConsoleCursorPosition(hStdOut, coord);
        cout << "2 PLAYERS";
    }
    else {
        SetConsoleTextAttribute(hStdOut,
          BACKGROUND_GREEN | BACKGROUND_BLUE | BACKGROUND_INTENSITY | BACKGROUND_RED);
        cout << "1 PLAYER";
        SetConsoleTextAttribute(hStdOut,
          FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
        coord = { 50, 11 };
        SetConsoleCursorPosition(hStdOut, coord);
        cout << "2 PLAYERS";
    }
}

[CodeInside]

Анатолий Медведев: Отличный код. Спасибо.

[CodeInside]

Анатолий Медведев: ещё лучше было б в функцию selectPlayer передавать указатель на bool и изменять значение уже в функции selectPlayer, а не в main.

[Анатолий Медведев]

CodeInside: это бессмысленно.

[CodeInside]

Анатолий Медведев: почему?
Мне кажется, так лучше.

#include <iostream>
#include <windows.h>
#include <conio.h>
using namespace std;

void selectPlayer(bool* palayer);

int main()
{
	char field[3][3];//игровое поле
	bool player2 = false;
	selectPlayer(&player2);

	char c = 0;
	while (c != 13)
	{
		c = _getch();
		if (c == 72 || c == 80)//38 и 40 - стрелки вверх и вниз у таблице ASCII
			selectPlayer(&player2);
	}
	return 0;
}

void clearScreen() {
	static HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
	SetConsoleTextAttribute(hStdOut,
		FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
	system("cls");
}

void selectPlayer(bool* player) {
	clearScreen();
	COORD coord = { 50, 10 };
	static HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
	SetConsoleCursorPosition(hStdOut, coord);
	if (*player) {
		SetConsoleTextAttribute(hStdOut,
			FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
		cout << "1 PLAYER";
		SetConsoleTextAttribute(hStdOut,
			BACKGROUND_GREEN | BACKGROUND_BLUE | BACKGROUND_INTENSITY | BACKGROUND_RED);
		coord = { 50, 11 };
		SetConsoleCursorPosition(hStdOut, coord);
		cout << "2 PLAYERS";
		*player = false;
	}
	else {
		SetConsoleTextAttribute(hStdOut,
			BACKGROUND_GREEN | BACKGROUND_BLUE | BACKGROUND_INTENSITY | BACKGROUND_RED);
		cout << "1 PLAYER";
		SetConsoleTextAttribute(hStdOut,
			FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
		coord = { 50, 11 };
		SetConsoleCursorPosition(hStdOut, coord);
		cout << "2 PLAYERS";
		*player = true;
	}
}

[Анатолий Медведев]

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

Answer 2

[Алек Оним]

Попробуйте перед очищением консоли задавать стандартный черный цвет фона:)

так - работает:

void Select1Player()
{	
  COORD coord = { 50, 10 };
  HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
  SetConsoleTextAttribute(hStdOut, 0);
  system("cls");
  SetConsoleTextAttribute(hStdOut,
    BACKGROUND_GREEN | BACKGROUND_BLUE | BACKGROUND_INTENSITY | BACKGROUND_RED);
  SetConsoleCursorPosition(hStdOut, coord);
  cout << "1 PLAYER";
  
  coord.X = 50;
  coord.Y = 11;
  SetConsoleTextAttribute(hStdOut,
    FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
  SetConsoleCursorPosition(hStdOut, coord);
  cout << "2 PLAYERS";
}

//выводит выбор параметров и выделяет "2 players"
void Select2Players()
{
  COORD coord = { 50, 10 };
  HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
  SetConsoleTextAttribute(hStdOut, 0);
  system("cls");
  SetConsoleTextAttribute(hStdOut,
    FOREGROUND_GREEN | FOREGROUND_BLUE | FOREGROUND_INTENSITY | FOREGROUND_RED);
  SetConsoleCursorPosition(hStdOut, coord);
  cout << "1 PLAYER";
  
  coord.X = 50;
  coord.Y = 11;
  SetConsoleTextAttribute(hStdOut,
    BACKGROUND_GREEN | BACKGROUND_BLUE | BACKGROUND_INTENSITY | BACKGROUND_RED);
  SetConsoleCursorPosition(hStdOut, coord);
  cout << "2 PLAYERS";
}