probleme de segmentation

Version imprimable

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:

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;
            }

04/07/2006, 07h40
joellel
Voilà la ligne coupable:
Code:

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:

1 2 scanf("%d",&T[i]);
04/07/2006, 08h24
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 :

Citation:

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 :

Citation:

warning: return type defaults to 'int'

D'ailleur au passage, pour le retour, préfère return EXIT_SUCCESS à la place du return 0.
04/07/2006, 08h54
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.
04/07/2006, 09h11
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).
04/07/2006, 09h21
PRomu@ld

Citation:

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

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

Citation:

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 ...
04/07/2006, 10h14
Gnux

Citation:

Envoyé par PRomu@ld

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

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 :mouarf: :aie:

Citation:

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
04/07/2006, 10h19
gege2061

Citation:

Envoyé par PRomu@ld

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

Citation:

Envoyé par PRomu@ld

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

C'est un euphémisme :mouarf:
Notepad2 fait ça très bien :ccool:
04/07/2006, 10h24
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...

Citation:

Envoyé par lorie75

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:

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.

04/07/2006, 11h04
PRomu@ld
Citation:

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:

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:

1 2 |x| <= x*x
04/07/2006, 11h42
Luther13
Citation:

Envoyé par lorie75

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)
Citation:
Envoyé par Emmanuel Delahaye

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.
04/07/2006, 11h50
Emmanuel Delahaye
Citation:
Envoyé par PRomu@ld

Ca n'est pas portable non plus ...

Code:

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:

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 !
04/07/2006, 11h52
Emmanuel Delahaye

Citation:

Envoyé par Luther13

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 !)
06/07/2006, 09h35
lorie75

merci a vous tous. j'ai bien notée tout ça.:D
10/07/2006, 20h31
Raiden

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