Problem with loop index

Question

Arthur Romeu 2020-3-9

0
链接

此问题的直接链接

https://ww2.mathworks.cn/matlabcentral/answers/509892-problem-with-loop-index

评论： Arthur Romeu 2020-3-9

采纳的回答： Alex Mcaulley

在 MATLAB Online 中打开

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?

Thanks in advance,

Arthur.

0 个评论
显示 -2更早的评论隐藏 -2更早的评论

请先登录，再进行评论。

请先登录，再回答此问题。

Answer 1

Alex Mcaulley 2020-3-9

0
链接

此回答的直接链接

https://ww2.mathworks.cn/matlabcentral/answers/509892-problem-with-loop-index#answer_419273

在 MATLAB Online 中打开

A simple way to do that avoiding the loop is:

idx = ismember(FinAti.Data,rm_zerohoras);
FinAti(idx,:) = [];

1 个评论
显示 -1更早的评论隐藏 -1更早的评论

Arthur Romeu 2020-3-9

Thanks!

请先登录，再进行评论。

Answer 2

Adam 2020-3-9

1
链接

此回答的直接链接

https://ww2.mathworks.cn/matlabcentral/answers/509892-problem-with-loop-index#answer_419269

在 MATLAB Online 中打开

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.