[DaMi25]

Here is the problem.
a. Write an application that instantiates a JFrame that contains a JButton. Disable the JButton after the user clicks it.
b. Modify the JFrameDisableButton program so that the JButton is not disabled until the user has clicked atleast 8 times. At that point, display a JLabel that indicates "That's enough!".

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

public class JFrameDisableButton extends JFrame implements ActionListener
{
  JButton button = new JButton("Enabled");
  JLabel label = new JLabel("");
  public JFrameDisableButton()
  {
    super("Enabled-Disabled");
    setLayout(new FlowLayout());
    add(button);
    add(label);
    button.addActionListener(this);
  }

  public void actionPerformed(ActionEvent e)
  {
  Object source = e.getSource();
  final int LIMIT = 8;
  int i;
  int times = 0;
  for(i = 0;i<8;i++)
  {
    if(source == button)
  {
    ++times;
    if(times == 8)
   {
    button.setEnabled(false);
   }
  }
  }
  
  }
  
  
  public static void main(String[] args)
  {
    JFrameDisableButton y = new JFrameDisableButton();
    final int FRAME_WIDTH = 300;
    final int FRAME_HEIGHT = 150; 
    y.setSize(FRAME_WIDTH, FRAME_HEIGHT);
    y.setVisible(true);
    y.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
  }
}

I just can't answer the problem and if you don't mind, can you tell me if I answered letter a correctly when it comes to instantiation of JFrame? Thank you.

[GregBrannon]

The variables important to accomplish what you're trying to do are local to the actionPerformed() method so are recreated and initialized each time the button is pressed. Declare and initialize the counter outside the actionPerformed() method.

This post has been edited by GregBrannon: 04 March 2013 - 04:28 AM

[codecsmash49]

What Greg said is correct, to allow variables to be left unchanged after each new click you have to remove them from the method. The location that you initializing your variables is the issue.

[ShroomiaCo]

Another issue aside from the instantation, you need to remove the for loop. What would happen is it will always set it to eight, even if it I pressed only once.

[pbl]

ShroomiaCo, on 04 March 2013 - 10:24 PM, said:

Another issue aside from the instantation, you need to remove the for loop. What would happen is it will always set it to eight, even if it I pressed only once.

I think that codecsmash49 had already answered your concern

codecsmash49, on 04 March 2013 - 11:22 AM, said:

The location that you initializing your variables is the issue.

This post has been edited by pbl: 05 March 2013 - 02:40 PM

[DaMi25]

Thanks to all of you! I appreciate your effort. Sorry for the late reply, i was busy the previous days! Keep up the good work.