"Klasse::Datenelement ist privat" angezeigt, aber ich bewege mich auf es mit eine member-Funktion?

Ich bin ziemlich neu in C++, und ich verstehe nicht, was das auslösen dieses Fehlers:

/home/---/Documents/C++/---_lab2/lab2c.cpp||In function ‘int main()’:|
Line 9: error: ‘float circle::x1’ is private
Line 58: error: within this context

Ich weiß, die Daten-member x1 (x2,y1,y2) ist privat, aber ich fahre auf das Objekt myObj mit Funktionen, die Mitglieder der Klasse circle, so sollten Sie nicht immer noch funktionieren? Kann mir jemand erklären, was hier falsch?

#include <iostream>
#include <cmath>
#define PI 3.14159

using namespace std;

class circle{

private:
float x1,y1,x2,y2;

protected:

float distance(float x1,float y1,float x2, float y2){
    return sqrt(fabs((x2-x1)*(x2-x1))+fabs((y2-y1)*(y2-y1)));
};

public:

float radius(float x1, float y1, float x2, float y2){
    float rad = distance(x1,y1,x2,y2);
    return rad;
};

float circumference(float rad){
    return 2*PI*rad;
};

float area(float rad){
    return PI*rad*rad;
};

float populate_classobj(float x1main,float x2main,float y1main,float y2main){
x1 = x1main;
x2 = x2main;
y1 = y1main;
y2 = y2main;
};

};

int main(){

circle myObj;
float x1main,x2main,y1main,y2main;
cout << "Coordinates of center" << endl;
cout << "X: ";
cin >> x1main;
cout << "Y: ";
cin >> y1main;
cout << "Coordinates of point on circle" << endl;
cout << "X: ";
cin >> x2main;
cout << "Y: ";
cin >> y2main;

myObj.populate_classobj(x1main,x2main,y1main,y2main);

cout << "Radius is " << myObj.radius(myObj.x1,myObj.y1,myObj.x2,myObj.y2) << endl;
cout << "Circumference is " << myObj.circumference(myObj.radius(myObj.x1,myObj.y1,myObj.x2,myObj.y2)) << endl;;
cout << "Area is " << myObj.area(myObj.radius(myObj.x1,myObj.y1,myObj.x2,myObj.y2)) << endl;



return 0;
}
Die meisten dieser code ist nicht erforderlich, um das problem anzuzeigen. In diesem besonderen Beispiel, das problem ist leicht zu sehen, aber im Allgemeinen schneiden Sie Ihren code bis hin zu den kleinsten kann man noch zeigt, was das problem ist.

InformationsquelleAutor nmar | 2012-09-30

  1. 3

    Du bist der versucht auf private Mitglieder außerhalb der Klasse, wenn Sie anrufen radius & andere Methoden.

    Aber Ihr eigentliches problem mit der Logik. Warum brauchen Sie, um die übergabe von Parametern an, zum Beispiel, die radius - Methode Ihrer Klasse:

    float radius(float x1, float y1, float x2, float y2){
    float rad = distance(x1,y1,x2,y2);
    return rad;
};

    Den Kreis bereits selbst enthalten, warum nicht einfach:

    float radius(){
    float rad = distance(x1,y1,x2,y2);
    return rad;
};

    Gleich mit:

    float circumference(){
    return 2*PI*radius();
};

float area(){
    return PI*radius()*radius();
};

    Beachten Sie auch, dass:

    circle myObj;

    erzeugt ein ungültiges Objekt. Sie sollten nicht haben zu rufen populate_classobj nur um ihn gültig zu machen. Stattdessen haben Sie einen richtigen Konstruktor:

    circle(float x1main,float x2main,float y1main,float y2main) :
x1(x1main),
x2(x2main),
y1(y1main),
y2(y2main)
{
};

    und erstellen Sie das Objekt als:

    circle myObj(x1main,x2main,y1main,y2main);

    InformationsquelleAutor Luchian Grigore

  2. 2

    Den verschiedenen Einfüge-Anweisungen am Ende der main versuchen myObj.x1, die versucht zu verwenden, das Mitglied x1 von myObj. Sie kann nicht, weil x1 ist privat. Es spielt keine Rolle, was der code tut mit diesem Wert; privat ist privat. Sie können den Zugriff auf den Wert von innen eine Elementfunktion oder eine friend-Funktion, aber nicht von außen.

    InformationsquelleAutor Pete Becker

  3. 2
    cout << "Radius is " << myObj.radius(myObj.x1,myObj.y1,myObj.x2,myObj.y2) << endl;
cout << "Circumference is " << myObj.circumference(myObj.radius(myObj.x1,myObj.y1,myObj.x2,myObj.y2)) << endl;;
cout << "Area is " << myObj.area(myObj.radius(myObj.x1,myObj.y1,myObj.x2,myObj.y2)) << endl;

    Können Sie keinen Zugriff auf eine private variable. Auch Sie sollten nicht zu tun haben.
    Ihre Methode Signatur myObj.radius() oder myObj.area() als x1 y1 x2 y2 sind bereits Mitglieder der circle myObj. So übergeben Sie Sie wieder als Argumente redundant ist.

    InformationsquelleAutor Lews Therin

Schreibe einen Kommentar