problem with reverse tabbing

[John Neel]

Hi

Please have a look at the following code.
The focus is shifted from component to component well, when we do tabbing.
But when we do reverse tabbing, the focus is not maintained properly.

Please advise me how to resolve this.

Thanks and Regards,
Janardan

 
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.border.*;
 
public class SampleFrame extends JFrame implements ActionListener,
  FocusListener {
  private JPanel jPanel1 = new JPanel();
  private JPanel jPanel2 = new JPanel();
  private JPanel jPanel3 = new JPanel();
  private JPanel jPanel4 = new JPanel();
  private JTextField jTextField1 = new JTextField();
  private JTextField jTextField2 = new JTextField();
  private JRadioButton jRadioButton1 = new JRadioButton();
  private JTextField jTextField5 = new JTextField();
  private JTextField jTextField6 = new JTextField();
  private JRadioButton jRadioButton2 = new JRadioButton();
  private JTextField jTextField7 = new JTextField();
  private JTextField jTextField8 = new JTextField();
  private JRadioButton jRadioButton3 = new JRadioButton();
  private JRadioButton jRadioButton4 = new JRadioButton();
  private JTextField jTextField9 = new JTextField();
  private JTextField jTextField10 = new JTextField();
  private JTextField jTextField3 = new JTextField();
  private JTextField jTextField4 = new JTextField();
  private ButtonGroup group1 = new ButtonGroup();
  private ButtonGroup group2 = new ButtonGroup();
 
  public SampleFrame() {
    try {
      jbInit();
      postJbInit();
    } catch(Exception e) {
      e.printStackTrace();
    }
  }
 
  private void jbInit() throws Exception {
    this.setSize(new Dimension(401, 219));
    this.getContentPane().setLayout(null);
    jPanel1.setLayout(null);
    jPanel1.setBounds(new Rectangle(0, 0, 395, 195));
    jPanel1.setBorder(BorderFactory.createBevelBorder(BevelBorder.RAISED));
    jPanel2.setBounds(new Rectangle(10, 10, 375, 45));
    jPanel2.setBorder(BorderFactory.createEtchedBorder(EtchedBorder.LOWERED));
    jPanel2.setLayout(null);
    jPanel3.setBounds(new Rectangle(10, 60, 375, 75));
    jPanel3.setBorder(BorderFactory.createEtchedBorder(EtchedBorder.LOWERED));
    jPanel3.setLayout(null);
    jPanel4.setBounds(new Rectangle(10, 140, 375, 45));
    jPanel4.setBorder(BorderFactory.createEtchedBorder(EtchedBorder.LOWERED));
    jPanel4.setLayout(null);
    jTextField1.setBounds(new Rectangle(30, 10, 165, 25));
    jTextField1.setNextFocusableComponent(jTextField2);
    jTextField2.setBounds(new Rectangle(200, 10, 165, 25));
    jTextField2.setNextFocusableComponent(jRadioButton1);
    jRadioButton1.setBounds(new Rectangle(10, 10, 15, 25));
    jRadioButton1.setNextFocusableComponent(jTextField5);
    jTextField5.setBounds(new Rectangle(30, 10, 100, 25));
    jTextField5.setNextFocusableComponent(jRadioButton2);
    jTextField6.setBounds(new Rectangle(30, 40, 100, 25));
    jTextField6.setNextFocusableComponent(jRadioButton4);
    jRadioButton2.setBounds(new Rectangle(10, 40, 15, 25));
    jRadioButton2.setNextFocusableComponent(jTextField6);
    jTextField7.setBounds(new Rectangle(160, 40, 100, 25));
    jTextField7.setNextFocusableComponent(jTextField9);
    jTextField8.setBounds(new Rectangle(160, 10, 100, 25));
    jTextField8.setNextFocusableComponent(jTextField10);
    jRadioButton3.setBounds(new Rectangle(140, 40, 15, 25));
    jRadioButton3.setNextFocusableComponent(jTextField7);
    jRadioButton4.setBounds(new Rectangle(140, 10, 15, 25));
    jRadioButton4.setNextFocusableComponent(jTextField8);
    jTextField9.setBounds(new Rectangle(265, 40, 100, 25));
    jTextField9.setNextFocusableComponent(jTextField4);
    jTextField10.setBounds(new Rectangle(265, 10, 100, 25));
    jTextField10.setNextFocusableComponent(jRadioButton3);
    jTextField3.setBounds(new Rectangle(200, 10, 165, 25));
    jTextField3.setNextFocusableComponent(jTextField1);
    jTextField4.setBounds(new Rectangle(30, 10, 165, 25));
    jTextField4.setNextFocusableComponent(jTextField3);
    jPanel4.add(jTextField4, null);
    jPanel4.add(jTextField3, null);
    jPanel1.add(jPanel4, null);
    jPanel1.add(jPanel3, null);
    jPanel3.add(jTextField10, null);
    jPanel3.add(jTextField9, null);
    jPanel3.add(jRadioButton4, null);
    jPanel3.add(jRadioButton3, null);
    jPanel3.add(jTextField8, null);
    jPanel3.add(jTextField7, null);
    jPanel3.add(jRadioButton2, null);
    jPanel3.add(jTextField6, null);
    jPanel3.add(jTextField5, null);
    jPanel3.add(jRadioButton1, null);
    jPanel2.add(jTextField2, null);
    jPanel2.add(jTextField1, null);
    jPanel1.add(jPanel2, null);
    this.getContentPane().add(jPanel1, null);
  }
 
  private void postJbInit() {
    group1.add(jRadioButton1);
    group1.add(jRadioButton2);
    jRadioButton1.setSelected(true);
    group2.add(jRadioButton3);
    group2.add(jRadioButton4);
    jRadioButton3.setSelected(true);
 
    jRadioButton1.setActionCommand("1");
    jRadioButton1.addActionListener(this);
    jRadioButton2.setActionCommand("2");
    jRadioButton2.addActionListener(this);
    jRadioButton3.setActionCommand("3");
    jRadioButton3.addActionListener(this);
    jRadioButton4.setActionCommand("4");
    jRadioButton4.addActionListener(this);
 
    jRadioButton1.addFocusListener(this);
    jRadioButton2.addFocusListener(this);
    jRadioButton3.addFocusListener(this);
    jRadioButton4.addFocusListener(this);
 
    alterStates();
  }
 
  public void actionPerformed(ActionEvent ae) {
    alterStates();
  }
 
  private void alterStates() {
    if(jRadioButton1.isSelected()) {
      jTextField5.setEnabled(true);
      jTextField5.setBackground(Color.white);
      jTextField6.setEnabled(false);
      jTextField6.setBackground(Color.lightGray);
    } else {
      jTextField6.setEnabled(true);
      jTextField6.setBackground(Color.white);
      jTextField5.setEnabled(false);
      jTextField5.setBackground(Color.lightGray);
    }
 
    if(jRadioButton4.isSelected()) {
      jTextField8.setEnabled(true);
      jTextField8.setBackground(Color.white);
      jTextField10.setEnabled(true);
      jTextField10.setBackground(Color.white);
 
      jTextField7.setEnabled(false);
      jTextField7.setBackground(Color.lightGray);
      jTextField9.setEnabled(false);
      jTextField9.setBackground(Color.lightGray);
    } else {
      jTextField8.setEnabled(false);
      jTextField8.setBackground(Color.lightGray);
      jTextField10.setEnabled(false);
      jTextField10.setBackground(Color.lightGray);
 
      jTextField7.setEnabled(true);
      jTextField7.setBackground(Color.white);
      jTextField9.setEnabled(true);
      jTextField9.setBackground(Color.white);
    }
  }
 
  public void focusGained(FocusEvent fe) {
    Component comp = fe.getComponent();
 
    if(comp instanceof JRadioButton) {
      ((JRadioButton) comp).setSelected(true);
      ((JRadioButton) comp).getNextFocusableComponent().requestFocus();
    }
 
    alterStates();
  }
 
  public void focusLost(FocusEvent e) {
  }
 
  public static void main(String[] args) {
    SampleFrame sampleFrame = new SampleFrame();
    sampleFrame.show();
  }
}
 

[Craig Wood]

I'm using j2se 1.5, commented out all the setNextFocusableComponent calls and changed the show to setVisible and your code compiled and ran okay. The reverse tabbing worked well this way. Check the JComponent api Method Detail for getNextFocusableComponent. It is deprecated as of j2se 1.4. If your api doesn't have the link to the new focus system page, here it is: How to use the Focus SubSystem.