Ошибка инициализации массива под шахматную доску

@elykov · Регистрация: 17.02.2017

Студворк — интернет-сервис помощи студентам

Здравствуйте, решил написать шахматы, застрял на одном месте: при инициализации массива происходит что-то непонятное и при выводе значений выдает совсем не то либо исключения. На данный момент заполняется весь массив вместо 2 строк. Массив здесь - это шахматная доска; он динамический, родительского класса, в него должны записываются элементы - классы-наследники, и все действия в программе должны проходить с ним. Буду благодарен за объяснение такого поведения массива либо другой путь, всем добра)
Вот код

C++
1
2
3
4
5
6
7
8
9
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
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
#include <conio.h>
#include <iomanip>
#include <iostream>
 
using namespace std;
 
class Coord
{
public:
    Coord() {}
    Coord(int _x, int _y);
    int x, y;
    ~Coord() {}
}
 
class Figure : public Coord
{
public:
    bool color; //true - white
    Figure() {}
    Figure(int _x, int _y, bool _color) : Coord(_x, _y) { color = _color; }
    ~Figure() {}
public:
    virtual void newPosition(int i, int j);
    virtual int whichFigure() { return 0; }
}
 
class Pawn : public Figure // пешка
{
public:
    Pawn() {}
    ~Pawn() {}
    Pawn(int _x, int _y, bool _color) : Figure(_x, _y, _color) {}
    int posibleMoves(Coord *coord, Figure **figures);
    int whichFigure() { return 1; }
}
 
using namespace std;
 
int main(int argc, char* argv[])
{
    bool b = true;
    
    Figure **a;
    a = new Figure*[8];
    
    for (int i = 0; i < 8; i++) {   // (3)
        a[i] = new Figure[8];     // инициализация указателей
    }
 
    for (int i = 0; i < 8; i++) {   // (3)
        a[1, i] = new Pawn(1, i, b);     // инициализация указателей
    }
 
    b = false;
 
    for (int i = 0; i < 8; i++) {   // (3)
        a[6, i] = new Pawn(6, i, b);     // инициализация указателей
    }
 
    int value;
    for (int i = 0; i < 8; i++)
    {
        for (int j = 0; j < 8; j++)
        {
            if (a[i, j] == nullptr) continue;
            value = a[i, j]->whichFigure();
            switch (value)
            {
                case 0:
                {
                    cout << setw(6) << "empty";
                    break;
                }
                case 1:
                {
                    cout << setw(6) << "pawn";
                    break;
                }
            }
            
        }
        cout << endl;
    }
 
    _getch();
    return 0;
}

@Encephalopathy · 24.07.2017, 21:23

У вас

C++
1
Coord(int _x, int _y);

объявлена но неопределенна.Так и должно быть?

@igdev · 24.07.2017, 21:37

elykov, Думаю, должно быть так:

C++
1
2
3
4
5
Coord(int _x, int _y)
{
    x = _x;
    y = _y;
}

@elykov · 24.07.2017, 22:16 **[ТС]**

да, я решил весь код не вставлять, а только самое главное

@igdev · 25.07.2017, 13:56

Сообщение от elykov

да, я решил весь код не вставлять, а только самое главное

И? Проблема решена или еще нет?

@elykov · 25.07.2017, 16:47 **[ТС]**

Сообщение от igdev

И? Проблема решена или еще нет?

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

@igdev · 25.07.2017, 16:51

elykov, Выложите весь ваш код.

@elykov · 25.07.2017, 20:41 **[ТС]**

Сообщение от igdev

Выложите весь ваш код.

C++
1
2
3
4
5
6
7
8
9
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
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
#include "stdafx.h"
#include <conio.h>
#include <iomanip>
#include <iostream>
 
using namespace std;
 
class Coord
{
public:
    Coord() {}
    Coord(int _x, int _y)
    {
        x = _x;
        y = _y;
    }
    int x, y;
    ~Coord() {}
};
 
class Figure : public Coord
{
public:
    bool color; //true - white
    Figure() {}
    Figure(int _x, int _y, bool _color) : Coord(_x, _y) { color = _color; }
    ~Figure() {}
public:
    virtual void newPosition(int i, int j)
    {
        x = i;
        y = j;
    }
    virtual int whichFigure() { return 0; }
};
 
class Pawn : public Figure // пешка
{
public:
    Pawn() {}
    ~Pawn() {}
    Pawn(int _x, int _y, bool _color) : Figure(_x, _y, _color) {}
    int posibleMoves(Coord *coord, Figure **figures);
    int whichFigure() { return 1; }
};
 
using namespace std;
 
int main(int argc, char* argv[])
{
    bool b = true;
 
    Figure **a;
    a = new Figure*[8];
 
    for (int i = 0; i < 8; i++) {   // (3)
        a[i] = new Figure[8];     // инициализация указателей
    }
 
    for (int i = 0; i < 8; i++) {   // (3)
        a[1, i] = new Pawn(1, i, b);     // инициализация указателей
    }
 
    b = false;
 
    for (int i = 0; i < 8; i++) {   // (3)
        a[6, i] = new Pawn(6, i, b);     // инициализация указателей
    }
 
    int value;
    for (int i = 0; i < 8; i++)
    {
        for (int j = 0; j < 8; j++)
        {
            if (a[i, j] == nullptr) continue;
            value = a[i, j]->whichFigure();
            switch (value)
            {
            case 0:
            {
                      cout << setw(6) << "empty";
                      break;
            }
            case 1:
            {
                      cout << setw(6) << "pawn";
                      break;
            }
            }
 
        }
        cout << endl;
    }
 
    _getch();
    return 0;
}

@RamonN · 25.07.2017, 20:57

У меня ваш код работает вот так. Что он должен выводить?

@igdev · 25.07.2017, 21:36

Не правильная инициализация массива:

C++
1
2
3
for (int i = 0; i < 8; i++) {   // (3)
        a[1, i] = new Pawn(1, i, b);     // инициализация указателей
    }

Это не паскаль.
Должно быть:

C++
1
2
3
for (int i = 0; i < 8; i++) {   // (3)
        a[0][i] = new Pawn(1, i, b);     // инициализация указателей
    }

В С++ индексация массивов начинается с нуля.

Добавлено через 1 минуту
Здесь почти правильно:

C++
1
2
3
 for (int i = 0; i < 8; i++) {   // (3)
        a[6, i] = new Pawn(6, i, b);     // инициализация указателей
    }

Однако, должно быть следующее:

C++
1
2
3
 for (int i = 0; i < 8; i++) {   // (3)
        a[6][i] = new Pawn(6, i, b);     // инициализация указателей
    }

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

Сообщение от igdev

Это не паскаль.
Должно быть:

C++
1
2
3
for (int i = 0; i < 8; i++) { * // (3)
* * * * a[0][i] = new Pawn(1, i, b); * * // инициализация указателей
* * }

Ошибся. Это же ряд пешек. Должно быть так:

C++
1
2
3
for (int i = 0; i < 8; i++) {   // (3)
        a[1][i] = new Pawn(1, i, b);     // инициализация указателей
    }

Добавлено через 3 минуты
Здесь также:

C++
1
2
if (a[i, j] == nullptr) continue;
            value = a[i, j]->whichFigure();

Исправляем:

C++
1
2
if (a[i][j] == nullptr) continue;
            value = a[i][j]->whichFigure();

@elykov · 28.07.2017, 01:48 **[ТС]**

igdev, у меня ругается компилятор, когда я делаю так

C++
1
2
3
for (int i = 0; i < 8; i++) { * // (3)
* * * * a[1][i] = new Pawn(1, i, b); * * // инициализация указателей
* * }

вместо

C++
1
2
3
for (int i = 0; i < 8; i++) {   // (3)
        a[1, i] = new Pawn(1, i, b);     // инициализация указателей
    }

а именно так:
error C2679: бинарный "=": не найден оператор, принимающий правый операнд типа "Pawn *" (или приемлемое преобразование отсутствует)

и так при всех заменах [i, j] на [i][j]

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

Сообщение от RamonN

Что он должен выводить?

Должен выводить
empty empty empty empty empty empty empty empty
pawn pawn pawn pawn pawn pawn pawn pawn
empty empty empty empty empty empty empty empty
empty empty empty empty empty empty empty empty
empty empty empty empty empty empty empty empty
empty empty empty empty empty empty empty empty
pawn pawn pawn pawn pawn pawn pawn pawn
empty empty empty empty empty empty empty empty

@igdev · 28.07.2017, 11:21

Я с двумерными массивами указателей на производные классы никогда не работал.

И сейчас что-то не получается что-то скрутить до кучи.
С вектором указателей все проще:

C++
1
2
3
4
5
6
std::vector<Figure*>figures;
for (int i = 0; i < 8; i++) {   // (3)
      figures.push_back(new Pawn(1, i, b));     // инициализация указателей
}
for (int  i = 0; i < 8; i++)
     delete figures[i];

И все хорошо. С двумерным массивом указателей, по идеи, все тоже должно быть хорошо. Но, видимо, я что-то не знаю.
Может, gru74ik, GbaLog-, Renji смогут обьяснить начинающим, как правильно сделать двумерный массив указателей на обьекты производных классов?

@GbaLog- · 28.07.2017, 11:55

C++
1
2
3
4
5
6
7
Figure ** figures = new Figure*[8];
for (int i = 0; i < 8; ++i)
  figures[i] = new Pawn(1, i, b);
 
for (int i = 0; i < 8; ++i)
  delete figures[i];
delete [] figures;

@proghelp4you · 28.07.2017, 11:59

igdev, а что мешает использовать двумерный вектор указателей на Figure? Зачем усложнять себе задачу?

@John Prick · 28.07.2017, 12:04

Не по теме:

Это же шахматы, там же нотация есть :)

C++
1
2
3
4
5
6
std::map<std::string, std::map<int, Figure*> > board;
board["a"][2] = new Pawn;
board["b"][2] = new Pawn;
// ...
board["a"][1] = new Rook;
// ...

Интересно, будет такое работать? :)

@igdev · 28.07.2017, 12:19

proghelp4you,

Сообщение от proghelp4you

а что мешает использовать двумерный вектор указателей на Figure? Зачем усложнять себе задачу?

Собственно, ничего. Однако, я не являюсь ТС. Я помогаю человеку (ТС) разобраться. У меня просто возникли некоторые трудности. Я же тоже еще только учусь.

Добавлено через 3 минуты
GbaLog-,

Сообщение от GbaLog-

C++
1
2
3
4
5
6
Figure ** figures = new Figure*[8];
for (int i = 0; i < 8; ++i)
* figures[i] = new Pawn(1, i, b);
for (int i = 0; i < 8; ++i)
* delete figures[i];
delete [] figures;

В вашем примере вся доска будет заставлена пешками. ТС нужно, чтобы пешками были заставлены только вторая сверху строка (с индексом 1) и вторая снизу строка (с индексом 6).

@GbaLog- · 28.07.2017, 12:22

Не по теме:

Сообщение от John Prick

Интересно, будет такое работать?

будет.
только ключ в первой мапе лучше сделать char.

Сообщение от igdev

как правильно сделать двумерный массив указателей на обьекты производных классов?

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

Добавлено через 54 секунды

Сообщение от igdev

В вашем примере вся доска будет заставлена пешками. ТС нужно, чтобы пешками были заставлены только вторая сверху строка (с индексом 1) и вторая снизу строка (с индексом 6).

нафига там вообще динамический массив?
шахматная доска имеет свойство менять размеры?

@igdev · 28.07.2017, 12:31

Сообщение от GbaLog-

нафига там вообще динамический массив?
шахматная доска имеет свойство менять размеры?

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

@GbaLog- · 28.07.2017, 12:37

что-то я тупанул, как раз понятно, зачем там динамический массив.
полиморфизм же.

Новые блоги и статьи Все статьи Все блоги /
Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .	Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .
Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и напряжениями. Надо найти токи в ветвях. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и решает её. Последовательность действий:. . .	Новый CodeBlocs. Версия 25.03 palva 04.01.2026 Оказывается, недавно вышла новая версия CodeBlocks за номером 25. 03. Когда-то давно я возился с только что вышедшей тогда версией 20. 03. С тех пор я давно снёс всё с компьютера и забыл. Теперь. . .	Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .

@elykov 0 / 0 / 0 Регистрация: 17.02.2017 Сообщений: 7
	24.07.2017, 22:16 [ТС]
	да, я решил весь код не вставлять, а только самое главное 0

@igdev 79 / 67 / 28 Регистрация: 22.04.2016 Сообщений: 384
	25.07.2017, 16:51
	elykov, Выложите весь ваш код. 0

@RamonN 32 / 32 / 19 Регистрация: 13.07.2011 Сообщений: 140
	25.07.2017, 20:57
	У меня ваш код работает вот так. Что он должен выводить? Миниатюры 0

@proghelp4you 62 / 62 / 34 Регистрация: 09.07.2017 Сообщений: 475
	28.07.2017, 11:59
	igdev, а что мешает использовать двумерный вектор указателей на Figure? Зачем усложнять себе задачу? 0

@GbaLog- Любитель чаепитий 3745 / 1801 / 566 Регистрация: 24.08.2014 Сообщений: 6,020 Записей в блоге: 1
	28.07.2017, 12:37
	что-то я тупанул, как раз понятно, зачем там динамический массив. полиморфизм же. 0