Discussion :

[joffrey575]

Bonjour à tous,

Actuellement je cherche à convertir un tableau de char en double à défaut de savoir stocker directement un double depuis une lecture de fichier.
Avec le compilateur gcc et la fonction atof la conversion char vers double fonctionne très bien.
Avec gcc++, ça marche moins bien apparemment.

Je souhaite convertir cette chaine de caractère en double : 335.677250. J'obtiens 335,000000 après être passé par la fonction atof ou strtod.

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
 
 
#include <iostream>
#include <string>
#include <fstream>
#include <stdio.h>
 
using namespace std;
 
    //char nomfich[21];
    char donnee[100];
    char donnee_stock[100][100];
    double don;
    int n=0;
 
    FILE * entree;
 
    entree = fopen ("test.txt","r");
 
    do
    {   fseek (entree,12,SEEK_CUR);
        fread (donnee,11,1,entree); //normalement position 23
        strcpy(donnee_stock[n],donnee);
        n++;
    }
    while( !feof(entree));
 
    fclose(entree);
 
    //on affiche la chaine de caractère 
    std::cout << donnee_stock[0] << endl;
 
    //conversion de la chaîne de caractère en double
    //don = std::atof(donnee_stock[1]);
    don = strtod(donnee_stock[0],NULL);
 
    //affichage en double incorrecte 335,000000
    cout.precision(6);
    std::cout << fixed << don << std::endl;

[joffrey575]

Problème résolu.

Visiblement c'est juste la caractère "." à remplacer par "," !

Au lieu de mettre ceci avant la conversion : 335.677250 mettre plutôt 335,677250

C'est normal mais bon

[darkman19320]

Salut,
Je pense que tu te prends bien la tête à moins que ce soit un exercice...

Voici un petit exemple écrit avec des fonctionnalités offertent par le 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
#include <iostream>
#include <fstream>
 
int main()
{
   std::ifstream  ifs("test.txt");
   if(ifs.is_open())
   {
      double res;
      ifs >> res;
      std::cout << res << std::endl;
   }
 
   ifs.close(); // optionel
 
  return 0;
}

[joffrey575]

Dans un fichier .txt, j'ai 2048 fois ceci :
1498.000000,335.677250
1498.000000,335.677250
1498.000000,335.677250
...

J'aimerais stocker cette valeur "1498.000000" dans un tableau ainsi que celle-ci dans un autre "335.677250".

Effectivement ton exemple est plus cours.

[koala01]

Salut,

Dans un fichier .txt, j'ai 2048 fois ceci :
1498.000000,335.677250
1498.000000,335.677250
1498.000000,335.677250
...

J'aimerais stocker cette valeur "1498.000000" dans un tableau ainsi que celle-ci dans un autre "335.677250".

Effectivement ton exemple est plus cours.

La solution la plus simple est toujours la moins compliquée...

Dés lors, que penserais tu d'un code proche de

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
 
std::vector<double> tabA;
std::vector<double> tabB;
tabA.resize(2048); // permet d'éviter d'avoir ... 2048 augmentations de taille lors du push_back
tabA.resize(2048);
std::ifstream ifs("fichier.txt");
size_t pos{0};
while( ifs && pos){
    double a;
    double b;
    ifs>> a>>b;
    if(ifs){
        tabA.push_back(a);
        tabB.push_back( b);
    }
}

Il y a moyen de faire plus compliqué, bien sur, en utilisant getline puis en convertissant les valeurs à l'aide de std::stod (seule solution robuste pour la conversion de chaine de caractères en double )... Mais ceci devrait déjà faire l'affaire

[Invité]

Bonsoir,

Code :

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

1
2
tabA.resize(2048); // permet d'éviter d'avoir ... 2048 augmentations de taille lors du push_back
tabA.resize(2048);

reserve plutôt non ? Ou on va se retrouver avec 4096 valeurs.

Code :

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

1
2
size_t pos{0};
while( ifs && pos){

On va pas aller bien loin. Et à quoi sert ce pos justement ?

J'aurais tendance à sortir les déclarations de a et b de la boucle et fusionner les while et if comme ceci :

Code :

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

1
2
3
4
5
6
double a;
double b;
while(ifs >> a >> b){    
    tabA.push_back(a);
    tabB.push_back(b);
}

[LittleWhite]

Bonjour,

Code :

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

1
2
tabA.resize(2048); // permet d'éviter d'avoir ... 2048 augmentations de taille lors du push_back
tabA.resize(2048);

Alors, heureusement, les 2048 augmentation de taille, cela n'arrive dans aucune implémentation que je connaisse (bon, je connais disons que celle de GCC en réalité ). Il a tendance à faire juste le double de la taille actuelle, lors du push_back() et qu'il a besoin de mémoire.

reserve ou resize ? Et bah, les deux font bien l'allocation de mémoire, donc ça, c'est bon. La différence entre les deux : le resize(), c'est comme avoir 2048 push_back() utilisant le constructeur par défaut pour construire l'élément. Cela veut dire que vous avez les 2048 éléments, prêt à être utilisé. Le reserve(), vous avez bien un vector de capacity() 2048, mais si vous faites un v[20], vous avez une erreur de segmentation. En effet, le vector est comme vide, même si la mémoire est allouée. Il faut donc le remplir avec des push_back(), qui tant que l'on ne dépasse pas 2048 éléments, ne provoqueront pas d'allocations.
Dans les deux cas, faire deux fois le reserve/resize() ne change absolument rien. Cela n'ajoute pas. Il y a même un cas, ou cela peux provoquer une troncation d'un vecteur ayant plus de 2048 éléments (le resize, mais faut vérifier la documentation).

En vérifiant le code, oui, c'est mieux d'avoir un reserve() au lieu du resize(), car le prochain push_back() après le resize() fera que le vector aura une taille de 2049 élément (ce qui provoquera une allocation de mémoire pour agrandir le vecteur).

[joffrey575]

Merci pour vos réponses.

Le format du fichier.txt est le suivant :

Code :

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

1
2
 
1500.000000;335.677250

Il est possible si ça nous facilite la tâche de mettre "," à la place de ";".


J'ai essayé le code ci-après :

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
 
void MainWindow::on_pushButton_clicked()
{
 
    int t=0;
 
    std::vector<double> tabA;
    std::vector<double> tabB;
    tabA.reserve(2048); // permet d'éviter d'avoir ... 2048 augmentations de taille lors du push_back
    tabB.reserve(2048);
    std::ifstream ifs("text.txt");
    double a;
    double b;
    while(ifs >> a >> b){
        tabA.push_back(a);
        tabB.push_back(b);
    }
 
    for (t=0 ; t<=1023 ; t++)
    {
    std::cout << fixed << "pixels" << t << "  :  "  << tabA[t] << " ; " << tabB[t] << std::endl;
    }

Lors des exécutions du programme, j'ai observé la réponse suivante :

Code :

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

1
2
3
4
 
pixels0  :  0.000000 ; 0.000000
...
pixels1023  :  0.000000 ; 0.000000

Quelques exécutions peut être trop rapide entre deux, j'avais des valeurs hasardeuses. Ca doit surement venir du fait qu'il faut libérer la mémoire. D'ailleurs je me demande en C, il faut mettre free et le pointeur pour libérer la mémoire c'est bien ça?


J'aimerais acheter un livre pour le langage C++ au même titre que celui que j'ai en langage C : "programmer en langage C (Cours et exercices corrigés) edition eyrolles de Claude Delannoy. Je me demandais si ses autres livres sont tout aussi complet!

Pour le moment je reste sur ma version j'acquière une chaîne de caractère que je transforme en double. Si je ne trouve pas de solution plus simple, je vais devoir me complexifier la tâche car 1500 ne sera pas toujours 1500 mais aussi bien 500 que 50 ...