クラスの属性値をメソッドの既定のパラメーターとして使用する代わりに、

Question

私はこのような何かを達成したい：

非静的メンバは、デフォルト引数

に使用してはならない：C++標準が言うように、これは不可能です

class C 
{ 
    int m_nVal; 
public: 
    C(int nVal) : m_nVal(nVal){}    

    void foo(int nVal = m_nVal) 
    { 
     // use nVal, if provided; otherwise use m_nVal 
    } 
}; 

C c(1); 
c.foo(); // use 1 
c.foo(2); // use 2 

を

私が持っているオプション：

（1）オーバーロードfoo()：

class C 
{ 
    int m_nVal; 
public: 
    C(int nVal) : m_nVal(nVal){} 

    void foo() 
    { 
     // use m_nVal 
    } 

    void foo(int nVal) 
    { 
     // use nVal 
    } 
}; 

（2）静的メンバを使用します

class C 
{ 
    static int m_nVal; 
public:   
    void foo(int nVal = m_nVal) 
    { 
     // use nVal, if provided; otherwise use m_nVal 
    } 
}; 

を私はm_nVal静的メンバをしたくないので、オプション1は、一つだけのようです。

これを達成する他の方法はありますか？

Answer 1

インターフェイスを変更したい場合は、他の方法があります。あなたはboost::optionalを使用することができます。

// untested: 
void foo(boost::optional<int> val = boost::optional<int>()) { 
    int value; 
    if (val) value = *val; 
    else value = m_val; 
    // Now use `value` in the function 
} 

あなたはブーストを使用することができない場合は、あなたがあなた自身のNULL可能ラッパーを書くことができます。タイプ（int）と、それが設定されているかどうかを決定するフラグを格納するだけです。

次のオプションは、引数がオプションであることをマークするために、ポインタを使用している：

void foo(int *pval = 0) { 
    int value = (pval? *pval : m_val); 
    // use value from here on 
} 

しかし、ポインタを持つオプションは、関数の引数として右辺値の使用を禁止する（あなたは正しいが必要、すなわち、関数を呼び出す変数は、foo(1)を実行することはできませんが、むしろint x = 1; foo(&x);を実行する必要があります。これは痛みの一種です）。

最後に、2つのオーバーロード、最初にないとだけ転送しない引数と1をとるものを提供するというあなたのアプローチを使用することができます。

void foo(int val) { 
    // actual implementation 
} 
void foo() { 
    foo(m_val); 
} 

これは実際には最良の選択肢かもしれません...

Answer 2

2つのオプションは同等ではありません。メンバをstaticにすることは、それをメソッドのデフォルトとして使用するかどうかを決定するべきではありません。

m_nValがインスタンスではなくクラスに論理的にバインドされている場合は、staticにします。

m_nValがクラスの各オブジェクトに固有の場合は、最初のオプションを使用しないでください。

Answer 3

class C 
    { 
     int m_nVal; 
    public: 

     C(int nVal) : m_nVal(nVal){}    

     void foo(int nVal = -1) 
     { 
     if(nVal == -1) 
      nVal = m_nVal; 

      // use nVal, if provided; otherwise use m_nVal 
     } 
    }; 

C c(1); 
c.foo(); // use 1 
c.foo(2); // use 2 

Answer 4

デフォルトのパラメータを渡すと、コンパイラはそれを渡す必要があります。つまり、m_nValの場合、コンパイラはthis->m_nValを使用します。これは `foo（this-> m_nVal）; 'を意味します。

これは私が何を意味するかです：m_nValプライベートデータは、クラス外部からアクセスすることができるようになる

c.foo(c.m_nVal); // use 1 

、および基本的なC++のルールを破ります。