Discussion :

[crazy_zakaria]

Bonjour tout le monde,
Je viens de créer un programme qui lit les éléments d'un tableau, calcule leur somme, et détermine leur min et max
Voici l'algorithme:

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
#include <stdio.h>
#define TMax 20
main()
{
	int i=0,T[i],N,max,min,imax,imin,som=0;
	//Détérmination de la taille du tableau
	do
		{
			printf("Veuillez entrer la taille du tableau\n");
			scanf("%d",&N);
		}
	while (N<0||N>TMax);
	//Écriture des éléments du tableau
	for (i=0;i<N;i++)
		{
			printf("Veuillez donner le %d eme élément\n",i);
			scanf("%d",&T[i]);
		}
	//Calcul de la somme
	for (i=0;i<N;i++);
		{
			som=som+T[i];
		}
	printf("La somme est: %d\n",som);
	//Détérmination du min et du max
	for (i=0;i<N;i++)
		{
			min=max=T[0];
			if (T[i]>max) 
				{
					max=T[i];
					imax=i;
				}
			if (T[i]<min) 
				{
					min=T[i];
					imin=i;
				}
		}
	printf("Le max est: %d et son indice est: %d\n",max,imax);
	printf("Le min est: %d et son indice est: %d\n",min,imin);
return(0);
}

Mais le résultat est un peu bizarre
Regardez un peu:

Le compilateur me donne aussi une alerte disant: " warning return type defaults to 'int' "
Quelle est mon erreur au juste?
Et merci d'avence

[Médinoc]

La déclaration de T devrait avoir la taille TMax et non pas i.

De plus, main() doit être déclarée ainsi, et retourner un int:

Code C :

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

int main(void)

[SofEvans]

Code :

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

1
2
3
4
5
	//Calcul de la somme
	for (i=0;i<N;i++);
		{
			som=som+T[i];
		}

Enleve aussi le ; a la fin du for.

Pour info

Code :

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

for (i=0 ; i<N ; i++);

revient a ecrire

Code :

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

1
2
3
for (i=0 ; i<N ; i++)
{
}

Ton code peut s'ecrire de la maniere suivante :

Code :

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

1
2
3
4
 
    //Calcul de la somme
    for (i=0;i<N;i++)
        som += T[i];

[crazy_zakaria]

Merci les amis,
Mais reste encore l'indice du min qui me donne toujours un nombre astronomique 134520820
Que faut-il faire dans ce cas?

[Pouet_forever]

Je pense que ton problème doit venir de là :

Code :

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

int i=0,T[i]

Tu déclares un tableau de 0 cases ... lol

[Médinoc]

Oui, je l'ai déjà dit.
Et en plus, ça ne compile qu'en C99.

gcc est passé à "C99 par défaut" ou quoi?

[crazy_zakaria]

Non j'ai corrigé mais le problème persiste encore

[Pouet_forever]

Essaye de sortir "min=max=T[0];" de ta boucle ^^

Code :

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

1
2
for (i=0;i<N;i++)
    min=max=T[0];

@Médinoc : Ah oui j'avais pas vu
Chez moi GCC compile en C99 par défaut

[crazy_zakaria]

Qu'est-ce que ça veut dire C99? Est-ce que ça a un rapport avec le fait que je travaille sous Ubuntu ou quoi? Si oui, que faire?
Soyez gentils avec moi svp je ne suis qu'un débutant

[Médinoc]

C99 est le "nouveau" standard du C, qui notamment introduit les "Variable-Length Arrays": Des tableaux donc la taille n'a pas besoin d'être connue à la compilation.

Malheureusement, ça engendre énormément de confusion, et on se retrouve facilement (comme ici) avec des tableaux qui ont une mauvaise taille.

[SofEvans]

Repost nous ton code, car chez moi (apres plusieurs modification) cela marche sans probleme.

Donc, il y a quelque chose qui ne va pas.