Discussion :

[monsrhf]

je suis nouveau sur c++, je travaille sur une class Point avec des setters et des getters et des méthodes mais je reçois l'erreur suivante :
no matching function for call to 'Point::affiche(Point&)'
le fichier Point.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
 
#ifndef POINT_H
#define POINT_H
 
 
class Point
{
    public:
        Point();
        virtual ~Point();
        Point(const Point& other);
        void affiche();
        float deplace(float x, float y);
        float getAbscisse() const;
        float getOrdonnee() const;
        void setAbscisse(float abs);
        void setOrdonnee(float ord);
    private:
        float abscisse;
        float ordonnee;
};
 
#endif // POINT_H

le fichier Point.cpp est comme suite:

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
#include "Point.h"
#include <iostream>
Point::Point()
{
    //ctor
}
 
Point::~Point()
{
    //dtor
}
 
Point::Point(const Point& other)
{
    //copy ctor
}
Point::affiche(Point p)
{
    cout<<"les cordonnées du point sont:"<<p.getAbscisse()<<p.getOrdonnee();
}
Point::deplace(Point p)
{
    float d;
    cout<<"pour déplacer le point entrer un taux de déplacement"<<endl;
    cin>>d;
    cout<<" "<<endl;
    p.setAbscisse(abscisse)=p.setAbscisse(abscisse)+d;
    p.setOrdonnee(ordonnee)=p.setOrdonnee(ordonnee)+d;
}
Point::setAbscisse(float abs)
{
    abs=abscisse;
}
Point::setOrdonnee(float ord)
{
    ord=ordonnee;
}
Point::getAbscisse() const
{
    return abscisse;
}
Point::getOrdonnee() const
{
    return ordonnee;
}

le fichier main.cpp est:

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 "Point.h"
#include <cstdlib>
 
using namespace std;
 
int main()
{
    Point p;
    float abs, ord;
cout<<"entrer les cordonnées du point"<<endl;
cin>>abs>>ord;
cout<<" "<<endl;
p.setAbscisse(abs);
p.setOrdonnee(ord);
p.affiche(p);
p.deplace(abs,ord);
p.affiche(p);
    return 0;
}

[Daïmanu]

Bonjour.

Le compilateur t'indique simplement que la définition de la fonction Point::affiche est différente de sa déclaration.

En effet tu as déclaré void Point::affiche() dans ton fichier header, mais tu as déclaré /*rien*/ Point::affiche(Point) dans le fichier cpp.

Il faut donc mettre la même déclaration dans les deux fichiers.
La meilleure fonction dans te cas sera void affiche() const dans le .h et void Point::affiche() const {} dans le .cpp. Ce const dans ce indique que la fonction ne modifiera pas l'objet sur lequel elle est appelée.

[monsrhf]

j'ai apporté les modifications que tu m'as dit mais le problème persiste toujours malheureusement!

[Daïmanu]

Quel message d'erreur le compilateur te donne ?

Ton code présente plusieurs incohérences :

• La fonction affiche que je t'ai donné s'utilise comme ceci p.affiche(), mais dans le main tu l'utilises ainsi p.affiche(p)
• Dans ton fichier c++ il manque les type de retour :

float Point::getAbscisse() const
void Point::setAbscisse(float) const
pareil pour l’ordonnée et pour le déplacement

• La méthode déplacer() a le même défaut que afficher() (prototypes que ne correspondent pas), mais elle est également fausse. Elle devrait uniquement modifier les coordonnées de l'objet courant et ne rien retourner.
• Les méthodes set* sont erronées également, tu fais l'assignation à l'envers.
• En outre, le destructeur n'a pas besoin d'être virtuel (mais c'est une remarque, pas une erreur)

Je t'invite à corriger ces erreurs et à nous montrer le nouveau code ainsi que les erreurs du compilateur si il y en a.

[monsrhf]

effectivement il y avait des erreurs sur mon code j'ai essayé de corriger les erreurs, mais cette fois j'ai reçu l'erreur suivante :'cout' was not declared in this scope. et le voici mon code:
le fichier Point.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
 
#ifndef POINT_H
#define POINT_H
 
class Point
{
    public:
        Point();
        virtual ~Point();
        Point(const Point& other);
        void affiche(Point p) const;
        void deplace(float x, float y);
        float getAbscisse() const;
        float getOrdonnee() const;
        void setAbscisse(float abs);
        void setOrdonnee(float ord);
    private:
        float abscisse;
        float ordonnee;
};
 
#endif // POINT_H

le fichier Point.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
#include "Point.h"
#include <iostream>
Point::Point()
{}
 
Point::~Point()
{}
 
Point::Point(const Point& other)
{}
 
void Point::affiche(Point p) const
{
    cout<<"les cordonnées du point sont:"<<p.getAbscisse()<<p.getOrdonnee();
}
 
void Point::deplace(float x, float y)
{
    float d;
    cout<<"pour déplacer le point entrer un taux de déplacement"<<endl;
    cin>>d;
    cout<<" "<<endl;
    x=x+d;
    y=y+d;
    p.setAbscisse(x);
    p.setOrdonnee(y);
 
}
 
void Point::setAbscisse(float abs)
{
    abscisse=abs;
}
 
void Point::setOrdonnee(float ord)
{
    ordonnee=ord;
}
 
float Point::getAbscisse() const
{
    return abscisse;
}
 
float Point::getOrdonnee() const
{
    return ordonnee;
}

le fichier main.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
 
#include <iostream>
#include "Point.h"
#include <cstdlib>
 
using namespace std;
 
int main()
{
    Point p;
    float abs, ord;
cout<<"entrer les cordonnées du point"<<endl;
cin>>abs>>ord;
cout<<" "<<endl;
p.setAbscisse(abs);
p.setOrdonnee(ord);
p.affiche(p);
p.deplace(abs,ord);
p.affiche(p);
    return 0;
}

[Daïmanu]

Il manque simplement la directive using namespace dans ton fichier cpp.

[monsrhf]

merci beaucoup vous m'avez beaucoup aidez à réaliser mon premier projet objet.