C++のgetenv（）ラッパー関数は、値を設定しない

Question

getenv()状態にcplusplus.comドキュメント...

ポインタがその内容または有効性をさらに呼び出しによって変更することができる内部メモリブロックにポイントを返さ〜getenv

...「コンテンツを保持したい場合はコピーしてください」私はいくつかの変数を取得する必要があるためので、私は少しラッパー関数のカップルを書いた：

#include <iostream> 
#include <string.h> 

using namespace std; 

void getEnv (char *val, const char *var) { 
    val = nullptr; 
    char *enVar = getenv(var); 
    if (enVar != nullptr) { 
     val = new char[strlen(enVar) + 1]; 
     strcpy(val, enVar); 
    } 
} 

void getEnv (int &val, const char *var) { 
    val = -1; 
    char *enVar = getenv(var); 
    if (enVar != nullptr) { 
     val = atoi(enVar); 
    } 
} 

int main() { 
    char *textMode = nullptr; 
    int cLen = 0; 

    getEnv(cLen, "CONTENT_LENGTH"); 
    cout << cLen << endl << endl; 

    getEnv(textMode, "TEXT_MODE"); 
    if (textMode == nullptr) 
     cout << "Not set."; 
    else 
     cout << "[" << textMode << "]<br>\n"; 

    return 0; 
} 

予想通りintバージョンでは動作しますが、私は戻ってcharバージョンから何を取得し、私は何も意味しない：I場合*textModeを宣言時に初期化しないでください。初期化されていないポインタのままです。

これは正しいポインタですか？右？私はそれが分かっています。ポインタが必要です。私はこれらのうちの1つを理解するだろうが、ちょっと - 少なくとも私は仕事にリンクされたリストを持っている！わーい！

Answer 1

第2の関数は、val（int）を参考にして、となり、期待通りに渡される変数を変更することができます。

最初の関数はval（char*）の値をvoid getEnv (char *val, const char *var)とします。したがって、valを変更すると、渡される変数には影響しません。簡単な解決策は、それを単に参考にしておくことです。void getEnv (char *&val, const char *var)

Answer 2

私のコメントとそれらに対するOPの応答に従います。

#include <iostream> 
#include <string.h> 

using namespace std; 

// Use a class to encapsulate the data need to be captured 
// in an environment variable.  
class EnvironmentVariable 
{ 
    public: 

     EnvironmentVariable(char const* name) : name_(name), isSet_(false) 
     { 
     char *val = getenv(name); 
     if (val != nullptr) 
     { 
      isSet_ = true; 
      this->value_ = val; 
     } 
     } 

     bool isSet() const 
     { 
     return isSet_; 
     } 

     void getValue(char const*& val) const 
     { 
     if (isSet_) 
     { 
      val = this->value_.c_str(); 
     } 
     else 
     { 
      val = nullptr; 
     } 
     } 

     void getValue(int& val) const 
     { 
     if (isSet_) 
     { 
      val = stoi(this->value_); 
     } 
     else 
     { 
      val = 0; // Find a suitable default value 
     } 
     } 

    private: 
     std::string name_; 
     std::string value_; 
     bool isSet_; 
}; 

int main() { 
    char const* textMode = nullptr; 
    int cLen = 0; 

    EnvironmentVariable env1("CONTENT_LENGTH"); 

    env1.getValue(cLen); 
    cout << cLen << endl << endl; 

    EnvironmentVariable env2("TEXT_MODE"); 
    env2.getValue(textMode); 

    if (textMode == nullptr) 
     cout << "Not set.\n"; 
    else 
     cout << "[" << textMode << "]<br>\n"; 

    return 0; 
}