addBindings only be effective once, then no response with another click.

[Jacky Luk]

I called the following code in the constructor, it worked, but once the Ctrl-V is hitted, data is copied, another Ctrl-V keystroke causes nothing to happen.
Why is that?
Here is the code.

protected void addBindings() {
        
          
        jSplitPane1.getInputMap().put(KeyStroke.getKeyStroke(
                                        KeyEvent.VK_V, Event.CTRL_MASK),
                                        "check");
        
        jSplitPane1.getActionMap().put("check", new AbstractAction() {
            public void actionPerformed(ActionEvent e) {
                TextTransfer tx = new TextTransfer();
                String input = tx.getClipboardContents();
                List<String> tokens = new ArrayList<String>();
               
                tokens = readString(input);
                    
                    
                    
                     
                
           
                jTextFirstName.setText(tokens.get(0).toString().trim());
                jTextLastName.setText(tokens.get(1).toString().trim());
                jTextHKID.setText(tokens.get(2).toString().trim());
                jTextPhone.setText(tokens.get(3).toString().trim() );
                jTextAddress.setText(tokens.get(4).toString().trim() );
                
                // Danger!
                jComboDistrict.setSelectedItem(tokens.get(5).toString().trim());

                jFormattedDOB.setText(tokens.get(6).toString().trim());
                jTextAge.setText(tokens.get(7).toString().trim());
                
                // Danger!
                jComboSex.setSelectedItem(tokens.get(8).toString().trim());
                
                jTextQualification.setText(tokens.get(9).toString().trim());
                jTextEmail.setText(tokens.get(10).toString().trim());
                // Danger!
                jComboHospName.setSelectedItem(tokens.get(11).toString().trim());
                try {
                    updateAgeField();
                } catch (ParseException ex) {
                    Logger.getLogger(bokss_patient_frame.class.getName()).log(Level.SEVERE, null, ex);
                }
                  
            }
        });
        
         
    }

[Michael Dunn]

you're using the getInputMap() that requires the component to have focus,
and if updateAgeField() gives the focus to some other component, it won't
work the 2nd time.

but that's just a guess - we can't run the code to test it.

[Jacky Luk]

Michael Dunn wrote:you're using the getInputMap() that requires the component to have focus,
and if updateAgeField() gives the focus to some other component, it won't
work the 2nd time.

but that's just a guess - we can't run the code to test it.

Hmmm, this guess is a bit off. I commented out that line, still it doesn't allow me to cut-n-paste twice

  private List<String> readString(String input)
    {
         String line = input;
         List<String> output = new ArrayList<String>();

        String otherThanQuote = " [^\"] ";
        String quotedString = String.format(" \" %s* \" ", otherThanQuote);
        String regex = String.format("(?x) "+ // enable comments, ignore white spaces
                ",                         "+ // match a comma
                "(?=                       "+ // start positive look ahead
                "  (                       "+ //   start group 1
                "    %s*                   "+ //     match 'otherThanQuote' zero or more times
                "    %s                    "+ //     match 'quotedString'
                "  )*                      "+ //   end group 1 and repeat it zero or more times
                "  %s*                     "+ //   match 'otherThanQuote'
                "  $                       "+ // match the end of the string
                ")                         ", // stop positive look ahead
                otherThanQuote, quotedString, otherThanQuote);

        String[] tokens = line.split(regex);
        for(String t : tokens) {
             output.add(t);
        }
        return output;
    }
  

Thanks
Jack

[Jacky Luk]

I started to inspect this event handler

 public void Clear()
    {
        jTextID.setText(String.valueOf(db.getMaxID()));
        jTextFirstName.setText("".trim());
        jTextLastName.setText("".trim());
        jTextHKID.setText("".trim() );
        jTextPhone.setText("".trim() );
        jTextAddress.setText("".trim() );
        jComboDistrict.setSelectedIndex(0);

        jFormattedDOB.setText("".trim());
        jTextAge.setText("".trim());
        jComboSex.setSelectedIndex(0);
        jTextQualification.setText("".trim());
        jTextEmail.setText("".trim());
        jComboHospName.setSelectedIndex(0);
 
    }

The cut-n-paste stuff only stopped working when I clicked on this button. But I would like to allow the user to clear all the fields on the form.
So this button is inevitable to be clicked. Any ideas how to change my program?
Thanks
Jack

[Michael Dunn]

read my reply again, and use the correct input map

[Jacky Luk]

Michael Dunn wrote:read my reply again, and use the correct input map

Thanks Michael
This time, the actionPerformed method is never called.

 private void addBindings() {
        
          
        jSplitPane1.getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW).put(KeyStroke.getKeyStroke(
                                        KeyEvent.VK_V, Event.CTRL_MASK),
                                        "pressed");
        
        jSplitPane1.getActionMap().put("pressed", new AbstractAction() {
            public void actionPerformed(ActionEvent e) {
                TextTransfer tx = new TextTransfer();
                String input = tx.getClipboardContents();
                List<String> tokens = new ArrayList<String>();
               
                tokens = readString(input);
                    
                    
                    
                     
                
           
                jTextFirstName.setText(tokens.get(0).toString().trim());
                jTextLastName.setText(tokens.get(1).toString().trim());
                jTextHKID.setText(tokens.get(2).toString().trim());
                jTextPhone.setText(tokens.get(3).toString().trim() );
                jTextAddress.setText(tokens.get(4).toString().trim() );
                
                // Danger!
                jComboDistrict.setSelectedItem(tokens.get(5).toString().trim());

                jFormattedDOB.setText(tokens.get(6).toString().trim());
                jTextAge.setText(tokens.get(7).toString().trim());
                
                // Danger!
                jComboSex.setSelectedItem(tokens.get(8).toString().trim());
                
                jTextQualification.setText(tokens.get(9).toString().trim());
                jTextEmail.setText(tokens.get(10).toString().trim());
                // Danger!
                jComboHospName.setSelectedItem(tokens.get(11).toString().trim());
                try {
                    updateAgeField();
                } catch (ParseException ex) {
                    Logger.getLogger(bokss_patient_frame.class.getName()).log(Level.SEVERE, null, ex);
                }
             }
        });
        
         
    }

[Jacky Luk]

Anyone would like to offer some help. That'll be nice of you.

Thanks
Jack

[Tony Docherty]

Without seeing how these components are laid out it's hard to give precise advice but Michael's comments seem correct to me.
I don't know what JSplitPane1 is but I'm guessing from the name it's a split panel in which case it won't get focus so you should probably use jSplitPane1.getInputMap(JComponent.WHEN_ANCESTOR_OF_FOCUSED_COMPONENT...
Note though that this will also stop working if you have other visible panels that aren't children of this panel and the focus moves to a component in one of those other panels.

[Jacky Luk]

Tony Docherty wrote:Without seeing how these components are laid out it's hard to give precise advice but Michael's comments seem correct to me.
I don't know what JSplitPane1 is but I'm guessing from the name it's a split panel in which case it won't get focus so you should probably use jSplitPane1.getInputMap(JComponent.WHEN_ANCESTOR_OF_FOCUSED_COMPONENT...
Note though that this will also stop working if you have other visible panels that aren't children of this panel and the focus moves to a component in one of those other panels.

One thing remarkable is that when I excluded the parameter of getInputMap, the cut-n-paste action worked just fine. When I included anyone of them. It stopped working.
The JFrame is divided by this split Pane, with a Panel over the top and a scroll panel at the bottom(in the middle), then there is another Panel over the bottom (inside the split panel), which is a JTabbedPane

[Tony Docherty]

One thing remarkable is that when I excluded the parameter of getInputMap, the cut-n-paste action worked just fine.

Not so remarkable: cut, copy and paste support is built into the swing text components - see this tutorial http://docs.oracle.com/javase/tutorial/uiswing/dnd/cutpaste.html

I assumed you wanted to take over the default CCP handling to do something esoteric, if that is the case please provide an SSCCE and a full description of what you are trying to achieve.

[Michael Dunn]

the problem appears to be if any textComponent has the focus, Ctrl-V is processed/consumed by the component before the bindings trigger.

[Jacky Luk]

Michael Dunn wrote:the problem appears to be if any textComponent has the focus, Ctrl-V is processed/consumed by the component before the bindings trigger.

Michael's guess is correct. It seems to be working now.
Thanks
Jack