Discussion :

[lorie75]

bonjour a tous,
je suis une nouvelle inscrite ici, et je suis nouvelle dans la programmation, et l'un de mes premiers programmes en c ne fonctionne pas! je voudrais savoir pourquoi?(je travaille sous windows xp)
merci de votre aide .
voila mon programme:

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
  # include<stdio.h>
  # include<stdlib.h>
   # define N 10
            main (void)
            {
                 int T[N];
                 int i;
                   printf("donnez %d nombres entiers:\n",N);
 
 
                 for (i = 0; i < N; i++)
 
                   scanf("%d",T[i]);
 
                   printf("Nombre     Carré\n");
                 for (i=0;i<N;i++)
                  printf("%d          %d\n",T[i],T[i]*T[i]);
                 system("pause");
            return 0;
            }

[joellel]

Voilà la ligne coupable:

Code :

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

1
2
 
scanf("%d",T[i]);

Pour pouvoir charger la valeur saisie dans T[i], la fonction scanf() a besoin de connaitre son adresse en mémoire. Il faut donc lui passer l'adresse de T[i], de la manière suivante:

Code :

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

1
2
scanf("%d",&T[i]);

[PRomu@ld]

Je ne reviendrai pas sur l'erreur (qui vient bien du scanf) mais sur deux choses. Comme tu débutes, il faut prendre de bonnes habitudes.

Tout d'abord, ton erreur aurait pu être détectée à la compilation, en effet, lorsque je compile le programme sur ma machine j'ai ceci :

warning: format '%d' expects type 'int *', but argument 2 has type 'int'

Il faut donc absolument que tu configures ton compilateur pour lui faire afficher le maximum de message d'erreur et d'avertissement.

Ensuite, il y a une deuxième chose : la construction de ta fonction main, elle doit retourner un entier, il faut donc que tu lui précises. Par exemple ce genre de warning peut te mettre la puce à l'oreille :

warning: return type defaults to 'int'

D'ailleur au passage, pour le retour, préfère return EXIT_SUCCESS à la place du return 0.

[Médinoc]

De plus, sache que conseiller la fonction scanf() aux débutants est une pratique d'un autre âge : Cette fonction n'est pas une fonction pour les débutants et est assez difficile à utiliser correctement.
Préfère-lui la fonction fgets() pour lire une chaîne de caractères sur l'entrée standard et la fonction strtol() pour la convertir en entier.

[Gnux]

Salut
C'est pas très pratique là de plus. Pourquoi ne pas inclure math.h et utiliser la fonction sqrt() ? Tu devrais aussi prendre l'habitude d'indenter ton code, soit en utilisant un éditeur qui le fait pour toi (comme Vim) soit en utilisant un prog qui peut le faire pour toi (comme GNU Indent).

[PRomu@ld]

Pourquoi ne pas inclure math.h et utiliser la fonction sqrt() ?

Parce que ça ne répond pas au problème ! Le programme affiche les carrés des nombres entrés, pas leur racine ...

soit en utilisant un éditeur qui le fait pour toi (comme Vim)

Ca existe sous windows ? (en natif, j'entend bien, pas en installant cygwin ou des outils de ce genre ...) En plus, ça n'est pas l'éditeur le plus convivial qui soit ...

[Gnux]

Parce que ça ne répond pas au problème ! Le programme affiche les carrés des nombres entrés, pas leur racine ...

Pardon là j'avoue il était un peu trop tot pour moi encore. J'ai vu ça après mais bon...C'était pour voir si tu suivais

Ca existe sous windows ? (en natif, j'entend bien, pas en installant cygwin ou des outils de ce genre ...) En plus, ça n'est pas l'éditeur le plus convivial qui soit ...

J'utilise Vim là sous Windows et je n'ai pas installer de lib cygwin ou autre. Ca fonctionne bien comme ça. L'apprentissage est pas facile au début mais ça vaut vraiment le coup à la fin

[gege2061]

Ca existe sous windows ? (en natif, j'entend bien, pas en installant cygwin ou des outils de ce genre ...)

http://www.vim.org/download.php#pc

En plus, ça n'est pas l'éditeur le plus convivial qui soit ...

C'est un euphémisme
Notepad2 fait ça très bien

[Médinoc]

Si on rentre dans les discussion sur les éditeurs:

• Unixoïde, mode console : vi (connais pas le fonctionnement/l'interface de vim).
• Unixoïde, mode graphique : nedit
• Windows : Un EDI ou UltraEdit

PS: Pour le carré d'un entier, je pense que la bête multiplication est préférable aux fonctions de math.h qui utilisent des flottants...

[Emmanuel Delahaye]

j<...> l'un de mes premiers programmes en c ne fonctionne pas! je voudrais savoir pourquoi?(je travaille sous windows xp)

• Ta présentation est horrible !
• scanf() attend l'adresse d'un int.
• En C99, les types retournés par les fonctions doivent être explicites.
• Je recommande l'usage systématique des {} dans les structures de code.
• le system("pause"); de la fin est inutile si tu utilises autre chose que Dev-C++ (Code::Blocks, par exemple)

Ton code corrigé :

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
 
#include<stdio.h>
#include<stdlib.h>
#define N 10
 
int main (void)
{
   int T[N] = {0};
   int i;
   printf("donnez %d nombres entiers:\n", N);
 
   for (i = 0; i < N; i++)
   {
      scanf("%d", T + i);
   }
   printf("Nombre Carre\n");
   for (i = 0;i < N;i++)
   {
      printf("%6d %5d\n", T[i], T[i]*T[i]);
   }
   return 0;
}

Nota. Il est déconseillé d'utiliser scanf() si on ne le maitrise pas. (tapes a<enter> pour voir...). Je recommande une combinaison de fgets() et de la ou les fonctions de nettoyage et/ou de conversions qui vont bien.

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

A lire et poser des questions si tu ne comprends pas.

[PRomu@ld]

le system("pause"); de la fin est inutile si tu utilises autre chose que Dev-C++ (Code::Blocks, par exemple)

Ca n'est pas portable non plus ...

Code :

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

printf("%6d %5d\n", T[i], T[i]*T[i]);

Pourquoi affecter 6 pour le nombre et 5 pour son carré ? Le contraire aurait été plus judicieux, on a en effet :

Code :

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

1
2
 
|x| <= x*x

[Luther13]

je suis une nouvelle inscrite ici, et je suis nouvelle dans la programmation, et l'un de mes premiers programmes en c ne fonctionne pas! je voudrais savoir pourquoi?(je travaille sous windows xp)

• Ta présentation est horrible !...

la Pauvre ! Mais je trouve pas ça si horrible que ça.
Juste trop naturel et la nature n'est pas parfois bien faite.

[Emmanuel Delahaye]

Ca n'est pas portable non plus ...

Code :

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

printf("%6d %5d\n", T[i], T[i]*T[i]);

Pourquoi affecter 6 pour le nombre et 5 pour son carré ? Le contraire aurait été plus judicieux, on a en effet :

Code :

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

1
2
 
|x| <= x*x

Arf, rien à voir ! Question de présentation. Pour être aligné avec "Nombre" et "Carre"...

Sinon, oui, on peut définir des largeurs de colonnes d'une façon moins ... empirique !

[Emmanuel Delahaye]

la Pauvre ! Mais je trouve pas ça si horrible que ça.
Juste trop naturel et la nature n'est pas parfois bien faite.

Arf ! Horreur et incompréhension !

Je ne parlais pas de cette présentation là, sur laquelle je n'ai pas d'avis particulier, mais de la présentation du code (et de l'application, par ailleurs), évidemment (c'est quand même un des sujets du forum !)

[lorie75]

merci a vous tous. j'ai bien notée tout ça.

[Raiden]

N'oublie pas un petit si ton probleme est résolu