Unity c# multiple functions share the same behavior?

Question

In my game, there are tons of places where I need to play audios after functions complete, for example:

class Player{
    public AudioClip clip;
    void WalkToDestination()
    {
         //walk code here

         AudioManager.Play(clip);
    }
}

class GameManger{  
    public AudioClip clip;
    void AfterCompleteLevel()
    {
          //play level completion animation
           AudioManager.Play(clip);
    }
}

Since there are so many functions requiring playing sound, and every time I have to add public AduioSource clip into the class, and AudioManager.Play(clip); into the body, there are too much repetitive work. Is there any good design pattern for this?

This code should not even compile. I think you should modify it to reflect what you have now. — Programmer
– Programmer, Commented Oct 25, 2016 at 1:15

Gasper · Accepted Answer · 2016-10-25 12:45:26Z

2

Define a base class, and put shared functions therein:

public class PlayerBase 
{
     protected SomeClass clip;
     public void PlaySound()
     {
         AudioManager.Play(clip);
     }
}

class Player : PlayerBase {}
class GameManger : PlayerBase {}

The call PlaySound() directly in the derived Player or GameManager class.

edited Oct 25, 2016 at 12:45

Gasper

3,1001 gold badge14 silver badges16 bronze badges

answered Oct 25, 2016 at 3:05

David

16.4k22 gold badges57 silver badges68 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

ywj7931 Over a year ago

Yeah, that is a solution, but i feel they are not that similar, the only similar behavior is playing music

Kardux · Accepted Answer · 2016-10-25 08:45:33Z

2

What you're probably looking for is called a Singleton : they are very useful when using audio managers or such things.

You can declare one this way :

public class YourClassName: MonoBehaviour
{
    public static YourClassName instance;

    public void Awake()
    {
        if (!instance)
        {
            instance = this;
        }
    }

    public void YourMethodToCall()
    {
        //Do what you want here
    }
}

Then from another script you can access it using :

if (YourClassName.instance)
{
    YourClassName.instance.YourMethodToCall();
}

edited Oct 25, 2016 at 8:45

answered Oct 25, 2016 at 8:37

Kardux

2,1571 gold badge21 silver badges21 bronze badges

4 Comments

Scott Chamberlain Over a year ago

Instead of a singleton class it is better to have a "pre-load" scene and load the audio manager there. You drop the static instance and you instead use a Object.FindObjectOfType<YourClassName>();. See this answer for a full explination on how to do it.

Kardux Over a year ago

@ScottChamberlain Agreed that this is another way to do so : I personally "always" use a kind of preload scene (usually the one containing custom splash screens, intro video, etc...) to set all my managers :)

Scott Chamberlain Over a year ago

The preload scene should not be a splash screen IMHO. You want it to quickly run even when debugging, if you make it your splash screen then you have to wait for the splash to load before debugging every time.

Kardux Over a year ago

@ScottChamberlain That's why on every piece of script that performs unnecessary (for the game) long actions such as video playing, using a defined symbol like DEBUG allows for easy and fast debugging without having the pain to restore everything before final build ;)

Collectives™ on Stack Overflow

Unity c# multiple functions share the same behavior?

2 Answers 2

1 Comment

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related