The JButton works for only click

[sampath surineni]

Hi I am coding an alarm application.
I use a JButton to set the alarm. I use the same JButton to warn the user about choosing any file other than an "mp3" file for alarm ringer. But the button works only once.
Here is the code

set.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                IsTime it = new IsTime();
                if (set.getText() == "Start") {
                    int hh = Integer.parseInt(hrs.getSelectedItem().toString());
                    int mm = Integer.parseInt(min.getSelectedItem().toString());
                    int apm;
                    if (ampm.getSelectedItem() == "AM") {
                        apm = Calendar.AM;
                    } else {
                        apm = Calendar.PM;
                    }
                    if (getFile.fiLoc == null) {
                        JOptionPane.showMessageDialog(backFrame, "Please select a tone for the alarm");
}
                } else {
                    mp3Player.plyr.close();
                }
            }
        });

But I want the button to work as many times as user selects a file other than an mp3 file. Can somebody please help me with this

[Jesper de Jong]

Use .equals() instead of == to compare strings:

if (set.getText().equals("Start")) {  

// ...

if (ampm.getSelectedItem().equals("AM")) {  

See:
AvoidTheEqualityOperator
Difference between == and .equals

[Unnar Björnsson]

Remember the (==) comparison is object comparison, it checks if the objects on both sides of the == signs are the same.
The .equals() method compares the string value of the string objects, which is what you want to compare.

[sampath surineni]

Jesper de Jong wrote:Use .equals() instead of == to compare strings:

if (set.getText().equals("Start")) {  

// ...

if (ampm.getSelectedItem().equals("AM")) {  

See:
AvoidTheEqualityOperator
Difference between == and .equals

Thank you very much for the suggestion. Can you help me out with my main problem also?

[sampath surineni]

Unnar Björnsson wrote:Remember the (==) comparison is object comparison, it checks if the objects on both sides of the == signs are the same.
The .equals() method compares the string value of the string objects, which is what you want to compare.

Thank you,can you help me out with my main problem?

[Keith Lynn]

Can you elaborate on what happens when you click the second time?

[sampath surineni]

Keith Lynn wrote:Can you elaborate on what happens when you click the second time?

The second time I press the button nothing is happening

[Keith Lynn]

Have you tried printing out something at the beginning of actionPerformed to see if it may be getting there?

[sampath surineni]

Keith Lynn wrote:Have you tried printing out something at the beginning of actionPerformed to see if it may be getting there?

But its working first time.Do I have to put it in any loop?

[Keith Lynn]

It's possible that on the second time you click the button, the condition in the if is false.