While Pressing Ctrl+v it is pasting twice

Question

The following is my piece of code
import javax.swing.*;
import java.awt.event.*;
import java.awt.*;
import java.awt.datatransfer.*;
public class CopyPasteExample extends JFrame implements KeyListener
     JTextArea textArea;
     String s;
     JMenuBar menuBar;
     JMenu Edit;
     JMenuItem cut,copy,paste;
     EditorActions editorActions;
     public CopyPasteExample()
     textArea = new JTextArea(200,300);
     editorActions = new EditorActions(this);
     add(textArea,BorderLayout.CENTER);
     menuBar = new JMenuBar();
     Edit = new JMenu("Edit");
     Edit.add(editorActions.cutAction);
     Edit.add(editorActions.copyAction);
     Edit.add(editorActions.pasteAction);
     menuBar.add(Edit);
     setJMenuBar(menuBar);
     setSize(400,400);
     setVisible(true);
     textArea.addKeyListener(this);
     public void keyPressed(KeyEvent e)
          if(e.getKeyCode() == KeyEvent.VK_C)
               s = textArea.getSelectedText();
     public void keyReleased(KeyEvent e)
          if(e.getKeyCode() == KeyEvent.VK_V)
               appendText(s);
     public void keyTyped(KeyEvent e)
     public void doCopy()
          s = textArea.getSelectedText();
     System.out.println("The Selected COPY STRING"+s);
     public void doCut()
     public void doPaste()
          appendText(s);
     public void appendText(String s)
System.out.println("The selq"+textArea.getSelectedText());
if(textArea.getSelectedText() != null)
          textArea.replaceSelection(s);
System.out.println("Entered IF");
else
System.out.println("Enteringi else");
textArea.insert(s, i);
public static void main(String args[])
          CopyPasteExample copy = new CopyPasteExample();
class EditorActions
public AbstractAction cutAction;
public AbstractAction copyAction;
public AbstractAction pasteAction;
     public EditorActions(CopyPasteExample example)
     final CopyPasteExample copy=example;
     setAction(copyAction = new AbstractAction()
          {public void actionPerformed(ActionEvent e)
               {copy.doCopy();}},
               "copy","VK_C","C");
     setAction(cutAction = new AbstractAction()
          {public void actionPerformed(ActionEvent e)
               {copy.doCut();}},
               "cut","VK_X","X");
     setAction(copyAction = new AbstractAction()
          {public void actionPerformed(ActionEvent e)
               {copy.doPaste();}},
               "copy","VK_V","P");
public static void setAction(Action action, String label,String mnemonic,
String acceleratorKey)
     if (acceleratorKey != null && acceleratorKey.length() > 0)
action.putValue(Action.ACCELERATOR_KEY,
KeyStroke.getKeyStroke(acceleratorKey));
if (label != null && label.length() > 0)
action.putValue(Action.NAME, label);
if (mnemonic != null && mnemonic.length() > 0)
KeyStroke key = KeyStroke.getKeyStroke(mnemonic);
if (key != null)
action.putValue(Action.MNEMONIC_KEY,
new Integer(key.getKeyCode()));
Please help me out in this issue

Answer 1

That's because you coded it all twice. You have one implementation via accelerators and another via key listeners. You only need one of those.