Нарушение прав доступа при чтении по адресу, указатели

@metrovlad2033 · Регистрация: 23.05.2016

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

Доброго времени суток, дорогие форумчане! Проблема в следующем: мне дано было задание сделать игру Го(кто не знает это что-то вроде китайских шахмат-шашек) в консоли с возможностью играть с компьютером! Словом у меня есть класс element, который представляет собой набор данных где есть 4 ссылки на соседние элементы(right, left, top, down), значение(value), координаты на доске 19х19(x, y), дыхания(breathes - соседние места где ничего не стоит). После того, как я реализовал возможность сделать ход со стороны игрока, мне нужно проверить, не хочет ли поставить фишку в место окруженное вражескими(по правилам это запрещено). Эта проверка выглядит так:

C++

bool checkForRools(unint irow, unint icolumn, char whoIsValue)
{
    if (board[irow][icolumn].top->value == whoIsValue && board[irow][icolumn].right->value == whoIsValue && board[irow][icolumn].down->value == whoIsValue && board[irow][icolumn].left->value == whoIsValue)//ВОТ ТА МНОГОСТРАДАЛЬНАЯ СТРОКА
    {
        cout << "Your motion doesn't match with rool N_1 - " << rools[0] << board[irow][icolumn].value;
        return false;
    }
    else
    {
        return true;
    }
}
//--------------------------------------------
//А вот определения класса элемент
class element
{
public:
    unint id;
    char value;
    element *top;
    element *left;
    element *right;
    element *down;
    unint x, y, breathes;
    void givingValue(char irow, char icolumn, char ivalue);
    element()
    {
        id = 0;
        value = ' ';
        x = 0;
        y = 0;
        breathes = 4;
    }
    ~element()
    {
        id = 0;
        value = ' ';
        x = 0;
        y = 0;
        breathes = 0;
    }
};
//--------------------------------------------------------------------------------
// Это функция, которая инициализирует указатели на соседние фишки
void element::givingValue(char irow, char icolumn, char ivalue)
{
    x = irow;
    y = icolumn;
    value = ivalue;
    top = &board[irow - 1][icolumn];
    left = &board[irow][icolumn + 1];
    right = &board[irow][icolumn - 1];
    down = &board[irow + 1][icolumn];
}

Вот что выводит:

@metrovlad2033 · 16.11.2016, 00:10 **[ТС]**

Если что ошибка в самой длинном условии if вначале!

MrGluck · 16.11.2016, 00:17

А вы уверены, что на момент обращения все указатели типа board[irow][icolumn].top/right/down действительны?

C++

1	if (board[irow][icolumn].top && board[irow][icolumn].top->value->value == ...)

Зайдите в отладку, посмотрите значения irow, icolumn. Нет ли выхода за границы массива.

@metrovlad2033 · 16.11.2016, 16:42 **[ТС]**

Я с помощью точки останова обнаружил что:

C++

1	board[irow][icolumn].top = board[irow][icolumn].right = board[irow][icolumn].down = board[irow][icolumn].left \| 0x00000000 \| NULL

Я просто не пойму, почему инициализируя все указатели в функции givingValue они остаются нулевыми?(Я точно вызываю эту функцию)

Добавлено через 6 минут
Один из моих преподавателей сказал, что решением проблемы может быть инициализация в цикле, но как по мне это ничего не поменяет! Ведь итак им всем даются значения и зачем это еще делать в цикле?... Могу еще выложить код функции, которая получает значения координат нового хода и как раз вызывает функцию инициализации указателей... Вот она:

C++

void userTurns(void)
{
    row = 0;
    CHcolumn = 0;
    char answer = ' ';
    bool check = true;
    if (userMotions == 160)
    {
        cout << "Do you want to pass your turn? (y/n)";
        cin >> answer;
        if (answer == 'y')
        {
            userPass++;
        }
    }
    if (answer != 'y')
    {
        bool result1 = true;
        bool result2 = true;
        do
        {
            cout << "\nEnter coords for your new element:\n\t";
            cout << "1) row = ";
            while (!(cin >> row) || (cin.peek() != '\n') || row < 1 || row > 19)
            {
                cin.clear();
                cin.sync();
                while (cin.get() != '\n');
                cout << "\t1) row = ";
            }
            cout << "\t2) column = ";
            while (!(cin >> CHcolumn) || (cin.peek() != '\n') || CHcolumn < 65 || CHcolumn > 83)
            {
                cin.clear();
                cin.sync();
                while (cin.get() != '\n');
                cout << "\t1) column = ";
            }
            --row;
            column = CHcolumn - 65;
            result1 = checkForPosition(row, column);
            result2 = checkForRools(row, column, computerColor);
        } while (!result1 || !result2);
            currentElement = board[row][column].id;
            board[row][column].givingValue(row, column, userColor);
        userMotions++;
    }

Если есть еще вопросы - задавайте, разьясню все что угодно!

@metrovlad2033 · 16.11.2016, 16:55 **[ТС]**

Вот подробнее об ошибке:

Нарушение прав доступа при чтении по адресу, указатели

@Mathist · 16.11.2016, 17:03

metrovlad2033,

C++

            result1 = checkForPosition(row, column);
            result2 = checkForRools(row, column, computerColor);
        } while (!result1 || !result2);
            currentElement = board[row][column].id;
            board[row][column].givingValue(row, column, userColor);

у тебя функция checkForRools вызывается раньше givingValue.

@metrovlad2033 · 16.11.2016, 20:24 **[ТС]**

БОЛЬШОЕ ВАМ СПАСИБО!!!

@metrovlad2033 0 / 0 / 0 Регистрация: 23.05.2016 Сообщений: 22
	16.11.2016, 00:10 [ТС]	2
	Если что ошибка в самой длинном условии if вначале! 0

@metrovlad2033 0 / 0 / 0 Регистрация: 23.05.2016 Сообщений: 22
	16.11.2016, 16:55 [ТС]	5
	Вот подробнее об ошибке: 0

@metrovlad2033 0 / 0 / 0 Регистрация: 23.05.2016 Сообщений: 22
	16.11.2016, 20:24 [ТС]	7
	БОЛЬШОЕ ВАМ СПАСИБО!!! 0

Опции темы

Нарушение прав доступа при чтении по адресу, указатели

Решение