Executing two timer functions continuously. GUI.
显示 更早的评论
Dear experts,
this is Carlos Andreu. Several months ago, You helped me to refresh continously a textbox with a timer function. This solution has worked perfectly. Now, I would like to add other timer function in order to refresh two text boxes at the same time. I have tried it with the following code:
function myGUI(handles,hObject)
TimerH = timer('UserData', handles.text5, 'TimerFcn', @myTimerFcn, 'Period', 0.5, ...
'ExecutionMode', 'fixedRate');
drawnow;
TimerH2 = timer('UserData', handles.text9, 'TimerFcn', @myTimerFcn2, 'Period', 0.6, ...
'ExecutionMode', 'fixedRate');
drawnow;
start([TimerH TimerH2]);
function myTimerFcn(TimerH,eventData,handles,hObject)
global distance
global flag_tracker
if flag_tracker == 1
TextH = get(TimerH, 'UserData');
distancia = Guardar_pos(TextH);
pause(0.5)
set(TextH,'String',num2str(distance))
drawnow();
else
TextH = get(TimerH, 'UserData');
set(TextH,'String','Measuring...');
end
function myTimerFcn2(TimerH2,eventData,handles,hObject)
global distance2
global flag_tracker
if flag_tracker == 1
TextH2 = get(TimerH2, 'UserData');
distancia_fija = Guardar_pos3(TextH2);
pause(0.5)
set(TextH2,'String',num2str(distance2))
drawnow();
else
TextH2 = get(TimerH2, 'UserData');
set(TextH2,'String','Measuring...');
end
The problem is the textbox of timerH works, but textbox of timerH2 doesn't.
Please, Can you help me?
Thank you in advance,
Carlos.
采纳的回答
Jan
2017-11-29
Using global variables is a bad idea in general. If you see a way to avoid them, do this.
You did not mention, what is not working. Do you get an error message? Is the timer function entered?
Why do you calculate the values.
distancia = Guardar_pos(TextH);
and
distancia_fija = Guardar_pos3(TextH2);
when the variables are not used at all? What is "Guardar_pos" and "Guarder_pos3"?
If you define the TimerFcn as "@myTimerFcn2", it is called with 2 inputs: The handle of the timer and the EventData struct. Then requesting 4 inputs is not smart, although it is not an error, when the not provided inputs are not used in the function:
function myTimerFcn2(TimerH2,eventData,handles,hObject)
% Better:
function myTimerFcn2(TimerH2,eventData)
There is no need for a drawnow after creating a timer object. But after setting a string, drawnow is required to see the changes. A simplified timer callback:
function myTimerFcn2(TimerH2,eventData)
global distance2 flag_tracker
TextH2 = get(TimerH2, 'UserData');
if flag_tracker == 1
% distancia_fija = Guardar_pos3(TextH2); % Is this needed at all?
% pause(0.5)
set(TextH2, 'String', num2str(distance2))
else
set(TextH2, 'String', 'Measuring...');
end
drawnow;
2 个评论
Carlos
2017-11-29
Jan
2017-11-30
@Carlos: You observe that myTimerFcn2 does not change the contents of a specific text field. This can have different reasons:
- The timer is not running
- The string is changed, but always to the same number
- A different text field is accessed, perhaps hidden behind another one.
You can check these 3 problems using the debugger. Set a breakpoint inside myTimerFcn2, e.g. on the "if flag_tracker==1" line. Then start the code. Is the function entered at all? Then point 1 can be excluded. Step forward until distancia_fija is calculated. It looks strange, that you need the handle of the text field to calculate a distance. Is the expected value replied by Guardar_pos3()? IF so, point 2 is excluded. Now try to set the string of the field manually during the dbugging:
set(TextH2, 'String', 'Hello!')
Does it work? Then point 3 is not the problem also.
The only difference between the two callback function is "Guardar_pos" and "Guardar_pos3". If this difference could be avoided, you could use myTimerFcn as callback for both timer objects.
Some notes to the code (partially repeated):
- In "myTimerFcn2(TimerH2,eventData,handles,hObject)" the variables "handles" and "hObject" are neither used nor defined. This is not a bug but confusing. But as soon as "handles" or "hObject" would be used, the code would crash. Prefer "myTimerFcn2(TimerH2,eventData)".
- The drawnow is not called after setting the string to 'Measuring...'. Then this change might not be displayed. Move all redundant code out of the different IF-branches for clarity:
function myTimerFcn2(TimerH2,eventData)
global flag_tracker
TextH2 = get(TimerH2, 'UserData');
if flag_tracker == 1
distancia_fija = Guardar_pos3(TextH2);
set(TextH2,'String',num2str(distancia_fija))
else
set(TextH2,'String','Measuring...');
end
drawnow();
Leaner code without redundant commands offers less chances for typos and if the code must be changed, you do not have to apply the changes repeatedly, but once only, which is less prone to bugs.
更多回答（0 个）
类别
在 帮助中心 和 File Exchange 中查找有关 Entering Commands 的更多信息
Translated by 
