Déclaration des variables

Version imprimable

13/07/2008, 15h48
Telemak

Déclaration des variables
Bonjour à tous,
Petite question sur la déclaration des variables : peut elle se faire n'importe où à l'intérieur d'un programme ou en général après le main..? Car j'essaie de déclarer mes variables après un case et celà produit une erreur..

Exemple :

int main()
Code:

1 2 3 int main () float CapitalPrete; int nbAnneesPlacement;
Là pas de problème...
Code:

1 2 3 case 1: float CapitalPrete; int nbAnneesPlacement;
Là le compilateur me génère des erreurs du fait de l'absence de déclaration des variable...

Pour moi il s'agit principalement d'une lisibilité de programme...

Merci d'avance
a+
13/07/2008, 15h59
wiztricks

{ }

De mémoire le langage doit permettre la déclaration de variable à l'ouverture d'un block.

Il devrait suffire de coder:

Code:

case "label" : { int i; i = 1; } break;

- W
14/07/2008, 18h16
Telemak

Je viens d'essayer... à priori ça ne passe pas à la compilation...

Messages d'erreurs :

Citation:

MenuPrincipal.c:74: attention : unused variable ‘InteretSimple’
MenuPrincipal.c:73: attention : unused variable ‘TauxPlacement’
MenuPrincipal.c:72: attention : unused variable ‘nbAnneesPlacement’
MenuPrincipal.c:71: attention : unused variable ‘CapitalPrete’

etc..

a+
14/07/2008, 18h26
diogene

Ces messages signifient simplement que tu as déclaré des variables dont tu ne te sers pas (peut-être parce que le programme ou la fonction n'est pas encore terminé ?)
14/07/2008, 18h56
Telemak

En fait j'ai pas mis le reste des messages d'erreurs...

Citation:

MenuPrincipal.c:75: attention : unused variable ‘InteretSimple’
MenuPrincipal.c:74: attention : unused variable ‘TauxPlacement’
MenuPrincipal.c:73: attention : unused variable ‘nbAnneesPlacement’
MenuPrincipal.c:72: attention : unused variable ‘CapitalPrete’
MenuPrincipal.c:80: erreur: ‘CapitalPrete’ undeclared (first use in this function)
MenuPrincipal.c:80: erreur: (Each undeclared identifier is reported only once
MenuPrincipal.c:80: erreur: for each function it appears in.)
MenuPrincipal.c:82: erreur: ‘nbAnneesPlacement’ undeclared (first use in this function)
MenuPrincipal.c:84: erreur: ‘TauxPlacement’ undeclared (first use in this function)
MenuPrincipal.c:85: erreur: ‘InteretSimple’ undeclared (first use in this function)

et le compilateur s'arrête à la compilation...
Si je met la déclaration des variables après le main(), le programme compile et fonctionne sans problèmes...

Pour moi il s'agit essentiellement d'une lisibilité du programme, car si je doits mettre toutes les variables que je déclare en début de programme...ça va être chaud...:D
14/07/2008, 19h30
gl

Citation:

Envoyé par Telemak

et le compilateur s'arrête à la compilation...
Si je met la déclaration des variables après le main(), le programme compile et fonctionne sans problèmes...

Pour moi il s'agit essentiellement d'une lisibilité du programme, car si je doits mettre toutes les variables que je déclare en début de programme...ça va être chaud...:D

Pourrais-tu poster le code preentant le probleme ?

Voilà ... dans la foulée...j'en profite également ( bien que ce ne soit pas le sujet du message...sinon je referait un nouveau post..) au niveau du test de la condition "if" qui sélectionne la période, mon programme ne s'arrête pas à la demande du taux ( j'ai mis en rouge)... Je viens juste de rajouter ces lignes dans mon programme mais je n'ai pas cherché plus loin pour l'instant...possible qu'il s'agisse d'une simple erreur...mais n'ayant plus trop le temps à rester dessus ce soir... je m'y repencherais dessus...à un autre moment...

a+

Code:

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
#include <stdio.h>
#include <stdlib.h>
 
/*
 * 
 */
int main(int argc, char** argv) {
 
    //int nn=0;  
 
    int xPrincipal;                  // Choix du menu principal
    int xInteret;                    // Choix du menu intérêts
 
 
    // Calcul de l'intérêt simple....
           float CapitalPrete;         // Variable du montant prêté, placé..
           int nbAnneesPlacement;      // Variable de la durée   
           char periode;               // Variable de calcul de la période
           float division;             // Variable de calcul 
           float TauxPlacement;        // Variable du taux de placement
           float InteretSimple;        // Variable de résultat
    // printf("\x1B[2J" ); // Pour effacer l'écran
 
    system("clear");
 
    printf("\n\n\n\n\n");
    printf("        Intérêts  =   1\n");
    printf("        Taux      =   2\n");
    printf("        Crédits   =   3\n");
 
    printf("\n\n");
    printf("\t\t Indiquez votre choix : ");
 
    //ChoixMenuPrincipal = getchar();
    scanf("%d",&xPrincipal);
    //xPrincipal = scanf("%d",&nn);
 
    switch (xPrincipal) {
 
        case 1:
 
            system("clear");
 
            printf("\n\n");
            printf("Calcul de l'intérêt simple  = 1\n");
            printf("Calcul de l'intérêt sur une durée de placement = 2\n");
            printf("\n\n");
            printf("Retour Menu Principal = 0\n\n");
            printf("Sélectionnez votre choix : ");
 
           // ChoixMenuInteret = getchar();
            scanf("%d",&xInteret);
 
          //  xInteret = scanf("%d,&nn");
 
 
 
////////////////// Calcul sur les intérêt /////////////////////////////////            
            switch (xInteret) {
 
                case 0:
                    printf("Ok -> retour menu principal\n");
 
                break;
 
                case 1: 
           //    { 
           //     float CapitalPrete;         // Variable du montant prêté, placé..
           //       int nbAnneesPlacement;      // Variable de la durée     
           //      float TauxPlacement;        // Variable du taux de placement
           //       float InteretSimple;        // Variable de résultat
           //     }
                    printf("CACUL DE L'INTERET SIMPLE\n");
                    printf("\n\n\n");
                    printf("Montant du Capital : ");
                    scanf("%f",&CapitalPrete);
                 //   printf("Nombre année(s) de la durée du prêt ( ou du placement) : ");
 
                    printf("Indiquez la période de calcul (A)nnée(s) - (M)ois - (J)our(s) : ");
                    scanf("%c",&periode);
                 if (periode = "A")
                        division = 100;
                    else if (periode = "M")
                        division = 1200;
                    else if (periode = "J")
                        division = 36000; 
 
                    scanf("%d",&nbAnneesPlacement);
                    printf("Taux du prêt ( ou du placement) en %% : ");
                    scanf("%f",&TauxPlacement);
                    InteretSimple = ((CapitalPrete*TauxPlacement*nbAnneesPlacement)/division);
 
              printf("\n\n");
              printf("Un capital de %5.3f ",CapitalPrete);
              printf("placé ou prêté pendant %d ", nbAnneesPlacement);
              printf("an(s), au taux de %5.3f ", TauxPlacement);
              printf(" %% ");
              printf("fournira au prêteur un intérêt égal à : %5.3f ", InteretSimple);
              printf(" €uros\n");
              CapitalPrete = CapitalPrete+InteretSimple;
              printf("et l'emprunteur, à l'expiration du délai de %d", nbAnneesPlacement);
              printf(" an(s), devra remettre à son prêteur : %5.3f",CapitalPrete);
              printf(" €uros");
              printf("\n\n");
 
 
                break;        
 
            }
 
    }
 
 
 
 
 
    return (EXIT_SUCCESS);

Code:

1
2
3
4
5
6
                    if (periode == 'A')
                        division = 100;
                    else if (periode == 'M')
                        division = 1200;
                    else if (periode == 'J')
                        division = 36000;

1- la comparaison se fait avec ==, pas =
2- une constante caractère est entre simple guillemet. Entre double guillemets, on met les constantes chaînes de caractères

Citation:

Envoyé par Telemak

Voilà ... dans la foulée...

Ce code ne présenté pas le problème évoqué précédemment. Je suppose que le code ayant le problème est celui commenté dans le case ?

Si c'est effectivement le cas, il est évident qu'il y a un problème. Pourquoi refermes-tu le bloc juste après la déclaration des variables ? Celles-ci n'existent pas en dehors du bloc, d'où le problème.

Le bloc doit se poursuivre jusqu'à la fin du traitement. En clair, ce n'est pas
Code:

1 2 3 4 5 6 7 8 9 case 1: { float CapitalPrete; // Variable du montant prêté, placé.. int nbAnneesPlacement; // Variable de la durée float TauxPlacement; // Variable du taux de placement float InteretSimple; // Variable de résultat } ... break;
mais
Code:

1 2 3 4 5 6 7 8 9 case 1: { float CapitalPrete; // Variable du montant prêté, placé.. int nbAnneesPlacement; // Variable de la durée float TauxPlacement; // Variable du taux de placement float InteretSimple; // Variable de résultat ... } break;

Citation:

Envoyé par Telemak

Voilà <...>

Ce code est illisible, incomplet et ne compile pas en C90. Voici ton code rendu compilable (et présentable par la même occasion). J'ai déplacé les variables comme tu le voulais (je conseille de porsuivre en séparant le code en fonctions...) :

Code:

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
 
#include <stdio.h>
#include <stdlib.h>
 
int main (void)
{
/* Choix du menu principal */
   int xPrincipal;
/* Choix du menu intérêts */
   int xInteret;
 
/* Calcul de l'intérêt simple.... */
 
   printf ("\n\n\n\n\n");
   printf ("        Intérêts  =   1\n");
   printf ("        Taux      =   2\n");
   printf ("        Crédits   =   3\n");
 
   printf ("\n\n");
   printf ("\t\t Indiquez votre choix : ");
 
   scanf ("%d", &xPrincipal);
 
   switch (xPrincipal)
   {
 
   case 1:
 
      printf ("\n\n");
      printf ("Calcul de l'intérêt simple  = 1\n");
      printf ("Calcul de l'intérêt sur une durée de placement = 2\n");
      printf ("\n\n");
      printf ("Retour Menu Principal = 0\n\n");
      printf ("Sélectionnez votre choix : ");
 
      scanf ("%d", &xInteret);
 
/* //////////////// Calcul sur les intérêt ///////////////////////////////// */
      switch (xInteret)
      {
 
      case 0:
         printf ("Ok -> retour menu principal\n");
 
         break;
 
      case 1:
         {
            float division = 0;
            float CapitalPrete;
            int nbAnneesPlacement;
            char periode;
            float TauxPlacement;
            float InteretSimple;
 
            printf ("CACUL DE L'INTERET SIMPLE\n");
            printf ("\n\n\n");
            printf ("Montant du Capital : ");
            scanf ("%f", &CapitalPrete);
            printf
               ("Indiquez la période de calcul (A)nnée(s) - (M)ois - (J)our(s) : ");
            scanf ("%c", &periode);
            if (periode == 'A')
               division = 100;
            else if (periode == 'M')
               division = 1200;
            else if (periode == 'J')
               division = 36000;
 
            scanf ("%d", &nbAnneesPlacement);
            printf ("Taux du prêt ( ou du placement) en %% : ");
            scanf ("%f", &TauxPlacement);
            InteretSimple =
               ((CapitalPrete * TauxPlacement * nbAnneesPlacement) /
                division);
 
            printf ("\n\n");
            printf ("Un capital de %5.3f ", CapitalPrete);
            printf ("placé ou prêté pendant %d ", nbAnneesPlacement);
            printf ("an(s), au taux de %5.3f ", TauxPlacement);
            printf (" %% ");
            printf ("fournira au prêteur un intérêt égal à : %5.3f ",
                    InteretSimple);
            printf (" €uros\n");
            CapitalPrete = CapitalPrete + InteretSimple;
            printf ("et l'emprunteur, à l'expiration du délai de %d",
                    nbAnneesPlacement);
            printf (" an(s), devra remettre à son prêteur : %5.3f",
                    CapitalPrete);
            printf (" €uros");
            printf ("\n\n");
         }
         break;
 
      }
   }
   return EXIT_SUCCESS;
}

ce code est instable à cause de l'usage naïf qui est fait de scanf().

Soit tu apprends à t'en servir correctement :

http://xrenault.developpez.com/tutoriels/c/scanf/

ce qui est assez compliqué, soit tu utilises fgets(), puis purge() (code maintes fois publié sur ce forum) et les fonctions de conversions qui vont bien (strtod() etc.)

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

Je conseille l'usage de double à la place de float pour une meilleure précisioin dans les calculs (exigé par l'administration en matière financière).

ok, effectivement par rapport à = et == ...j'avoue ne plus avoir continué par la suite..d'ailleurs j'ai un signalement de celà au moment de la compilation..

Pour les parenthèses au niveau de la déclaration des variables ..c'est également rectifié...

Citation:

ce code est instable à cause de l'usage naïf qui est fait de scanf().

Je pensaits qu'il y avait assez de stabilité concernant cette fonction...
Est ce pour celà que, lorsque j'exécute le programme, celui-ci au lieu de me demander la suite des entrées (cf : scanf("%d",&nbAnneesPlacement) ), continue à faire le calcul sans qu'aucune donnée supplémentaire n'est été saisie...?
Y figure bien un scanf qui devrait etre en attente d'une saisie..non.?
Code:

1 2 3 4 5 6 7 8 9 10 11 printf("Indiquez la période de calcul (A)nnée(s) - (M)ois - (J)our(s) : "); scanf("%c",&periode); if (periode == 'A') division = 100; else if (periode == 'M') division = 1200; else if (periode == 'J') division = 36000; printf("Nombre année(s) de la durée du prêt ( ou du placement) : "); scanf("%d",&nbAnneesPlacement);
Citation:

Ce code est illisible, incomplet .

Tu veuts dire par rapport à l'allignement du code, la mise en forme, la nomination des variales ..etc..ainsi que les mises en remarques ..// etc...?

Merci
a+

16/07/2008, 09h17
bulki
Citation:

Envoyé par Telemak

Tu veuts dire par rapport à l'allignement du code, la mise en forme, la nomination des variales ..etc..ainsi que les mises en remarques ..// etc...?

Oui, ça le rend très difficile à lire. Les espaces sont n'importes comment aussi :s, des switch dans des switch, c'est pas terrible... m'enfin, c'est le métier qui rentre comme on dit ;). En choisissant un bon nom de variable, tu évites un commentaire inutile comme "Variable du taux de placement".

Btw, plutot que d'écrire ça:
Code:

1 2 3 4 printf(" %% "); printf("fournira au prêteur un intérêt égal à : %5.3f ", InteretSimple); printf(" €uros\n");
Tu peux écrire ça:
Code:

1 2 3 4 printf(" %% " "fournira au prêteur un intérêt égal à : %5.3f " " €uros\n", IntertSimple);
edit: indentation a un peu sauté :P
16/07/2008, 11h59
Telemak

En fait il s'agit d'une première ébauche..essentiellement pour "apréhender le langage..
C'est pour celà que le nom des variables ainsi que l'indentation n'est pas vraiment au top..
Il s'agit plus d'un brouillon pour moi. Certains préfère faire celà dès le début, ce que je comprends.. Pour moi il ne s'agit pas pour l'instant d'en faire un programme comment dire...utilisable ou en production..
Comme tu dits..c'est le métier qui rentre...;)

Pour les switch dans les switch ..je suis également d'accord avec toi.. Pas forcément l'idéal...Mais ne voyant pas pour l'instant au niveau de mes connaissances dans ce langage le moyen de procéder autrement, hormis la création de fonctions...mais qui ne s'apparenterais pas être la manière la plus judicieuse dans mon cas. Surtout si je "barde" le programme de centaines de formules de calculs différentes, je pense qu'au fil du temps avec les fonctions celà alourdirait pas mal...D'ailleurs celà avait donné lieu à une discution lors d'un autre tread...
http://www.developpez.net/forums/sho...d.php?t=586032

Comme cité plus haut, il s'agit pour moi essentiellement de faire un petit tour du langage, sans plus...
Tous les chemins mênent à Rome... ;)

a+
16/07/2008, 13h20
droggo

Gao,

Citation:

Envoyé par Telemak

Surtout si je "barde" le programme de centaines de formules de calculs différentes, je pense qu'au fil du temps avec les fonctions celà alourdirait pas mal...

c'est plutôt en n'en faisant pas que ça alourdit. :)