ｃ言語の乱数について

Question

実現したいこと

ｃで乱数が変化するようにしたい

前提

ここに質問の内容を詳しく書いてください。
（例）
3つのx,y座標をもった構造体のリストを作成するプログラムを書いています。
リストの要素は5です。
3つの座標をランダムに発生させたいのですが
三角形のアドレス:7673392
0:x=404.470870 y=381.138218
1:x=404.470870 y=381.138218
2:x=404.470870 y=381.138218

三角形のアドレス:7673584
0:x=510.618854 y=10.476547
1:x=510.618854 y=10.476547
2:x=510.618854 y=10.476547

というように要素ごとには違う値が入っているのですが座標ごとには同じ値が入っています。
srand((unsigned int)time(NULL));
の挿入位置の問題とはわかっているのですが様々な場所に変更したり増やしたりしても改善されなかったのでご助言頂きたいです。

該当のソースコード

#include<stdio.h>
#include<string.h>
#include<stdlib.h>
#include <ctype.h>
#include<conio.h>
#include<math.h>
#include <time.h>
#include <unistd.h>
#define N_MIN 3
#define N_MAX 100
#define ZAHYO_MIN 55.0
#define ZAHYO_MAX 550.0

struct zahyo{
	double x;
	double y;
};
typedef struct zahyo Coordinate;


struct sankaku{
	Coordinate v[3];
	struct sankaku *next;		
};
typedef struct sankaku Triangle;

Triangle *node_malloc(Triangle *random,int n,int i){
	Triangle *a = (Triangle*)malloc(sizeof(Triangle));
	a = random;
	
	for(int b=0; b<3;b++){
		a->v[b].x=(double) fmod(rand(),(ZAHYO_MAX - ZAHYO_MIN + 1.0) + ZAHYO_MIN) + (double)rand()/RAND_MAX;	
		a->v[b].y=(double) fmod(rand(),(ZAHYO_MAX - ZAHYO_MIN + 1.0) + ZAHYO_MIN) + (double)rand()/RAND_MAX;
	}

	if(n>i){	
		Triangle *b = (Triangle*)malloc(sizeof(Triangle));
		a->next=b;		
		a->next=node_malloc(a->next,n,i+1);
	}
	
	return a;
	
}

int main(){	
	int i=1;
	int n=5;
	srand((unsigned int)time(NULL));
	Triangle *sankaku;

	sankaku = (Triangle*)malloc(sizeof(Triangle));//領域の確保
	sankaku=node_malloc(sankaku,n,i);

	while(sankaku->next!=NULL){
		printf("三角形のアドレス:%d\n",sankaku);
		for(int j=0; j<3; j++){
			printf("%d:x=%f   y=%f\n",j,sankaku->v[i].x,sankaku->v[i].y);
		}
		puts("\n");
		sankaku=sankaku->next;
	}
	return 0;
	
}

試したこと

srand((unsigned int)time(NULL));
を31行目の次に挿入したがむしろ全ての座標が同じ乱数になった

Answer 1

58行目のソースですが

			printf("%d:x=%f   y=%f\n",j,sankaku->v[i].x,sankaku->v[i].y);

ではなく

			printf("%d:x=%f   y=%f\n",j,sankaku->v[j].x,sankaku->v[j].y);

ではないでしょうか。(v[i]⇒v[j])

Comments

[nikiita]

おっしゃる通りです。。。
デバック用にあるループから別のループにprintf文を移動させた時に変数を変えるのを失念しておりました。
変更したところ、それぞれの座標が別の乱数になりました。
乱数が変化しない=srand()のミスであると決めつけて視野が狭くなっていました。
ありがとうございます。

Answer 2

Triangle *node_malloc(Triangle *random,int n,int i){
	Triangle *a = (Triangle*)malloc(sizeof(Triangle)); // メモリを確保し、そのアドレスをaに保持しておきながら
	a = random; // わざわざそれを書き潰す
        ...

...なにがしたかったのでしょう?

Comments

[nikiita]

当方、ポインタと動的なメモリ確保の理解が足りておらず色んなサイトを見よう見まねで書いてエラーがでたり思わぬ挙動になればその都度書き直す,,,という書き方をしているため有識者の方から見れば意図がわからない・気持ち悪いコードを書いているかもしれません。申し訳ないです。
ご指摘を受けた箇所は今回出た不具合はちゃんとa=randomをするだけでaが確保されてるか心配で過剰な書き方をしてしまった,,,という感じです(伝わらなかったら申し訳ございません)

[episteme]

> a=randomをするだけでaが確保されてるか心配で過剰な書き方をしてしまった,,,

mallocで確保した領域はfreeせぬ限り解放されんので、解放されぬままメモリ内に居座ります。やっちゃダメです。