Discussion :

[fab87]

Bonjour,

Lors du débugage, je m'aperçois qu'il y un problème pour l'utilisation de getters.

Voici mon code :

MainWindow.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
class MainWindow : public QMainWindow
{
    Q_OBJECT
 
public:
        MainWindow(QWidget *parent = 0);
 
        int getNbPoints() ;
        double getTableauXParfait(int) ;
        double getTableauYParfait(int) ;
        double getTableauZParfait(int) ;
        double getTableauXDefaut(int) ;
        double getTableauYDefaut(int) ;
        double getTableauZDefaut(int) ;
        double getTableauErreurX(int) ;
        double getTableauErreurY(int) ;
        double getTableauErreurZ(int) ;
        double getTableauErreurTotale(int) ;

MainWindow.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
double MainWindow::getTableauXParfait(int i)  { return *(tableauXParfait+i); }
double MainWindow::getTableauYParfait(int i)  { return *(tableauYParfait+i); }
double MainWindow::getTableauZParfait(int i)  { return *(tableauZParfait+i); }
double MainWindow::getTableauXDefaut(int i)  { return *(tableauXDefaut+i); }
double MainWindow::getTableauYDefaut(int i)  { return *(tableauYDefaut+i); }
double MainWindow::getTableauZDefaut(int i)  { return *(tableauZDefaut+i); }
double MainWindow::getTableauErreurX(int i)  { return *(tableauErreurX+i); }
double MainWindow::getTableauErreurY(int i)  { return *(tableauErreurY+i); }
double MainWindow::getTableauErreurZ(int i)  { return *(tableauErreurZ+i); }
double MainWindow::getTableauErreurTotale(int i)  { return *(tableauErreurTotale+i);} 
 
MainWindow::MainWindow(QWidget *parent) : QMainWindow(parent)
{
    setFixedSize(820, 820);
    uiMW.setupUi(this);
 
    fic = new Fichiers(this);
    point = new Point(this);
 
    graphique = new Graphique(this);
...
}
 
void MainWindow::calculer()
{
graphique->dessinerCourbesParfaiteDefaut();
graphique->show();
}

Graphique.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
22
23
24
25
26
27
28
29
30
31
32
33
34
35
#ifndef GRAPHIQUE_H
#define GRAPHIQUE_H
 
#include "qwt_plot.h"
#include <qwt_plot_curve.h>
#include <qwt_plot_zoomer.h>
#include <qwt_plot_grid.h>
#include "maFenetre.h"
#include "ihm.h"
#include "ihmgraphique.h"
 
class MainWindow;
 
class Graphique: public QDialog
{
    Q_OBJECT
 
    private:
 
        QwtPlotCurve *courbeParfaite;
        QwtPlotCurve *courbeDefaut;
        QwtPlotGrid grille;
        QwtPlotZoomer *myZoomer;
 
        Ui::ihmGraphique uigraphe;
        MainWindow *mw;
 
    public:
        Graphique(QWidget *parent = 0);
        void dessinerCourbesParfaiteDefaut(int);
        void dessinerCourbesErreurs();
 
};
 
#endif // GRAPHIQUE_H

Graphique.cpp

Code :

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

1
2
3
4
5
6
7
8
9
10
...
for(int i=0;i<mw->getNbPoints();i++)
    {
        xParfait.push_back(mw->getTableauXParfait(i));
        yParfait.push_back(mw->getTableauYParfait(i));

        xDefaut.push_back(mw->getTableauXDefaut(i));
        yDefaut.push_back(mw->getTableauYDefaut(i));
    }
...

Ce qui est en gras est apparemment faux mais je ne vois pas où est le soucis.
J'ai déclaré un pointeur sur ma fenêtre principale dans Graphique.h pour récupérer les tableaux grâce à mes getters...

[ymoreau]

Sans le message d'erreur du compilateur on ne peut pas vraiment t'aider...

[fab87]

Le programme compile mais le débogueur me dit qu'il y a un problème au niveau des getters.

[dourouc05]

Le programme compile mais le débogueur me dit qu'il y a un problème au niveau des getters.

Sans le message d'erreur on ne peut pas vraiment t'aider...

Là, tu nous demandes de chercher une aiguille dans une botte de foin... voire toute la ferme. Quels sont les messages d'erreur ?

[fab87]

Il y a des segmentation fault au niveau des getters. Je ne peux malheureusement pas vous en dire plus, j'ai bien peur justement que c'est une aiguille dans une botte de foin!

[ymoreau]

Code :

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

double getTableauXParfait(int i)  { return *(tableauXParfait+i); }

Je crains que tu ne doives revoir tes bases en C/C++, tu retournes là ce qui est pointé par l'adresse de valeur tableauXParfait à laquelle on ajoute i.

http://c.developpez.com/faq/?page=pointeurs

[fab87]

Ah, mais tableauXParfait[i] est bien équivalent à *(tableauXParfait+i) nan?

Pourtant, j'étais assez sûr de moi là... çà ne me choque pas

[superjaja]

Pour info ils sont définit ou tes tableaux (tableauXParfait,tableauXParfait...) car je ne les vois pas en variable membre de ta classe MainWindow ?

[fab87]

Normal, je n'ai pas mis tout mon code

En membres privés de MainWindow :

Code :

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

1
2
3
double *tableauXParfait; 
        double *tableauYParfait; 
       ...

[ymoreau]

Ah, mais tableauXParfait[i] est bien équivalent à *(tableauXParfait+i) nan?

En effet c'est équivalent (mais c'est moche ^^), et comme ta variable est un pointeur ça devrait fonctionner, si tu alloues bien ton tableau avant d'essayer d'y accéder, et si i est bien strictement inférieur à la taille de ton tableau.

Dans tous les cas on évite ce genre de choses en C++ car ça mène à des seg fault assez facilement quand on est pas assez prudent, et que des classes ont été écrites pour nous faciliter la vie et éviter ce genre de plantage :
http://qt.developpez.com/doc/4.7/qvector/

[fab87]

J'ai bien alloué tous les tableaux dans le constructeur de MainWindow...
Je croyais que j'avais fait une erreur tu m'as dit tout à l'heure

[superjaja]

Tu fais de l'allocation à un moment sur tes tableaux ou pas ? (un new, un malloc...) . Sinon comme le dit YoniBlond tu vas surement dans un zonne mémoire non alloué pour ton tableau (index trop grand).

[fab87]

Oui je fais des new sur mes tableaux avant de passer dans la méthode dessinerCourbesParfaiteDefaut() dans le slot calculer()
Voici le code :

tableauXParfait = new double[nbPoints]; tableauYParfait = new double[nbPoints];
...

graphique->dessinerCourbesParfaiteDefaut();
graphique->show();