Действие присваивания, элементу динамического массива, внутри функции, присваивает это значение всем копиям массива. Что делать?

Question

[yunikeil]

Помогите, пожалуйста, как избавится от таких фокусов, с int** array[i][j] внутри функций, и как оно вообще работает?

Мой код

#include <iostream>
#include <Windows.h>
using std::cout;
using std::cin;
using std::endl;

void prog(int** a) {
    int** g = a;
    int** gg = a;

    for (int i = 0; i < 3; ++i){
        for (int j = 0; j < 3; ++j) {
            cout << a[i][j] << " ";
        }
         cout << endl;
    }

    g[1][1] = 1;
    cout << endl;

    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            cout << g[i][j] << " ";
        }
        cout << endl;
    }
    cout << endl;
    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            cout << gg[i][j] << " ";
        }
        cout << endl;
    }
    cout << endl;
    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            cout << a[i][j] << " ";
        }
        cout << endl;
    }
}

int main() {
    setlocale(0, "");
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);

    int** meza = new int* [3];
    for (int i = 0; i < 3; ++i)
        meza[i] = new int[3];

    for (int i = 0; i < 3; ++i)
        for (int j = 0; j < 3; ++j)
            meza[i][j] = 0;

    prog(meza);

    cout << "\n\nEnd of program, xx: " << endl;
    return 0;
}

Comments

[Евгений Шатунов]

yunikeil , в твоем коде нет копий массива.
В твоем коде есть выделение памяти под массив массивов в главной функции, после чего идет передача указателя на эту память в функцию prog. Внутри prog этот указатель пишется в еще две переменные и все. Память массива массивов у тебя одна для всего твоего процесса.

Answer 1

[galaxy]

g и gg - это не копии массива, а копии указателя. Выделяйте память под новые массивы и копируйте с помощью memcpy

Comments

[yunikeil]

Изменил на такой код:

#include <iostream>
#include <Windows.h>
#include <malloc.h>
using std::cout;
using std::cin;
using std::endl;

void prog(int** a) {

    int** g = (int**)malloc(3 * 3 * sizeof(int));
    int** gg = (int**)malloc(3 * 3 * sizeof(int));
    memcpy(g, a, 3 * 3 * sizeof(int));
    memcpy(gg, a, 3 * 3 * sizeof(int));

    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            cout << a[i][j] << " ";
        }
        cout << endl;
    }
 
    g[1][1] = 1;
    cout << endl;

    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            cout << g[i][j] << " ";
        }
        cout << endl;
    }
    cout << endl;
    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            cout << gg[i][j] << " ";
        }
        cout << endl;
    }
    cout << endl;
    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            cout << a[i][j] << " ";
        }
        cout << endl;
    }
}

int main() {
    setlocale(0, "");
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);

    int** meza = new int* [3];
    for (int i = 0; i < 3; ++i)
        meza[i] = new int[3];

    for (int i = 0; i < 3; ++i)
        for (int j = 0; j < 3; ++j)
            meza[i][j] = 0;

    prog(meza);

    cout << "\n\nEnd of program, xx: " << endl;
    return 0;
}

проблема не ушла

[galaxy]

yunikeil, mega - это не матрица int 3x3, а массив из 3 указателей на массивы из int. Когда вы копируете просто кусок памяти по указателю a, вы копируете указатели на три массива (т.е. g и gg - указывают туда же).
Копировать нужно отдельно три массива a[0], a[1], a[2]. Ну или делайте действительно матрицу 3x3, только адресация поменяется: a[i*3+j]

Answer 2

[yunikeil]

Просто заменил с запись указателя, циклом, проблема решена

#include <iostream>
#include <Windows.h>
using std::cout;
using std::cin;
using std::endl;

void prog(int** a) {


    int** g = new int* [3];
    for (int i = 0; i < 3; ++i)
        g[i] = new int[3];

    int** gg = new int* [3];
    for (int i = 0; i < 3; ++i)
        gg[i] = new int[3];

    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            g[i][j] = a[i][j];
        }
    }

    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            gg[i][j] = a[i][j];
        }
    }

    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            cout << a[i][j] << " ";
        }
        cout << endl;
    }
 
    g[1][1] = 1;
    cout << endl;

    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            cout << g[i][j] << " ";
        }
        cout << endl;
    }
    cout << endl;
    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            cout << gg[i][j] << " ";
        }
        cout << endl;
    }
    cout << endl;
    for (int i = 0; i < 3; ++i) {
        for (int j = 0; j < 3; ++j) {
            cout << a[i][j] << " ";
        }
        cout << endl;
    }
}

int main() {
    setlocale(0, "");
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);

    int** meza = new int* [3];
    for (int i = 0; i < 3; ++i)
        meza[i] = new int[3];

    for (int i = 0; i < 3; ++i)
        for (int j = 0; j < 3; ++j)
            meza[i][j] = 0;

    prog(meza);

    cout << "\n\nEnd of program, xx: " << endl;
    return 0;
}