VC++　配列のポインタについて

以下のようなプログラムを作りましたがうまくいきません。

関数ｂにて配列の一番最初の出力を行ってますが
printfの最初のみ正しく出力があり
それ以降は値がおかしくなります。

その原因として
ローカル変数であるA
が関数aの終了とともに消えてしまっているのではないかと考えています。
アドバイス頂けたらと思います。
よろしくお願いいたします。

long* a();
void b(long* hairetu);

int main(){

long* hairetu;

hairetu = a();

b(hairetu)

}

a(){

int A[100];
for (int i = 0; i < 100;i++ ){

A[i] = i;
}

return A;
}

b(long* hairetu){

printf("配列 %d\n",hairetu[0]);
printf("配列 %d\n",hairetu[0]);
printf("配列 %d\n",hairetu[0]);
}

行動規範の内容に同意します

回答2件

a(){
int A[100];
for (int i = 0; i < 100;i++ ){
A[i] = i;
}
return A;
}

これは期待通りには動作しないでしょう。主だった問題点は次の二つです。

関数内で寿命を終える変数のアドレスを返していること
返り値の型を省略していること (intがデフォルトの型として用いられています)

いくつか警告が出ている筈です。無視して実行してはいけません。

簡単には、次のように配列を呼び出し側で用意してやると良いでしょう。

C
1int main(void) {
2    long hairetu[100];
3    a(hairetu, 100);
4    b(hairetu);
5}
6
7void a(long *hairetu, size_t size) {
8    ...
9}

コードの貼り方について

teratailには、コードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押して下さい。

投稿2020/01/26 11:27

編集2020/01/26 11:30

LouiS0616

総合スコア35660

KKKM

2020/01/26 11:41

ご回答いただきましてありがとうございます。今回サンプルとして乗せた部分は実際に使っていたソースの要約？に近い形になるように手打ちしたものです。見直ししたつもりが細部の記入ミスがありました。＞＞返り値の型を省略していることこれは実際には生じていませんでした。従ってやはり原因は＞＞関数内で寿命を終える変数のアドレスを返していることにあるようです。またこの問題の改善策として、 Aをstaticにしようか,グローバルにしようか悩んでいましたか main 関数内で定義する方法は思いつきませんでした。大変貴重なご意見有難うございます。 mainで宣言するように変更しようと思います。最後に主題とは異なりますが。先ほどのAをstaticにした場合にはデータが保持されますがもしaの関数を何度も呼び出し、尚且つ代入する範囲を狭めたり広めたりした場合には前の値が残っている可能性がある（staticは初期化されないため）と思うので staticはやめたほうがいいと思うのですが認識はあっているでしょうか。

LouiS0616

2020/01/26 11:59 編集

まず表面的な指摘から。『staticは初期化されない』というのは誤りです。寿命があるうちに領域内のデータが勝手に書き換わらないのはどんな変数でも同じです。static変数はプログラム全体と同じ寿命を持ちます。一方、関数内で宣言した配列は初期化されていませんので、心配なようであれば long hairetu = {} などと明示的に書く必要があります。この場合全要素が0として初期化されます。 --- > もしaの関数を何度も呼び出し、尚且つ代入する範囲を狭めたり広めたりした場合確かに同じ配列を何度も使いまわしたいのであれば、static変数は避けた方が良いでしょう。呼び出し側で必要な長さの配列を用意するのが無難です。あるいは、C++ならば可変長配列(vector)を簡単に利用できますので、それを使っても良いです。コーディングが楽になります。

行動規範の内容に同意します

こんにちは。

C++には便利な機能があるので使いましょう。

C++
1#include <cstdio>
2#include <array>
3
4typedef std::array<int, 100> Hairetsu;
5
6Hairetsu a();
7void b(Hairetsu const& hairetu);
8
9int main()
10{
11    Hairetsu hairetu = a();
12    b(hairetu);
13}
14
15Hairetsu a()
16{
17    Hairetsu A;
18    for (int i=0; i < A.size(); ++i)
19    {
20        A[i] =  A.size()-i;
21    }
22
23    return A;
24}
25
26void b(Hairetsu const& hairetu)
27{
28    printf("配列 %d\n",hairetu[0]);
29    printf("配列 %d\n",hairetu[0]);
30    printf("配列 %d\n",hairetu[0]);
31}