implements IMenuTask, IEventNotification

[reven]

is there anyway to detect what is initializing the plugin and only create the things needed for each?  ie for IEventNotification i only use the one method Notify(EventTypes eventType, string eventText) and it uses nothing else of the plugin (nothing at all needs to be initiated) but everything is being created twice since it implements both of them.  anyway around this?  i could create another dll just for the IEventNotification, but that seems a little silly.

i may try making a few more things static and see if they have been created yet, but i cant do this for everything.  anyone got any ideas?

[jasonf]

Would using a Singleton help here?  There would only be 1 instance ever created.  Your IMenuTask and IEventNotification would serve as a pass-thru layer to an inner Singleton class.

To make a class into a singleton, you first make the constructors private.  Then you usually have a private static instance of the class that you expose through a public static method, which instantiates the object if it hasn't already been, and returns the private static instance.

<table border="0" align="center" width="95%" cellpadding="0" cellspacing="0"><tr><td>Code Sample </td></tr><tr><td id="CODE">
public class Class1
{
  private Class1()
  {
  }

  private static Class1 instance = null;

  public static Class1 GetInstance()
  {
     if (instance == null)
     {
        instance = new Class1();
     }
     return instance;
  }

  public void SomePublicMethod()
  {}
}
[/QUOTE]

[reven]

and of course the obivous solution. thanks jasonf, yeah that will work. now i feel like an idiot

[Jeff]

What happens when GBPVR tries to create an instance of your class. I assume it doesn't know about the GteInstance method. I tried to accomplish the same thing by checking for an existing instance in my constructor and if I had already been ccreated I simply returned.

<table border="0" align="center" width="95%" cellpadding="0" cellspacing="0"><tr><td>Code Sample </td></tr><tr><td id="CODE"> public VAPanel()
{
// Don't run in the config app
if (PluginHelperFactory.getPluginHelper() == null) return;

if (instance != null) return;

instance = this;[/QUOTE]

It seems to work.

Jeff

[jasonf]

In the case of using a Singleton for a plugin, it's not the plugin itself that is the Singleton, but rather, it's some inner class that the plugin proxies.

<table border="0" align="center" width="95%" cellpadding="0" cellspacing="0"><tr><td>Code Sample </td></tr><tr><td id="CODE">
public class Plugin : IMenuTask
{
  public Plugin() {}

  public void Activate() { SingletonClass.GetInstance().Activate(); }

  public void render(out bool requiresMore)
  {
     SingletonClass.GetInstance().render(out requiresMore);
  }
}
[/QUOTE]

[reven]

i just added another class that implements IEventNotification, since that class doesnt need to communicate or share any common values/objects with the IMenuTask class, its like they are two different plugins just compiled into the same dll. i feel like such an idiot for not thinkign of this

the IEventNotification class just updates one field in the database, it sets a video file to seen when a user plays a video file, nothing special, just so if a user plays a video from anywhere in gbpvr it is displayed as seen (not sure how many people will find this feature useful, but i love it, i always forget what ive seen and what i havent).