Discussion :

[amateurc]

Bonjour j'essai de faire un programme qui me semble assez simple à réaliser mais que ne fonctionne pas voici 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
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
 
main()
{
	srand((unsigned)time(NULL));
	int i, nblancer, *tab;
 
	printf("Saisir le nombre de lancer : ");
	scanf("%d",&nblancer);
	for (i=0;i<=nblancer-1;i++)
	{
        tab = (int *)calloc (2,sizeof(int *));
        tab[i]=rand()%10 +1;
	}
	for (i=0;i<=nblancer-1;i++)
		printf("valeur du lancer %2d : %d\n", i+1, tab[i]);
}

j'arrive a faire les premiers lancer mais pas les autres au dessus de 4 sa plante est ce que vous pouvez m'indiquer mon erreur

[orfix]

Salut,

• main retourne un int
• Pourquoi est-il déconseillé d'utiliser scanf ?
• On libère toujours la mémoire allouer après usage via un free
• Ton allocation devrait se faire à l'extérieur de la boucle ( avant ) car à chaque itération tu as des fuites mémoires...sinon d'après ton affichage il faut mettre nblancer et non pas 2 ...non ?

Code :

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

1
2
3
4
 
tab = calloc (nblancer,sizeof(int) );
/* Ou alors mieux...*/
tab = calloc (nblancer,sizeof *tab );

Bon courage !

[amateurc]

D'accord je vois ou ce trouvait le problème en fait je voulais qu'a chaque nouveau tour de la boucle je rajoute un emplacement mémoire est ce que cela est possible ?

[Aldian]

C'est parfaitement possible, mais d'une complexité inutile pour ce petit programme.

Tu peux par exemple faire un realloc à chaque boucle, mais ce n'est pas conseillé car c'est couteux en ressources. En général, on fait des realloc de temps en temps (genre toutes les 100 cases) selon le compromis choisi bouffage de ressources/bouffage de mémoire.


Tu peux aussi faire une liste linéaire chainée. Il te suffit pour cela de faire une structure avec un champ int et un champ pointeur vers ce type de structure. Ensuite à chaque itération tu alloue une nouvelle strucuture, tu mets son pointeur à NULL, et tu dis au pointeur de la structure de la boucle d'avant de pointer sur la structure que tu viens de créer.

[amateurc]

D'accord merci de vos réponses