Q&A
sortinit(&arr)直してみたのですが"セグメンテーション違反です"というエラーが出ました。
arrが未定義で捜査しているからかなと思ったのですがダメでした。
ですが、 arr = sortinit(); でいったらちゃんと実行できました。ありがとうございます。
もしよろしければ&arrでなぜセグメンテーション違反がでるのか教えていただければ嬉しいです。
c言語でプログラムを作っているのですが、構造体の中のポインタを参照するとセグメンテーション違反と出ました。
調べてみたらこんな記事があって解決方法も載ってました。
http://oshiete.goo.ne.jp/qa/3144134.html
この中の4の回答者様と同じようにたぶん実態がないんだと思います。
そこでいろいろ試してみてこういうものを作ったのですが、こうするとprintfで変な値を指しています。たぶん環境によって変わると思いますが、自分の場合は./a.outを指していました。
この挙動はなぜ起こるのですか？また、このように別の関数で構造体を宣言して、その後、別の関数からポインタで参照する方法を教えてください。
お願いします。
c
1#define DEBUG 2 3 #include<stdio.h> 4 5 #include<stdlib.h> 6 7 8 9struct Sort{ 10 11 int no; 12 13 char *name; 14 15 16 17 char *avg; 18 19 //char *maximax; 20 21 char *saiaku_saidai; 22 23 24 25 char *alias; 26 27}; 28void sortinit(struct Sort *); 29 30int main(void){ 31struct Sort *arr; 32void *fes; 33 34sortinit(arr); 35//struct Sort *aa=(struct Sort *)fes; 36 37 printf("%s",arr->name); 38 39 return 0; 40 41} 42 43 void sortinit(struct Sort *in){ 44 struct Sort sortArr[]={ 45 46 { 47 48 1,"バブルソート","nasi","n^2","基本交換法" 49 50 },{ 51 52 2,"選択ソート","n^2","n^2","基本三法" 53 54 },{ 55 56 3,"挿入ソート","n+d","n^2","基本三法" 57 58 },{ 59 60 4,"シェルソート","nasi","nlog^2n","" 61 62 },{ 63 64 5,"マージソート","nlogn","nlogn","" 65 66 },{ 67 68 6,"ヒープソート","nlogn","nlogn","" 69 70 },{ 71 72 7,"クイックソート","nlogn","n^2","" 73 74 },{ 75 76 8,"ただの(偏った)二分木","n+1/2","n","線形探索法" 77 78 },{ 79 80 9,"完全二分木","","","" 81 82 },{ 83 84 10,"2分探索法","","","" 85 86 } 87 88 }; 89in=sortArr; 90//struct Sort *in2=(struct Sort *)in; 91 92}
回答2件
0
ベストアンサー
いくつか問題点があります。
- まず、関数を抜けた後も参照したい配列は、
staticにするか関数の外側で定義してください。普通に関数内で宣言した変数（自動変数）は、関数を抜けた瞬間に無効となります（無理やりポインタで参照した場合、結果は未定義です）。 - C言語では引数はすべて値渡しなので、
sortinit(arr);としてもmain側のarrは書き換わりません（未定義な値を使って関数を呼びだした段階で、動作は未定義です）。とりあえず動かしたいのであれば、上のと合わせてsortArrをmainより前に定義してしまうのが手っ取り早いです。
投稿2016/06/01 07:41
総合スコア146676
0
printf("%s",arr->name); の arr は未初期化なので何処を指しているか不明で動作未定義です。
sortinit の中では引数で受け取っている in に代入していますが
この影響範囲は sortinit の中だけであり呼び出し元の main の arr には何の影響もありません。
arrの初期化という事が意図なら
sortinit(&arr);
もしくは
arr = sortinit();
のような構成にすべきですね。
もう１つの問題点は（今回の直接原因ではありませんが） sortinit の中で宣言された sortArr[]
を sortinit の外で参照しようとしている点です。（参照できてはいませんでしたが）
もしこのようにするのなら sortArr に static をつけるなど何かしらの対策が必要です。
「関数の中で宣言されたローカルなauto変数は原則的にスタック上に確保されその関数終了と共に消失する」ので関数終了後に仮に元々あったハズの場所が判明してそこにアクセスできたとしてもそこにその時点で何が入っているかは不明でありそんなアクセスはしてはいけません。
投稿2016/06/01 07:43
編集2016/06/01 07:45
総合スコア2160
sortinit(&arr);
とするのなら
void sortinit(struct Sort *in);
も
void sortinit(struct Sort **in);
に変えなければなりませんしその中の
in=sortArr;
も
*in=sortArr;
とし、さらにsortArrの宣言も
struct Sort sortArr[]={
ではなく
static struct Sort sortArr[]={
のように静的配置すべきですが、それら全て対処した上でもエラーが出ているという事でしょうか？
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
退会済みユーザー
2016/06/01 08:20 編集
退会済みユーザー
2016/06/01 08:31 編集
2016/06/01 08:34
退会済みユーザー
2016/06/01 09:30