Serena - I think that you are missing an end at the end of your first double for loop block that sets the colours of the pushbuttons, and that missing end has been put at the end of the next double for loop block to make some of the pushbuttons visible. This causes two problems since both sets of double for loops use the same indexing variables i and j (as an aside, you may want to consider renaming these to something more meaningful since MATLAB also uses these to represent the imaginary number). This most likely causes a problem with the line
set(L(i,j),'callback',{@my_funct,L,A});
only being assigned to the (5,5) pushbutton control since it occurs outside of the previous double for loop. I suggest that you include this in the first block and end it appropriately as
for r = 1: size(A,1) % rows
for c = 1: size(A,2) % columns
set(L(r,c),'callback',{@my_funct});
if A(r,c) == 0
set(L(r,c),'BackgroundColor', 'w')
set(L(r,c), 'visible', 'off')
elseif A(r,c) == 1
set(L(r,c),'BackgroundColor','b')
set(L(r,c), 'visible', 'off')
elseif A(r,c) == 2
set(L(r,c),'BackgroundColor','y')
set(L(r,c),'String','GOAL')
set(L(r,c), 'visible', 'on')
elseif A(r,c) == 3
set(L(r,c),'BackgroundColor','g')
end
end
end
So now every pushbutton will be assigned the callback. Since the above outer for loop as been terminated with the end then you will have to remove the redundant end from the second block of code.
Note the change to your callback
set(L(r,c),'callback',{@my_funct});
Since the callback is a nested function within create_maze, you do not need to pass in parameters that are declared or are local to create_maze. The callback will have access to these elements since it is nested. This will require you to change the callback signature to
function [] = my_funct(source, callbackdata)
So that should get the callback to fire. The next step will be to revisit your callback logic - you probably only want "do" anything if the user is pressing a white pushbutton. You can use the source input along with the property BackgroundColor to retrieve the colour as
bgColor = get(source,'BackgroundColor');
if all(bgColor)
% pushbutton is white
% do something
end
Since the bgColor is a 1x3 array indicating the RGB color, if it is all ones then the button background is white.
Now you presumably need to determine the neighbours of this white pushbutton only. How can you do that since all you have is the source (which is the handle to the pushbutton)? Since the handles are of a floating precision (double) data type, you don't want to be looking through each element of L and comparing each to this handle. So how else can you determine where in L (and so A) this handle is? Hint - look back at how you assign the callback to the pushbutton and consider using the r and c as the unique identifier into L and A.
