は私がのCString ::フォーマットを使用するか

Question

が、私はこの

コード1のような2コード持ってsprintf_sべき：CStringの:: Formatメソッドを

CString cStr; 
char* ToString(CString pszFormat, ...) 
{ 
    va_list argList; 
    va_start(argList, pszFormat); 
    cStr.FormatV(_T(pszFormat), argList); 
    va_end(argList); 
    return (LPTSTR)(LPCTSTR)cStr; 
    //Note that this will return the pointer to the cstring content 
} 

コード2：sprintf_s方法

char strChar[100]; 
char* ToString(char const* const _Format, ...) 
{ 
    va_list argList; 
    va_start(argList, _Format); 
    vsprintf_s(strChar, _Format, argList); 
    va_end(argList); 
    return strChar; 
    //Note that this will return the pointer to the string content 
} 

でコード1、私は完全に安全だと感じています - 私は長さが長すぎるかもしれないことを恐れる必要はありません。しかし、コード1はパフォーマンスを低下させる恐れがあります。私はそれがメモリリークを引き起こすかどうかわからない。そして、Cstringに動的な長さがあると、多分、それは誰のビジネスのようにメモリを割り振り、解放するだろうと思います。

私はコード2を思いついています。しかし、コード2では、_Formatを長すぎるような長さの文字列を1000に渡すと危険に直面します。プログラムが 'バッファーが小さすぎます'というエラーでクラッシュします。

私が優れている1かわからない：CStringの::フォーマットまたはsprintf_s？ sprintf_sが実際にパフォーマンスを向上させ、CString :: Formatがパフォーマンスに悪い場合は、sprintf_sで「バッファが小さすぎる」のを防ぐためにさらに努力します。しかし、sprintf_sがそれほど価値がない場合は、CString :: Formatを使用します。

は、読書のためにありがとうございます。あなたはstrCharのバッファがオーバーフローかもしれないと心配している場合

Answer 1

、なぜ単にvsnprintf_s（）を使用していません。 2番目の引数は、出力バッファに書き込まれる文字数を制限します。あなたは、この余分なsizeOfBufferフィールドを受け取り、vsnprintf_s（）にそれを渡すためにあなたの「ToStringメソッド（）」関数を変更することができます。

参照[詳細およびバッファオーバーランを防止するための他の方法のためにhttps://msdn.microsoft.com/en-us/library/d3xd30zz.aspx][1]。