C++削除のエラーは[]ポインタを通じて

Question

環境を反復した後に：Windows 7のプロのx64、マイクロソフトのVisual Studio 2015のEnterprise、バージョン14.0.25424.00アップデート上記のコードを実行している3

int testFunction() 
{ 
    std::string _orig = "[188 80% (1/2)O:152]"; 
    std::string _orig2 = "[999 99% (1/1)O:999]"; 

    char *orig = NULL; 
    char *orig2 = NULL; 

    orig = new char[_orig.length() + 1]; 
    strcpy(orig, _orig.c_str()); 

    orig2 = new char[_orig2.length() + 1]; 
    strcpy(orig2, _orig2.c_str()); 

    *orig++; 
    *orig2++; 

    int a = atoi(orig); 
    int b = atoi(orig2); 

    delete[] orig; 
    delete[] orig2; 

    return 0; 
} 

は「_CrtIsValidHeapPointer（ブロックでクラッシュ） "エラー。

私が反復しない（* orig ++と* orig2 ++）なら、問題はありません。

私の質問は、どのようにポインタを反復処理すればいいのですか？そして、私がそれらと関係がある必要があるときには、正しく削除してください。

Answer 1

割り当てたポインタは削除されませんでした。

deleteは、newが返す元のメモリアドレスで呼び出される必要があります。 orig++を行ったので、deleteのアドレスを指摘できません！

反復処理は、間接参照に、アレイサブスクリプションを使用してインデックスで行われ、することができます。

*(orig+i) = 'a'; 

それとも、同じ上に別のポインタを取得することができます。これを行うことと同じである

orig[i] = 'a'; 

データを作成し、これを変更します。

char* pOrig = orig; 
++pOrig; 

なぜあなたはただ、++それ自体では、繰り返しを行うだろう

*orig++; // why dereferencing? 

を書きました。

Answer 2

私はポインタを反復処理することができますし、私はそれらと一緒に行う必要があることを終えたら、正しく削除してください。

ポインタのコピーを作成します。

char* orig = new char[size]; 
char* i = orig; 
*i++ = 'a'; 
delete orig; 

Aは、おそらくより一般的なイディオムは一時的なデリファレンスにある：

for(int i = 0; i < size - 1; i++) 
    orig[i] = 'a'; 

---

私は[使用std::stringに大好きです]しかし、私はstd :: stringで動作しませんatoi（）を使用する必要があります

あなたは間違っています。 atoiはstd::stringと同じです。 strcpyと同じように、単にstd::string::c_str()を使用してください。 newでメモリのブロックを割り当てる理由はまったくありません。

Answer 3

int testFunction() 
{ 
    std::string _orig = "[188 80% (1/2)O:152]"; 

    int a = 0; 
    for (std::string::iterator it = _orig.begin(); it != _orig.end(); ++it) 
    { 
     if (isdigit((char)*it)) 
      a = (atoi(it._Ptr)); 
    } 

    return 0; 
} 

私はそれを得ました。私がこの結論に至るのを助けてくれた皆様に感謝します。 std :: stringでの滞在は、実際には最良のアプローチでした。

Answer 4

生ポインタを使用しないでください。あなたのコードは簡単です：

std::string orig = "[188 80% (1/2)O:152]"; 
std::string orig2 = "[999 99% (1/1)O:999]"; 

int a = atoi(orig.c_str() + 1); 
int b = atoi(orig2.c_str() + 1); 

あなたの間違いは、元のポインタの代わりにシフトポインタを削除しようとしています。その結果、ヒープマネージャーは割り当てられたブロック情報を間違って取得し、通常は割り当てられたポインターの前に置かれ、ヒープの破損が発生します。