C++プログラムchar配列の削除時に終了する

Question

配列の要素を削除する際に問題があります。 以下のコードを見ることができます。

delete[] payload[x].dataが2回目に実行された後で、2番目のforループが終了すると、プログラムは終了します。

struct s_payload{ 
     u_char data[300]; 
    } 
    . 
    . 
    u_char sample[300] ="Lorem ipsum dolor sit amet, eu mea laudem impetus."; 
    int _bufferSizeConnection = 1000; 
    int testCounter = 0; 
    s_payload* activeBuffer = new s_payload[_bufferSizeConnection]; 
    for (;;) { 
     for (long int counter = 0; counter < _bufferSizeConnection; counter++) {   
      memcpy(activeBuffer[counter].data, sample, 299); 
     } 
     //_buffersConnection is std::queue 
     _buffersConnection.push(activeBuffer); 
     testCounter += 1; 
     if (testCounter == 15) { 
      LOG_INFO(">testcounter %d", testCounter); 
      break; 
     }    
     activeBuffer = new s_payload[_bufferSizeConnection];      
    }   
    for (int i = 0; i < 15; i++) { 
     //taking one of the activeBuffer from _buffersConnection 
     s_payload* payload = _buffersConnection.wait_and_pop();   
     for (int x = 0; x < _bufferSizeConnection; x++) { 
      u_char* current= payload[x].data; 
      delete[] payload[x].data; //Exit at the second running of this line; 
     } 
     delete[] payload; 
    } 

このコードで何が問題になっていますか？ ありがとうございます。

Answer 1

あなたはactiveBuffer = new s_payload[...]と言っているので、合法的にdelete[] activeBufferと言うことができます。ただし、dataは、s_payload要素内のポインタではなく、配列で、activeBufferです。したがって、delete payload[x]またはdelete[] payload,dataのいずれかになると、delete payload[x].data —になる必要はありません。

しかし、自分の持つものにパッチを当てるよりも、アプローチを考え直すことをおすすめします。固定サイズのバッファ配列をプッシュしてポップしています—本当に必要ですか？ std::stringまたはstd::vector<unsigned char>をプッシュしてポップアップするだけで、メモリ管理について心配する必要はありませんか？例えば、std::vectorは、任意の量のデータを保持することができる。

ライブラリーにできるだけ多くの作業をさせて、そうする必要はありません！