Discussion :

[torquemada]

Bonjour à tous ,
Avant de me lancer dans l'étude des pointeurs ,j'ai du mal à comprendre pourquoi ce code produit un résultat faux(sa valeur intermédiaire est exacte) pour la variable minute qui est
pourtant déclarée en globale ,et dans mon esprit devrait pouvoir être atteinte par la fonction main.
En vous remerciant de m'avoir consacré de votre temps.

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
 
#include <stdio.h>
#include <stdlib.h>
#include<math.h>
 
int heures = 0,minutes=90;
void decoupeMinutes (int minutes);
 
 
int main()
{
   decoupeMinutes(minutes);
   printf("%d Heures",heures);
   printf("%d Minutes",minutes);
    return 0;
}
 void decoupeMinutes(int minutes)
  {
 
      heures = minutes/60;
       minutes=minutes%60;
       printf("%d minutes",minutes);
  }

[Médinoc]

La variable locale minutes cache la variable globale.
Supprime complètement les variables globales et utilise des pointeurs dans tes fonctions, à la place.

[Sve@r]

Salut

Généralement, sauf très rares exceptions, les variables globales ne se justifient pas. Généralement elles induisent des incompréhensions sur certains comportements (d'ailleurs tu en as un bon exemple puisque tu ne comprenais pas que la variable "minutes" de ta fonction prenait le pas sur la variable globale "minutes").

Voici 2 exemples qui font ce que tu désires mais sans passer par les globales

Code c :

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
#include <stdio.h>
#include <stdlib.h>
#include<math.h>
 
void decoupeMinutes (int*, int*);
 
int main()
{
   int heures = 0,minutes=90;
   decoupeMinutes(&heures, &minutes);
   printf("%d Heures",heures);
   printf("%d Minutes",minutes);
   return 0;
}
 
void decoupeMinutes(int *heures, int *minutes)
{
      *heures = (*minutes)/60;
      *minutes=(*minutes)%60;
}

Code c :

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
#include <stdio.h>
#include <stdlib.h>
#include<math.h>
 
typedef struct {
    int hh;
    int mn;
} t_time;
 
void decoupeMinutes (t_time*);
 
int main()
{
   t_time heure={0, 90};
 
   decoupeMinutes(&heure);
   printf("%d Heures",heure.hh);
   printf("%d Minutes",heure.mn);
   return 0;
}
 
void decoupeMinutes(t_time *t)
{
      t->hh = t->mm / 60;
      t->mn=t->mn % 60;
}

Ces deux exemples nécessitent toutefois de passer par des pointeurs, seule façon qu'une fonction "appelée" puisse modifier des variables situées dans une fonction "appelante". Donc l'adresse de "heures" (&heures) passe dans la fonction dans le pointeur "heures". La fonction ayant l'adresse de cette variable peut aller taper à cet endroit (*heures) pour le modifier.

Le second exemple induit l'utilisation de structure permettant de regrouper dans une même notion logique des éléments allant ensembles (heure+minutes) tout en conservant la faculté de les traiter individuellement.

Maintenant, si tu veux quand-même utiliser des globales (tu en as tout à fait le droit), alors il faut le faire correctement et essayer de lever au maximum les ambigüités

Code c :

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
#include <stdio.h>
#include <stdlib.h>
#include<math.h>
 
int heures = 0,minutes=90;
void decoupeMinutes();
 
int main()
{
   decoupeMinutes();
   printf("%d Heures",heures);
   printf("%d Minutes",minutes);
   return 0;
}
 
void decoupeMinutes()
{
    extern int heures;
    extern int minutes; 
 
    heures = minutes/60;
    minutes=minutes%60;
    printf("%d minutes",minutes);
}

En redéclarant dans ta fonction des éléments que tu indiques comme déjà existant ailleurs ("extern"), cela te permet de connaitre tes variables utilisées et de ne pas avoir de souci de nom si tu dois en créer une autre.

Accessoirement, comme les variables sont globales, cela ne sert à rien de les passer comme paramètres...