"Classe :: Membro dei dati è un errore privato", ma lo sto funzionando con una funzione membro?
-
11-12-2019 - |
Domanda
Sono abbastanza nuovo per C ++, e non capisco cosa stia attivando questo errore:
/home/---/Documents/C++/---_lab2/lab2c.cpp||In function ‘int main()’:|
Line 9: error: ‘float circle::x1’ is private
Line 58: error: within this context
.
Conosco anche il membro dei dati x1 (anche x2, y1, y2) è privato, ma sto operando sull'oggetto myobj usando le funzioni che sono membri del cerchio di classe, quindi non dovrebbero ancora funzionare?Qualcuno può spiegarmi cosa c'è che non va qui?
#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;
}
. Soluzione
Stai tentando di accedere ai membri private
al di fuori della classe quando chiami radius
e altri metodi.
Ma Il tuo vero problema è con la logica.Perché hai bisogno di passare i parametri a, ad esempio, il metodo radius
della tua classe:
.
float radius(float x1, float y1, float x2, float y2){
float rad = distance(x1,y1,x2,y2);
return rad;
};
Il cerchio è già autosufficiente, perché non solo:
.
float radius(){
float rad = distance(x1,y1,x2,y2);
return rad;
};
Lo stesso con:
.
float circumference(){
return 2*PI*radius();
};
float area(){
return PI*radius()*radius();
};
Inoltre, nota che:
.
circle myObj;
crea un oggetto non valido.Non dovresti chiamare populate_classobj
solo per renderlo valido.Invece, avere un costruttore adeguato:
.
circle(float x1main,float x2main,float y1main,float y2main) :
x1(x1main),
x2(x2main),
y1(y1main),
y2(y2main)
{
};
e crea l'oggetto come:
.circle myObj(x1main,x2main,y1main,y2main);
Altri suggerimenti
Le varie istruzioni di inserimento alla fine del main
tentano di utilizzare myObj.x1
, che tenta di utilizzare il membro x1
di myObj
.Non possono, perché x1
è privata.Non importa quale sia il codice con quel valore;il privato è privato.È possibile accedere al valore da Inside una funzione membro o una funzione di un amico, ma non dall'esterno.
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;
.
Non è possibile accedere a una variabile privata.Inoltre non dovresti farlo.
La firma del metodo dovrebbe essere myObj.radius()
o myObj.area()
come x1 y1 x2 y2
sono già membri del circle myObj
.Quindi passarli di nuovo come gli argomenti sono ridondanti.