tiles[x][y+1], если y является максимальным допустимым значением, не будет NULL, кроме как по милости. Это выходит за пределы, и как только вы выходите за пределы, все ставки прекращаются. Вы вызвали неопределенное поведение , и почти все может произойти. Даже то, что вы ожидали.

То же самое относится к сообщенному месту крушения, tiles[x - 1][y - 1].

Правка: опущено решение. Не полезно.

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

void assign_if(Type & neighbour, int x, int y)
{
    if(x >= 0 && x < width && y >= 0 && y < height)
    neighbour = tiles[x][y];
}

и назовите его

assign_if(neighbours[0], x, y+1);

и позже

assign_if(neighbours[0], x-1, y-1);

Редактировать: украсть это у Боба для полноты

Невозможно вернуть сырой массив из функции. Массив выходит из области видимости и указатель на него становится недействительным. Либо передайте массив в качестве другого параметра, либо используйте std::array или std::vector , оба из которых могут быть возвращены. Благодаря Copy Elision умный компилятор, вероятно, уменьшит затраты на копирование.

Пример:

std::array TileManager::GetNeighbours(int x, int y)
{
    std::array neighbours;
    ...
    return neighbours;
}

Редактировать по оригинальному постеру. Вот мое решение:

std::array TileManager::GetNeighbours(int c, int r)
{
    std::array neighbours;

    const int y[] = { -1, -1, -1,  1, 1, 1,  0, 0 };// 8 shifts to neighbors
    const int x[] = { -1,  0,  1, -1, 0, 1, -1, 1 };// used in functions 

    for (int i = 0; i < 8; ++i)// visit the 8 spaces around it
        if (inField(r + y[i], c + x[i]))
            neighbours[i] = tiles[r + y[i]][c + x[i]];
        else
            neighbours[i] = Tile::Type::Void;

    return neighbours;
}

bool TileManager::inField(int r, int c)
{
    if (r < 0 || r >= 25) return false;
    if (c < 0 || c >= 25) return false;
    return true;
}

Редактировать: Caveat

Этот ответ напрямую связан с решением поставленной задачи. Смотрите ответ Kaz для описания более практичного решения, которое торгует немного памяти, чтобы полностью исключить необходимость тестирования и генерации массива neighbours.