C++ ポインタのchar配列に追加するには

Question

char型のポインタの末尾にchar1つ（0x0D）を追加したいのですが、
以下のような方法の他にもっとシンプルな方法はありますか？

  char src[7] = "abcdef";
  char* test = &src;

  const int size = sizeof(src) + 1;
  char dst[size] = {};
  ::memcpy(dst, test, sizeof(test));
  dst[size - 1] = 0x0D;   // 0x0Dを末尾に追加

Answer 1

char src[] = "abcdef";
char dst[N];
sprintf(dst, "%s%c", src, 0x0d);

※ ただし src が'\0'を含まないこと。

[追記] てかC++なら

std::string dst = "abcdef";
dst += (char)0x0d;

でよくね?

Comments

[yumetodo]

それあまり安全ではないです・・・。なぜならばdstのサイズがsrcのサイズより少なくとも1大きいことを保証しないといけませんが、その依存関係はわかりにくく、見落としやすいです。

[episteme]

そこが問題なんかなー...ならやっぱ std::string 使うが楽

[yumetodo]

それはそう、std::stringをパフォーマンス的に使えないとかそういう事情かなとおもって回答してましたけど。

[episteme]

そーゆー事情なんかなー...

[退会済みユーザー]

お二方ありがとうございます！
std::stringの方法でうまくいきそうです。

[episteme]

そっち!? やっぱそっち!?

[yumetodo]

あれれっ！

[yumetodo]

もしかして: 深読みしすぎた

[退会済みユーザー]

すみません。いろいろ方法があって、その上c++の文字操作が苦手なので混乱してました。
今回の回答とてもためになりました。
ありがとうございます。

Answer 2

基本方針としてはそれでいいと思いますが、sizeof(command)ってなんですかね・・・？
dst[size - 1] = 0x0D;って文字列終端NULL文字の位置を書き換えてるので、NULL終端しなくなりますよ。dst[size - 2] = 0x0D;では。

---

追記

std::stringを使えるなら絶対に使ってください。pure Cのようなやり方で文字列を操作するメリットはC++には存在しません。パフォーマンス的な最適化を考えるときはstd::string_viewの出番となります。どちらもどうしてもうまくはまらないというときにはmemcpyするというのは手段としてありえます。優先順位を間違えないでください。