# Correct error in logical indexing

2 views (last 30 days)
Morten Jørgensen on 23 Apr 2019
Edited: Guillaume on 23 Apr 2019
Hi, I can't figure out why i get the wrong result in this loop
I now when k=3 it should be 1, but I get a 0.
it gives me a array only containing of 0
thanks a lot for your help.
for k = length(PassCoord)
X(k,1) = mBall(PassCoord(k,1)) > mBall(PassCoord(k,2))
end

Alex Mcaulley on 23 Apr 2019
Your loop only runs for k = length(PassCoord), you need to put the limits of k (1:length(PassCoord)):
for k = 1:length(PassCoord)
X(k,1) = mBall(PassCoord(k,1)) > mBall(PassCoord(k,2))
end

Morten Jørgensen on 23 Apr 2019
arh, thanks a lot :)
Guillaume on 23 Apr 2019
DO not use length on a matrix. See explanation in my answer.
And also, do not use the loop anyway, it's long-winded and unnecessary.

Guillaume on 23 Apr 2019
• You use length on a matrix. If your matrix happens to only have one row, your code will error since length will return the number of columns in that case. Here length means the number of rows, so use size(PassCoord, 1) to explicitly request the number of rows.
• What's the point of the loop> the whole code is equivalent to:
X = mBall(PassCoord(:, 1)) > mBall(PassCoord(:, 2));
As for what you don't get the result you expect, we can't tell without knowing what's in mBall and PassCoord. Either the code you wrote doesn't do what you meant it to do or the inputs are different from what you think. So explain and attach the two variables in a mat file.