How to avoid recursive and or infinity loop monogame return function

Question

This piece of code is returning on a infinite base how to fix?

public GuiElement MoveElement(int x, int y)
{
    GUIRect = new Rectangle(GUIRect.X += x,GUIRect.Y += y,GUIRect.Width,GUIRect.Height);
    return MoveElement(x,y);
}

this is where i set the values i have in my Other class

        gm.Add(new GuiElement("BackGround"));      

        gm.Add(new GuiElement("Play_Button").MoveElement(150, 50));

        gm.Add(new GuiElement("HighScore_Button"));

        gm.Add(new GuiElement("Quit_Button"));

basicly i want a to be able to move some elements using that MoveElement

it creates a infinite loop

An unhandled exception of type 'System.StackOverflowException' occurred in giftman.exe

Answer 1

You don't need to use recursion for this (and without a base case, you'll get an infinite loop, like you are).

Just change your method to:

public GuiElement MoveElement(int x, int y)
{
    GUIRect = new Rectangle(GUIRect.X += x,GUIRect.Y += y,GUIRect.Width,GUIRect.Height);
    return this;
}

Or better yet, add a constructor to set the position of your GuiElement, so you can just call it like this:

gm.Add(new GuiElement("Play_Button", 150, 50));

Or you can just change your MoveElement method to not return anything at all:

public void MoveElement(int x, int y)
{
    GUIRect = new Rectangle(GUIRect.X += x,GUIRect.Y += y,GUIRect.Width,GUIRect.Height);
}

Then when creating it you can do this:

GuiElement playButton = new GuiElement("Play_Button");
playButton.MoveElement(150, 50);
gm.Add(playButton);