ダブルポインタについて

Question

main関数で生成したダブルポインタ変数を、関数の引数に渡して、動的なサイズの配列を作って、main関数からデータアクセスできるようなプログラムを書きたいのですが、
main関数のprintfのところで、index１個目のデータにはアクセスできるのですが、
その後のindexにアクセスすると、セグメンテーション違反になってしまいます。
どのように、アクセスすれば良いのでしょうか。

#include <stdio.h>
#include <malloc.h>

void setData(int** data, int size);

int main(void) {
    int** data = new int*;
    setData(data, 10);
    for (int i=0; i<10; i++) {
        printf("%d\n", *data[i]);
    }
}

void setData(int** data, int size) {
    *data = (int*)malloc(sizeof(int) * size);
    for (int i=0; i<size; i++) {
        (*data)[i] = i+1;
    }
}

Answer 1

普通に考えると

#include <stdio.h>
#include <malloc.h>

void setData(int** data, int size);

int main(void) {
    int* data;  // ここと
    setData(&data, 10);
    for (int i=0; i<10; i++) {
        printf("%d\n", data[i]); // ここが違う
    }
}

void setData(int** data, int size) {
    *data = (int*)malloc(sizeof(int) * size);
    for (int i=0; i<size; i++) {
        (*data)[i] = i+1;
    }
}

もしくは

#include <stdio.h>
#include <malloc.h>

void setData(int** data, int size);

int main(void) {
    int** data = new int*;
    setData(data, 10);
    for (int i=0; i<10; i++) {
        printf("%d\n", (*data)[i]); // こうか
        printf("%d\n", data[0][i]); // これでもよい
    }
}

void setData(int** data, int size) {
    *data = (int*)malloc(sizeof(int) * size);
    for (int i=0; i<size; i++) {
        (*data)[i] = i+1;
    }
}

演算子*は演算子[]よりも優先度が低いので後回しにされるので
*data[i]は*(data[i])と等価です。

---

int **data = new int*;の是非
個人的にはなしです。解放がめんどくさいですし、今回の場合には必要性が見えません。
必要な場合でも、可能な限りstd::vectorやstd::arrayでどうにかできないか考えるべきでしょう。

というわけでC++風に

#include <stdio.h>
#include <vector>

void setData(std::vector<int> &data, size_t size);

int main(void) {
    std::vector<int> data;
    setData(data, 10);
    for(int& x : data)
        printf("%d\n", x);
}
void setData(std::vector<int> &data, size_t size){
    data = std::vector<int>(size);
    for (size_t i = 0; i < size; i++)
        data[i] = i + 1;
}

Comments

[usop]

>というわけでC++風に　のコード
本当にそのとおりだと思います！
仕事で、C言語の関数(setDataの部分。共有用に改変)を使わなければならず、
データの動的読み込み部分を見ると、あれあれ、ダブルポインタだ。あれ、こういうアクセスの仕方でよかったんだっけか。みたいになって笑

個人的には、ポインタに配列がつくと、なんだかよくわからなくなってしまうので、
*(*data+i) = i+1;
のほうが、解りやすい気がしてます。

Answer 2

自己解決しました。
☓ printf("%d\n", *data[i]);
○ printf("%d\n", (*data)[i]);