関数間で文字列を転送するC++

Question

私はランダムなモンスタージェネレータを作成し、次にモンスターのパラメータを印刷する必要があるコースのプログラムを作成しています。

私が生成し、データを印刷するmainで、このループを使用します。

for (g=0; g < m; g++) 
    { 
     tab[g] = &createRandomMonster(g); 
     printMonsterData(*tab[g]); 
    } 

この関数モンスター生成するために：印刷用

Monster createRandomMonster(int g) 
{ 
    ostringstream buffer; 
    buffer << ++g; 
    string name = buffer.str(); 
    Monster nowypotwor{ name, MonsterType(rand() % 6 + 1), rand() % 25 + 5, (double)(10.0 + (rand()/(double)(RAND_MAX))*(30.0 - 10.0)) }; 
    return nowypotwor; 
} 

そして、この機能を：

void printMonsterData(const Monster &potwor) 
{ 
    cout << "potwor: " << potwor.name << endl << "typ: " << getTypeName(MonsterType(potwor.type)) << endl << "atak: " << potwor.attack << endl << "zycie: " << potwor.health << endl << endl; 
} 

私はモンスターの名前を印刷しません。 printMonsterDataがcreateRandomMonsterにある場合は行いますが、それは私が行う必要があるmainで行います。

Answer 1

モンスターのリターンをキャプチャする必要があります。すなわち、 モンスターaMonster = createRandomMonster（g）; タブ[g] = & aMonsterと言うことができます。

EDITでも、このソリューションではタブ[g]はforループの寿命、つまりaMonsterの寿命の間のみ有効です。これは参照による操作の別の結果です。あなたがメインの最も外側のスコープにモンスターを宣言し、それに戻るならば、この制限を回避することができます...

タブがモンスター配列ではなくモンスター*配列である理由はありますか？

tab[g] = &createRandomMonster(g); 

createRandomMonster戻りMonsterクラス：それは

Answer 2

これは間違っているあなたはすぐにそれを参照して印刷するためにそれを渡すために欽慕特に以来...私は何かが欠けていない限り、後者のように、より理にかなってそれは返されるものは、完全なステートメントの終わりの直後に破棄される一時的なオブジェクトです（セミコロンの後ろには、セミコロン;の文字です）。したがって、この一時的なオブジェクトのアドレスを&で取得すると、実際にはすぐに破棄される一時的なアドレスが取得され、tab[g]に未定義のビヘイビアが発生します。

私はあなたがtabにポインタとしてごMonsterオブジェクトを格納いけないが、その値によって示唆している：

std::vector<Monster> g; 

あなたはポインタは、その後、それらを保存する必要がある場合：

std::vector<std::unique_ptr<Monster>> g; 

とにcreateRandomMonsterを変更します。

std::unique_ptr<Monster> createRandomMonster(int g) 
{ 
    ostringstream buffer; 
    buffer << ++g; 
    string name = buffer.str(); 
    std::unique_ptr<Monster> nowypotwor = std::make_unique<Monster>(name, MonsterType(rand() % 6 + 1), rand() % 25 + 5, (double)(10.0 + (rand()/(double)(RAND_MAX))*(30.0 - 10.0))); 
    return nowypotwor; 
} 

および

tab[g] = createRandomMonster(g); 

Answer 3

あなたは関数がスマートポインタ（例えばを保持するために、「タブ」に変更、いっそのモンスター（モンスター*）へのポインタを返す、またはcreateRandomMonsterにする必要があります。 shared_ptr）を通常のポインタの代わりに使用し、その関数からそのポインタを返します。

std::vector<std::shared_ptr<Monster>> tab; 

std::shared_ptr<Monster> createRandomMonster(int g) 
{ 
    ostringstream buffer; 
    buffer << ++g; 
    string name = buffer.str(); 

    return std::make_shared<Monster>(name, 
     MonsterType(rand() % 6 + 1), rand() % 25 + 5, 
     (double)(10.0 + (rand()/(double)(RAND_MAX))*(30.0 - 10.0))); 
} 

for (g=0; g < m; g++) 
{ 
    tab[g] = createRandomMonster(g); 
    printMonsterData(*tab[g]); 
} 

あなたは多型の挙動を持つ必要がない場合は、それはベクトルの値を保持し、タブに右辺値をコピーするのが最善です：

std::vector<Monster> tab; 

for (g=0; g < m; g++) 
{ 
    tab[g] = createRandomMonster(g); 
    printMonsterData(tab[g]); 
} 

Answer 4

は、関数に文字列を転送するには、のいずれかに従うことができます

1）Cスタイル - 配列（静的/グローバル/ローカル）を宣言し、そのアドレス（またはC++のような参照）をポインタとして呼び出された関数に渡します。動的に割り当てられている場合は、代わりにポインタを渡します

2）C++スタイル - std :: stringを使用して、必要に応じてすべての拡張を実行します。これは、境界線を書き留めることを心配する必要はなく、あなたが関数から復帰したとき、または等価性を後で実行するときの魔法