So apologies if my script here seems super messy or just plane wrong, I am super new to coding in C#. I've been having issues with the UI for a couple hours and I finally kinda got it working. The script right now is supposed to open up a menu whenever a player gets close enough to a Vector3 location. (Left out that part of the script so it wouldn't be too much)

 

So I can get the menu to activate perfectly fine. That part pretty much works as intended. But for some reason the "else if " line wants to fire too, even when the logic is wrong.

 

Pretty sure this is user error but I'm honestly at a loss.  Any help would be awesome! Code can be found below.

 

https://pastebin.com/jRXPFrrp

You are checking the position of all 8 hotel locations. That means you are very likely to be away from at least one of these positions at all times, hence your code is firing constantly. You could add a boolean and only show your "warning" when you are not close to any hotel. Like this:

 

while (true)
{
    GameFiber.Yield();
    bool isCloseToAnyHotel = false;
    for (int index = 0; index < 8; index++) 
    {
        if (Vector3.Distance2D(hotelLocation[index], Game.LocalPlayer.Character.AbovePosition) < 20 ) // Only activate if the player is close enough.
        {                      
            isCloseToAnyHotel = true; 
            Game.DisplayHelp("Menu Should Be activated");
            if (activeMenu == !true)
            {
                mainMenu.Visible = true;
                activeMenu = true;
                GameFiber.StartNew(delegate
                {
                    while (true)
                    {
                        GameFiber.Yield();
                        _menuPool.ProcessMenus();
                        
                    }
                });
                
            }        
        }
    }

    if (!isCloseToAnyHotel)
    {
        // Now only fired when you are not close to any hotel.
        Game.DisplayNotification("You are away from a hotel.");
    }
}

 

8 hours ago, LMS said:

You are checking the position of all 8 hotel locations. That means you are very likely to be away from at least one of these positions at all times, hence your code is firing constantly. You could add a boolean and only show your "warning" when you are not close to any hotel. Like this:

 

Dude, you are awesome! Thanks a bunch. I hadn't thought of it like that when I was typing the code. A thousand times, thank you.