C++でスマートポインタを使用してメモリを確保したい

Question

C++でスマートポインタを利用してnバイト分のメモリを確保し、失敗した場合はエラー処理を書きました

それを読み込んだファイルデータの保存などの方法で、生のポインタとして使用する場合、
もっと優良(一般的)な書き方はありますか？(Byte型を使うなど)

色々書いた結果一応動くコードが完成しただけなので少し不安です

int n=100;

std::unique_ptr<void*> buffer(new(std::nothrow) void* [n]);
if (buffer == nullptr){//メモリを確保できなかった場合の処理
}

追記:
すみません、上のソースコードはポインタの配列を確保するコードだったようなのですが、
memcpy等のコピー先に使えるような連続したメモリが必要で、配列としては使用しません

Answer 1

僕ならこうする:

#include <iostream> // cout, endl
#include <memory>   // uniqur_ptr, make_unique
#include <cstdint>  // uint8_t

int main() {
  int n = 10;
  const char data[] = "abcdefghij";
  std::unique_ptr<uint8_t[]> buffer = std::make_unique<uint8_t[]>(n);

  for ( int i = 0; i <= n; ++i ) {
    buffer[i] = data[i];
  }
  // できたかな?
  for ( int i = 0; i < n; ++i ) {
    std::cout << buffer[i] << ' ';
  }
  std::cout << '[' << buffer.get() << ']' << std::endl;
} 

Comments

[nicoyou]

ご回答ありがとうございます
サンプルプログラムでメモリを配列として確保されていますが、配列として使用しない場合も配列として確保したほうが良いのでしょうか？
それともmake_uniqueを使用するために仕方なくで配列として確保されているのでしょうか?

[episteme]

え？ 連続した領域なら配列と同義ちゃうん?
先頭ポインタが欲しいならサンプルのとおり get() で得られるし。

[nicoyou]

軽く調べてみたところ、std::unique_ptr<uint8_t[]>としてメモリを確保した場合
[]のオーバーロードが使用できるようになると書いていたので、わざわざ使わない[]を付ける必要性はないのではないかと思いました (質問文のコードは間違っていたようなので追記しました)
std::unique_ptr<uint8_t> buffer = std::make_unique<uint8_t>(n);
(動作するのかは分かっていません)

>え？ 連続した領域なら配列と同義ちゃうん?
そうだと思いますm(__)m

[episteme]

型Tがintを引数とするコンストラクタを持っていた場合、make_unique<T>(n) のnがコンストラクタ引数なのか配列の個数なのか曖昧になるやん。

[nicoyou]

理解しました

Answer 2

指摘事項が2点あります。

std::unique_ptr<void*> buffer(new(std::nothrow) void* [n]);

n個のvoid*なので、これはポインタの配列を取得します。

if (buffer == nullptr){//メモリを確保できなかった場合の処理

std::unique_ptr::operator boolがあります

std::unique_ptr<std::byte> buffer(new(std::nothrow) std::byte[n]);
if(!buffer){
  std::cerr << "bad alloc" << std::endl;
}

例外を使用したくない場合を除き、一般的にはstd::vector<std::byte>で十分です。

追記

std::vectorは連続した領域を確保する事が仕様で定められていますので
(なお、std::vector<bool>は除く)

    std::unique_ptr<std::byte> buffer(new(std::nothrow) std::byte[n]);    
    memcpy(buffer.get(),"hello",6);
    std::cout << (char*)buffer.get();

    std::vector<std::byte> v(n);
    memcpy(v.data(), "world", 6);
    std::cout << (char*)v.data() << std::endl;

    std::cout << (char*)&*buffer << (char*)&v[0];

memcpy可能です。

Comments

[nicoyou]

ご回答ありがとうございます
質問文のコードは正しくなかったようで追記しました
おそらくstd::vector<std::byte>は使えないと思います

[episteme]

＞おそらくstd::vector<std::byte>は使えないと思います
なんで？
std::vector<std::byte> buffer(n);
std::byte* ptr = &buffer[0];
これ↑ではダメなん？

[nicoyou]

>std::vectorは連続した領域を確保する事が仕様で定められていますので
一方向listの様なものだと思っていました
覚えておきますm(__)m

[episteme]

vector, array, string は連続領域が確保されることが言語(ライブラリ)仕様で規定されています。ご安心を。
# 単方向リストは std::forward_list が用意されています

[nicoyou]

お二方のご回答ありがとうございました
どちらも参考になったのですが、メモリ確保できなかった処理を含めて、私のコードの間違いを指摘して修正していただいたのでasmさんをベストアンサーに選ばさせていただきました

[asm]

ただし、キャパシティ以上に拡張した際に移動しますので
コンストラクタやreserveでの指定をサボったりすると駄目な事もあります

[nicoyou]

>vector, array, string は連続領域が確保されることが言語(ライブラリ)仕様で規定されています。ご安心を。
># 単方向リストは std::forward_list が用意されています

stringやarrayは感覚的に理解していたのですが、vectorは知りませんでした
std::forward_listの存在も覚えておこうと思います