C++ tryブロックのオブジェクトを作成してstd :: arrayに格納します

Question

私はPlayerオブジェクトを持っています。そのオブジェクトはそのコンストラクタ内で例外をスローする可能性がありますので、私の主な機能ではtryブロックの中に2 Playerオブジェクトを作成しています。 私はそのようなstd::arrayにおけるこれら2つのPlayer S格納したい：問題は、私はtryブロックの外に配列を使用することはできないだろうということです、と私はすべてのmainを置くことを聞いた

try 
{ 
    Player p1(10, 10, ikazuchi); 
    Player p2(70, 10, hibiki); 
    std::array<Player, 2> players = {p1, p2}; 
} 

をtryブロック内のコードはしばしば悪い考えです。

p1とp2がそこに存在しないため、私はstd::arrayをtryブロックの後に宣言できません。

std::vectorで問題を解決できますが、コンパイル時にアレイのサイズを知っているときにはstd::arrayを使用する方が良いと読んだことがあります。

デフォルトのコンストラクタを作成してオブジェクトを作成し、tryブロック内にそれらを埋め込むことができましたが、コンストラクタですべてを作成する方が適切なようです。

そのためのベストプラクティスは何ですか？

Answer 1

ダイナミックアロケーションなどを使用して、いつでもboost::optional<std::array<Player, 2>>のような問題を回避することができますが、実際の問題は次のとおりです。つまり、プレーヤーオブジェクトの1つが構築に失敗し、例外がスローされたとします。その後、players配列で何をしますか？それは法的な状態ではなく、少なくとも例外がスローされなければそれが期待される状態にはありません。

tryの範囲を大きくすることに問題はありません。それは、例外があなたの進歩を妨げるすべてのものをカバーするべきです。物理的にたくさんのソースコードがあるのが好きでない場合は、コードを関数に移動します。とにかく、これは良いアイデアかもしれません。すべてがうまくいくと仮定し、エラーを処理することが主な役割を果たす関数です。

playersの内部オブジェクトが無効な状態（コンストラクタが投げられた状態）にある場合、意味のある方法がある場合は、その状態のオブジェクトを最初に配置することができます（tryブロック）、tryの中にそれらを割り当ててください。

質問は、あなたが不要なデフォルトコンストラクタを提供したくないと述べたが、私はどちらかplayerがtryの内側に属していることを主張し、またはPlayerはデフォルトコンストラクタ（または「正しく初期化されていません表現の同等の方法）が必要です。

Answer 2

以下のようにすることができます。

int main() { 
    std::array<int, 2> array; 
    try { 
    array[0]=1; 
    array[1]=1; 
    } catch (...) { 
    } 
    return 0; 
} 

利用Playerの代わりint。

Answer 3

あなたのプレイヤーがコピー/移動され、自明に構成されている場合は、配列を作成して割り当てます。

std::array<Player, 2> players; 
try{ 
    players[0]=Player(10, 10, ikazuchi); 
    players[1]=Player(70, 10, hibiki); 
} catch ... 

代わりにあなたが行うことができます：

std::optional<std::array<Player, 2>> players; 
try{ 
    Player p1(10, 10, ikazuchi); 
    Player p2(70, 10, hibiki); 
    players.emplace(std::array<Player, 2>{{std::move(p1),std::move(p2)}}); 
} catch ... 

いますが、ブーストのオプションのように、オプションの非C++ 17のバージョンを確認する必要があるかもしれません。

別のアプローチは、次のとおりです。

auto players = [&]()->std::array<Player, 2>{ 
    try { 
    Player p1(10, 10, ikazuchi); 
    Player p2(70, 10, hibiki); 
    return {{std::move(p1),std::move(p2)}}; 
    } catch (some_error){ 
    throw some_other_error; 
    } 
}(); 

が、これは、配列またはスローのいずれかによって終了する必要があります。