Question

Instructions

The application JFacts contains a JFrame, a label, and six random facts. Every time the user clicks the JButton, remove replace the label's text and add a different one. The facts should cycle in order, fact 1 to fact 6. When the JButton is pressed on face 6, the program should display fact 1 again.

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

public class JFacts extends JFrame implements ActionListener {
FlowLayout flow = new FlowLayout();
JButton b = new JButton("Press to change fact");
JLabel stars = new JLabel("**********************************************");
JLabel fact = new JLabel();
String[] quotes = new String[] { "Only seven prisoners were in the Bastille when it was stormed. ",
"A Tale of Two Cities is a novel set during the French Revolution. ",
"Perhaps 40,000 people were executed at the guillotine. ",
"A loaf of bread cost a weeks wages prior to the French Revolution. ",
"Thomas Jefferson was the U.S. Minister to France during the Revolution. ",
"Protestant and Jewish religions were illegal in France before 1879. ", };
int counter = 0;

public JFacts() {
// Write your code here
this.setLayout(flow);
this.add(stars);
this.add(fact);
this.add(b);
b.addActionListener(new ActionListener() {

@Override
public void actionPerformed(ActionEvent e) {
// TODO Auto-generated method stub
//Get test and set to label
fact.setText(quotes[counter]);
//Increment counter
counter++;
//Mod by 6 to reset counter to 0;
counter = counter % 6;
}
});
}

public static void main(String[] args) {
JFacts rFrame = new JFacts();
rFrame.setSize(440, 300);
rFrame.setVisible(true);
}

@Override
public void actionPerformed(ActionEvent e) {
// TODO Auto-generated method stub

}

}

Someone tried helping me out with this code this code works but it's giving me a wierd error so it wont work for the test casses

Test Contents

public class CodevolveTestRunner {

   public static void main(String[] args) throws InterruptedException {
    JFacts jf = new JFacts();
    jf.setSize(440, 100);
    jf.setVisible(true);
    Thread.sleep(1000);
    // performs the click actions
    for(int i = 0; i < 6; ++i) {
       if(!jf.fact.getText().equals(jf.quotes[i])) {
              System.err.print("The label should read " + jf.quotes[i] + " but displays " + jf.fact.getText());
        }
       Thread.sleep(1000);
       jf.b.doClick();
      Thread.sleep(1000);
    }
    Thread.sleep(2000);
    System.exit(0);
   }
}

Errors

The label should read Only seven prisoners were in the Bastille when it was stormed.  but displays The label should read A Tale of Two Cities is a novel set during the French Revolution.  but displays Only seven prisoners were in the Bastille when it was stormed. The label should read Perhaps 40,000 people were executed at the guillotine.  but displays A Tale of Two Cities is a novel set during the French Revolution. The label should read A loaf of bread cost a weeks wages prior to the French Revolution.  but displays Perhaps 40,000 people were executed at the guillotine. The label should read Thomas Jefferson was the U.S. Minister to France during the Revolution.  but displays A loaf of bread cost a weeks wages prior to the French Revolution. The label should read Protestant and Jewish religions were illegal in France before 1879.  but displays Thomas Jefferson was the U.S. Minister to France during the Revolution. 

Answer 1

There were couple of simple mistakes in your code. Initially during start up, the program doesn’t show a fact unless the button is clicked, but the test program expects the JFacts program to display the first fact even before the button click. So I have fixed the code to display it initially, and whenever the button is pressed, the counter is updated before setting the new fact. Here is the completed code for this problem. Comments are included, go through it, learn how things work and let me know if you have any doubts or if you need anything to change. If you are satisfied with the solution, please rate the answer. Thanks

// JFacts.java

import javax.swing.*;

import java.awt.*;

import java.awt.event.*;

public class JFacts extends JFrame implements ActionListener {

      FlowLayout flow = new FlowLayout();

      JButton b = new JButton("Press to change fact");

      JLabel stars = new JLabel("**********************************************");

      JLabel fact = new JLabel();

      String[] quotes = new String[] {

                  "Only seven prisoners were in the Bastille when it was stormed. ",

                  "A Tale of Two Cities is a novel set during the French Revolution. ",

                  "Perhaps 40,000 people were executed at the guillotine. ",

                  "A loaf of bread cost a weeks wages prior to the French Revolution. ",

                  "Thomas Jefferson was the U.S. Minister to France during the Revolution. ",

                  "Protestant and Jewish religions were illegal in France before 1879. ", };

      int counter = 0;

      public JFacts() {

            // Write your code here

            this.setLayout(flow);

            this.add(stars);

            this.add(fact);

            this.add(b);

            // you should display the fact initially (before button press)

            fact.setText(quotes[counter]);

            b.addActionListener(new ActionListener() {

                  @Override

                  public void actionPerformed(ActionEvent e) {

                        // whenever the button is clicked, the counter should be updated

                        // before setting the text

                        // Increment counter

                        counter++;

                        // Mod by 6 to reset counter to 0;

                        counter = counter % 6;

                        // Get test and set to label

                        fact.setText(quotes[counter]);

                  }

            });

      }

      public static void main(String[] args) {

            JFacts rFrame = new JFacts();

            rFrame.setSize(440, 300);

            rFrame.setVisible(true);

      }

      @Override

      public void actionPerformed(ActionEvent e) {

      }

}