how can I improve this code???!!!

Question

f4r3in 2019-11-13

0
链接

此问题的直接链接

https://ww2.mathworks.cn/matlabcentral/answers/490956-how-can-i-improve-this-code

编辑： Daniel M 2019-11-14

hi , I write a script and in this script below code take time than other codes in my script.

I need your knowledge to know how can I improve this script. (I know it takes only 0.11 sec to run but in large scale inputs it takes too long and also I want to learn that how can I write a script efficient.)

margin=zeros;
l=1;
C_in= 3405;
for i=1:1824
    for j=1:10
        temp=COPT(i,3)*COPT_load(j,2);
        if temp>d
            %this line takes time %from here
            margin(l,1)=COPT(i,2)-COPT_load(j,1);
            %to here
            margin(l,2)=temp;
            l=l+1;
        end
    end
end

So how can I improve my code ???

2 个评论
显示无隐藏无

the cyclist 2019-11-13

Can you upload a *.mat file with the data, so that we can test on your actual data?

f4r3in 2019-11-14

I'm so sorry. this is my project and I cant share it in internet.

but when I click on "run and time" button in MATLAB I understand that these lines take time in my code.

I know you want to help me but for some reasons I cant share it, Sorry.

请先登录，再进行评论。

请先登录，再回答此问题。

Answer 1

Daniel M 2019-11-13

2
链接

此回答的直接链接

https://ww2.mathworks.cn/matlabcentral/answers/490956-how-can-i-improve-this-code#answer_401407

编辑：Daniel M 2019-11-13

在 MATLAB Online 中打开

You should try preallocating margin before the loop. You have only set the value to one element, of zero. Since it's not clear how big margin will be at the end of your loops, you should preallocate it to be the maximum size (1824*10 = 18240). Then at the end, only keep the first 1:l (L) rows using

% after your loops
margin = margin(1:l,:);

Let us know if it's still slow after that, and include a mat file so we can test.

4 个评论
显示 2更早的评论隐藏 2更早的评论

Steven Lord 2019-11-14

在 MATLAB Online 中打开

You don't need to know the exact size of the array you want to preallocate when you write the code, if you can determine or calculate the size of the array from information available before you enter the loop(s). I'm going to guess the limits on i and j are related to the size of COPT?

assert(ismatrix(COPT)) % rather than a 3-D or N-D (N > 2) array
sz = size(COPT);
% Preallocate A to be the same size as COPT
A = zeros(sz);
for whichrow = 1:sz(1)
    for whichcol = 1:sz(2)
        % Do stuff with A(whichrow, whichcol) and COPT(whichrow, whichcol)
    end
end

I have no idea how large COPT will be when you run that code, but I do know A will start off the same size as COPT.

Daniel M 2019-11-14

编辑：Daniel M 2019-11-14

Incremental growth of arrays and dynamic reallocation are tricky things in MATLAB (this is when you cannot know the size that an array will be before the loop occurs). You may be interested in the utility growdata on the File Exchange, where John D'Errico talks about and implements solutions to this problem:

https://www.mathworks.com/matlabcentral/fileexchange/8334-incremental-growth-of-an-array-revisited

请先登录，再进行评论。

how can I improve this code???!!!

2 个评论
显示无隐藏无

采纳的回答

4 个评论
显示 2更早的评论隐藏 2更早的评论

更多回答（0 个）

另请参阅

类别

标签

Community Treasure Hunt

how can I improve this code???!!!

2 个评论 显示 无隐藏 无

采纳的回答

4 个评论 显示 2更早的评论隐藏 2更早的评论

更多回答（0 个）

另请参阅

类别

标签

Community Treasure Hunt

2 个评论
显示无隐藏无

4 个评论
显示 2更早的评论隐藏 2更早的评论