Guide handing objects to functions

Using Guide I have two radiobuttons. When pushing one the other has to be unchecked. So far this is my solution:
function distButton_Callback(hObject, eventdata, handles)
checkThis = get(handles.distButton, 'Value');
checkThat = get(handles.fftButton, 'Value');
if checkThis == checkThat
set(handles.fftButton, 'Value', ~checkThis);
end
Since there are four button pairs a function would save a couple of lines.
Calling the function:
checkThis('distButton', 'fftButton', handles);
However combining handles. with a string does not seem to convert this to 'distButton'
> Reference to non-existent field 'this'.
function checkThis(this, that, handles)
checkThis = get(handles.this, 'Value');
checkThat = get(handles.that, 'Value');
if checkThis == checkThat
set(handles.that, 'Value', ~checkThis);
end
It's probably a simple matter of syntax..

2 Commenti

Adam
Adam il 17 Ago 2018
Why are you not just placing the radio buttons inside a uibuttongroup which does the mutual exclusivity for you?
Since this is my first time working with guide, I was simply not aware of that. Thanks for the advice.

Accedi per commentare.

 Risposta accettata

ES
ES il 17 Ago 2018
function checkThis(this, that, handles)
checkThis = get(eval(['handles.', this]), 'Value');
checkThat = get(eval(['handles.', that]), 'Value');
if checkThis == checkThat
set(eval(['handles.', that]), 'Value', ~checkThis);
end

3 Commenti

checkThis = get(handles.( this ), 'Value');
checkThat = get(handles.( that ), 'Value');
would be a lot better. Don't use eval when you don't have to (i.e. never use it!)
Stephen23
Stephen23 il 17 Ago 2018
Modificato: Stephen23 il 17 Ago 2018
Do NOT follow this answer! There is absolutely no point in using ugly eval like that, unless you want to force yourself into writing slow, complex, buggy, hard-to-debug code.
It is so easy to write simpler and much more efficient code:
function checkThis(this, that, handles)
checkThis = get(handles.(this),'Value');
checkThat = get(handles.(that),'Value');
if checkThis==checkThat
set(handles.(that),'Value',~checkThis);
end
And, as Adam already wrote, a uibuttongroup would be even simpler.
Much appreciated. I will take this into account.

Accedi per commentare.

Più risposte (0)

Categorie

Scopri di più su Interactive Control and Callbacks in Centro assistenza e File Exchange

Richiesto:

il 17 Ago 2018

Commentato:

il 17 Ago 2018

Community Treasure Hunt

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

Start Hunting!

Translated by