Discussion :

[goldensun]

Bonjour

le but de l'exo c'était de faire un programme qui permet d'entrer le nombre de valeur que l'on va entrer, puis de entrer ces valeurs, de les sommer, et aussi de dire qu'elle est la valeur min et la valeur max.
J'ai réussit à faire le programme mais j'ai une erreur de segmentation lorsqu'on doit sortir de la boucle.

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
#include <stdio.h>
 
int som=0;
 
void min_max(int *x, int *b, int *t,int *i){
if (*i==0) {
*b=*t;
*x=*t;
}
 
else
{
{if (*t>*x) {
*x=*t;}
if (*t<*b)
{*b=*t;}
}
}
}
void somme(int *som,int *t)
{*som=*som+*t;
}
 
 
int main()
{
int a;
int i;
int x, b;
printf("entrer la quantitée:\n");
scanf("%d",&a);
for(i=0;i<a;i++)
{
printf("saisissez la valeur : \n");
scanf("%d",&a+1+i);
somme(&som,&a+1+i);
printf("la somme vaut %d\n",som);
min_max(&x,&b,&a+i+1,&i);
}
printf("la max vaut %d et le min vaut %d",x,b);
return 0;
}

[Invité(e)]

Bonjour,

Que veut dire cette ligne ?

Code :

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

scanf("%d", &a + 1 + i);

On écrit en gros après a, mais dans un espace qui n'a pas été réservé. Pourquoi ne pas utiliser une autre variable ?

Il faudrait aussi changer le nom des variables en n_valeurs, derniere_valeur, min, et max, ça serait plus clair...

[goldensun]

Bonjour,

Que veut dire cette ligne ?

Code :

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

scanf("%d", &a + 1 + i);

On écrit en gros après a, mais dans un espace qui n'a pas été réservé. Pourquoi ne pas utiliser une autre variable ?

Il faudrait aussi changer le nom des variables en n_valeurs, derniere_valeur, min, et max, ça serait plus clair...

C'est tout à fait ça, je ne sais pas comment faire pour reserver l'espace étant donné que l'on fixe le nombre de valeur à entrer avec le scanf, donc j'ai stocké les infos les une à la suite des autres. C'est surement la cause de l'erreur de segmentation.


Dsl pour la presentation
, c'est vrai qu'il n'était pas necessaire de faire une globale, j'aurai pu définir som dans le main.

[diogene]

C'est tout à fait ça, je ne sais pas comment faire pour reserver l'espace étant donné que l'on fixe le nombre de valeur à entrer avec le scanf,

Si le nombre de valeurs est fixé à l'exécution, tournes-toi vers l'allocation dynamique

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
  int a;
  int * tab;
....
  printf ("entrer la quantitée:\n");
  scanf ("%d", &a);
  tab = malloc (a*sizeof *tab);
  if(tab != NULL)
  {
    for(i=0;i<a;i++)
    {
       printf("saisissez la valeur : \n");
       scanf("%d",tab+i); 
....

[Emmanuel Delahaye]

Si le nombre de valeurs est fixé à l'exécution, tournes-toi vers l'allocation dynamique

Ne pas se laisser intoxiquer par les désirs farfelus du P.O. Pour le moment, il n'est nul besoin de tableaux ici...

[Emmanuel Delahaye]

le but de l'exo c'était de faire un programme qui permet d'entrer le nombre de valeur que l'on va entrer, puis de entrer ces valeurs, de les sommer, et aussi de dire qu'elle est la valeur min et la valeur max.
J'ai réussit à faire le programme mais j'ai une erreur de segmentation lorsqu'on doit sortir de la boucle.

Un minimum de présentation montre qu'on respecte le lecteur...

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
49
 
#include <stdio.h>
 
int som = 0;
 
static void min_max (int *x, int *b, int *t, int *i)
{
   if (*i == 0)
   {
      *b = *t;
      *x = *t;
   }
   else
   {
      {
         if (*t > *x)
         {
            *x = *t;
         }
         if (*t < *b)
         {
            *b = *t;
         }
      }
   }
}
static void somme (int *som, int *t)
{
   *som = *som + *t;
}
 
int main (void)
{
   int a;
   int i;
   int x, b;
   printf ("entrer la quantitée:\n");
   scanf ("%d", &a);
   for (i = 0; i < a; i++)
   {
      printf ("saisissez la valeur : \n");
      scanf ("%d", &a + 1 + i);
      somme (&som, &a + 1 + i);
      printf ("la somme vaut %d\n", som);
      min_max (&x, &b, &a + i + 1, &i);
   }
   printf ("la max vaut %d et le min vaut %d", x, b);
   return 0;
}

• Une globale ? Pourquoi ?
• Que signifie pour toi '&a + 1 + i' ?
• Attention à l'abus de pointeurs. une analyse rapide montre qu'on peut en supprimer la moitié.
• Tu as brisé une règle simple qui veut qu'une variable ne serve qu'à une chose. Or ici, tu utilises 'a' pour 2 fonctions. Le comportement est ... étrange ...

ceci me parait raisonnable :

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
 
entrer la quantitee:
3
saisissez la valeur :
1
la somme vaut 1
saisissez la valeur :
2
la somme vaut 3
saisissez la valeur :
3
la somme vaut 6
la max vaut 3 et le min vaut 1
 
Process returned 0 (0x0)   execution time : 6.124 s
Press any key to continue.

avec :

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
49
50
 
#include <stdio.h>
 
static void min_max (int *x, int *b, int t, int i)
{
   if (i == 0)
   {
      *b = t;
      *x = t;
   }
   else
   {
      {
         if (t > *x)
         {
            *x = t;
         }
         if (t < *b)
         {
            *b = t;
         }
      }
   }
}
static void somme (int *som, int t)
{
   *som = *som + t;
}
 
int main (void)
{
   int a;
   int i;
   int x, b;
   int som = 0;
 
   printf ("entrer la quantitee:\n");
   scanf ("%d", &a);
   for (i = 0; i < a; i++)
   {
      int v;
      printf ("saisissez la valeur : \n");
      scanf ("%d", &v);
      somme (&som, v);
      printf ("la somme vaut %d\n", som);
      min_max (&x, &b, v, i);
   }
   printf ("la max vaut %d et le min vaut %d\n", x, b);
   return 0;
}

[goldensun]

Une globale ? Pourquoi ?
Que signifie pour toi '&a + 1 + i' ?

C'est les adresses qui suivent celle de a.

Attention à l'abus de pointeurs. une analyse rapide montre qu'on peut en supprimer la moitié.

ça signifit juste que je ne maitrise pas encore les pointeurs.

Tu as brisé une règle simple qui veut qu'une variable ne serve qu'à une chose. Or ici, tu utilises 'a' pour 2 fonctions. Le comportement est ... étrange ...

Il s'agit du même a, il me semble.

[Emmanuel Delahaye]

C'est les adresses qui suivent celle de a.

Et tu penses qu'elles sont valides ?

Il s'agit du même a, il me semble.

Oui, et c'est bien le problème Dans main(), l'un sert pour déterminer le nombre de valeurs et le même sert pour saisir la valeur. La boucle fait donc n'importe quoi.

Si je saisie a=3 pour 3 valeurs, puis a=1 comme première valeur, la boucle s'arrête... J'ai mis du temps à comprendre pourquoi...

[goldensun]

Et tu penses qu'elles sont valides ?

Nan car il y a des chances que l'espace mémoire de cette adresse soit déjà occupé.

Oui, et c'est bien le problème Dans main(), l'un sert pour déterminer le nombre de valeurs et le même sert pour saisir la valeur. La boucle fait donc n'importe quoi.

Si je saisie a=3 pour 3 valeurs, puis a=1 comme première valeur, la boucle s'arrête... J'ai mis du temps à comprendre pourquoi...

Il saisit dans les adresses qui suivent donc la valeur de a reste intacte "&a+1+i" c'est dans les cases mémoires qui ont des adresses de ce type que les infos sont stockés, donc je ne touche pas au a après la saisie.

[Emmanuel Delahaye]

Nan car il y a des chances que l'espace mémoire de cette adresse soit déjà occupé.

C'est surtout une adresse invalide. C'est donc bien une erreur que j'ai corrigée.

Il saisit dans les adresses qui suivent donc la valeur de a reste intacte "&a+1+i" c'est dans les cases mémoires qui ont des adresses de ce type que les infos sont stockés, donc je ne touche pas au a après la saisie.

Oui, mais comme j'ai corrigé, je tombe sur ce bug.

Je ne te demande pas de justifier tes erreurs, mais de les corriger.