# Problem with loop index

3 Ansichten (letzte 30 Tage)
Arthur Romeu am 9 Mär. 2020
Kommentiert: Arthur Romeu am 9 Mär. 2020
Greetings everyone,
I am trying to do a simple loop with this piece of code:
for k = 1:size(rm_zerohoras,1)
for j = 1:size(FinAti,1)
if rm_zerohoras(k) == FinAti.Data(j)
FinAti(j,:) = [];
end
end
end
rm_zerohoras contains datetimes that I need to delete from FinAti. My intention is to, whenever the loop finds the corresponding datetime on a row in FinAti, the whole row gets deleted on FinAti. I'll attach both rm_zerohoras and FinAti for better understanding. The current problem is that when I run the program, the following error appears:
"Error using tabular/dotParenReference (line 108)
Index exceeds the number of array elements (474)."
I think it has something to do with the size of FinAti which is shrinking whenever the loop passes through. But how do I overcome this?
Arthur.
##### 0 Kommentare-2 ältere Kommentare anzeigen-2 ältere Kommentare ausblenden

Melden Sie sich an, um zu kommentieren.

### Akzeptierte Antwort

Alex Mcaulley am 9 Mär. 2020
A simple way to do that avoiding the loop is:
idx = ismember(FinAti.Data,rm_zerohoras);
FinAti(idx,:) = [];
##### 1 Kommentar-1 ältere Kommentare anzeigen-1 ältere Kommentare ausblenden
Arthur Romeu am 9 Mär. 2020
Thanks!

Melden Sie sich an, um zu kommentieren.

### Weitere Antworten (1)

Calculate the indices of all the rows you want to delete , then delete them all in one go afterwards, as e.g.
rowsToDelete = [1 5 7 8];
FinAti( rowsToDelete, : ) = [];
There's probbaly a vectorised way to calculate those indices, but if not simply store them in your loop instead of actually deleting rows.
If the number of occurences isn't huge the cost of having an array of indices growing in a loop is negligible, despite that it will give a code warning.
You should never delete elements of an array you are iterating forwards through - it never ends well. You can do it if you iterate backwards, but still my first suggestion is much cleaner as deleting a row at a time will be a lot slower than just deleting them all in one go.
##### 1 Kommentar-1 ältere Kommentare anzeigen-1 ältere Kommentare ausblenden
Arthur Romeu am 9 Mär. 2020
Thank you!
Your explanation was really clear. I've learned a lot!

Melden Sie sich an, um zu kommentieren.

### Kategorien

Mehr zu Loops and Conditional Statements finden Sie in Help Center und File Exchange

### Community Treasure Hunt

Find the treasures in MATLAB Central and discover how the community can help you!

Start Hunting!

Translated by