1

Вот есть функция:

int checkField(int chessPlate[8][8], int x, int y) {
    int near = 4;
    if (chessPlate[x][y + 1] == 1) {
        near--;
        cout << "1";
    }
    if (chessPlate[x][y - 1] == 1) {
        near--;
        cout << "2";
    }
    if (chessPlate[x + 1][y] == 1) {//здесь что-то не так
        near--;
        cout << "3";
        cout << chessPlate[x + 2][y] << " " << x + 1;
    }
    if (chessPlate[x - 1][y] == 1) {
        near--;
        cout << "4";
    }
    return near;
}

есть матрица 8х8,и в позиции 7,7 есть 1 (остальные ячейки 0).В функцию передаю эту матрицу,и числа 7 и 7.Должно вернуть 4,но возвращает 3.Как так?

Вот весь код:

#include "pch.h"
#include <iostream>
using namespace std;
int checkField(int chessPlate[8][8], int x, int y) {
    int near = 4;
    if (chessPlate[x][y + 1] == 1) {
        near--;
        cout << "1";
    }
    if (chessPlate[x][y - 1] == 1) {
        near--;
        cout << "2";
    }
    if (chessPlate[x + 1][y] == 1) {//здесь что-то не так
        near--;
        cout << "3";
        cout << chessPlate[x + 2][y] << " " << x + 1;
    }
    if (chessPlate[x - 1][y] == 1) {
        near--;
        cout << "4";
    }
    return near;
}
int main()
{
    int NumOfPix;
    cin >> NumOfPix;
    int chessPlate[8][8] = { {0,0,0,0,0,0,0,0},
    {0,0,0,0,0,0,0,0},
    {0,0,0,0,0,0,0,0},
    {0,0,0,0,0,0,0,0},
    {0,0,0,0,0,0,0,0},
    {0,0,0,0,0,0,0,0},
    {0,0,0,0,0,0,0,0},
    {0,0,0,0,0,0,0,0} };
    int **Pos= new int*[NumOfPix];
    for (int i = 0; i < NumOfPix; i++)//записываем в матрицу значения
    {
        Pos[i] = new int[2];
        cin >> Pos[i][0] >> Pos[i][1];
        Pos[i][0]--;
        Pos[i][1]--;
        chessPlate[Pos[i][0]] [Pos[i][1]] = 1;

    }
    for (int i = 0; i < 8; i++)
    {
        for (int j = 0; j < 8; j++)
        {
            cout << chessPlate[i][j] << " ";
        }
        cout << endl;
    }
    int res = 0;
    for (int i = 0; i < NumOfPix; i++)//выщитываем периметр
    {
        res+=checkField(chessPlate, Pos[i][0], Pos[i][1]);
    }
    cout << res; 
}

res должен быть равен 4 при вводе 1,8 8

4
  • Достаточно просто int chessPlate[8][8] = {}; чтобы инициализировать всю матрицу нулями
    – IR42
    23 сен 2018 в 17:26
  • Что такое "ввод 1,8 8"? И куда же и кем читается эта запятая? 23 сен 2018 в 17:29
  • Если вы сами понимаете что написали и чего хотите добиваться, будьте добры хотябы рассказать какое действие должна выполнять ваша функция. А то с таким кодом вы легко выходите за пределы массива, и непонятно для чего вам нужен двумерный массив размером NumOfPix * 2 и много другое. А результат вы получаете такой, поскольку выполняется одно из условий в функции 23 сен 2018 в 17:41
  • функция должна возвращать количество ячеек с нулями вокруг заданной(макс.число - 4)
    – OneOrigiN
    23 сен 2018 в 17:49

1 ответ 1

0

Вы пишете, что передаете в функцию матрицу и числа 7 и 7 - все верно? Смотрим что вы написали в той функции:

if (chessPlate[x][y + 1] == 1) {

это вторая строчка. Подставляем ваши значения:

if (chessPlate[7][7 + 1] == 1) {

как видим вы пытаетесь обратиться к элементу матрицы [7][8] но ваша матрица размерностью 8 на 8, тобишь самый нижний правый элемент это [7][7]. Тобишь у вас налицо выход за пределы массива.

5
  • Да,но выход за нраницы массива не равняется же 1,верно?
    – OneOrigiN
    23 сен 2018 в 17:18
  • Если бы вы были внимательны,то у меня 2 выхода за границы массива,а вычитается только один раз
    – OneOrigiN
    23 сен 2018 в 17:21
  • @One-Original-Name: Почему не равняется? Выход за пределы массива - неопределенное поведение, которое может делать что угодно. Что там у вас "вычитается один раз" значения не имеет. Неопределенное поведение - дальше смотреть нечего. 23 сен 2018 в 17:31
  • я ж могу добавить условие выхода за границу массива?Если да,то подскажите как
    – OneOrigiN
    23 сен 2018 в 17:37
  • @One-Original-Name вам следует вообще не выходить за пределы массива. Ну а что касается количества выходов за пределы - я не занимался подсчетом, да и не анализировал код ниже того, что написал - этого достаточно, чтобы понять в чем проблема. 23 сен 2018 в 18:25

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.