propreté de code

Bonsoir , j'ai écrit un programme composé de 2 fichiers qui affiche une table de conversion des euros en francs , j'aimerais juste que quelqu'un de super maniaque et bon en C me dise si il y a de petites choses à régler pour que le code soit plus propre , merci :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

#include <stdio.h>   int my_euros(void);   int main(void)   {   my_euros(void);   return 0;   }

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

#include <stdio.h>   int my_euros(void)   {   float euro = 1;   for ( euro = 0; euro <500; euro = euro + 20 ) { printf("%6.2f %6.2f", euro, euro * 6.5); } return 0; }

---

PS : j'ai juste le droit aux boucles et à printf , j'ai pas fait d'erreur çà va ?

je ne suis ni maniaque ni bon en C mais il me semble qu'1 euro = 6.55957F :frenchy: (et ça serait bien de mettre ça dans une constante).

oui bon ce genre d'erreur n'a pas d'importance lol , ce qui m'intéresse surtout c'est voir si j'ai pas fait de minuscules erreur au niveau du code en C...,,au passage , si j'ai :

float euro = 1;

for ( euro = 0; euro < 500....


est ce que je peux l'écrire comme ceci :


for ( float euro = 1; euro < 500...

je veux dire que je peux pas déclarer une variable au début de la boucle for , si ?

je ne programme plus qu'en C++ où on peut faire ce genre de chose. Il me semble que ça passe en C maintenant.

sinon si tu veux des critiques :
- l'indentation de ton prog est à revoir
- met le rapport EURO/FRANC dans une constante.
- pourquoi 2 fonctions ?
- il n'y a jamais de retour à la ligne dans tes affichages ?
- pourquoi euro est un float ? il est toujours entier
- utilise l'opérateur +=
[ajout] quand tu appeles ta fonction my_euros(), void est inutile - d'ailleurs je suis étonné que ça compile.

ok peux tu corriger çà s'il te plait ?

j'ai fait 2 fichiers juste pour m'entrainer à compiler avec un makefile .

et 6.5 c'est un flottant pas un entier il me semble , les nombres à virgules sont des flottants .

en plus.

-<stdio.h> n'est pas utile dans le fichier où se trouve le main.
-inutile d'initialiser euroà 1 pour le remettre de suite à 0 aprés.
-Pourquoi ne pas utiliser un fichier d'entete pour les prototypes de fonction et ainsi l'inclure dans les parties de programme qui les utilisent ?

"-Pourquoi ne pas utiliser un fichier d'entete pour les prototypes de fonction et ainsi l'inclure dans les parties de programme qui les utilisent ?"

peux tu préciser ce point et me donner l'exemple avec mon fichier , çà m'intéresse , merci .

Citation:

---

Envoyé par jmv

je ne programme plus qu'en C++ où on peut faire ce genre de chose. Il me semble que ça passe en C maintenant.

---

Définir une variable dans un for(), non. C'est du C++. A ma connaissance, même C99 ne le permet pas.

Pour répondre à Bidouba, c'est difficile de juger sur un exemple aussi trivial, qui ne nécessite pas de fonction en premier lieu. De plus, la valeur retournée par my_euros (drole de nom, aucun rapport avec ce qu'elle fait) est inutile. Comme il n'y pas d'erreur prévue dans la fonction, autant qu'elle ne retourne rien (void). Mais comme je l'écris plus haut, la fonction est inutile (j'imagine que c'est encore un test pour la compilation séparée).
Sinon, ton code est propre mais mal indenté, et tu abuses des retours à la ligne.

Citation:

---

Envoyé par Bibouda

peux tu préciser ce point et me donner l'exemple avec mon fichier , çà m'intéresse , merci .

---

Ouvre un livre de C ou un tutoriel sur l'Internet (liens sur ce site), par pitié! Tu ne vas pas apprendre en posant des questions ici.

Citation:

---

Envoyé par DaZumba

Définir une variable dans un for(), non. C'est du C++. A ma connaissance, même C99 ne le permet pas.

---

Bien au contraire :

• C90 : impossible pas de mixe code/déclaration
• C99 : Ok, la variable n'existe plus à la de la boucle
• C++ : a éviter, la porté de la variable n'est pas définie par la norme (donc on ne sais pas si la variable existe toujours après la boucle, ça dépend des compilateurs)

pour ta variable euro, puisque tu l'initialises dans ta boucle tu n'a pas besoin d'initialiser cette derniere au début. Par exemple:

Code:

---

1 2 3 4 5 6 7

float euro;   for( euro = 0; euro < 500; euro += 20 ) { // ton code }

---

Citation:

---

Envoyé par Bibouda

est ce que je peux l'écrire comme ceci :

for ( float euro = 1; euro < 500...

je veux dire que je peux pas déclarer une variable au début de la boucle for , si ?

---

En C99, c'est possible.

Citation:

---

Envoyé par Bibouda

Bonsoir , j'ai écrit un programme composé de 2 fichiers qui affiche une table de conversion des euros en francs , j'aimerais juste que quelqu'un de super maniaque et bon en C me dise si il y a de petites choses à régler pour que le code soit plus propre , merci :

---

• Ne compile pas.
  Citation:

  ---

  Code:

  ---

  my_euros(void);

  ---

  ---

  Tu ne peux pas mettre (void) en paramètre quand tu appelles une fonction. C'est juste ().
  
  Code:

  ---

  my_euros();

  ---

• Le découpage en module n'est pas propre. Le prototype séparé doit être placé dans un header.
  http://emmanuel-delahaye.developpez....ganiser_source
  http://emmanuel-delahaye.developpez.....htm#organiser
• La présentation est horrible. Pas d'indentation, lignes blanches injustifiées...
• Les nombre flottants par défaut sont 'double'. Les float obligent à d'incessantes conversions internes et à une perte de précision. Ne sont utiles que pour du stockage de masse.
• Pourquoi initialiser une variable deux fois de suite ? Ceinture et bretelles ?
• Le nombre de Francs par Euros est fixé officiellement à 6.55957
• Bien que ce soit correct, en C, on ne dit pas 'a = a + 2' mais 'a += 2'.
• La sortie texte n'est pas terminée par un '\n' ni par un fflush(stdout).
  http://emmanuel-delahaye.developpez....#fflush_stdout
• Le format "6.2f" est insuffisant pour la colonne des Euros. Il faut au moins "7.2f".

Je recommande ceci :

Code:

---

1 2 3 4 5 6 7 8 9

#ifndef H_X #define H_X   /* x.h */   int my_euros (void);   #endif

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

/* x.c */ #include "x.h"   #include <stdio.h>   #define F_PER_EURO 6.55957   int my_euros (void) { double euro;   for (euro = 0; euro < 500; euro += 20) { printf ("%6.2f %7.2f\n", euro, euro * F_PER_EURO); } return 0; }

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11

#include "x.h"   #include <stdio.h>   int main (void) { my_euros ();   return 0; }

---

Ce qui produit :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

0.00 0.00 20.00 131.19 40.00 262.38 60.00 393.57 80.00 524.77 100.00 655.96 120.00 787.15 140.00 918.34 160.00 1049.53 <...> 440.00 2886.21 460.00 3017.40 480.00 3148.59

---

Citation:

---

Envoyé par gege2061

Bien au contraire :

• C90 : impossible pas de mixe code/déclaration
• C99 : Ok, la variable n'existe plus à la de la boucle

---

Tu veux dire qu'en C99 on peut faire for(int i = 0; ...) ? Je ne trouve cela nul part dans le dernier draft de la norme...

Citation:

---

Envoyé par DaZumba

Citation:

---

Envoyé par gege2061

Bien au contraire :

• C90 : impossible pas de mixe code/déclaration
• C99 : Ok, la variable n'existe plus à la de la boucle

---

Tu veux dire qu'en C99 on peut faire for(int i = 0; ...) ? Je ne trouve cela nul part dans le dernier draft de la norme...

---

What are for loop intializers?
Il suffit d'essayer avec un compilateur bien réglé!

Citation:

---

Envoyé par DaZumba

Citation:

---

Envoyé par gege2061

Bien au contraire :

• C90 : impossible pas de mixe code/déclaration
• C99 : Ok, la variable n'existe plus à la de la boucle

---

Tu veux dire qu'en C99 on peut faire for(int i = 0; ...) ? Je ne trouve cela nul part dans le dernier draft de la norme...

---

Citation:

---

Envoyé par n1124

6.8.5 Iteration statements
Syntax
1 iteration-statement:
while ( expression ) statement
do statement while ( expression ) ;
for ( expressionopt ; expressionopt ; expressionopt ) statement
for ( declaration expressionopt ; expressionopt ) statement
Constraints
2 The controlling expression of an iteration statement shall have scalar type.
3 The declaration part of a for statement shall only declare identifiers for objects having
storage class auto or register.
Semantics
4 An iteration statement causes a statement called the loop body to be executed repeatedly
until the controlling expression compares equal to 0.
5 An iteration statement is a block whose scope is a strict subset of the scope of its
enclosing block. The loop body is also a block whose scope is a strict subset of the scope
of the iteration statement.

---

Merci pour le lien. Encore un ajout C99 que je n'utiliserai jamais...

Citation:

---

Envoyé par gege2061

Il suffit d'essayer avec un compilateur bien réglé!

---

Certes. Mon reglage par defaut de gcc inclus -ansi -pedantic, donc je ne risque pas d'utiliser ce genre de choses.

Edit: Merci Emmanuel, en survolant cette section, je n'avais pas observe le "declaration".

Citation:

---

Envoyé par DaZumba

Certes. Mon reglage par defaut de gcc inclus -ansi -pedantic, donc je ne risque pas d'utiliser ce genre de choses.

---

Suffit de remplacer le -ansi par -std=c99 ;)

Par contre pour ce qui ai du c++, mes vagues souvenirs sont inexactes puisque la norme a était revu et indique le même comportement qu'en C99 mais son utilisation fait couler le l'encre : Portee des variables pour boucle for