Discussion :

[Mysti¢]

Bonjour, donc je continue ma petite série d'exercices ^^
La il faillait faire un programme qui affiche une pyramide avec le nombre de ligne qu'on a spécifié au lancement du programme.
La console doit donc afficher:

Code :

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

1
2
3
4
5
6
7
8
Nombre de Lignes ?
>> 5                                 /* Choisi par l'utilisateur */
 
*
**
***
****
*****

Donc mon 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
#include <stdio.h>
main() {
  int i;
  int lines;
  printf("Choisissez le nombre de lignes que devra faire la pyramide \n>>");
  scanf("%i", &lines);
  for (i=0; i<=lines; i++)
    printline(i);
}
 
int printline(x) {
  int j;
  char etoile;
  etoile = '*';
  for (j = 0;  j < x; j++)
    printf("%c", etoile);
  printf("\n");
 }

Le code est fonctionnel, mais je voudrais savoir si il y'a pas des choses à éviter au niveau de la syntaxe voir inutiles..., la structure même du code (lisibilité).
Si c'est mieux de séparer en fonctions même pour les petites opérations...
Voilà

[DaZumba]

Tout d'abord, on ecrit syntaxe.
Ensuite:
dans main():
- le prototype de main() est incorrect;
- que se passe-t-il si l'utilisateur entre autre chose qu'en entier? Quel est le comportement de scanf()? Comment y remedier? (La reponse a cette derniere question est complexe);
- toute fonction utilisee doit etre definie avant son utilisation. Il manque donc le prototype de printline();
- main() doit retourner un entier.

dans printline():
- La definition de printline() est incorrecte, il manque le type de x;
- x n'est pas un nom de variable pratique, on ne sait pas a quoi elle correspond.

en general:
- meme si les accolades autour du bloc defini par for() sont optionnelles lorsqu'il n'y a qu'une expression dans le bloc, il est preferable de toujours les mettre (lisibilite, maintenance).

[Mysti¢]

Tout d'abord, on ecrit syntaxe.

Autant pour moi :X

le prototype de main() est incorrect;

Explicitement (pour un débutant) ça donne?

main() doit retourner un entier

Je vois souvent return 0; à la fin de certaines fonctions, même si j'en ai pas encore vraiment compris l'utilité.

Sinon je ne sais pas si c'est à force de coder, si t'es bilingue ou encore si tu as un clavier QWERTY, mais les accents ça existe.

Apart ça, merci pour tes recommandations que je vais d'ailleurs prendre en compte

[AL1986]

Explicitement (pour un débutant) ça donne?

Le fait est qu'il y a une norme à respecter ; il y a deux formes canoniques pour le main :

Code C :

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

int main(void)

et

Code C :

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

int main(int argc, char **argv)

Il y a des compilateurs qui refusent de compiler si le main n'est pas écrit de cette façon et y en a qui émettent des warnings.

Je vois souvent return 0; à la fin de certaines fonctions, même si j'en ai pas encore vraiment compris l'utilité.

Si mes souvenirs sont bons, le 0 correspond à la constante EXIT_SUCCESS qui signifie que le programme s'est exécuté comme il le faut.

- toute fonction utilisee doit etre definie avant son utilisation. Il manque donc le prototype de printline();

Il vaut mieux définir un .h pour chaque .c

[DaZumba]

Explicitement (pour un débutant) ça donne?

Il y a de nombreux exemples dans ce forum. La norme donne deux formes pour le prototype de main():

Code :

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

1
2
 
int main(void)

et

Code :

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

1
2
 
int main(int argc, char *argv[])

La seconde forme permet de recuperer les arguments donne a l'executable. Tu la rencontreras plus tard dans ta formation.

Je vois souvent return 0; à la fin de certaines fonctions, même si j'en ai pas encore vraiment compris l'utilité.

Puisque main() est declare comme retournant un entier, il faut qu'on retourne cet entier. Les valeurs possibles sont 0, EXIT_SUCCESS et EXIT_FAILURE (ces deux constantes etant definies dans l'entete standard stdlib.h). Pour l'instant, tu peux t'en tenir a zero.

Sinon je ne sais pas si c'est à force de coder, si t'es bilingue ou encore si tu as un clavier QWERTY, mais les accents ça existe.

Je travaille et je vis en Angleterre. Mes claviers sont en effet QWERTY.

[DaZumba]

Il vaut mieux définir un .h pour chaque .c

Je pense que Mysti¢ verra la compilation separee un peu plus tard. De plus, toutes les fonctions d'une unite de compilation ne sont pas declarees dans un entete (cf. fonctions qualifiees avec static).

[Emmanuel Delahaye]

Si mes souvenirs sont bons, le 0 correspond à la constante EXIT_SUCCESS qui signifie que le programme s'est exécuté comme il le faut.

A vrai dire exit(EXIT_SUCCESS) a la même sémantique que exit(0). Mais EXIT_SUCCESS n'a pas forcément la valeur 0. Ca dépend du système.

[Mysti¢]

Ok, merci pour vos réponses. Ca me permettra de partir sur de bonnes bases