Ошибка кода MATLAB

Question

Ниже приведена моя функция MATLAB, и в ней есть некоторая ошибка, поэтому пожелайте, чтобы любой эксперт мог исправить или указать на ошибку и способ ее улучшить. Я с помощью MATLAB R2015a

function L = remove_all(L,E) 
% remove_all(List,element) - delete all occurrences of E from L 
for Index = length(L.elements):-1:1 
    if isequal(E,L.elements{Index}) 
     L.elements(Index) = []; 
    end 
end 

Answer 1

Вы мутируют структуру в цикле for путем удаления элементов в этой структуре, но забывая о том, что длина изменения структуры, как вы удаляете элементы. Таким образом, длина структуры уменьшает с каждым удалением, и вы, в конечном счете, выходите из пределов при удалении элементов. В частности, вы использовали length, чтобы зафиксировать начальную длину списка, но по мере удаления элементов эта длина уже не совпадает и цикл for не знает этого факта. Таким образом, вы в конечном итоге получите ошибку вне границ из-за этого ошибочного удаления элементов.

Способ решения этой проблемы было бы сохранить все места, которые должны быть удалены из структуры и, когда цикл for делается, удалить их все сразу:

function L = remove_all(L,E) 
% remove_all(List,element) - delete all occurrences of E from L 
indices = []; %// New - keep the locations that need to be removed 
for Index = length(L.elements):-1:1 
    if isequal(E,L.elements{Index}) 
     indices = [indices; Index]; %// Add to list if equal 
    end 
end 
L.elements(indices) = []; %// Remove all entries at once 

Answer 2

Хотя ответ rayryeng наведен из ошибки и исправить ее, но я все равно хочу попробовать версию 1-линейный код с cellfun:

function L = remove_all(L,E) 
% remove_all(List,element) - delete all occurrences of E from L 
L.elements(cellfun(@(x) x == E, L.elements)) = []; 
end