The passed in argument is named “MenuInterface” but you already have a member variable called “MenuInterface”, thats shadowing, just rename the passed in argument to “InMenuInterface” or something similiar.
Because it can become ambiguous to the compiler what you really want.
Btw. this is c++, and “this” is not needed, so you are basically telling the compiler “MenuInterface = MenuInterface”, it doesn’t know, that you want the passed in argument.