probleme boucle infini dans tableau

Version imprimable

Bonjour

J'ai un programme qui calcule le prix de 3 panier différents avec 5 produits chacun.

Il faut que tableau qui s'affiche donne le prix d'un panier dans une supermarché ( sur 1 colonne ) , le prix moyen d'un produit ( sur 1 ligne )

J'aurais donc un tableau 6 lignes et 4 colonnes .

Problème arrivé au 5 eme produit du 3 eme supermarché , je saisi mon dernier et mon programme et affiche sur une ligne 4 valeurs sortie de nul part " 1 6 11 6 " ou " 1 6 11 2.33333 "

Je ne vois pas du tout vient l'erreur ?

voici le programme

Code:

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
# include <iostream.h>
 
int main()
 
{ 
	//declaration des variables 
 
	const int t1=6,t2=4;
	float super[t1][t2],ps,pp; 
	int l,c;
 
	// instructions principales
 
	// remplir le tableau
 
	cout<<" remplir le tableau : \n ";
 
	for (c=0;c<=t2-2;c++)
	{
        cout<<" supermarche "<<c<<" : \n";
		for(l=0;l<=t1-2;l++)
		{
			cout<<" produit "<<l<<" : ";
			cin>>super[l][c];
		}
	}
 
	cout<<" essai ";
 
	//affichage tableau 
 
	for (c=0;c<=t2-2;c++)
	{
		for(l=0;l<=t1-2;l++)
		{
			cout<<super[l][c];
			cout<<"\t";
		}
		cout<<"\n";
	} 
 
	// calcul prix panier par supermarche
 
	for (c=0;c<=t2-2;c++)
	{
        ps=0;
		for(l=0;l<=t1-2;l++)
		{
            ps=super[l][c]+ps;
            super[5][c]=ps;
        }
    }
 
	// calcul prix moyen produit  
 
    for (l=0;l<=t1-2;l++)
	{
        pp=0;
		for(c=0;c<=t2-2;c++)
		{
            pp=super[l][c]+pp;
            super[l][3]=(pp/3);   	
		}
	}
 
	// affichage tableau
 
	for (l=0;l<t1;c++)
	{
		for(c=0;c<t2;c++)
		{
			cout<<super[l][c];
			cout<<"\t";
		}
		cout<<"\n";
	}
 
    cin>>pp;
	return 0; 
}

30/01/2011, 15h01
ptyxs

Puisque vous êtes en C++ et non en C, pourquoi n'utilisez-vous std::vector au lieu d'un tableau ?
30/01/2011, 15h09
bertry
Salut,
Dans toutes tes boucles :
Code:

1 2 3 4 5 for (c=0;c<=t2-2;c++) { for(l=0;l<=t1-2;l++) { ...
tu n'initialise jamais ton tableau pour c = 5 et l = 3 puisque tu t'arrête à c = t1 - 2 = 4 et l = t2 - 2 = 2! Pourquoi changer brutalement tes boucles au dernier affichage du tableau :
Code:

1 2 3 4 5 for (l=0;l<t1;c++) { for(c=0;c<t2;c++) { ...
Tes valeurs sont forcément "sorties de nulle part", puisque tu ne les initialisent jamais!!
30/01/2011, 15h40
Invité

Citation:

Envoyé par ptyxs

Puisque vous êtes en C++ et non en C, pourquoi n'utilisez-vous std::vector au lieu d'un tableau ?

Je ne sais pas encore utiliser std :: vector a mon niveau , et je ne peux pas utiliser autre chose que les boucles for ( vu que c'est le but de l'exercice et imposé )

Citation:

Dans toutes tes boucles :
Code :

for (c=0;c<=t2-2;c++)
{
for(l=0;l<=t1-2;l++)
{
...

tu n'initialise jamais ton tableau pour c = 5 et l = 3 puisque tu t'arrête à c = t1 - 2 = 4 et l = t2 - 2 = 2! Pourquoi changer brutalement tes boucles au dernier affichage du tableau :
Code :

for (l=0;l<t1;c++)
{
for(c=0;c<t2;c++)
{
...

Tes valeurs sont forcément "sorties de nulle part", puisque tu ne les initialisent jamais!!

Non c'est t1 = 6 et t2 = 4 , je ne vois pas ou voulais en venir avec c et l ? Ces 2 lettres sont des indices qui varient

Pourriez donner un exemple de declaration ?

Merci d'avance
30/01/2011, 15h43
ptyxs

Citation:

Envoyé par tanaka59

Je ne sais pas encore utiliser std :: vector a mon niveau , et je ne peux pas utiliser autre chose que les boucles for ( vu que c'est le but de l'exercice et imposé )

std::vector est plus simple d'utilisation qu'un tableau à mon avis, justement ! C'est en partie pour cela qu'on l'a inventé, du reste !
En outre on peut parfaitement le parcourir avec une boucle for !

Tu es apparemment victime d'un enseignement mal conçu du C++....
30/01/2011, 15h56
ptyxs

J'ajoute une autre remarque : ton programme contient des "nombres magiques" (des constantes numériques) le mieux est de passer par des constantes ayant un nom intelligible si l'on veut s'y retrouver, du genre :

Code:

const int prix = 300;

....

Code:

depense= prix*nb_articles

(ce sont de simples exemples, pour donner une idée de ce que j'ai en tête, je ne veux pas dire que ces deux lignes doivent se retrouver dans ton programme, bien entendu)
D'autre part, tu utilises des variables dont le nom est lui aussi inintelligible (t, l etc...) : donne des noms compréhensibles à tes variables (sauf pour les index d'une boucle où l'on utilise d'ordinaire les lettre i, j etc.).

La conjonction de ces deux travers rend ton programme illisible pour les autres et probablement même pour toi !! Passer un peu de temps à choisir convenablement des noms de variable en vaut tout à fait la peine.

D'autre part en C++ on ne déclare pas les variables en bloc en tête du programme, mais au moment où on s'apprête à les utiliser ! Cela aussi accroît la lisibilité du programme pour toi et pour les autres.
Une boucle for, par exemple, contiendra donc la déclaration de son index, comme ceci :

Code:

for (int i = 0; i <= max; i++)

ou plus rigoureusement si tu parcours un tableau ou un std::vector :

Code:

for (size_t i = 0; i <= max; i++)
30/01/2011, 17h12
Invité

Citation:

Envoyé par ptyxs

std::vector est plus simple d'utilisation qu'un tableau à mon avis, justement ! C'est en partie pour cela qu'on l'a inventé, du reste !
En outre on peut parfaitement le parcourir avec une boucle for !

Tu es apparemment victime d'un enseignement mal conçu du C++....

Je confirme , le base que l'on nous enseigne on 10 ans d'age :(
30/01/2011, 19h10
Emmanuel Deloget

Citation:

Envoyé par tanaka59

Je confirme , le base que l'on nous enseigne on 10 ans d'age :(

C'est qu'avant de courrir, il faut quand même apprendre à marcher. On ne peut pas expliquer l'opérateur [] et en quoi std::vector<> est mieux qu'un tableau sans expliquer les tableaux d'abord :)
30/01/2011, 21h17
JolyLoic

Citation:

Envoyé par Emmanuel Deloget

C'est qu'avant de courrir, il faut quand même apprendre à marcher.

Dans son dernier livre, Bjarne reprend cette analogie, et je suis assez d'accord avec lui : Quand on regarde un enfant qui apprend à se mouvoir, on se rend compte qu'en fait, il apprend à courir bien avant que d'apprendre à marcher. Et bien entendu, il tombe à de multiples reprises, mais ce n'est pas grave, il se relève et a progressé. Ce n'est que bien plus tard quand il aura acquis une bonne expérience qu'il pourra maîtriser finement ses jambes pour mettre en œuvre l'équilibre subtil nécessaire à la marche.

J'enseigne dans les cours que je donne comment utiliser vector avant de parler de tableau C-style. Pour ça, il n'est nul besoin de détailler la surcharge des opérateurs, ni de faire un comparatif entre ces deux solutions. Il suffit d'expliquer comment utiliser vector, ce qui est bien plus simple que d'expliquer comment utiliser un tableau C-style.

L'apprentissage des tableaux C-style vient plus tard, au moment où je montre comment créer ses propres classes et comment mettre en place une sémantique de valeur.

Bien entendu, l'hétérogénéité des niveaux étant une constante dans des cours d'informatiques, certains ont déjà quelques notions de C ou de C++, et veulent utiliser les tableaux C-style dans les exercices que je donne. A ceux-là, et à ceux-là uniquement, j'explique des choses en avance de phase.