Discussion :

[a_lincoln54]

Bonjour à tous,

Le code suivant ne fonctionne pas (BCB C++) :

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
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
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
 
 
#include <vcl.h>
#include <string.h>
#include <conio.h>
#include <stdio.h>
#include <cstring.h>
#include <iostream.h>
#pragma hdrstop
 
 
#pragma argsused
int main(int argc, char* argv[])
{
 
//vecteurs d'entrée 
int const n = 50;
string InputVector[n] = {"0", "1e-3", "2e-5", "4e-3", "7e-7", "6e-4",
"1", "4e-3", "66e-5", "43e-3", "78e-7","12e-4","2", "2e-3", "10e-5", "20e-3", "90e-7", "70e-4"};
 
//vecteur de sortie
string OutputVector_1[6];
string OutputVector_2[6];
string OutputVector_3[6];
 
string CuVa = "";
 
	int p = -1;
	int r = 0;
 
//lecture du vecteur d'entrée
 
		for (int i=0; i <= n-1; i++)
		{
 
				CuVa = InputVector[i];
 
				cout << CuVa << "\t";
 
				if (CuVa == "0" || CuVa == "1" || CuVa == "2")
				{
				p++;
				r = 0;
				}
 
				if (p==0)
				OutputVector_1[r] = CuVa;
				if (p==1)
				OutputVector_2[r] = CuVa;
				if (p==2)
				OutputVector_3[r] = CuVa;
 
 
				r++;
 
 
		}
 
		// ecriture matricielle du vecteur de sortie
 
		int j = 0;
		for (int i=0; i <= p; i++)
		{
			j = 0;
			while(j <= 6)
			{
 
				if (i==0)
				{
				ShowMessage("val 1");
				CuVa = OutputVector_1[j];
				cout << CuVa;
 
				}
				if (i==1)
				{
				ShowMessage("val 2");
				CuVa = OutputVector_2[j];
				cout << CuVa;
				}
				if (i==2)
				{
				ShowMessage("val 3");
				CuVa = OutputVector_3[j];
				cout << CuVa;
				}
 
			 //cout << CuVa;
 
			 if (j != 6	)
			 cout << "\t";
			 else
			 cout << "\n";
 
			 j++;
			}
 
 
getch();
 
}

Ce code permet d'effectuer la chose suivante. Etant donné un vecteur string d'entrée qui contient un certain nombre d'éléments dont les éléments "0" "1" et "2", le but est de pouvoir classer ce vecteur dans trois vecteurs de sortie tel que chaque vecteur de sortie commence par l'un des éléments "0" "1" et "2".
Le problème est que la séquence de restitution des données "classées" ne fonctionne pas puisque les éléments restitués sont tous inexistants. On a en effet la sortie suivante :

0 1e-3 2e-5 4e-3 7e-7 6e-4 1 4e-3 66e-5 43e-3 78e-7 12e-4 2 2e-3 10e-5 20e-3 90e-7 70e-4

pour le premier cout - tout a fait normal puisque l'on relit le vecteur d'entrée - et seulement :

2 2e-3 10e-5 20e-3 90e-7 70e-4

pour la suite, alors que l'on devrait avoir :

0 1e-3 2e-5 4e-3 7e-7 6e-4
1 4e-3 66e-5 43e-3 78e-7 12e-4
2 2e-3 10e-5 20e-3 90e-7 70e-4

Grâce au ShowMessage, il est possible de vérifier que les trois vecteurs de sortie sont lus.

C'est certainement une erreur bidon, mais je ne vois pas du tout où elle pourrait être.

Je vous remercie pour votre aide.

[NewbiZ]

chaque vecteur de sortie commence par l'un des éléments "0" "1" et "2".

Code :

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

1
2
CuVa = InputVector[i];
if (CuVa == "0" || CuVa == "1" || CuVa == "2")

Tu fais la comparaison sur l'intégralité de la chaine, pas son premier caractère.

[jmv]

salut,

Code :

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

1
2
CuVa = InputVector[i];
if (CuVa == "0" || CuVa == "1" || CuVa == "2")

Tu fais la comparaison sur l'intégralité de la chaine, pas son premier caractère.

c'est justement ce que a_lincoln54 veut faire.

Il y a des erreurs dans ton code, une fois corrigé, ça troune sur DEV-C++

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
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
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
// utilisation des fichiers standards actuels
#include <string>
#include <iostream>
using namespace std;
 
int main(int argc, char* argv[])
{
 
//vecteurs d'entrée 
int const n = 50;
string InputVector[n] = {"0", "1e-3", "2e-5", "4e-3", "7e-7", "6e-4",
"1", "4e-3", "66e-5", "43e-3", "78e-7","12e-4","2", "2e-3", "10e-5", "20e-3", "90e-7", "70e-4"};
 
//vecteur de sortie
string OutputVector_1[6];
string OutputVector_2[6];
string OutputVector_3[6];
 
string CuVa = "";
 
	int p = -1;
	int r = 0;
 
//lecture du vecteur d'entrée
 
                // il n'y a que 18 places dans les vecteurs de sorties
		for (int i=0; i < 18; i++)
		{
 
				CuVa = InputVector[i];
 
				cout << CuVa << "\t";
 
				if (CuVa == "0" || CuVa == "1" || CuVa == "2")
				{
				p++;
				r = 0;
				}
 
				if (p==0)
				OutputVector_1[r] = CuVa;
				if (p==1)
				OutputVector_2[r] = CuVa;
				if (p==2)
				OutputVector_3[r] = CuVa;
 
 
				r++;
 
			
		}
 
		// ecriture matricielle du vecteur de sortie
               cout << endl; // Un retour à la ligne pour faire joli
		int j = 0;
		for (int i=0; i <= p; i++)
		{
			j = 0;

                        // L'indice va de 0 à 5
			while(j <= 5)
			{
 
				if (i==0)
				{
//				ShowMessage("val 1");
				CuVa = OutputVector_1[j];
				cout << CuVa;
 
				}
				if (i==1)
				{
//				ShowMessage("val 2");
				CuVa = OutputVector_2[j];
				cout << CuVa;
				}
				if (i==2)
				{
//				ShowMessage("val 3");
				CuVa = OutputVector_3[j];
				cout << CuVa;
				}
 
			 //cout << CuVa;
 
			 if (j != 5 ) // Meme probleme d'indice
			 cout << "\t";
			 else
			 cout << "\n";
 
			 j++;
			}
        }
			
 
    cin.ignore();
 
}

Pourquoi n'utilise-tu pas la class std::vector ?

a+
jmv

[a_lincoln54]

Bonjour,

Merci beaucoup pour votre réponse pertinente. Cela fonctionne maintenant !! Je pense que le fait de lire des cases de tableaux de string qui ne contiennent rien sont probablement à l'origine de ça (puisque dans la boucle de lecture, j'avais mis

Code :

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

 for (int i=0; i <= (n-1); i++)

avec n = 18; ce que vous avez corrigé en

Code :

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

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

. En fait, j'avais déjà essayé cette manip. mais il y avait toujours une ligne qui disparaissait lors de la lecture des données "classées". Il y a aussi le

Code :

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

 namespace std

, que je n'incluais jamais car cela marchait toujours sans !!

Pour ce qui est de la classe vector, je ne savais pas que cela existait. C'est vrai que cela a l'air d'être bien pratique d'autant plus que - si je reviens à mon code - le vecteur d'entrée est ici fixé, mais est en réalité indéterminé. Ce code fait partie d'une application plus vaste dont l'une des fonctions est de lire des données à partir d'un fichier texte - de taille à priori quelconque...

Comme je fais du développement à finalité scientifique, je n'ai pas trop le temps de chercher à optimiser le code (c'est un grand tord que je reconnais volontiers) et donc de trouver de nouvelles classes et méthodes. On se dit que tout marchera avec les bases (ultra classiques) du C standard mais bon, cet exemple démontre que l'on peut mieux faire...

Encore merci.

cdt