Discussion :

[JulDev]

Bonjour,

J’aimerais avoir votre avis pour optimiser le temps de calcul d’un code.

Pour résumer grossièrement mon code :
• J’ai une classe qui contient une méthode de calcul

Code :

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

1
2
3
4
5
6
 
Class object {
public:
    object(QString objectName = "Not set");
    double computeValue(const double param1, const double param2) {return lectureFichier(param1, param2);}
}

• Mon main

Code :

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

1
2
3
 
for(unsigned long long i=0 ; i<NOMBRE ; i++)
	object->computeValue(valeur1, valeur2);

avec NOMBRE qui peut être très très élevé (typiquement plusieurs millions) et valeur1 et valeur2 issus d’un calcul non « prévisible » (combinaison d’aléas).

• Une bibliothèque de fonctions :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
// Le .h
#ifndef BIBLIFUNCTIONS_H
#define BIBLIFUNCTIONS _H
#include <iostream>
#include <fstream>
 
double lectureFichier(const double param1, const double param2);
 
#endif

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
// Le .cpp
#include "bibliFunctions.h"
 
    double lectureFichier(const double param1, const double param2) {
        ifstream file(PATH_FILE, ios::in | ios::binary);
	Index = FUNCTION(param1, param2);
        file.seekg(index*SIZE_OF_DOUBLE);
        file.read(reinterpret_cast<char *>(&values), 4*SIZE_OF_DOUBLE);
        file.close();
 
	return FONCTION(values) ; // Calcule 1 valeur en utilisant les 4 valeurs lues dans le fichier
}

Le fichier est une série de 260000 doubles (implicitement organisés en un tableau de 720 par 360). C’est un binaire de 2Mo. Ces fonctions se trouvent dans une bibliothèque à part car elles sont appelées à plusieurs endroits. Je précise également que je ne fais que lire le fichier (je ne fais jamais d’écriture).

Mon problème est que j’ouvre et ferme de très nombreuses fois le ficher, ce qui me pénalise en temps de calcul.

J’ai d’abord pensé à faire ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
// Le .h
#ifndef BIBLIFUNCTIONS_H
#define BIBLIFUNCTIONS _H
#include <iostream>
#include <fstream>
 
static std::ifstream file(PATH_FILE, std::ios::in | std::ios::binary);
 
double lectureFichier(const double param1, const double param2);
 
#endif

Code :

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

1
2
3
4
5
6
7
8
9
10
 
// Le .cpp
#include "bibliFunctions.h"
 
    double lectureFichier(const double param1, const double param2) {
	Index = FUNCTION(param1, param2);
        file.seekg(index*SIZE_OF_DOUBLE, ios::beg);
        file.read(reinterpret_cast<char *>(&values), SIZE_OF_DOUBLE);
	return value ;
}

C’est bien plus rapide (je n’ouvre le fichier plus qu’une fois) mais je ne sais pas comment faire pour appeler le file.close().

Je pensais aussi stocker le fichier (2Mo) en mémoire pour y accéder encore plus vite (pour éviter de lire sur le disque dur). Cependant, je ne sais pas trop sous quel format/quelles méthodes je dois utiliser car c’est un fichier binaire :
• un istream avec les fonctions seekg et read ?
• un stringsteam (mais il faut convertir les string en double, c’est peut-être pas le plus rapide) ?
• autre format plus performant que je ne connais pas ?

Donc j’aimerais un petit conseil pour optimiser au mieux ce code. Merci d’avance !

[ternel]

C'est le moment de faire intervenir une classe RAII, voire de se souvenir que fstream en est une.

imagine une classe DataHolder ainsi faite:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
class DataHolder {
private:
	ifstream & stream;
public:
	DataHolder(ifstream& source) : stream(source) {}
 
	double computeValue(const double param1, const double param2) {
		Index = FUNCTION(param1, param2);
	        file.seekg(index*SIZE_OF_DOUBLE);
        	file.read(reinterpret_cast<char *>(&values), 4*SIZE_OF_DOUBLE);
		return FONCTION(values)
	}
};

Du coup, le main crée le stream, le donne à un DataHolder, et boucle sur celui-ci.
En étant plus attentionné, je m'aperçois que Object pourrait fort bien contenir ce stream, ou en avoir une référence.


Autre solution, changer la fonction pour lui donner la référence au stream.
le main deviendrai:

Code :

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

1
2
3
ifstream & file(...);
for(unsigned long long i=0 ; i<NOMBRE ; i++)
	object->computeValue(file, valeur1, valeur2);

PS: j'espère que le nom Object est seulement pour l'exemple, et que ton code n'est pas basé sur un GodObject

[Bousk]

Pourquoi tu rajouterais pas juste le fichier en question en paramètre de la méthode de lecture ?

Code :

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

1
2
3
4
5
 
// ouvrir fichier
// deadly loop
{// calcul}
// fermer fichier

[JulDev]

Bonjour, merci pour vos réponses.

Leternel, j'ai plusieurs questions :

Dans la solution 1:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
class DataHolder {
private:
	ifstream & stream;
public:
	DataHolder(ifstream& source) : stream(source) {}
 
	double computeValue(const double param1, const double param2) {
		Index = FUNCTION(param1, param2);
	        file.seekg(index*SIZE_OF_DOUBLE);
        	file.read(reinterpret_cast<char *>(&values), 4*SIZE_OF_DOUBLE);
		return FONCTION(values)
	}
};

Est-ce que ce ne devrait pas être stream.seekg() et stream.read() au lieu de file.seekg() et file.read() ?

J'ai cru comprendre que ifstream crée un "pointer" vers le fichier ? Est-ce que je dois comprendre de tes propositions que créer un ifstream est aussi efficace que de stocker le fichier en mémoire ?

Pour la solution 2, j'aurai au final une trentaine de fichiers qui seront tous utilisés par toutes mes classes donc la solution 1 me semble plus adaptée.

Oui, "object" est un nom que j'ai donné pour l'exemple.


Bousk,

ton approche me demande de réorganiser pas mal de choses dans ce programme.

[ternel]

Effectivement, c'est un copié collé malencontreux.
fstream est un peu mieux qu'un fichier, mais gère ça proprement.

créer un ifstream, c'est exactement ce que tu faisais.
Ma solution permet de garder cette variable le temps nécessaire.

La solution de Boosk revient sensiblement au même, sauf qu'il se passe de créer un Bidule™ pour garder la variable.
C'est encore meilleur.

[JulDev]

Ok. Merci pour toutes ces réponses !

[_zzyx_]

Un fichier de 2MB sur une machine ordinaire avec des Gigas de RAM, ca vaut le coup de le stocker en RAM

- Soit tu écrit un parser pour ton fichier binaire qui te le ré-écrit comme une suite d'objets/structures, c'est un peu de travail, mais ca te permet d'avoir quelques-chose bien propre

- Soit tu sais exactement où tu dois tapper dans le fichier, tu n'as pas peur de compter les bits (ce que tu fais déjà à coup de seek) bref que tu es un bad boy of programming dans ce cas là tu te contente de mettre ton fichier dans un vector<char> (en fait un vectorr<int8_t> histoire d'être sur de la taille) voire (because STL containers are for pussies :p ) un tableau dynamque à l'ancienne (a tes risques et périls)