No appropriate method, property, or field 'Mariorun1' for class 'matlab.gr​aphics.pri​mitive.Ima​ge'.

Im trying to make a Mario game gui using an arduino joystick as a controller. For some reason I keep getting an error in the loop where the mario sprites alternates and mario moves horizontally. Can anyone help with this? The code below is in the opening function of the gui.

 a = arduino('COM8','Uno');
[Sprites Spritesmap SpritesAlpha] = imread('some_mario_stuff_and_more_by_pokekoks-d4wzrc5.png');
handles.Sprites.Sprites = Sprites;
handles.Sprites.Mariorun1 = Sprites(181:1:213,89:1:121,:);
handles.Sprites.Mariorun2 = Sprites(219:1:251,9:1:41,:);
handles.Sprites.Mariorun3Jump = Sprites(220:1:254,127:1:161,:);
handles.Sprites.MarioStanding = Sprites(181:1:213,11:1:43,:);

Stage_UC = imread('Mario1_1.png');
Mario_Structure = load('mario_stages.mat');
handles.Template = Mario_Structure.Stage1_1.statMapInd; % or whatever it's called.
Stage = Stage_UC(1:1:227,:,:);
imshow(Stage);
axes(handles.TempAxis)
imshow(handles.Template);
axes(handles.MarioAxis)
imshow(Stage);

 axes(handles.MarioSpriteAxis)
 handles.Sprites = imagesc(handles.Sprites.MarioStanding);
 handles.Sprites.AlphaData = SpritesAlpha(181:1:213,11:1:43,:);
 axis off

 configurePin(a,'A0','AnalogInput')
 for time = [1:1:10000];
     A2Voltage = readVoltage(a, 'A0');
     CurrentX = 0;
if A2Voltage >= 3  
 handles.Sprites = imagesc(handles.Sprites.Mariorun1);
 handles.Sprites.AlphaData = SpritesAlpha(181:1:213,89:1:121,:);
 axis off
 pause(.04)
 handles.Sprites = imagesc(handles.Sprites.Mariorun2);
 handles.Sprites.AlphaData = SpritesAlpha(219:1:251,9:1:41,:);
 axis off
 pause(.04)
 handles.Sprites = imagesc(handles.Sprites.Mariorun1);
 handles.Sprites.AlphaData = SpritesAlpha(181:1:213,89:1:121,:);
 axis off
 pause(.04)
 handles.Sprites = imagesc(handles.Sprites.Mariorun3Jump);
 handles.Sprites.AlphaData = SpritesAlpha(220:1:254,127:1:161,:);
 axis off
 CurrentX = CurrentX + 1
 set(handles.MarioSpriteAxis, 'Position' ,[CurrentX 21.385 1.8 0.615]);
elseif A2Voltage < 2
 set(handles.MarioSpriteAxis,'xdir','reverse')
 handles.Sprites = imagesc(handles.Sprites.Mariorun1);
 handles.Sprites.AlphaData = SpritesAlpha(181:1:213,89:1:121,:);
 axis off
 pause(.04)
 handles.Sprites = imagesc(handles.Sprites.Mariorun2);
 handles.Sprites.AlphaData = SpritesAlpha(219:1:251,9:1:41,:);
 axis off
 pause(.04)
 handles.Sprites = imagesc(handles.Sprites.Mariorun1);
 handles.Sprites.AlphaData = SpritesAlpha(181:1:213,89:1:121,:);
 axis off
 pause(.04)
 handles.Sprites = imagesc(handles.Sprites.Mariorun3Jump);
 handles.Sprites.AlphaData = SpritesAlpha(220:1:254,127:1:161,:);
 axis off
     CurrentX = CurrentX - 1 
     set(handles.MarioSpriteAxis, 'Position' ,[CurrentX 21.385 1.8 0.615])
end
 end

if true
  % code
end
%Update handles structure
guidata(hObject, handles);

Answer 1

[Walter Roberson]

You have, for example,

   handles.Sprites = imagesc(handles.Sprites.Mariorun1);

This replaces all of handles.Sprites with the output of imagesc(), which is going to be a primitive image() object. image() objects do not have Mariorun2 properties, so when you do

   handles.Sprites = imagesc(handles.Sprites.Mariorun2);

a few lines down, there is no Mariorun2 property to access.

[Curtis Vannor]

Thank you for replying. So what should i do in this case?

[Walter Roberson]

create one image() object for each possible variety of sprite. set the visible 'off' if it is not visible, 'on' when it becomes visible. To move it, set its XData and YData properties.

I guess you might have to copyobj() sometimes, if you need more than one sprite with the same appearance.

[Curtis Vannor]

Do you mean imagesc()?

[Walter Roberson]

imagesc and image and imshow all create image() objects

[Curtis Vannor]

so i should make separate axes for each variety?

[Walter Roberson]

The only reason to create a different axes for each variety would be if the different varieties needed to have different color maps. But even then it would be better to instead convert the pseudocolor into RGB and put all of them into the same axes.

Ah... You should have exactly one axes for each "layer". That is, if you have two things that can collide, they should be on the same axes. However if you have some things that are always on top of the others, those should go into a different axes. Clouds that do not interfere with what they are passing over, for example.

[Curtis Vannor]

Ok so how would I set visibility without using axes?

[Walter Roberson]

Each image() object has its own Visible property.

[Curtis Vannor]

This is what I tried:

 axes(handles.SpriteAxis)
 M = imagesc(handles.Sprites.MarioStanding);
 M.AlphaData = SpritesAlpha(181:1:213,11:1:43,:);
 A = imagesc(handles.Sprites.Mariorun1);
 A.AlphaData = SpritesAlpha(181:1:213,11:1:43,:);
 R = imagesc(handles.Sprites.Mariorun2);
 R.AlphaData = SpritesAlpha(181:1:213,11:1:43,:); 
 I = imagesc(handles.Sprites.Mariorun3Jump);
 I.AlphaData = SpritesAlpha(181:1:213,11:1:43,:);
 axis off

 a = arduino('COM8','Uno');

 configurePin(a,'A0','AnalogInput')
 for time = [1:1:10000];
     A2Voltage = readVoltage(a, 'A0');
     CurrentX = 0;
if A2Voltage >= 3  
set(M,'visible','off');
set(A,'visible','on');
set(R,'visible','off');
set(I,'visible','off');
axis off
 pause(.04)
set(M,'visible','off');
set(A,'visible','off');
set(R,'visible','on');
set(I,'visible','off');
 axis off
 pause(.04)
set(M,'visible','off');
set(A,'visible','on');
set(R,'visible','off');
set(I,'visible','off');
 axis off
 pause(.04)
 set(M,'visible','off');
set(A,'visible','off');
set(R,'visible','off');
set(I,'visible','on');
 axis off
 CurrentX = CurrentX + 1

im getting an invalid or deleted object error

[Walter Roberson]

You need a "hold on" after your first imagesc()

[Curtis Vannor]

Ok now im getting this error:

Warning: Error creating or updating Image Error in value of property AlphaData Array is wrong shape or size

Here's the code:

 axes(handles.SpriteAxis)
 M = imagesc(handles.Sprites.MarioStanding);
 M.AlphaData = SpritesAlpha(181:1:213,11:1:43,:);
 hold on;
 A = imagesc(handles.Sprites.Mariorun1);
 A.AlphaData = SpritesAlpha(181:1:213,11:1:43,:);
 R = imagesc(handles.Sprites.Mariorun2);
 R.AlphaData = SpritesAlpha(181:1:213,11:1:43,:); 
 I = imagesc(handles.Sprites.Mariorun3Jump);
 I.AlphaData = SpritesAlpha(181:1:213,11:1:43,:);
 axis off

 a = arduino('COM8','Uno');

 configurePin(a,'A0','AnalogInput')
 for time = [1:1:10000];
     A2Voltage = readVoltage(a, 'A0');
     CurrentX = 0;
if A2Voltage >= 3  
set(M,'visible','off');
set(A,'visible','on');
set(R,'visible','off');
set(I,'visible','off');
axis off
 pause(.04)
set(M,'visible','off');
set(A,'visible','off');
set(R,'visible','on');
set(I,'visible','off');
 axis off
 pause(.04)
set(M,'visible','off');
set(A,'visible','on');
set(R,'visible','off');
set(I,'visible','off');
 axis off
 pause(.04)
 set(M,'visible','off');
set(A,'visible','off');
set(R,'visible','off');
set(I,'visible','on');
 axis off
 CurrentX = CurrentX + 1

[Walter Roberson]

handles.Sprites.Mariorun3Jump = Sprites(220:1:254,127:1:161,:);

That is 35 x 35

   I.AlphaData = SpritesAlpha(181:1:213,11:1:43,:);

That is 33 x 33

[Curtis Vannor]

Ok thank you. Can you take a look at this? I'm trying to use a color template in order to declare what can and cannot be touched by mario. Its not allowing me to use logicals in order to set the different colors on the template to different values.

axes(handles.TempAxis)
imshow(handles.Template); 

axes(handles.MarioAxis)
 imshow(Stage); 

   axes(handles.SpriteAxis)
   M = imagesc(handles.Sprites.MarioStanding);
   M.AlphaData = SpritesAlpha(181:1:213,11:1:43,:);
   hold on;
   A = imagesc(handles.Sprites.Mariorun1);
   A.AlphaData = SpritesAlpha(181:1:213,89:1:121,:);
   R = imagesc(handles.Sprites.Mariorun2);
   R.AlphaData = SpritesAlpha(219:1:251,9:1:41,:); 
   I = imagesc(handles.Sprites.Mariorun3Jump);
   I.AlphaData = SpritesAlpha(220:1:254,127:1:161,:);
   axis off

   [Rows Columns Color] = size(handles.Template)
   for RowIndex = 1:1:Rows
       for ColumnIndex = 1:1:Columns
           if handles.Template(RowIndex, ColumnIndex, 2) <= 22 &&...
             handles.Template(RowIndex, ColumnIndex, 3) <= 24
               handles.Template(RowIndex,ColumnIndex,:) == 1;
           else
              handles.Template(RowIndex,ColumnIndex,:) == 0;     
           end
       end
   end

   configurePin(a,'A0','AnalogInput')
        CurrentX = 0;
   for time = [1:1:10000];
       A2Voltage = readVoltage(a, 'A0');
  if A2Voltage >= 3
   Row = floor(handles.SpriteAxis.Position(2)/29);
   Column = floor((handles.SpriteAxis.Position(1)+15)/29);
     set(handles.SpriteAxis,'xdir','normal')
  set(M,'visible','off');
  set(A,'visible','on');
  set(R,'visible','off');
  set(I,'visible','off');
  axis off
   pause(.04)
  set(M,'visible','off');
  set(A,'visible','off');
  set(R,'visible','on');
  set(I,'visible','off');
   axis off
   pause(.04)
  set(M,'visible','off');
  set(A,'visible','on');
  set(R,'visible','off');
  set(I,'visible','off');
   axis off
   pause(.04)
   set(M,'visible','off');
  set(A,'visible','off');
  set(R,'visible','off');
  set(I,'visible','on');
   axis off
    if  (handles.Template(Row,Column) == 0)
      handles.SpriteAxis.Position(1) = handles.SpriteAxis.Position(1)+15;
    end  
  elseif A2Voltage < 2
  Row = floor(handles.SpriteAxis.Position(2)/29);
  Column = floor((handles.SpriteAxis.Position(1)-15)/29);
  set(handles.SpriteAxis,'xdir','reverse')
  set(M,'visible','off');
  set(A,'visible','on');
  set(R,'visible','off');
  set(I,'visible','off');
  axis off
   pause(.04)
  set(M,'visible','off');
  set(A,'visible','off');
  set(R,'visible','on');
  set(I,'visible','off');
   axis off
   pause(.04)
  set(M,'visible','off');
  set(A,'visible','on');
  set(R,'visible','off');
  set(I,'visible','off');
   axis off
   pause(.04)
   set(M,'visible','off');
  set(A,'visible','off');
  set(R,'visible','off');
  set(I,'visible','on');
   axis off
    if  (handles.Template(Row,Column) == 0)
        handles.SpriteAxis.Position(1) = handles.SpriteAxis.Position(1)-15;
    end
  end
   end

[Walter Roberson]

You have

             handles.Template(RowIndex, ColumnIndex, 3) <= 24
               handles.Template(RowIndex,ColumnIndex,:) == 1;
           else
              handles.Template(RowIndex,ColumnIndex,:) == 0;     
           end

Reminder that == is comparison; if you wanted assignment you would use =

[Curtis Vannor]

I tried that but when I changed this:

if  (handles.Template(Row,Column) = 4)
      handles.SpriteAxis.Position(1) = handles.SpriteAxis.Position(1)-15;
  end

im getting this error:

The expression to the left of the equals sign is not a valid target for an assignment.

[Walter Roberson]

axes(handles.TempAxis)
imshow(handles.Template); 

axes(handles.MarioAxis)
 imshow(Stage); 

     axes(handles.SpriteAxis)
     M = imagesc(handles.Sprites.MarioStanding);
     M.AlphaData = SpritesAlpha(181:1:213,11:1:43,:);
     hold on;
     A = imagesc(handles.Sprites.Mariorun1);
     A.AlphaData = SpritesAlpha(181:1:213,89:1:121,:);
     R = imagesc(handles.Sprites.Mariorun2);
     R.AlphaData = SpritesAlpha(219:1:251,9:1:41,:); 
     I = imagesc(handles.Sprites.Mariorun3Jump);
     I.AlphaData = SpritesAlpha(220:1:254,127:1:161,:);
     axis off

     [Rows Columns Color] = size(handles.Template)
     for RowIndex = 1:1:Rows
         for ColumnIndex = 1:1:Columns
             if handles.Template(RowIndex, ColumnIndex, 2) <= 22 &&...
               handles.Template(RowIndex, ColumnIndex, 3) <= 24
                 handles.Template(RowIndex,ColumnIndex,:) = 1;
             else
                handles.Template(RowIndex,ColumnIndex,:) = 0;     
             end
         end
     end

     configurePin(a,'A0','AnalogInput')
          CurrentX = 0;
     for time = [1:1:10000];
         A2Voltage = readVoltage(a, 'A0');
    if A2Voltage >= 3
     Row = floor(handles.SpriteAxis.Position(2)/29);
     Column = floor((handles.SpriteAxis.Position(1)+15)/29);
       set(handles.SpriteAxis,'xdir','normal')
    set(M,'visible','off');
    set(A,'visible','on');
    set(R,'visible','off');
    set(I,'visible','off');
    axis off
     pause(.04)
    set(M,'visible','off');
    set(A,'visible','off');
    set(R,'visible','on');
    set(I,'visible','off');
     axis off
     pause(.04)
    set(M,'visible','off');
    set(A,'visible','on');
    set(R,'visible','off');
    set(I,'visible','off');
     axis off
     pause(.04)
     set(M,'visible','off');
    set(A,'visible','off');
    set(R,'visible','off');
    set(I,'visible','on');
     axis off
      if  (handles.Template(Row,Column) == 0)
        handles.SpriteAxis.Position(1) = handles.SpriteAxis.Position(1)+15;
      end  
    elseif A2Voltage < 2
    Row = floor(handles.SpriteAxis.Position(2)/29);
    Column = floor((handles.SpriteAxis.Position(1)-15)/29);
    set(handles.SpriteAxis,'xdir','reverse')
    set(M,'visible','off');
    set(A,'visible','on');
    set(R,'visible','off');
    set(I,'visible','off');
    axis off
     pause(.04)
    set(M,'visible','off');
    set(A,'visible','off');
    set(R,'visible','on');
    set(I,'visible','off');
     axis off
     pause(.04)
    set(M,'visible','off');
    set(A,'visible','on');
    set(R,'visible','off');
    set(I,'visible','off');
     axis off
     pause(.04)
     set(M,'visible','off');
    set(A,'visible','off');
    set(R,'visible','off');
    set(I,'visible','on');
     axis off
      if  (handles.Template(Row,Column) == 0)
          handles.SpriteAxis.Position(1) = handles.SpriteAxis.Position(1)-15;
      end
    end
     end

[Curtis Vannor]

Im still getting: Subscript indices must either be real positive integers or logicals.

for: if (handles.Template(Row,Column) == 0);

[Walter Roberson]

One problem is that your handles.Template appears to be RGB but on that line you are accessing it as if it were grayscale. That will not fail itself: it would be the same as handles.Template(Row,Column,1) . But it indicates that you need to look at that portion of the code more closely.

[Walter Roberson]

You have

      Row = floor(handles.SpriteAxis.Position(2)/29);
    Column = floor((handles.SpriteAxis.Position(1)-15)/29);

What reason do you have to expect that those will not be 0? Or even -1 for Column ?

[Curtis Vannor]

Ok I understand. But I am curious if there is a way to check the color of the template axis in the area in front of mario before mario approaches that are in order to dictate whether or not mario can interact with that area. The stage is as shown:

And the template under this is as shown:

The solid objects are shown in blue and green on the template. Can you help me with this?