Beziehungen zwischen Klassen - wie strukturiere ich das?

[Student]

Hi,

(sorry für den Titel. Ich wusste es nicht besser zu umschreiben :roll: )

Wieder einmal ein Problemchen ;-)

Ich habe derzeit folgende Klassen (irrelevanten Code habe ich mal rausgeholt).

public class MyPasswordField extends JPasswordField 
{

    private PasswordNumCapsStateHandler m_passwordNumCapsHandler = new DefaultPasswordNumCapsStateHandler();


    public void setPasswordNumCapsHandler( PasswordNumCapsStateHandler passPasswordNumCapsHandler )
    {
        m_passwordNumCapsHandler = passPasswordNumCapsHandler;
    }

    public PasswordNumCapsStateHandler getPasswordNumCapsHandler()
    {
        return m_passwordNumCapsHandler;
    }
    
    
    public void setStatusbarIcon( Icon statusbarIcon )
    {
        //
    }

}

final class LoginView extends JFrame
{

   private XYZPasswordField m_txtPasswort = new XYZPasswordField(24);


   // Vorbelegung des Labels mit einem transparenten GIF
   private JLabel m_warningNumCaps = new JLabel(XYZRessourceLoader.getImageIcon(XYZRessourceLoader.IMG_HOOK_INVERS));


   m_txtPasswort.setPasswordNumCapsHandler( new DefaultPasswordNumCapsStateHandler() );

   m_txtPasswort.addKeyListener( new KeyListener()
   {
          public void keyReleased( KeyEvent e )
          {
              if ( ( e.getKeyCode() == KeyEvent.VK_CAPS_LOCK )
                    ||
                   ( e.getKeyCode() == KeyEvent.VK_NUM_LOCK ))
              {
                  m_txtPasswort.getPasswordNumCapsHandler().onNumCapsStateChange( 1 );
              }
          }

          public void keyPressed( KeyEvent e )
          {}

          public void keyTyped( KeyEvent e )
          {}

      });

   }

   public void setNumCapsWarning( Icon numCapsIcon )
   {
       m_warningNumCaps.setIcon( numCapsIcon );
   }

}

public class DefaultPasswordNumCapsStateHandler implements PasswordNumCapsStateHandler
{

    public void onNumCapsStateChange( int status )
    {
        if( status == 1 )
        {
            // Irgendwie muss das Icon ja im LoginView angezeigt werden. Frage: wie mache ich das
            // geschickt? Referenz an DefaultPasswordNumCapsStateHandler übergeben kann ja nicht
            // 'geschickt' sein, oder?
        }
        else
        {
            ((XYZParentFrame) App.getApplicationController().getWindowManager().getMDIParent()).
addTrayComponent( XYZRessourceLoader.getImageIcon(XYZRessourceLoader.ALARM) );
        }
    }
}

So. In der letzten Klasse steht es ja schon.
Wenn ich also in den IF-Zweig komme, dann soll ein Icon auf dem Label m_warningNumCaps in der Klasse LoginView angezeigt werden, ansonsten ein Icon in der StatusBar. ( beim Login gibt es noch keine StatusBar :idea: ).

Ich weiß jetzt nicht, wie ich das am cleversten mache. Ich könnte nuatürlich an die Klasse 'DefaultPasswordNumCapsStateHandler' eine Referenz der Klasse 'LoginView' übergeben, aber ist das wirklich 'gut'?

Ich streube mich irgendwie gegen diese Methode.

Ich danke euch für Hilfe und Ratschläge.

Grüße Ben ;-)

 

[bygones]

eine möglichkeit wäre dass der Handler das Label kennt, also beim erzeugen des handlers übergibst du das label...

 

[Student]

nun gut. das label ist eben nur eine ausnahme.
normalerweise wird das icon eben in die statusleiste eingebunden ... wäre das nicht zu speziell wenn ich da jetzt das label übergeben würde?

oder soll ich den konstruktor überladen?

 

[bygones]

mhm. wäre ne möhlichkeit, aber gefällt mir nicht so gut. sobald du was drittes hast zum icon setzen müsste vielleicht noch ein konstruktor dazu kommen.

warum implementierst du den Listener nicht als anonyme Klasse wie den KeyListener ?

 

[Student]

hi.
ja das hier ist echt ne blöde lösungsmethode. gefällt mir auch nicht:

public class DefaultPasswordNumCapsStateHandler implements PasswordNumCapsStateHandler
{
    private JLabel m_warningLoginView;

    public DefaultPasswordNumCapsStateHandler()
    {}

    public DefaultPasswordNumCapsStateHandler( JLabel warningLoginView )
    {
        m_warningLoginView = warningLoginView;
    }

    public void onNumCapsStateChange( int status )
    {
        if( status == 1 )
        {
            m_warningLoginView.setIcon( XYZRessourceLoader.getImageIcon(XYZRessourceLoader.IMG_ALARM) );
        }
        else
        {
            ((XYZParentFrame) App.getApplicationController().getWindowManager().getMDIParent()). 
addTrayComponent( XYZRessourceLoader.getImageIcon(XYZRessourceLoader.IMG_ALARM) );
        }
    }
}

könntest du deinen vorschlag etwas genauer erläutern?
ich danke dir.

grüße ben.

 

[bygones]

weiter möglichkeit:

public interface IconCallBack {
 public void changeIcon(IconEvent event);
}

public class IconEvent {
  public static final int LABEL = 0;
  public static final int STATUS = 1;
  
  private Icon icon;
  private int kind;

  public IconEvent(Icon icon, int kind) {
   // die attribute setzen;
  }

   // getter Methoden
}
public class LoginView extends JFrame implements IconCallBack {
  // so wie bisher

 m_txtPasswort.setPasswordNumCapsHandler( new DefaultPasswordNumCapsStateHandler(this) );
 
  ...

  public void changeIcon(IconEvent e) {
    if(e.getKind == IconEvent.LABEL) {
     label.setIcon(e.getIcon());
    }
    else {
      status.setIcon(e.getIcon());
    }
}

public class DefaultPasswordNumCapsStateHandler implements PasswordNumCapsStateHandler {
    private IconCallBack caller;

    public DefaultPasswordNumCapsStateHandler(IconCallBack caller) {
      this.caller = caller;
    }

    public void onNumCapsStateChange( int status ) {
        IconEvent e = null;
        if( status == 1 ) {
           e = new IconEvent(irgendEinIcon, IconEvent.LABEL);
        }
        else {
            e = new IconEvent(anderesIcon, IconEvent.STATUS);
        }
        caller.changeIcon(e);
    }
}

auch ne möglichkeit

is zwar komplizierter, aber so wären die beiden Klassen unabhäniger von einander

 

[Student]

danke .. bekomm ich noch ne kurze erläuterung ;-)
edit:
stopp .. ich muss erst mal selbst testen.


frage:
was geschieht hier:

  public void changeIcon(IconEvent e) { 
    if(e.getKind == IconEvent.LABEL) { 
     label.setIcon(e.getIcon()); 
    } 
    else { 
      status.setIcon(e.getIcon()); 
    }

also ... die objekte 'label' und 'status'. wo kommen die denn her?

 

[bygones]

danke .. bekomm ich noch ne kurze erläuterung ;-)

das Interface IconCallBack (btw. alles dumme namen, nur als bsp) definiert die methode changeIcon. Damit der Frame und der Handler nicht direkt von einander abhängig sind regeln sie die kommunikation über das Interface. Der Handler benachrichtigt den caller, wenn dieser sein Icon ändern soll. Was für ein Icon wie gesetzt wird wird in der Event klasse definiert.

Der Frame implementiert das interface also muss er die methode implementieren. Wird sie aufgerufen überprüft der Frame in der methode welches Icon wohin gesetzt werden soll.

verständlich ?

 

[bygones]

frage:
was geschieht hier:

  public void changeIcon(IconEvent e) { 
    if(e.getKind == IconEvent.LABEL) { 
     label.setIcon(e.getIcon()); 
    } 
    else { 
      status.setIcon(e.getIcon()); 
    }

also ... die objekte 'label' und 'status'. wo kommen die denn her?

da sollen die Objekte sein, deren Icon gesetzt werden sollen, weiß nicht wie die bei dir heißen....

 

[Student]

ok. danke. ich habe das jetzt mal angepasst und bin nun zum nächsten punkt gekommen.
ich überprüfe CAPS_LOCK und NUM_LOCK

ich überprüfe das mit einem KeyEvent. wenn als die NUM-taste gedrückt wird, dann überprüfe ich ob sie jetzt aktiv ist oder nicht.

und genau da liegt der haken.
wie bekomme ich denn heraus, ob NUM_LOCK aktiviert oder dekativiert ist?

der listener hängt an einem passwordfield ... wenn ich jetzt aus dem feld für den benutzernamen per tabulator in das feld springe und z.b. NUM inaktiv ist .. und ich dann auf NUM drücke .. dann sagt er mir .. "hey, stopp. NUM_LOCK ist aktiv". aber das stimmt ja gar nicht ..

mein code:

      txtPasswort.addKeyListener( new KeyListener()
      {
          public void keyReleased( KeyEvent e )
          {
              if ( ( e.getKeyCode() == KeyEvent.VK_CAPS_LOCK )
                    ||
                   ( e.getKeyCode() == KeyEvent.VK_NUM_LOCK ))
              {
                  txtPasswort.getPasswordNumCapsHandler().onNumCapsStateChange( IconEvent.LOGINVIEW , e.getKeyCode() );
              }
          }

      txtPasswort.addFocusListener(new FocusListener()
      {
         public void focusGained(FocusEvent e)
         {
             txtPasswort.selectAll();
             txtPasswort.requestFocus();
             System.out.println( "focusGained" );
             txtPasswort.getPasswordNumCapsHandler().onNumCapsStateChange( IconEvent.LOGINVIEW  );
         }

         public void focusLost(FocusEvent e)
         {
             System.out.println( "focusLost" );
             txtPasswort.getPasswordNumCapsHandler().onNumCapsStateChange( IconEvent.LOGINVIEW );
         }

      });

public class DefaultPasswordNumCapsStateHandler implements PasswordNumCapsStateHandler
{
    private IconCallBack m_caller;
    private boolean m_num  = true;
    private boolean m_caps = false;

    public DefaultPasswordNumCapsStateHandler( IconCallBack caller )
    {
      m_caller = caller;
    }

    private void checkNumCapsKeyPressed( int keyCode )
    {
        if( keyCode == KeyEvent.VK_CAPS_LOCK )
        {
            m_caps = !m_caps;
            System.out.println( "keyCode == KeyEvent.VK_CAPS_LOCK" );
        }

        if( keyCode == KeyEvent.VK_NUM_LOCK )
        {
            m_num = !m_num;
            System.out.println( "keyCode == KeyEvent.VK_NUM_LOCK" );
        }
    }


    private void checkNumCaps( int status )
    {
        IconEvent event = null;

        if( status == IconEvent.LOGINVIEW )
        {
            event = new IconEvent( SHDRessourceLoader.getImageIcon(SHDRessourceLoader.IMG_ALARM) , IconEvent.LOGINVIEW);
        }
        else
        {
            event = new IconEvent( SHDRessourceLoader.getImageIcon(SHDRessourceLoader.IMG_SCHLUESSEL) , IconEvent.STATUSBAR);
        }

        if( m_num && !m_caps)
        {
            event = new IconEvent( SHDRessourceLoader.getImageIcon(SHDRessourceLoader.IMG_HOOK_INVERS) , IconEvent.NO);
        }

        m_caller.changeIcon( event );
    }

    public void onNumCapsStateChange( int status )
    {
        checkNumCaps( status );
    }


    public void onNumCapsStateChange( int status, int keyCode )
    {
        checkNumCapsKeyPressed( keyCode );
        checkNumCaps( status );
    }

}

da weiß ich jetzt echt nicht mehr weiter. :roll:

 

[Student]

nachtrag:
kann man irgendwie "so tun, als ob eine taste gedrückt wurde" und dann überprüfen, ob z.b. der buchstabe gerade groß oder klein geschrieben wurde?

 

[bygones]

ich weiß nicht ob es einen controller gibt der dir sagt ob NumLock aktiv ist oder nicht.... mit der Klasse Robot kannst du einen Tastendruck simulieren

 

[Student]

hm .. ist windowsspezifisch hab ich mal gelesen, oder?
nungut. wenn es nicht anders geht ;-)

danke. ich werde es mal testen.

 

[Student]

hey,

also ich habe mir das mal angesehen:

private boolean checkIfCapsLockIsActive()
    {
        Robot robot = null;
        String testChar = "";

        try
        {
            robot = new Robot();
        }
        catch( AWTException exc )
        {
            // ...
        }

        robot.keyPress( KeyEvent.VK_A );

       // ...

        return ( testChar.equals( "A" ) );
    }

also ich muss doch jetzt noch irgendwo einen listener anhängen, aber wo? ???:L

 

[bygones]

es geht scheinbar auch einfacher...
schau mal hier
http://java.sun.com/docs/books/tutorial/uiswing/events/keylistener.html

 

[Student]

hey,

also ich habe das jetzt so gemacht .. ;-)

public class DefaultPasswordNumCapsStateHandler implements PasswordNumCapsStateHandler,
                                                           KeyListener
{
    /**
       *   weiterer Code
       */

    public boolean checkIfCapsLockIsActive()
    {
        Robot robot = null;
        String testChar = "";

        try
        {
            robot = new Robot();
        }
        catch( AWTException exc )
        {
        }

        robot.keyPress( KeyEvent.VK_A );

        return ( testChar.equals( "A" ) );
    }


    /**
     *  Implementierung der Methoden aus dem Interface KeyListener
     */
    public void keyPressed( KeyEvent event )
    {
        int id = event.getID();

        if( id == KeyEvent.KEY_TYPED )
        {
            System.out.println( KeyEvent.getKeyText(event.getKeyCode()) );
        }
    }

    public void keyReleased( KeyEvent event )
    {
    }

    public void keyTyped( KeyEvent event )
    {
    }
}

jut. problem ist jetzt.
Das alles hängt ja an dem PasswordField - also der Listener für den Default...Handler.

Ich gehe so vor, dass ich beim 'Gewinnen' des Fokus

public void focusGained(FocusEvent e)
         {
             // ...

             txtPasswort.getPasswordNumCapsStateHandler().checkIfCapsLockIsActive(); 

             // ..         
         }

die Methode aufrufe.

Problem dabei .. ich simuliere ja einen Tastendruck, was natürlich zur Folge hat, dass in dem PasswordField ein Zeichen angezeigt wird.
Das ist aber ja nicht die Absicht . :roll:

Hat da eventuell jemand einen Tipp, wie ich das umgehen kann?

Danke ..

 

[bygones]

hast du meinen Link nicht gesehen - du brauchst keinen Robot um zu testen ob das NumLock aktiv ist

 

[Student]

klar. ich bin dem link sogar gefolgt :roll:
öhem .. eventuell reichen meine englischkenntnisse nicht aus .. ich lese nochmals ..

 

[Student]

jo. sorry.
also .. ich muss ganz ehrlich gestehen, dass ich aus dem text nicht herauslesen kann, wie das gehen soll. :?

 

[bygones]

ok - muss es zurücknehmen - du kannst zwar per getKeyCode testen ob die Num taste gedrückt wurde, aber nicht, ob sie aktiv ist oder nicht... sry

 

[Student]

ok ;-) :bae:

nun gut. dann könnten wir ja hier nochmal drauf zurückkommen ;-)

Problem dabei .. ich simuliere ja einen Tastendruck, was natürlich zur Folge hat, dass in dem PasswordField ein Zeichen angezeigt wird.
Das ist aber ja nicht die Absicht . :roll:

Hat da eventuell jemand einen Tipp, wie ich das umgehen kann?

:arrow: ;-)

 

[bygones]

das kannst du nicht umgehen... es wird ja nicht simuliert dass eine Taste gedrückt wird, es wird ja eine gedrückt... kein Element darf in dem Moment den Focus haben...

mhm - there's got to be a better method ?!

 

[Student]

jep. das kann doch net so schwer sein ... ???:L

 

[Student]

mhm - there's got to be a better method ?!

jawohl. und ich habe sie endlich gefunden.

Toolkit.getDefaultToolkit().getLockingKeyState( KeyEvent.VK_NUM_LOCK )

Quelle:
:arrow: http://java.sun.com/j2se/1.5.0/docs/api/java/awt/Toolkit.html#getLockingKeyState(int)

Problem gelöst.
Danke @dbac. :toll: