Hi Could any one tell me what is wrong in my

Question

Prince Of Persia [field = CE,DM = BLOCK]

Hi Could any one tell me what is wrong in my

code?

for(auto it = this->holder.end()-1;it >= this->holder.begin();--it){
if(*it == 0){
it = this->holder.erase(it);
}else{
break;
}
}
this->holder.shrink_to_fit();

it gets
double free or corruption (out)
Signal: SIGABRT (Aborted)
at line
this->holder.shrink_to_fit();

0

20.01.2022

5 ответов

30 просмотров

Prince Of Persia [field = CE,DM = BLOCK] Автор вопроса

I got used reverse iterator that's not much different from normal iterator type of holder is std::vector<unsigned int> and I use vector::erase i don't think there would be and difference between the method and std::erase I want to erase all 0 element from the end-1(last element) to begin and stop deleting when I got any other value

0

20.01.2022

Ghasem Ramezani

Prince Of Persia [field = CE,DM = BLOCK]
I got used reverse iterator that's not much differ...

Just use Reverse Iterators + std::remove + std::erase instead of those buggy loops. https://gist.githubusercontent.com/another-ghasem/565dfd86c6abaccd031e04be16963314/raw/5ef21487a63968e711d041d291a9cd7af8b8cec6/main.cc

0

20.01.2022

Prince Of Persia [field = CE,DM = BLOCK] Автор вопроса

Ghasem Ramezani

Just use Reverse Iterators + std::remove + std::er...

but I don't want to erase 0s in middle nor in beginning

0

20.01.2022

Ghasem Ramezani

Prince Of Persia [field = CE,DM = BLOCK]
but I don't want to erase 0s in middle nor in begi...

Sorry, I didn't see that.

0

20.01.2022

Ghasem Ramezani · Accepted Answer

Ghasem Ramezani

https://stackoverflow.com/help/minimal-reproducible-example What is type of the holder ? Why you are using raw-loop instead of std::remove + std::erase or at least Reverse Iterators? What do you want to do?

0

20.01.2022

Похожие чаты

Hi Could any one tell me what is wrong in my

5 ответов

Похожие вопросы