Java-Gaming.org Java4K winners: [ by our judges | by the community ]         
Featured games (67)
games approved by the League of Dukes
Games in Showcase (∞)
games submitted by our members



News: Read the Java Gaming Resources, or peek at the official Java tutorials
 
    Home     Help   Search   Login   Register   
Pages: [1]
  Print  
  discussing my first try of a custom listener  (Read 1413 times)
0 Members and 2 Guests are viewing this topic.
Offline Serethos

Sr. Member
**

Posts: 302


Java games rock!


« on: 2003-11-24 08:16:29 »

i want to realize a messaging system over a custom-created listener model
to use only the very basics of a listener pattern, i want to create them right off from zero point, without using component listeners/ action listener

so heres my first try :

1  
2  
3  
4  
5  
6  
7  
8  
9  
10  
11  
12  
13  
14  
15  
16  
17  
18  
19  
20  
21  
22  
23  
24  
25  
26  
27  
28  
29  
30  
31  
32  
33  
34  
35  
36  
37  
38  
39  
40  
41  
42  
43  
44  
45  
46  
47  
48  
49  
50  
51  
52  
53  
54  
public interface MyListener
{
     public void notifyit(Object e);
}


class MrMoo
{
      MyListener listener;
     
      public MyListener getListener()
      {
            return listener;
      }
     
      public void addListener(MyListener l)
      {
            if(l!=null) listener = l;
      }
     
      public void removeListener()
      {
            listener = null;
      }
     
      public void doit()
      {
            listener.notifyit(this);
      }
     
}



public class ListenerTest
{
     
      public static void main(String args[])
      {
            MrMoo m = new MrMoo();
           
            MyListener l = new MyListener()
            {
                  public void notifyit(Object e)  
                        {
                             System.out.println("!!!");
                        }
            };
           
            m.addListener(l);
            m.doit();
      }
     
}



i need some response if im on the right way, if there could be done something better etc
Offline kevglass
« League of Dukes »

JGO Kernel
*****

Posts: 5214
Medals: 49


Mentally unstable, best avoided.


« Reply #1 on: 2003-11-24 09:17:11 »

Looks ok, addListener() kinda implies the possibility of multiple listeners though. So either keep a list of the listeners and notify each one when required or change the method to setListener(). If you leave it as it you might want to null check the listener before doing the notification.

Looks good tho,

Kev

Offline swpalmer

JGO Kernel
*****

Posts: 3438
Medals: 4


Where's the Kaboom?


« Reply #2 on: 2003-11-24 21:58:30 »

If you keep with a single listener then you will want the 'addListener' method to return status if the listener was already set... as it is you silently fail to change the listener to what is requested if there already is one.

Generally you have a list of listeners though.

Games published by our own members! Go get 'em!
Offline cfmdobbie

JGO Wizard
****

Posts: 1257


Who, me?


« Reply #3 on: 2003-11-25 04:12:01 »

...but if you do want to have a single listener, change the addListener method to setListener, and have it either silently override any currently set listener or throw an exception if a second call is made.  Preferably the former.

Edit: Oh, and then you don't need the removeListener method, as setListener(null) would be the best way to do it.

Hellomynameis Charlie Dobbie.
Offline Serethos

Sr. Member
**

Posts: 302


Java games rock!


« Reply #4 on: 2003-11-25 05:46:51 »

very nice, thx for the answers. i hope i have now the 'permission' to use it as official communication system for my game objects... =)
Offline swpalmer

JGO Kernel
*****

Posts: 3438
Medals: 4


Where's the Kaboom?


« Reply #5 on: 2003-11-25 07:46:19 »

For the single listener case, rather than throw an exception if there is a current listener.. have it return the current listener.  Much more robust that way.

Pages: [1]
  Print  
 
 
Jump to:  

Powered by MySQL Powered by PHP Powered by SMF 1.1.16 | SMF © 2011, Simple Machines Valid XHTML 1.0! Valid CSS!
Page created in 0.069 seconds with 19 queries.