[C] 構造体ポインタの生成

Question

###前提・実現したいこと
構造体を用いて三角形を生成したいですが三角形生成関数*creat_triangleでsegmentation faultが起きます。return直前のprintfでエラーが起きていないため原因がわかりません。
ご教授よろしくお願いします。

###実行結果

1  15192  11665  54390
2   6278  55415  85806
3  29180  39806  78076
Segmentation fault: 11

###該当のソースコード

#include<stdio.h>
#include<stdlib.h>
#include<time.h>
#include<math.h>

#define XMAX 100000
#define YMAX 100000
#define ZMAX 100000

typedef struct POINT {
  int x;
  int y;
  int z;
} point_t;

typedef struct TRIANGLE {
  point_t p[3];
} triangle_t;

//三角形生成
triangle_t *create_triangle()
{
  int x[3], y[3], z[3]; //各座標
  double s[3]; //各辺の長さ
  triangle_t *tr; //三角形
  int i;

  srand((unsigned)time(NULL));
  
  for (i = 0; i < 3; i++) {
    x[i] = rand() % XMAX;
    y[i] = rand() % YMAX;
    z[i] = rand() % ZMAX;
    //printf("%4d %4d %4d\n", x[i], y[i], z[i]);
 }

  s[0] = sqrt(pow((double)(x[1] - x[0]), 2) + pow((double)(y[1] - y[0]), 2) + pow((double)(z[1] - z[0]), 2)); 
  s[1] = sqrt(pow((double)(x[2] - x[1]), 2) + pow((double)(y[2] - y[1]), 2) + pow((double)(z[2] - z[1]), 2)); 
  s[2] = sqrt(pow((double)(x[0] - x[2]), 2) + pow((double)(y[0] - y[2]), 2) + pow((double)(z[0] - z[2]), 2)); 
  //for (i = 0; i < 3; i++) printf("%f\n", s[i]);

  //三角形成立条件
  if (s[0] + s[1] > s[2] && s[1] + s[2] > s[0] && s[2] + s[0] > s[1]) {
    for (i = 0; i < 3; i++) {
      tr->p[i].x = x[i];
      tr->p[i].y = y[i];
      tr->p[i].z = z[i];
    }

    for (i = 0; i < 3; i++) printf("%d %6d %6d %6d\n", i + 1, tr->p[i].x, tr->p[i].y, tr->p[i].z);
    return tr;
  }
  else return NULL;
}

int main()
{
  triangle_t *tr;

  tr = create_triangle();

  return 0;
}

Answer 1

triangle_t *trはポインターだけで、中身がありません。
そしてそのポインターが不正なアドレスを指しているため、segmentation faultが発生しています。

動的に生成するなら、malloc関数などを利用して生成しますが、
呼び出し側でポインターではないtriangle_tを作って、
それをポインターでcreate_triangle関数に渡す方が簡単です。

後者の場合、関数はvoid init_triangle(triangle_t *)に変えたほうが良いですね。

Comments

[退会済みユーザー]

解決しました。ありがとうございます。