## Simple Question about Optimization of Nested IF-FOR loops

### Mohsen (view profile)

on 18 Jun 2013

Does any one know how to optimize this code so that it runs faster:

``` for i=1:iNZ;
if iPointsinSlice>0;
for m=1:iNX;
for l=1:iNY;
DoseCubeU(m,l,i)=100*SumDose(m,l,i)/RX_Dose;
end
end
end
end
end```

Thanks a lot!

## Products

### Kye Taylor (view profile)

on 18 Jun 2013

First of all, it appears that each time through the outer-most loop (the first one), the variable iPointsinSlice does not change. Therefore, you can simplify to

```for i=1:iNZ;
for m=1:iNX;
for l=1:iNY;
DoseCubeU(m,l,i)=100*SumDose(m,l,i)/RX_Dose;
end
end
end
end
```

but i bet you actually mean to have some expression that depends on i execute before the loop that iterates over m.

Mohsen

### Mohsen (view profile)

on 18 Jun 2013

You are right!

Here is the entire code for this part.

iNZ=11; iNCUr=55;

UrPointX=zeros(iNCUr);

UrPointY=zeros(iNCUr); UrContourZ=[ 0 0 0 0 0 -5 -5 -5 -5 -5 -10 -10 -10 -10 -10 -10 -15 -15 -15 -15 -15 -15 -20 -20 -20 -20 -20 -20 -25 -25 -25 -25 -30 -30 -30 -30 -30 -30 -35 -35 -35 -35 -35 -35 -40 -40 -40 -40 -40 -40 -45 -45 -45 -45 -45]; ZSlice=[ 5 0 -5 -10 -15 -20 -25 -30 -35 -40 -45];

UrContourX=[ 69.103 70.633 71.05 70.911 68.964 69.938 68.408 69.103 69.938 70.633 67.713 68.686 69.659 70.007 70.285 69.52 67.018 68.269 69.381 69.938 69.103 68.964 69.729 67.713 67.921 68.547 69.868 70.216 70.494 68.408 68.825 70.355 70.633 70.702 70.007 68.964 68.2 68.964 70.911 69.52 69.52 67.991 69.103 70.077 70.077 69.242 68.478 69.173 70.772 70.772 70.633 68.964 69.242 70.633 70.633];

UrContourY=[ 44.801 45.914 47.026 47.026 46.887 50.919 49.946 48.556 48.417 49.112 52.24 50.711 50.641 50.989 51.754 53.144 54.534 52.866 52.866 54.395 55.369 55.369 55.994 55.508 54.187 53.352 53.422 55.508 56.62 56.481 53.839 53.978 54.117 55.438 56.272 56.272 55.786 53.7 54.256 55.925 56.064 55.09 53.561 53.839 53.492 53.422 52.935 51.614 52.24 52.588 51.058 50.78 49.668 50.224 50.919];

Offset=[ 6.8947; -16.648; 5];

`    for i=1:iNZ;`
`        iPointsinSlice=1;`
```        for j=1:iNCUr;
if UrContourZ(j)==ZSlice(i);
iPointsinSlice = iPointsinSlice+1;
UrPointX(iPointsinSlice-1)=1 + (UrContourX(j)-Offset(1,1));
UrPointY(iPointsinSlice-1)=1 + (UrContourY(j)-Offset(2,1));
end
end```
```        if iPointsinSlice>0;
indx=round(iPointsinSlice);
indy=round(iPointsinSlice);```
```            indx(1:iPointsinSlice-1)=UrPointX(1:iPointsinSlice-1)/DresX;
indy(1:iPointsinSlice-1)=UrPointY(1:iPointsinSlice-1)/DresY;```
`            SliceMaskUr = poly2mask(indx,indy,iNNX,iNNY);`
```            for m=1:iNX;
for l=1:iNY;
DoseCubeU(m,l,i)=100*SumDose(m,l,i)/RX_Dose;
end
end
end            ```
```        end
end```

My goal is to optimize this code and I would highly appreciate your help.

Thanks a lot! :)