Is ActionListener and Key Listener related?

[Kunal Lakhani]

I coded an application where there is 2 textfields having keylistener and a submit button with action listener.

For key and action listener, here is the coding
public void keyPressed(KeyEvent e) {

if(e.getSource()==currentPaymenttext){
			if(e.getKeyCode()==KeyEvent.VK_ENTER){
		//on pressing enter in the textfield perform some code related to currentPaymenttext
}
}

if(e.getSource()==enrollmentNotext){
		if(e.getKeyCode()==KeyEvent.VK_ENTER){
		//on pressing enter in the textfield perform some code related to enrollmentNotext
}
}


}

public void actionPerformed(ActionEvent ae) {
		
		if(ae.getActionCommand().equals("Pay")){
			//on action through pay, perform this
		}
}

Its working fine. If i put any value in currentPaymenttext in press enter, its works. Now, (without pressing PAY button), again i change the value in currentPaymenttext and press enter, its updated with new value.

But, when i submit the form through PAY button(ActionListener), the code inside

if(ae.getActionCommand().equals("Pay")){ }

runs twice. On little bit of research i found that the number of times i change the value inside currentPaymenttext textfield in press enter, those many times submit code (i.e the code inside

if(ae.getActionCommand().equals("Pay")){ } 

)runs, thus saving the same thing inside db those many number of times,

What is the mistake here?

[Michael Dunn]

> On little bit of research i found that the number of times i change the value inside currentPaymenttext textfield in press enter, those many times submit code

sounds like each click of the button adds an extra actionListener.

[Kunal Lakhani]

Thanks for your reply Michael. Yes, it adds additional ActionListener.

This was the previous coding

JButton sub = new JButton("Submit");
sub.addActionListener(this);


//some codes
if(dues>=0){
			overpaid.setVisible(false);
			subm.addActionListener(this);
		} else {
			overpaid.setVisible(true);
			subm.removeActionListener(this);
		}

On removing subm.addActionListener(this);, subm.removeActionListener(this); from the if block, the problem is solved. But, i want that, if the dues >=0, then only sub button would work

[Michael Dunn]

> i want that, if the dues >=0, then only sub button would work

are you sure?
your posted code adds the actionListener to subm if(dues >= 0) is true

have you thought about enable/disable subm based on if(dues >= 0)

[Kunal Lakhani]

Thanks Michael Dunn. It works much better and is easy to use