Reading data from text box

I am simply trying to get user inputted data from a text box and read that data in a push button. Shouldn't it be as simple as a get(handles.DisplayNumber,'String'); function in the push button?

function DisplayNumber_CreateFcn(hObject, eventdata, handles) %text box

if ispc && isequal(get(hObject,'BackgroundColor'), get(0,'defaultUicontrolBackgroundColor'))
  set(hObject,'BackgroundColor','white');
end

% --- Executes on button press in Lookup.
function Lookup_Callback(hObject, eventdata, handles) %Push button 
N = get(handles.DisplayNumber,'String');

[Naz]

Yes.

[Muhammad]

This is a very stupid answer to a question to reading an editbox. no where it is mentioned how editbox data is to be retrieved. He is talking about background color and everything and not the main thing that is required.

[Jan]

No, Muhammad, I do not agree. The answer, independent of which answer you actually mean, is neither stupid nor very stupid. Naz's answer solves the problem exhaustively and Steven has accepted it as a solution.

It would be polite if you do not post harsh opinions about the quality of other answers, but offer better solutions, if you think you know one. Thanks.

Answer 1

[Naz]

Yes. However, as soon as the button function is executed, your N is deleted. If you want to keep the data in memory, you should include the data in the main struct:

handles.N = get(handles.DisplayNumber,'String');

and then make sure to save it before leaving the fucntion:

guidata(hObject, handles);

And from that point on you should reference your variable as handles.N

The reading of the value from the textbox is correct. I would delete your button and the textbox and create new ones just to make sure I reference the correct objects. Otherwise, go over the objects and check if the tags correspond to the names you reference in the code. Also, every time you make changes in the GUI, you need to save it in addition to your .m file

[Steven]

Firstly I appreciate your response.

I am not using the variable N anywhere other than inside that function so I didn't name is handles.N but I suppose I could just in case I wanted to use it somewhere else. The problem I am having though is when I press my push button I get a "Invalid handle object." error.

[Walter Roberson]

What do you want done with the string once you have fetched it? Callbacks are not able to pass values back to their caller (because they do not have a caller.) Therefore if you want to use the value of the string outside of the callback, you must export it somehow. Storing it in the handles structure the way Naz shows is one common way of doing that. It doesn't matter whether you name it N or something else as long as the name is a valid field name and does not overwrite anything.

I think we would need to see more code to figure out the cause of the "Invalid handle object" error.

[Steven]

The purpose of the entire program is to take a picture from my computer modify it bases on some information contained in displaylist variable and display it.

Okay the following is the entire Guide I am working with

function DisplayNumber_Callback(hObject, eventdata, handles)

function DisplayNumber_CreateFcn(hObject, eventdata, handles)

if ispc && isequal(get(hObject,'BackgroundColor'), get(0,'defaultUicontrolBackgroundColor'))

set(hObject,'BackgroundColor','white');

end

% --- Executes on button press in Lookup.

function Lookup_Callback(hObject, eventdata, handles)

N = get(handles.DisplayNumber,'String');

[displaylist,type] = lookup(N);

the lookup function is written by me and is as follows

function [displaylist,type] = lookup(N)

N = num2str(N);

String = ['Defect Data\displaylist',N];

load(String)

I plan on taking the output of the lookup function and running an Imshow function to display an image all under the lookup push button so I won't need to store it as a handle although I am not opposed to doing that. I am pretty sure what ever I am doing wrong is a simple mistake. I debugged the program and the error occurs on the line where I am trying to extract the information from N.

[Steven]

Started from scratch and it worked

must have edited the wrong thing thanks for everyone's help!!

Answer 2

[Eric Keller]

don't you have to load the "handles" data? I don't use that functionality, but that's my understanding of how it works

[Steven]

Eric thank you for your response.

I thought that was what the get function was essentially doing. I have only ever used the load function to load variable, picture, ect ... into my code that were already in the directory or store somewhere on my computer not a user inputted source. However, I did try it with load instead of get and got an error.

[Eric Keller]

I see I'm wrong from the code sample. I always just debug/set a setpoint and see what the handles are. You are dereferencing the handles structure incorrectly somehow, only your debugger knows for sure. It will become obvious when you do this.

[Walter Roberson]

I can tell from the code that Steven is using GUIDE; GUIDE callbacks automatically pass the handles structure in.