Discussion :

[TNT89]

Bonjour,

Alors voilà un std::vector qui fait planter toute l'application quand je veux ajouter une chaine de caractère définie....

Code :

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

1
2
3
4
5
6
7
8
9
 
void newline(void)
{
   log_list.push_back("->         : ");  // Ne fonctionne pas 
   log_size++;
   last = log_list.end();                   // Ne fonctionne pas non plus
 
   return ;
}

Et pourtant l'initialisation fonctionne

Code :

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

1
2
3
4
5
 
   log_list.clear();
   log_list.push_back("->         : ");
   log_size = 1;
   last = log_list.begin();

Si une âme charitable avait un petit indice...
Merci d'avance...

[Mat007]

Salut,

C'est impossible à dire sans plus d'informations (ou une bonne boule de crystal).
Pour augmenter les chances d'obtenir de l'aide il est recommandé de :
. fournir un code complet reproduisant le problème (si possible il suffit de faire un copier-coller dans un fichier vide)
. fournir un code qui compile
. indiquer précisément quelle est l'erreur
Sans aucun de ces trois éléments, c'est pas gagné...

MAT.

[TNT89]

L'erreur c'est un Segmentation Fault

Le début de la classe

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
 
class LoG
        {
            private:
                /// Data
                    std::vector< std::string >            log_list;
                    int                                   log_size;
                    std::fstream                          log_file;
                    std::vector< std::string >::iterator  last;
                    long                                  start;
                    bool                                  working;
            public:
                /// Functions
                    LoG( std::string filename)
                    {
                        working = false;
                        log_file.open(filename.c_str(), std::fstream::out | std::fstream::ate);
 
                        if(!log_file.is_open())
                            return ;
 
                        working = true;
                        start = time(NULL);
 
                        log_list.clear();
                        log_list.push_back("->         : ");
                        log_size = 1;
                        last = log_list.begin();
                    }

L'opérateur censé faire le boulot( j'ai aussi des problèmes avec des templates...)

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
 
template<>
        LoG& LoG::operator<< <char>(char obj)
        {
            if(obj==ENDLOG)  // ENDLOG c'est '\n'
            {
                (*last) += '\n';
 
                newline();
 
                if( log_size >= MAX_LOG_STRING )
                    flush();
            }
            else
                (*last) += obj;
 
            return (*this);
        }

La fonction d'ajout des const char* fonctionne quant à elle

[Invité(e)]

Bonjour,

Le problème n'est pas reproduit chez moi.

Voici le code testé :

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
#include <vector>
#include <iostream>
#include <string>
#include <fstream>
using namespace std;
 
class LoG
{
private:
   /// Data
   std::vector< std::string >            log_list;
   int                                   log_size;
   std::fstream                          log_file;
   std::vector< std::string >::iterator  last;
   long                                  start;
   bool                                  working;
public:
   /// Functions
   LoG( std::string filename)
   {
      working = false;
      log_file.open(filename.c_str(), std::fstream::out | std::fstream::ate);
 
      if(!log_file.is_open())
         return ;
 
      working = true;
      start = time(NULL);
 
      log_list.clear();
      log_list.push_back("->         : ");
      log_size = 1;
      last = log_list.begin();
   }
   void newline(void)
   {
      log_list.push_back("->         : ");  // Ne fonctionne pas
      log_size++;
      last = log_list.end();                   // Ne fonctionne pas non plus
 
      return ;
   }
 
};
 
int main()
{
   LoG log("log");
 
   log.newline();
   log.newline();
   log.newline();
 
   return 0;
}

Par contre, quand j'ajoute cette méthode et que je tente de l'utiliser,

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
   template <class T>
        LoG& LoG::operator<< (T obj)
        {
            if(obj=='\n')  // ENDLOG c'est '\n'
            {
                (*last) += '\n';
 
                newline();
 
                if( log_size >= 2 )
                    log_file.flush();
            }
            else
                (*last) += obj;
 
            return (*this);
        }

les problèmes arrivent.

Alors, le point qui à mon avis pose problème est

Code :

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

last = log_list.end();

En effet, il ne me semble pas que cet iterateur pointe sur un objet valide.

Si tu veux travailler avec le dernier objet d'un vector, il y a les méthodes last() ou rbegin().

[r0d]

Je ne suis pas sûr que ça vienne de là, mais effectivement, vector::end() renvoie un iterateur spécial dont le déréférencement a un résultat indéterminé.

Autrement dit:

Code :

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

1
2
3
4
std::vector<T> v;
// on rempli v
std::vector::iterator it = v.end();
*it //... INTERDIT!

[TNT89]

Bon, et bien il semble que j'ai bien oublier comment fonctionne les iterateurs! en effet je me rappelle maintenant le petit :

Code :

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

1
2
3
 
it--;
*it; //OK

Enfin j'attend de voir si cela résout tous les problèmes. Merci beaucoup