値は変更されません

Question

// Example program 
#include <iostream> 
#include <string> 
using namespace std; 

class circle { 

    private: 
    double r; 
    double const pi=3.14; 

    public: 

    circle(double rad){ 

     r=rad; 

     } 

    void periArea(){ 
     double p,s; 

     p=2*pi*r; 
     s=pi*pi*r; 
     cout<<"\nS= "<<s<<endl; 
     cout<<"\nP= "<<p<<endl; 
     } 

     friend void changeRad(circle circle, double newrad); 
    }; 

    void changeRad(circle circle, double newrad){ 

     circle.r=newrad; 

     } 

int main() 
{ 
    double inpr,newr; 
    cout<<"input radius: "; 
    cin>>inpr; 
    circle c1(inpr); 
    c1.periArea(); 
    cout<<"\ninput new radius: "; 
    cin>>newr; 
    changeRad(c1,newr); 
    c1.periArea(); 
} 

私は周辺と面積を計算するクラスサークルを定義する必要があるこのcppコードを取得してから、フレンド関数を使用して半径を変更し、再度面積と周囲を計算します。しかし、変更関数の後でも同じpとsの値が得られます。

Answer 1

ぞんざいに、あなたは参照による円を渡すためにあなたの関数を変更する必要があります。

void changeRad(circle& circle, double newrad){ 

（&の使用に注意してください）。それ以外の場合は、circleインスタンスのコピーが関数に渡され、そのコピーに対する変更は元のものに反映されません。

しかし、物事の通常方法は、あなたが呼び出しサイトで

c1.changeRad(newr); 

を使用するようにコードを配置することです。

また、doubleを使用すると、piをわずか3人の有効数字に保存することはかなりひどいです。私のお気に入りの方法は、（M_PIはPOSIX、ISOでないです！）があなたのためにパイを定義していません C++標準以来

const double PI = std::atan(1.0) * 4; 

を使用することです。

Answer 2

あなたはその名の型の名前と一致した場合には、パラメータの有効な宣言ですが、それにもかかわらず、これが混乱することができていることを考慮してください参照

friend void changeRad(circle &circle, double newrad); 
          ^^^ 

によってオブジェクトに型サークルを渡す必要がありますパラメータの名前は型の名前を隠します。だから、また、機能クラスの非静的メンバを作成する方がよい例として

friend void changeRad(circle &c, double newrad); 
           ^^ 

のパラメータの名前の名前を変更することをお勧めします。たとえば

void changeRad(double newrad);