Проблемы с реализацией игры "Морской бой": найти и исправить ошибки в коде

@Алексаша · Регистрация: 30.03.2016

Доброго времени суток. У меня вопрос по игре "Морской бой". Я вывел поле с написанными координатами сверху и слева. Затем предлагаю пользователю ввести координаты, на этих коорд. должен вывестись крестик. Но он не выводится!!! Пожалуйста, если не трудно, помогите). Вот код:

C++

#include <iostream>
 
using namespace std;
 
 
// Глобальные переменные
const int SIZE1 = 9;
const int SIZE2 = 7;
 
char field[SIZE1][SIZE2];
 
bool GAME = true;
 
int num1;   //Номер колонки
int num2;  //номер строки
 
int mas1[SIZE2] = { 1, 2, 3, 4, 5, 6, 7 };
 
 
 
void DrawField()
{
    for (int i = 0; i < SIZE2; ++i)
    {
        cout << ' ' << mas1[i];
    }
 
    cout << endl;
 
    for (int i = 0; i < SIZE1; ++i)
    {   
        cout << (int)(i + 1);
        for (int j = 0; j < SIZE2; ++j)
        {
            
            field[i][j] = '.';
            cout << field[i][j] << ' ';
        }
        cout << endl;
 
    }
 
    
 
}
 
 
int main()
{
    setlocale(0, "");
 
    
    DrawField();
 
    while (GAME)
    {
        cout << "Введите номер колонки: ";
        cin >> num1;
 
        cout << "Введите номер строки: ";
        cin >> num2;
 
        field[num1][num2] = 'x';
 
        cout << endl;
 
        DrawField();
    }
 
    system("pause");
    return 0;
}

Не ругайте за глупые ошибки, если они будут. Я только учусь.

Это не весь код, чтоб игра была полностью сделана, но я потом допишу. Пока с этим помогите)).

@_Ivana · 08.01.2017, 19:57

C++

1 2	field[i][j] = '.'; cout << field[i][j] << ' ';

и зачем вам филд?

@dailydose · 08.01.2017, 20:05

C++

#include <iostream>
#include <vector>
 
using namespace std;
 
 
// Глобальные переменные
const int SIZE1 = 9;
const int SIZE2 = 7;
 
//char field[SIZE1][SIZE2] = { '.' };
std::vector< std::vector<char> > field(SIZE1, std::vector<char>(SIZE2, '.'));
 
bool GAME = true;
 
int num;   //Номер колонки
int symb;  //номер строки
 
int mas1[SIZE2] = { 1, 2, 3, 4, 5, 6, 7 };
 
 
 
void DrawField()
{
    std::system("cls||clear");
 
    for (int i = 0; i < SIZE1; ++i)
    {
        cout << (int)(i + 1);
        for (int j = 0; j < SIZE2; ++j)
        {
       //   field[i][j] = '.';
            cout << field[i][j] << ' ';
        }
        cout << endl;
 
    }
 
 
 
}
 
 
int main()
{
    setlocale(0, "");
 
 
    DrawField();
 
    while (GAME)
    {
        cout << "Введите номер колонки: ";
        cin >> num;
 
        cout << "Введите номер строки: ";
        cin >> symb;
 
        field[num][symb] = 'x';
 
        cout << endl;
 
        DrawField();
    }
 
    system("pause");
    return 0;
}

Добавлено через 2 минуты

Сообщение от Алексаша

Это не весь код, чтоб игра была полностью сделана, но я потом допишу. Пока с этим помогите)).

Это правильный подход. Сам всегда так же делаю. Вроде это называется TDD, не люблю терминами кидаться)

P.S. Рекомендую освоить отладку.

@Алексаша · 09.01.2017, 20:01 **[ТС]**

dailydose, Спасибо огромное)))

Добавлено через 3 часа 5 минут
Можно ещё просьбу? Как сделать тоже самое, но без векторов. Мне как то с массивами удобнее, а до этого делал, не получалось(((

Добавлено через 20 часов 45 минут
У меня опять проблема((((. Почему, когда ввожу 7 и 8, или 7 и 9, то происходит ошибка? Вот код:

C++

#include <iostream>
#include <windows.h>
#include <vector>
 
using namespace std;
 
 
// Глобальные переменные
const int SIZE1 = 9;
const int SIZE2 = 7;
 
int MaxShot = 10;  //Максимальное кол-во выстрелов
int Shots;              //Сделанные выстрелы
 
//char field[SIZE1][SIZE2] = { '.' };
std::vector< std::vector<char> > field(SIZE1, std::vector<char>(SIZE2, '.'));
 
bool GAME = true;
 
int num;   //Номер колонки
int symb;  //номер строки
 
int mas1[SIZE2] = { 1, 2, 3, 4, 5, 6, 7 };
 
 
 
void Welcome()
{
    int a;
 
    for (int i = 0;; ++i)
    {
        cout << endl;
 
        cout << "Всем привет, это игра Морской Бой. Правила такие: " << endl;
        cout << "Где то на море спрятан вражеский корабль! Ваша задача за определённое кол-во \nвыстрелов и уничтожить его. Удачи!" << endl;
        cout << endl;
 
        cout << "Введите 1, чтобы начать игру: ";
        cin >> a;
 
        if (a == 1)
        {
            break;
        }
 
        else{
            continue;
        }
 
    }
}
 
void DrawField()
{
    
    std::system("cls||clear");
 
    for (int i = 0; i < SIZE2; ++i)
    {
        cout << ' ' << mas1[i];
    }
 
    cout << endl;
 
    for (int i = 0; i < SIZE1; ++i)
    {
        cout << (int)(i + 1);
        for (int j = 0; j < SIZE2; ++j)
        {
            //   field[i][j] = '.';
            cout << field[i][j] << ' ';
        }
        cout << endl;
 
    }
 
 
 
}
 
 
int main()
{
    setlocale(0, "");
 
    Welcome();
 
    DrawField();
 
    while (GAME)
    {
        cout << "                                              Вы использовали: " << Shots << " выстрелов" << endl;
 
        cout << "Введите номер колонки: ";
        cin >> num;
 
        cout << "Введите номер строки: ";
        cin >> symb;
 
        field[num - 1][symb - 1] = 'x';
 
        Shots++;
 
        if (Shots >= MaxShot)
        {
            cout << "Вы использовали все выстрелы. Пока" << endl;
            break;
        }
 
        if (num == 3 && symb == 3)
        {
            field[num - 1][symb - 1] = '#';
        }
 
        cout << endl;
 
        DrawField();
    }
 
    system("pause");
    return 0;
}

@dailydose · 10.01.2017, 08:18

Сообщение от Алексаша

Можно ещё просьбу? Как сделать тоже самое, но без векторов. Мне как то с массивами удобнее, а до этого делал, не получалось(((

да, можно.
всё тоже самое будет
я просто закомментировал char field[][] и сделал на векторах, чтобы сразу же проинициализировать его точками.
можете вернуть свой вариант с char и отдельно его заполнить точками в цикле, только не в функции вывода

@dailydose 669 / 215 / 88 Регистрация: 21.07.2016 Сообщений: 1,036 Записей в блоге: 2
	10.01.2017, 08:18	5
	Сообщение от Алексаша Можно ещё просьбу? Как сделать тоже самое, но без векторов. Мне как то с массивами удобнее, а до этого делал, не получалось((( да, можно. всё тоже самое будет я просто закомментировал char field[][] и сделал на векторах, чтобы сразу же проинициализировать его точками. можете вернуть свой вариант с char и отдельно его заполнить точками в цикле, только не в функции вывода 0

Опции темы