Something is wrong with this simple code.

2013 6 27
1 个回答
回答已采纳
3 次查看（30 天）

0 个投票

Hi, I am quite new here and I am trying to write a code. We have collected data from two channels.
One is actuation count of a relay which is like 0001111111111111111111111111111111111111111111111122222222222.... and a voltage signal that is like 24,24,23,5,3,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,5,10,20,24,24,24,...
I want to analyse the data, and count how many samples occur since the actuation count changes (in the code I compare it to a variable named "x")and the voltage data reaches 0. then I multiply the number of samples for the time of the sample rate to calculate the time it takes to reach 0 Volts totally. Then I add these values to an array "tclosearray". I will appreciate any help.
It says it is busy so I guess a loop is running nonstop.
This is the code:
tclosearray=[];
y=0;
x=0;
for k=1:30000
if UntitledActuation_Count(k)~=x
while UntitledVoltage(k)>=0
if k<= 30000
y=y+1;
k=k+1;
end;
end;
tclose=y*0.01;
tclosearray=[tclosearray;tclose];
y=0;
x=UntitledActuation_Count(k);
end;
end;

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

David Sanchez
David Sanchez 2013-6-27
Your guess has to be right. You have to be within an infinite loop. What does the function UntitledActuation_Count do?
Most likely the loop-break condition is never fulfilled. Make your code to present some data in command window to debug it.
Sergio
Sergio 2013-6-27
Hi David! UntitledActuation_Count is an array of data collected from a relay which works as a switch here. (Open/close)Do you know how to show some data in the command window?
(Sorry the GMAIL didn-t tell me you answered me!)
Thanks
Sean de Wolski
Sean de Wolski 2013-6-27
编辑：Sean de Wolski 2013-6-27
@Sergio,
use something like:
disp(UntitledVoltage(k))
Sergio
Sergio 2013-6-27
Thank you for the advice

请先登录，再进行评论。

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

 采纳的回答

David Sanchez
David Sanchez 2013-6-27

0 个投票

Make sure the data array
UntitledVoltage(k)
contains some negative data, otherwise the while loop will go on for ever. (see advice given by Sean de Wolski in his comment)

1 个评论
显示 -1更早的评论 隐藏 -1更早的评论

Sergio
Sergio 2013-6-27
This is the new code. Yes, it didn-t have 0, or negative data, so I changed it for >0.2 which is good after seeing the data. My bad. However, it stills say it is busy and doesn-t create the array with the solutions :(
tclosearray=[];
y=0;
x=0;
for k=1:numel(UntitledVoltage)-1
if UntitledActuation_Count(k)~=x
while UntitledVoltage(k)>0.2
if k<numel(UntitledVoltage)-1
y=y+1;
k=k+1;
end;
end;
tclose=y*0.01;
tclosearray=[tclosearray;tclose];
y=0;
x=UntitledActuation_Count(k);
end;
end;

请先登录，再进行评论。

更多回答（0 个）

类别

在 帮助中心 和 File Exchange 中查找有关 Loops and Conditional Statements 的更多信息

标签

Community Treasure Hunt

Find the treasures in MATLAB Central and discover how the community can help you!

Start Hunting!

Translated by