Delegate on multiple objects didn't work the way I thought

Question

I had an idea to use delegates for my buttons whenever the button is pressed I can notify everthing listening/subscribed(?) to it and do stuff (I feel like this is a scenario where delegates aren't needed). I have this code attached to 3 objects

public class ClassSelectButton : MonoBehaviour
{
    public delegate void ClassSelectDelegate();
    public event ClassSelectDelegate classSelectedEvent;

    public BaseClass classToGive;

    void Awake()
    {
        var button = GetComponent<Button>();
        if (button)
            button.onClick.AddListener(() => OnButtonPressed());
    }

    void OnButtonPressed()
    {
        if (classSelectedEvent != null)
            classSelectedEvent();
    }
}

public class Player : MonoBehaviour
{
    public BaseClass playerClass;

    void Start()
    {
        FindObjectOfType<ClassSelectButton>().classSelectedEvent += Test;
    }

    void Test(BaseClass newClass)
    {
        playerClass = newClass;
    }
}

Turns out this doesn't work the way I thought it would because when pressing the button only one of the delegates work and I realized that since there's multiple instances of this script there's duplicates of the delegate and so only one of them is being listened to. So what's an alternative way to go about doing this?

p.s: As I was writing this I realized another problem is probably because I'm using FindObjectOfType

Answer 1

Because your delegate is not a match. One of them has a void in parentheses and the other has a BaseClass parameter and the compiler rejects it. Note that you wrote below that playerClass is equivalent to BaseClass...

void Test(BaseClass newClass)
{
   playerClass = newClass;
}

And when the event is run on the other side, you have not defined any new class for it. This mismatch prevents code execution.

void OnButtonPressed()
{
   if (classSelectedEvent != null) classSelectedEvent("Where is new Class??");
}

---

For fix this issue

First upgrade your delegate as shown below and then move the class from the event run.

public delegate void ClassSelectDelegate(BaseClass baseClass);

Run the event along with the class parameter:

void OnButtonPressed() => classSelectedEvent?.Invoke(classToGive);

Answer 2

Please check this if query for id or mobile number etc can be worked for the similar code, if am not understating wrong:

As 'https://graph.microsoft.com/v1.0/me gives complete profile details just like code provided

const GRAPH_ENDPOINT = 'https://graph.microsoft.com/v1.0/me';

  getProfile() {
    this.http.get(GRAPH_ENDPOINT)
      .subscribe(profile => {
        this.profile = profile;
        console.log(profile)
      });
  }

same in microsoft graph api :

Just like that we can query for id, mobilephone number, and other details By filtering using select query See References for more query parameters: Get a user-Microsoft Graph v1.0 | Microsoft Docs -REFERENCE , Use query parameters

So to get mobilenumber in graph https://graph.microsoft.com/v1.0/me?$select=mobilePhone is used, same can be modified for code some thing like below Example:

 getMobileNumber() {
    this.http.get(GRAPH_ENDPOINT+'?$select=mobilePhone ').subscribe(
      mobilePhone => { 
        this. mobilePhone = mobilePhone; 
        console.log( this. mobilePhone);
    });
  }

Just like from graph

So to get id in graph https://graph.microsoft.com/v1.0/me?$select=id ,modify your code which uses this request.

Example:

getId(){
this.http.get(GRAPH_ENDPOINT+'?$select=id ').subscribe(
      Id=> { 
        this. Id= Id; 
        console.log( this. Id);

});
}

You can also make use of graph client

References:

1. Build Angular single-page apps with Microsoft Graph - Microsoft Graph | Microsoft Docs
2. azure - Obtaining Profile Photo from MS Graph API to Angular app - Stack Overflow from >> So reference
3. create-requests-typescript