質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.47%

  • C

    4659questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

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

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 567
退会済みユーザー

退会済みユーザー

前提・実現したいこと

構造体を用いて三角形を生成したいですが三角形生成関数*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;
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+2

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/07/18 20:00

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.47%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る

  • C

    4659questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。