# for loop mixing the results with the previous loops

2 views (last 30 days)
Boby S on 16 Dec 2019
Answered: Dheeraj Singh on 19 Dec 2019
Hi
I am using the following loop for evey 10 row of my array (20 row or more* 2 column). then, I want to save the results (Z or res).
converted = table2array(imported);
n=2;
for ind = 1:n
tmp = converted((n-1)*10+1:n*10,:);
X = tmp
X = [true;diff(converted(:,1))~=0];
Y = diff(find([X;true]));
V = accumarray(cumsum(X),~converted(:,2));
Z = [Y,V]
end
res(:,:,ind)= Z
The problem is the code also runs for row 11 (should be first 10 ) then for the next 10 row starts to run from row 12 (which should be 11-20).
This means mixing first 10 rows result with the second batch of 10 rows.
correct sample for 10 rows:
>> M = [0,0;0,1;1,1;1,0;1,1;1,0;0,1;1,1;0,1;0,1]
M =
0 0
0 1
1 1
1 0
1 1
1 0
0 1
1 1
0 1
0 1
>> X = [true;diff(M(:,1))~=0];
>> Y = diff(find([X;true]));
>> V = accumarray(cumsum(X),~M(:,2));
>> Z = [Y,V]
Z =
2 1
4 2
1 0
1 0
2 0

Dheeraj Singh on 19 Dec 2019
The issue is that for loop variable ind is outside the for loop
You can try the following code:
converted = table2array(imported);
n=2;
for ind = 1:n
(ind-1)*10+1
ind*10
tmp = converted((ind-1)*10+1:ind*10,:)
X = tmp
X = [true;diff(converted(:,1))~=0];
Y = diff(find([X;true]));
V = accumarray(cumsum(X),~converted(:,2));
Z = [Y,V]
res(:,:,ind)= Z;
end