Discussion :

[sophie284]

Salut à tous !

Je suis actuellement sur un devoir universitaire en programmation C++. Mais je bloque ne maîtrisant pas du tout ce langage je vous demande un peu d'aide.

Sujet : Le traitement consiste à ouvrir le fichier log.csv et recopier dans un autre fichier (le fichier log-copie.csv) toutes les lignes du fichier log.csv. Par contre, il ne faut pas recopier plusieurs fois les lignes contenant le même numPièce et la même ref mais prendre seulement la ligne contenant la quantité la plus élevée. Par exemple, la ligne N°7 et la ligne N°8 sont identiques car elles contiennent le même numPièce qui est 1 et la même ref qui est 14. Il faut garder la ligne 7 car elle contient la quantité la plus élevée, ici 9. La ligne à supprimer est donc la ligne N°8.

Je vous joint le document log.csv et voici, la première partie de mon code ou j'arrive à ouvrir le dossier, extraire les valeur, et nommé chaque valeur de chaque colonne. J'aimerai garder intacte la partie de code déjà réalié, puisqu'elle marche.

log.xlsx

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
 
 
#include <fstream> 
#include <string>
#include <iostream> // utilisation de la bibliothèque iostream : inputoutput stream
using namespace std; // utilisation de l'espace de nommage standard
 
int main()
{
 
	ifstream fichier ("log.csv", ios :: in); // on ouvre le fichier en lecture
	string numPiece;
	string ref ;
	string Quantite;
	int PositionPointVirgule;
	int PositionPointVirgule2;
	int Taille;
	string LignePrecedente;
 
	if(fichier) // si l'ouverture à reussi
	{
		string ligne; 
		while(getline(fichier, ligne)) // tant que l'on peut mettre la ligne dans "LigneLu"
			{
 
				cout<<ligne<<endl; // On affiche 
				PositionPointVirgule=ligne.find(';');
				PositionPointVirgule2=ligne.rfind(';');
				Taille = ligne.length();
				//cout<<"Taille: "<<Taille-1<<endl;
				//cout<<"PositionPointVirgule: "<<PositionPointVirgule2<<endl;
				numPiece=ligne.substr(0,PositionPointVirgule);
				ref=ligne.substr(PositionPointVirgule+1,PositionPointVirgule2-PositionPointVirgule-1);
				Quantite=ligne.substr(PositionPointVirgule2+1,(Taille-1)-PositionPointVirgule2);
				//cout << "Quantite : "<<Quantite<<endl;
 
				// if ligne précédante à la même référence, comparer les quantité de toutes les réf identiques
				// Garder uniquement la ligne qui à la quantité la plus grande 
				// côpier la ligne dans le dossier
			//}
		//do
			//{
				//LignePrecedente = ligne;
					//cout<<"Ligne précédente est :"<<LignePrecedente<<endl;
 
 
		fichier.close (); // on ferme le fichier
		}
	}
	else // sinon 
		cerr << "Impossible d'ouvrir le fichier" << endl;
	return 0;
}

Merci d'avance !

[bacelar]

Vous êtes complètement perdu.
Ici c'est du C++/CLI, dialecte du C++ spécifique à Windows .NET.
(à moins que le fichier xlsx en PS n'est pas une fausse manipulation, car C++/CLI permet d'avoir accès à .NET donc à facilement décortiquer les fichiers Excel 2003 et supérieur)

using namespace std; // utilisation de l'espace de nommage standard

être compatible avec des compilateurs ne respectant pas la première norme C++ de 1998, c'est dans le cahier des charges de votre code ?
Il a quel age votre prof ?
90 ans ?
Dégagez moi cette cochonnerie et utilisez "std::" là où c'est nécessaire. Mort bleu.

Le format CSV est bien plus complexe que ce que vous semblez connaitre.
Donc, pas la peine de réinventer une roue carrée :
http://lmgtfy.com/?q=c%2B%2B+csv+parser+semicolon

(gestion de l'en-tête, etc..., à vérifier, il y a souvent des roues carrées sur Internet )

Déjà, vous utilisez des std::string et pas des char[], votre prof a 90ans mais il connait la différence entre le C et le C++.

J'aimerai garder intacte la partie de code déjà réalisé, puisqu'elle marche.

Arrêtez d'être pusillanime.
Les zombis aussi marchent, c'est pas pour cela qu'ils battent le record du 100 m.

Votre code est bien trop fragile et il y manque l'essentiel (cast en entier, utilisation de map pour le tri, etc...).
A la rigueur, il servira à comparer les parsers de fichier CSV que vous trouverez sur le Net.

Avant de coder, vous deviez avoir fait l'algorithme à implémenter sur papier.

Utilisez des map de tuple<int,int> pour stocker le maximum de la paire <numPièce,ref>, puis utilisez le contenu de cette map pour construire le fichier en sortie.
https://fr.cppreference.com/w/cpp/container/map
https://fr.cppreference.com/w/cpp/utility/tuple

P.S.: c'est le genre de truc qu'on fait avec des outils d'ETL, pas avec un programme console C/C++ fait sur un coin de table. (Encore des exercices débiles)

[sophie284]

La première partie du code à été réalisé avec l'aide de notre prof, une modification de celui ci, voudrait dire qu'on à forcement demander de l'aide. Mais je comprend votre point de vue, il n'a pas 90 ans peut être 45 ans. Son cours n'est pas développé, nous avons seulement une deux séances de cours et il nous demande un programme comme celui-ci pour nous évaluer. Problème on ne maitrise pas du tout ce langage.

MAis votre message ne m'avance pas beaucoup sur ce que je dois faire

[bacelar]

MAis votre message ne m'avance pas beaucoup sur ce que je dois faire

What, c'est quasiment tout algo. que je donne :

Utilisez des map de tuple<int,int> pour stocker le maximum de la paire <numPièce,ref>, puis utilisez le contenu de cette map pour construire le fichier en sortie.
https://fr.cppreference.com/w/cpp/container/map
https://fr.cppreference.com/w/cpp/utility/tuple

Vous lisez une ligne, vous cherchez la paire <numPièce,ref> dans la map, :
elle y n'est pas, vous ajoutez la paire comme clé de l’élément de la map et la valeur de la troisième colonne du fichier comme valeur de l’élément de la map.
elle y est, vous vérifiez que la valeur associé est supérieur à la valeur de la troisième colonne du fichier, sinon, vous remplacez dans la map cette valeur.
Lisez la documentation de std::map et de std::tuple.
Ça tombera pas tout seul dans votre bec.

Une fois la map rempli avec toutes les lignes du fichier, vous copiez le contenu de la map dans le fichier de sortie, c'est tout.

Pour la conversion std::string en entier, pour la valeur de la 3ème colonne du fichier : "std::stoi":
http://www.cplusplus.com/reference/string/stoi/

[sophie284]

Je ne l'avais pas vu comme ca, merci je vais me pencher dessus longuement