数据结构 - C++删除表中相同元素,循环问题在哪?

Question

template <class T>
bool SeqList<T>::Delsame(T x)
{
    int i,j;
    for(int j=1;j<n+1;j++)
    {
        if (elements[j-1]==x)
        {
            for (int i=j-1;i<n;i++)
                {
                    elements[i]=elements[i+1];
                }
             n--;
        }
    }
    return true;
}

删除表中相同元素部分的代码,当表中所需删除元素只有1个的时候,运行正常.当所需删除元素有多个的时候,运行结果正确,但是会提示.exe已停止工作,所以想知道循环问题在哪??

Answer 1

n代表的是順序表元素個數嗎？如果是的話elements[i+1]這一句會取到elements[n]的，越界了。
BTW，第一個循環為什麼直接j++？如果刪除了一個元素，後面的往前移了，還進行j++，那豈不是會跳過一個元素。
你試一下兩個相鄰的相同元素能不能刪除。

Answer 2

樓上已經指出了錯誤，我來嘗試給出一個正確的實現

int find(int lo,int hi,const T& e){
    while(lo<hi--&&elem[hi]!=e);
    return hi;//若失败返回lo-1
}
void remove(int lo,int hi){
    while(hi<size) elem[lo++]=elem[hi++];
}
void deduplication(){
    int i=1;
    while(i<size){
        (find(0,i,elem[i])<0)?
            i++:remove(i);
    }
}