逆参照できないベクトル

Question

コメントを見て、コードを見てエラーが見つかりました。

私がこのエラーに直面してしまったいくつかの工夫の後に思える：

Debug error: vector iterator is not dereferencable.

私はそれがassingthreads内部のベクトルであることを100％確信しています。

これはエラーを生成します新しく追加されたコードです：for -loop、it = partVec.end()て

void historical::writeData(std::vector<std::vector<std::wstring>> in, const string& symbol) { 
     std::cout << "Sending data to database connector" << std::endl; 
     std::vector<std::vector<std::wstring>> temp; 

     while (!in.empty()) { 
      for (int i = 0; i < 5; i++) { 
       temp.push_back(in.back()); 
       in.pop_back(); 
      } 
      assignthreads(temp, symbol); 
      temp.clear(); 
     } 

    } 
    void historical::assignthreads(std::vector<std::vector<std::wstring>> partVec, const string& symbol) { 
     int i = 0; 
     std::thread threads[5]; 
     std::vector<std::vector<std::wstring>>::iterator it; 
     for (it = partVec.end(); 
      it != partVec.begin(); 
      it--) { 
      std::shared_ptr<database_con> sh_ptr(new database_con); 
      threads[i] = std::thread(&database_con::start, sh_ptr, *it, symbol); 
      partVec.pop_back(); 
      i++; 
     } 
     for (auto& th : threads) th.join(); 

    } 

Answer 1

あなたの最初の時間。定義により

あなたがvectorのend間接参照することはできませんが、お電話：

threads[i] = std::thread(&database_con::start, sh_ptr, *it, symbol); 

あなたはおそらく意図forループは、このようなrbeginとrendを逆イテレータを使用：

for(auto it = rbegin(partVec); it != rend(partVec); ++it) 

---

カップルの追加メモ：

1. は、参照することによって、あなたのvectorを渡しますvoid assignthreads(std::vector<std::vector<std::wstring>>& partVec, const string& symbol)
2. あなたはthreadsがpartVecと同じサイズであることを確認する必要があります。だから、どちらかの操作を行います。vector<thread> threads(size(partVec))かthreads後には定義されています：assignthreadsでforループとassert(size(threads) == size(partVec))

Answer 2

少なくとも一つの問題は、ベクトルのend()逆参照しようとしていることです。

for (it = partVec.end(); it != partVec.begin(); it--) { 
    // ... 
    threads[i] = std::thread(&database_con::start, sh_ptr, *it, symbol); 
    //             ^^^^ 
} 

ループの最初の繰り返しでは、これは未定義です。デバッガはあなたにそれを伝えるだけです。あなたは、ループを「逆転」したい

for (it = partVec.rbegin(); it != partVec.rend(); ++it) 

サイドノート一般的に反復しながらコンテナを変更することをお勧めされていません（rbegin()とrend()経由で利用可能）コンテナのreverse_iteratorを使用する場合は

それを介して（partVec.pop_back();を介して）。 vectorから削除されたもので何もしないように思われるので、内容を繰り返し処理したほうがよいかもしれません。std::vector<>::clear()を呼び出すと、ループの後にベクターからすべての内容が削除されます。