配列を生成する関数の引数として使用する配列のポインターを使用しようとしています。
void generateArray(int *a[], int *si){
srand(time(0));
for (int j=0;j<*si;j++)
*a[j]=(0+Rand()%9);
} //end generateArray;
int main() {
const int size=5;
int a[size];
generateArray(&a, &size);
return 0;
} //end main
しかし、これをコンパイルすると、次のメッセージが表示されます。
cannot convert `int (*)[5]' to `int**' for argument `1' to `void generateArray(int**, int*)'
あなたはそれを過度に複雑にしている-それはする必要があります:
void generateArray(int *a, int si)
{
for (int j = 0; j < si; j++)
a[j] = Rand() % 9;
}
int main()
{
const int size=5;
int a[size];
generateArray(a, size);
return 0;
}
関数にパラメーターとして配列を渡すと、配列の最初の要素へのポインターに減衰します。そのため、通常、配列にポインターを渡す必要はありません。
int *a[]
は、関数パラメーターとして使用される場合(ただし、通常の宣言では使用されません)、配列へのポインターではなく、ポインターへのポインターです(通常の宣言では、ポインターの配列です)。配列へのポインターは次のようになります。
int (*aptr)[N]
ここで、N
は特定の正の整数(変数ではない)です。
関数をテンプレートにすると、それを行うことができ、配列のサイズを渡す必要さえありません(自動的に推論されるため):
template<size_t SZ>
void generateArray(int (*aptr)[SZ])
{
for (size_t i=0; i<SZ; ++i)
(*aptr)[i] = Rand() % 9;
}
int main()
{
int a[5];
generateArray(&a);
}
参照を取ることもできます:
template<size_t SZ>
void generateArray(int (&arr)[SZ])
{
for (size_t i=0; i<SZ; ++i)
arr[i] = Rand() % 9;
}
int main()
{
int a[5];
generateArray(a);
}
配列を既に関数に渡すときにdecay to pointersであるため、配列を生成する関数に渡すために配列へのポインタを取る必要はありません。パラメーターint a[]
を作成し、関数内で通常の配列として使用するだけで、渡された配列に変更が加えられます。
void generateArray(int a[], int si) {
srand(time(0));
for (int j=0;j<*si;j++)
a[j]=(0+Rand()%9);
}
int main(){
const int size=5;
int a[size];
generateArray(a, size);
return 0;
}
補足として、サイズをポインタで渡す必要はありません。関数内でサイズを変更しないためです。さらに、定数へのポインターを、非定数へのポインターを予期するパラメーターに渡すことはお勧めできません。
これが役立つと思います。
関数の引数として渡されると、配列はポインターと同じように機能します。したがって、それらを参照する必要はありません。次のように入力します:int x[]
または int x[a]
。両方の方法が機能します。コンラッド・ルドルフが言っていたのと同じことだと思います。