Azzera filtri
Azzera filtri

Error in for loop: Index exceeds matrix dimensions.

37 visualizzazioni (ultimi 30 giorni)
Me
Me il 1 Ago 2024 alle 13:09
Modificato: VBBV il 1 Ago 2024 alle 14:47
Hi everyone! I need help with my code.I have a matrix (300x300) with some equal rows, and I need to eliminate the duplicate rows. I can't solve the error. Can anyone help me??Thank you very much!
Below I copy the part of the code with the error and a simplified example.
clear all
matrix_f=[1 2 3; 1 2 3; 4 5 6; 7 8 9; 7 8 9; 10 11 12];
for k=1:length(matrix_f(:,1))
if matrix_f(k,1)==matrix_f(k+1,1)
matrix_f(k+1,:)=[];
end
end
  1 Commento
VBBV
VBBV il 1 Ago 2024 alle 14:46
Modificato: VBBV il 1 Ago 2024 alle 14:47
@Me, you need to give the column dimension for the matrix in the for loop and use size function instead of length
clear all
matrix_f=[1 2 3; 1 2 3; 4 5 6; 7 8 9; 7 8 9; 10 11 12];
for k=1:size(matrix_f,2) % give the column dimension here
if isequal(matrix_f(k,:),matrix_f(k+1,:)) % use isequal function for check
matrix_f(k,:)=[]; % delete duplicate rows of numbers
end
end
matrix_f % result
matrix_f = 4x3
1 2 3 4 5 6 7 8 9 10 11 12
<mw-icon class=""></mw-icon>
<mw-icon class=""></mw-icon>

Accedi per commentare.

Risposta accettata

Aquatris
Aquatris il 1 Ago 2024 alle 14:08
Modificato: Aquatris il 1 Ago 2024 alle 14:09
You have essentially made two mistakes:
  1. matrix_f only has 6 rows. You setup up your for loop such that k is 1 2 3 4 5 6 . When k is 6, you are trying to reach the 7th row of matrix_f, which does not exist. So your for loop should iterate from 1 to [length(matrix_f(:,1)-1].
  2. Within the for loop you change the size of matrix_f by removing rows. If you want to do this type of manipulation, you should just store the index you want to remove and remove the rows once your for loop is done. Otherwise indexing becomes messed up and you will run into the same issue, trying to reach a row that does not exist.
  3. You only remove rows that are next to each other.
Of course vectorizing the code is always better as in @Torsten answer.
clear all
matrix_f=[1 2 3; 1 2 3; 4 5 6; 7 8 9; 7 8 9; 10 11 12];
idx2remove = [];
for k=1:(length(matrix_f(:,1))-1)
if matrix_f(k,1)==matrix_f(k+1,1)
idx2remove = [idx2remove;k];
end
end
matrix_f(idx2remove,:) = [];
matrix_f
matrix_f = 4x3
1 2 3 4 5 6 7 8 9 10 11 12
<mw-icon class=""></mw-icon>
<mw-icon class=""></mw-icon>

Più risposte (1)

Torsten
Torsten il 1 Ago 2024 alle 13:20
Modificato: Torsten il 1 Ago 2024 alle 13:22
matrix_f=[1 2 3; 1 2 3; 4 5 6; 7 8 9; 7 8 9; 10 11 12];
unique(matrix_f,'stable','rows')
ans = 4x3
1 2 3 4 5 6 7 8 9 10 11 12
<mw-icon class=""></mw-icon>
<mw-icon class=""></mw-icon>
Note that this code will remove all duplicate rows in the matrix, not only subsequent ones.

Community Treasure Hunt

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

Start Hunting!

Translated by