How to simplify the loops of a matrix

1 view (last 30 days)
Walker on 6 Feb 2020
Edited: Rik on 6 Feb 2020
Hello. I have an issue with a code performing some calculations in two loops . It is very slow since I am using two loops. Could you please help me solve this,
Clear all
nx = 100;
ny = 200;
phi = random(nx,ny);
num = 5;
phi1 =zeros(nx,ny);
for i =1:nx
for j = 1:ny
phi1 = mod(phi(i,j), num);
end
end
figure1
surf(phi1)

Robert U on 6 Feb 2020
Hi Xinyuan,
the code snippet you are providing does not make any use of the two for-loops. You can remove them, since the function mod() is applied to each element of the matrix "phi". The result is the same. Your code just overwrites the same matrix for nx x ny times.
clear all
nx = 100;
ny = 200;
phi = rand(nx,ny);
num = 5;
phi1 = mod(phi, num);
figure
surf(phi1)
Kind regards,
Robert
Walker on 6 Feb 2020
@Robert,@Rik, Thank you very much for your reply. I'm learning matlab. It helps me a lot. I understand that , for my case, the two loops are not necessary.
Robert U on 6 Feb 2020
Hi Xinyuan,
your introduced change illustrates the problem that you solved partially yourself posting the former code snippet:
Remove the two for-loops and the indexing. The function mod() performs the modulo-operator on each single element of the matrix phi, thus there is no loop necessary.
Kind regards,
Robert