Discussion :

[membreComplexe12]

Bonjour tous,

j'ai un soucis bizarre sur mon programme :

=> il fonctionne en debug (il effectue les 12 iterations qu'il doit effectuer) mais en mode release il ne fonctionne pas : il beug completement (il se ferme) à la 5eme iteration...

auriez vous une idée d'où cela peut venir ??

je suis carrement pommé sur ce coup...

[Ekleog]

En général, c'est que le programme est mal formé mais que le compilateur ne s'en rend pas compte. Du coup, une optimisation exécutée en mode release fait tout exploser.

Y a-t-il un warning à la compilation ? Purrait-on avoir le code de cette boucle ?
Sans plus d'informations, il nous sera dur de t'aider, en tout cas.

[membreComplexe12]

merci de ton aide. Pour le code : oui, voici :

Main.cpp

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
int main(int argc, char *argv[])
{
Input input;
Coupl coup;
nb=5;
for (unsigned int i=0;i<12;i++)
{
vector<string> temp; temp.clear();
temp=coup.get_temp(i,nb);
input.runInput(nb,temp,coup,i);
}

input.h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#ifndef InputH //to avoid interference between fonctions
#define InputH
#include <string>
#include <vector>
#include "Coup.h"
class Input
{
    public:
        Input();
        ~Input();        void runInput(int,std::vector<std::string>,Coup&,unsigned int);
    private:
};
#endif // InputH included

Coup.h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#ifndef COUPL_HPP_INCLUDED
#define COUPL_HPP_INCLUDED
#include <string>
#include <vector>
class Coup
{
    public:
        Coup();
        ~Coup();
        std::vector < std::string > get_temp(unsigned int,int);
    private:
};
#endif // COUP_HPP_INCLUDED

Coup.cpp

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
 
#include <iostream>
#include <string>
#include <vector> 
#include <fstream> 
#include <stdlib.h>
#include <sstream> 
#include "Coup.h"
using namespace std;
Coup::Coup()
{
}
Coup::~Coup()
{
}
vector <string> Coup::get_temp(unsigned int _g,int Nb)
{
    vector <string> temp; temp.clear();
    for (int j=0;j<Nb;j++)
    {
        temp.push_back( Tim[j] );
        temp.push_back( Tem[j][_g] );
    }
return temp;

Input.cpp

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
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
 
#include <iostream>   
#include <fstream>       
#include <sstream>  
#include <vector>     
#include <stdlib.h>
#include "Input.h"
#include "Coup.h"
#include "Mat.h"
#include "Elem.h"
#include "Start.h"
#include "Preci.h"
#include "main.h"
using namespace std;
Input::Input()
{
}
Input::~Input()
{
}
void Input::runInput(int coupling,vector<string> temp,Coup &_coup,unsigned int nb)
{
    vector<string> arg;              
    vector< vector<string> >argBis;   
    int a=0;
    bool testTemp=false,TestDistri=false;
    string command, dump;
    Mat _mat;
    Elem _elem;
    Preci _preci;
    Start start;
    string my_file = "texte.txt";
    ifstream dataBase(my_file.c_str(), ios::in);    
    if(dataBase)   
    {
        while (!dataBase.eof())   
        {
            getline(dataBase,dump);              
            arg.clear();
            parse(dump,arg);
            if (arg[0]=="#")
            {
            }
            else if (arg[0]=="mat")
            {
                a=0;
                a=arg.size()-1; 
                if (a==4)
                {
                }
                else if (a<4)
                {                    
                    break;
                }
                else if (a>4)
                {
                    break;
                }
                _mat.Add_Mat(arg);
            }
            else if (arg[0]=="elem")
            {
                a=0;
                a=arg.size()-1;
                if (a==5)
                {
                }
                else if (a<5)
                {
                   break;
                }
                else if (a>5)
                {
                    break;
                }
                _elem.Add_Elem(arg);
            }
            else if (arg[0]=="preci")
            {
                    _preci.Add_Preci(arg,_elem);
            }
            else if (arg[0]=="tempProfile")
            {
                if (TestTemp==true)
                {
                    break;
                }
                else if(coupling==1)
                {
                    argBis.push_back(temp);
                }
                else
                {
                    a=0;
                    a=arg.size()-1;
                    if (a%2!=0)    
                    {
                      break;
                    }
                    else
                    {
                    }
                    for (size_t i=1; i<arg.size()-3; i=i+2) 
                    {                       
                        if ( atof(arg[i+2].c_str()) >= atof(arg[i].c_str()) )
                        {
                        }
                        else
                        {
                            break;
                        }
                    }
                    argBis.push_back( arg );
                }
            }
            else if (arg[0]=="all")
            {
                _elem.Add_All(arg);
            }
        }
        TempProfile _tempProfile(argBis.back());
        dataBase.close(); 
        tempeFileBase.close();
        _initialBase.close();
start.Run(coupling,_coup,nb,_mat,_preci,_elem,_tempProfile,TestTemp,TestDistri);
    }
}

Start.h

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
 
#ifndef StartH
#define StartH
#include <vector>
#include <string>
#include "Elem.h"
#include "Mat.h"
#include "Preci.h"
#include "Temp.h"
#include "Coup.h"
class Start
{
    public:
            Start();
            ~Start();
            void Run(int,Coup&,unsigned int,Mat&,Preci&,Elem&,Temp&,bool,bool);
    private:
 
};
//-------------------------------------------------------------------------------
#endif // StartH INCLUDED

Start.cpp

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
 
#include <iostream>  
#include <fstream>    
#include <sstream>   
#include <vector>  
#include "Start.h"
#include "Cal.h"
#include "main.h"
#include <math.h> 
using namespace std;
Start::Start()
{
}
Start::~Start()
{
}
void Start::Run(int r,Coup &_coup,unsigned int Nb,Mat &mat,Preci &preci,Elem &elem,Temp &temp,bool dat, bool initial)
{
//"ici je suis presque certain qu'il n'y a pas d'erreur"
}

je n'ai mis que les bout de code que j'ai changé il n'y a pas trop longtemps. Le reste date de très vieux et il n'y a pas de soucis à priori.

[oodini]

Ce type de problème peut venir de variables primitives non initialisées. En mode debug, elles sont intialisées avec une valeur par defaut, mais en mode release, non.

Mets ton warning au niveau maximal.

[membreComplexe12]

merci beaucoup de prendre le temps de m'aider !

J'ai deux petites questions :
1°) qu'appel tu des variables primitives ?
2°) comment dire à mon IDE de sortir le plus de warning possible ?
(je suis sous Qt ou sous CodeBlock)

Voici les warning que j'ai :
1°) j'ai pas mal de variables inutilisées (normal car j'ai commenté du code que je n'utilise pas encore)
2°) j'ai pas mal de "multiline comment"
3°) j'ai aussi une fonction non void qui ne renvoi rien (en fait elle renvoi quelque chose car je l'a fait passer dans un "if" et si elle passe dans le if c'est +1 et si elle passe dans le else c'est -1.
4°) j'ai des warning du genre : comparaison between unsigned integer and integer (mais je ne pense pas que ça soit bien grave car lorsque j'ai ceci mais entier sont positifs)

Autres pistes :
en fait je n'ai pas beaucoup de warning et en plus ceux que j'ai sont à des endroits où je n'ai pratiquement jamais de soucis...

en fait je me demande si je n'ai pas un de ces soucis :

1°) j'ai inclue les headers ".h" c'importe comment et du coup il s'appel entre eux et ça fou le bordel ?

2°) ou je me demande si je ne dois pas initialiser mes objets où les détruire à la fin de chaque boucle...
je dis ceci car avant j'utilisé exactement le même programme mais sans boucle et tous marchait bien... maintenant que j'utilise une boucle ça plante...
du coup je me dis que lors d'une nouvelle itération les précédentes informations n'ont pas étés correctement effacées ?

[Ekleog]

[...]
Voici les warning que j'ai :
1°) j'ai pas mal de variables inutilisées (normal car j'ai commenté du code que je n'utilise pas encore)
2°) j'ai pas mal de "multiline comment"

Qu'est-ce que ce warning ? Serait-il possible d'avoir les termes exacts ? (jamais rencontré, celui-ci)

3°) j'ai aussi une fonction non void qui ne renvoi rien (en fait elle renvoi quelque chose car je l'a fait passer dans un "if" et si elle passe dans le if c'est +1 et si elle passe dans le else c'est -1.

En théorie, le compilateur est capable de se rendre compte de cet état de fait. Es-tu certain que tu renvoies bien quelque chose ? Un cas d'école de ce genre de bug dont on se croit protégé est quelque chose comme :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
int foo(int p) {
   if (p > 0) { // Si p est positif
      // 150 lignes de code pour oublier la comparaison ci-dessus
      return 1;
   }
   if (p < 0) { // Sinon, si p est négatif
      // 150 autres lignes de code
      return 0;
   }
}
 
// Et foo(0) ?

4°) j'ai des warning du genre : comparaison between unsigned integer and integer (mais je ne pense pas que ça soit bien grave car lorsque j'ai ceci mais entier sont positifs

Ce genre de warning annonce souvent des failles de sécurité. Si tu es certain que tes entiers sont positifs, pourquoi ne pas les mettre unsigned int ?

Autres pistes :
en fait je n'ai pas beaucoup de warning et en plus ceux que j'ai sont à des endroits où je n'ai pratiquement jamais de soucis...

en fait je me demande si je n'ai pas un de ces soucis :

1°) j'ai inclue les headers ".h" c'importe comment et du coup il s'appel entre eux et ça fou le bordel ?

Un problème de header se poserait à la compilation, pas à l'exécution. (même avec un compilateur du siècle dernier, je pense)

2°) ou je me demande si je ne dois pas initialiser mes objets où les détruire à la fin de chaque boucle...
je dis ceci car avant j'utilisé exactement le même programme mais sans boucle et tous marchait bien... maintenant que j'utilise une boucle ça plante...
du coup je me dis que lors d'une nouvelle itération les précédentes informations n'ont pas étés correctement effacées ?

De quelle boucle parles-tu ? (Dans 200 lignes de code, difficile de deviner laquelle.)

[oodini]

1°) qu'appel tu des variables primitives ?

int, double, etc. Par opposition à des instances de classe.

2°) comment dire à mon IDE de sortir le plus de warning possible ?
(je suis sous Qt ou sous CodeBlock)

Je ne connais pas ces environnements.

2°) j'ai pas mal de "multiline comment"

Je ne vois pas en quoi ça pose un problème au compilateur...

3°) j'ai aussi une fonction non void qui ne renvoi rien (en fait elle renvoi quelque chose car je l'a fait passer dans un "if" et si elle passe dans le if c'est +1 et si elle passe dans le else c'est -1.

Fais malgré tout un return. Ca ne mange pas de pain. Mets éventuellement un assert(false) juste avant si tu es sûr que tu n'es jamais supposé passer par là.

4°) j'ai des warning du genre : comparaison between unsigned integer and integer (mais je ne pense pas que ça soit bien grave car lorsque j'ai ceci mais entier sont positifs)

Si, ça peut être grave si ta valeur dépasse 32567.

[koala01]

merci beaucoup de prendre le temps de m'aider !

J'ai deux petites questions :
1°) qu'appel tu des variables primitives ?

Les types qui n'ont pas besoin de fichier d'en-tête pour etre compris du compilateur:

• Les entiers signé (par ordre de taille)char, short, int, long et long long (sur les compilateurs récents)
• les entiers non signés (par ordre de taille)unsigned char, unsigned short, unsigned int, unsigned long et unsigned long long (sur les compilateurs récents)
• les réels (par ordre de taille)float, double et long double(sur les compilateurs récents)
• et bien sur les booléens bool

2°) comment dire à mon IDE de sortir le plus de warning possible ?
(je suis sous Qt ou sous CodeBlock)

avec CodeBlocks:

• chosir le menu "Options->Compiler"
• dans la fenetre qui s'ouvre, choisir l'onglet "compiler settigns" puis le sous onglet "compiler flags"
• Éventuellement choisir "Warning" dans la liste déroulante
• cocher "Enable all compiler warnings (-Wall)
• cocher "Enable warnings demanded by strict ISO (-pedentic)
• cocher "Enable Effective C++ warning (-Weffc++)

Pour QtCreator
ouvrir le fichier *.pro correspondant à ton projet
rajouter la ligne
QMAKE_CXXFLAGS += -Wall -pedentic -Weffc++)

Voici les warning que j'ai :
1°) j'ai pas mal de variables inutilisées (normal car j'ai commenté du code que je n'utilise pas encore)

essaye de commenter aussi les variables qui ne sont utilisées que par le code commenté...

Cela te permettra au moins de limiter les avertissements et donc de te permettre de "repérer ceux qui sont vraiment importants

2°) j'ai pas mal de "multiline comment"

aucun problème

3°) j'ai aussi une fonction non void qui ne renvoi rien (en fait elle renvoi quelque chose car je l'a fait passer dans un "if" et si elle passe dans le if c'est +1 et si elle passe dans le else c'est -1.

Etant donné que l'instruction return a pour effet la sortie de la fonction, tu peux très bien envisager d'écrire ton code sous une forme proche de

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
int  foo()
{
    if(condition)
    {
        /* ce qu'il faut faire */
        return  1;
    }
    // on ne passe ici que si la condition était fausse
    /* ce qu'il faut faire */
    return  -1
}

4°) j'ai des warning du genre : comparaison between unsigned integer and integer (mais je ne pense pas que ça soit bien grave car lorsque j'ai ceci mais entier sont positifs)

C'est surtout le signe que tu risque d'avoir des affichages surprenants si tu utilises un entier signé là ou tu devrait utiliser un entier non signé.

C'est, souvent (mais pas uniquement) du à l'utilisation de la fonction size() des collections de la STL, qui renvoient un size_t (qui correspond à un unsigned int, unsigned long ou unsigned long long selon les systèmes), car la taille d'un tableau ne peut pas etre négative

Autres pistes :
en fait je n'ai pas beaucoup de warning et en plus ceux que j'ai sont à des endroits où je n'ai pratiquement jamais de soucis...

L'idéal est quand meme de résoudre tous les avertissements car ils sont le symptome du fait que quelque chose "risque de clocher" (de préférence le jour de la présentation officielle du programme )

1°) j'ai inclue les headers ".h" c'importe comment et du coup il s'appel entre eux et ça fou le bordel ?

A priori, non, si du moins, tu as bien mis les garde anti inclusion

2°) ou je me demande si je ne dois pas initialiser mes objets où les détruire à la fin de chaque boucle...

Déclare les variables dans la portée dans laquelle elles sont utilisées

Si tu as un objet qui doit impérativement etre initialisé dans une boucle, déclare le entre l'accolade ouvrante et l'accolade fermante de la boucle

je dis ceci car avant j'utilisé exactement le même programme mais sans boucle et tous marchait bien... maintenant que j'utilise une boucle ça plante...

Fait attention au fait que ca peut planter parfois très loin de la source du problème...

Il suffit parfois d'une incrémentation de pointeur là ou il aurait fallut incrémenter ce qui est pointé par le pointeur pour que "ca plante" cinq fonctions plus loin

du coup je me dis que lors d'une nouvelle itération les précédentes informations n'ont pas étés correctement effacées ?

Si ton objet n'est pas déclaré dans la boucle, c'est normal