C++でベクトルの要素を削除およびソートする

Question

この単純なコードを作成して、ベクトルの要素を削除およびソートしました。私はそれを何度もチェックしていますが、ソートと削除のアルゴリズムはどちらも個別に実行すると正しいですが、要素を削除してからベクトルを降順で並べ替えると問題があります。プログラムはゴミを '0'私が昇順で並べ替えるとうまくいきます。 ご協力いただければ幸いです。ありがとうございました。 は、ここでは、あなたの並べ替え一部とoff-by-oneエラーを持っているプログラム1

#include<iostream> 
#include<vector> 
#include<algorithm> 
using namespace std; 
int main() 
{ 
    bool flag = 1; 
    int size = 5,delind; 

    vector <int> v1; 
    v1.resize(size); 

    for(int i = 0; i<size; i++) 
    { 
     cout<<"\nenter a no. "; 
     cin>>v1[i]; 
    } 

    // index of the element to be deleted 
    cout << "\nEnter the index of no. to delete: "; 
    cin >> delind; 

    for(int i = delind; i < size; i++) 
    { 
     v1[i] = v1[i+1]; 
    } 
    --size; 
    v1.erase(v1.begin()+size); 

    for(int i =0; i < size; i++) 
    { 
     cout << endl << endl << v1[i]; 
    } 
    cout << "\n sorting"; 
    // sorting 

    while (flag == 1) 
    { 
     flag = 0; 

     for(int i = 0; i < size; i++) 
     { 
      if(v1[i] < v1[i+1]) // works fine with if(v1[i]>v1[i+1]) 
      { 
       flag = 1; 
       swap(v1[i],v1[i+1]); 
      } 
     } 
    } 

    for(int i = 0; i < size; i++) 
    { 
     cout << endl << endl << v1[i]; 
    } 
    return 0; 
} 

Answer 1

のサンプルランのイメージです。次の行ではこの時点sizeで

for(int i = 0;i<size;i++) 
{ 
    if(v1[i]<v1[i+1]) // works fine with if(v1[i]>v1[i+1]) 
    { 
    flag = 1; 
    swap(v1[i],v1[i+1]); 
    } 
} 

4であり、そしてv1.size()もforループi 4.ある通常のように0から3に進みます。しかし、あなたはv1[i+1]を使用しているので、ループはv1[4]にアクセスしようとします。これは範囲外であり、見ているガベージ値を与えます。ちょうどあなたのループ境界を修正し、それを修正する

：

for (int i = 0; i < size - 1; i++)