Discussion :

[chardonette]

Bonjour à tous.

Me revoilà avec une nouvelle question.
Je travaille sur des projets sous code ::blocks version 8.02, système d'exploitation windows vista (I know this is daube :s), dont les programmes main.c utilisent des fonctions random.

Ces fonctions random sont dans une boucle for , jusqu'à N points.

Pour certains programmes : lorsque le N points devient trop important, en lançant la compilation j'obtiens => "main.exe a cessé de fonctionner", puis plus rien.

Pour d'autres ,même en diminuant fortement N (voire, réduit à 10), j'obtiens "main.exe a cessé de fonctionner".

Savez-vous d'où cela provient, et que pourrais-je faire pour éviter cela et avoir des programmes qui compilent?

[Pouet_forever]

Je pense qu'avec ton code on pourrais t'aider, mais sans ça me paraît compliqué

[Melem]

Le problème c'est évidemment que ton code est buggé, il ne peut y avoir d'autre raison. Dommage qu'on ne peut pas le voir pour le corriger ...

[chardonette]

Ah oui, bien sûr ^^ .
Voilà le code.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <time.h>
 
main ()
{
 
int rm1_max, N1, i, Nt, eps;
double Veff, Veff_t, abs_rs1, abs_rs2,V_espace,xs[Nt], ys[Nt], zs[Nt], xdiff;
 
 
srand(time(NULL));
Nt=100;
rm1_max=10;
V_espace=2.0*rm1_max*2.0*rm1_max*4.0*rm1_max;
 
/* eps est la largeur du recouvrement; quand eps=2rm il y a recouvrement total, et pour eps=0 il ny a pas de recouvrement*/
/*eps peut varier de 0 à 2rm1_max*/
eps=0;
 
N1=0; xdiff=4.0*rm1_max-eps;
 
	for (i=0; i<=Nt; ++i){
/* on lance les "cailloux" dans le volume totale*/
 
    xs[i]=-2.0*rm1_max + (((double) rand() / RAND_MAX) * 4.0 * rm1_max);
    ys[i]=-rm1_max + (((double) rand() / RAND_MAX) * 2.0 * rm1_max);
    zs[i]=-rm1_max + (((double) rand() / RAND_MAX) * 2.0 * rm1_max);
 
/* on retranche pour que le zero soit au centre du systeme*/
 
/* abs_rs1 est la distance entre la source et lhydrophone 1 qui se trouve initialement aux coordonnees x=-rm1_max, y=rm1_max et z=rm1_max*/
/*au fur et a mesure que eps augmente, le centre des deux hydrophones se déplace de +eps/2 pour H1 en -rm1_max et -eps/2 pour H2 en +rm1_max*/
	abs_rs1=sqrt((xs[i]-(-rm1_max+(eps/2.0)))*(xs[i]-(-rm1_max+(eps/2.0)))+ys[i]*ys[i]+zs[i]*zs[i]);
	abs_rs2=sqrt((xs[i]-rm1_max-(eps/2.0))*(xs[i]-rm1_max-(eps/2.0))+ys[i]*ys[i]+zs[i]*zs[i]);
 
 
	/* Simple signal*/
	/*if (abs_rs1<=rm1_max && abs_rs2<=rm1_max)*/
	{N1=N1+1;}
 
Veff=V_espace*N1/Nt;
printf("N1: %d \n", N1);
printf("V1eff %lf \n", Veff);
 
}
}

[Pouet_forever]

Regarde tes déclarations et dis moi si ça te paraît correct :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
int rm1_max, N1, i, Nt, eps;
double Veff, Veff_t, abs_rs1, abs_rs2,V_espace,xs[Nt], ys[Nt], zs[Nt], xdiff;

Tu initialises des tableaux avec une variable non initialisée !
De plus je te déconseille de faire comme ça (les non adeptes du C99 vont te sauter dessus )

Fais un '#define Nt ' avec une valeur que tu auras définie, ou alors utilises l'allocation dynamique

[chardonette]

Oui super!
Avec

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

#define Nt 1000

le programme compile!
Merci encore.

ps : Je ne vous assure pas que je ne reviendrais pas pour d'autres questions, mais à l'avenir pour introduire des constante, je sais qu'il faudra utiliser #define