Cannot load a thread that loads chart from JFreeChart until clicked

I think I did everything right. All I want is to load the thread that contains chart after the main thread that loads the components and window is finished. But it didn’t work. Somehow I have to click jPanel1 (the panel that will load the chart) and the chart is loaded. Any help would be appreciated. What I’ve tried:

• Changing the thread to SwingWorker
• Changing the thread to invokeLater

public LaporanPenjualan() {
    initComponents();
    createFrame1();
}
    
private void createFrame1() {
    SwingWorker thread = new SwingWorker() {
        @Override
        protected Boolean doInBackground() throws Exception {
            JLabel loading = new JLabel("Loading, please wait",SwingConstants.CENTER);
            loading.setSize(500,500);
            loading.setAlignmentY(250);
            jPanel1.add(loading);
            final DefaultCategoryDataset dataset = new DefaultCategoryDataset();
            dataset.addValue(34.0, "Penjualan/produk", "Kue Enak");
            dataset.addValue(23.0, "Penjualan/produk", "Kue Enggak Enak");
            dataset.addValue(54.0, "Penjualan/produk", "Kue Sultan");
            final JFreeChart chart = ChartFactory.createBarChart(
                    "", 
                    "Produk",
                    "Penjualan",
                    dataset,
                    PlotOrientation.VERTICAL,
                    true,
                    true,
                    false
                );
            final ChartPanel chartPanel = new ChartPanel(chart);
            chartPanel.setSize(500,500);
            chartPanel.setPreferredSize(new Dimension(300, 300));
            jPanel1.removeAll();
            jPanel1.add(chartPanel);
            chartPanel.setVisible(true);
            return true;
        }
    };                 
    thread.execute();
}

Answer

Swing is single threaded and not thread safe. This means that you shouldn’t perform any long running or blocking operations with the context of the Event Dispatching Thread and you shouldn’t modify the state of the UI or any state the UI relies on, from outside the context of the EDT.

So, having said that, you’re createFrame1 might look something more like…

private void createFrame1() {
    JLabel loading = new JLabel("Loading, please wait", SwingConstants.CENTER);
    loading.setSize(500, 500);
    loading.setAlignmentY(250);
    jPanel1.add(loading);
    jPanel1.revalidate();
    jPanel1.repaint();

    SwingWorker<DefaultCategoryDataset, Void> thread;
    thread = new SwingWorker<>() {

        @Override
        protected DefaultCategoryDataset doInBackground() throws Exception {
            final DefaultCategoryDataset dataset = new DefaultCategoryDataset();
            dataset.addValue(34.0, "Penjualan/produk", "Kue Enak");
            dataset.addValue(23.0, "Penjualan/produk", "Kue Enggak Enak");
            dataset.addValue(54.0, "Penjualan/produk", "Kue Sultan");
            return dataset;
        }

        @Override
        protected void done() {
            try {
                DefaultCategoryDataset dataset = get();

                final JFreeChart chart = ChartFactory.createBarChart(
                        "",
                        "Produk",
                        "Penjualan",
                        dataset,
                        PlotOrientation.VERTICAL,
                        true,
                        true,
                        false
                );
                final ChartPanel chartPanel = new ChartPanel(chart);
                jPanel1.removeAll();
                jPanel1.add(chartPanel);
                chartPanel.setVisible(true);
            } catch (InterruptedException ex) {
                Logger.getLogger(Test.class.getName()).log(Level.SEVERE, null, ex);
            } catch (ExecutionException ex) {
                Logger.getLogger(Test.class.getName()).log(Level.SEVERE, null, ex);
            }
        }

    };
    thread.execute();
}

Alternatively, if loading the data takes time, you could publish each data point…

public class DataPoint {

    private double value;
    private Comparable rowKey;
    private Comparable columnKey;

    public DataPoint(double value, Comparable rowKey, Comparable columnKey) {
        this.value = value;
        this.rowKey = rowKey;
        this.columnKey = columnKey;
    }

    public double getValue() {
        return value;
    }

    public Comparable getRowKey() {
        return rowKey;
    }

    public Comparable getColumnKey() {
        return columnKey;
    }
}

private void createFrame1() {
    SwingWorker<Void, DataPoint> thread;
    DefaultCategoryDataset dataSet = new DefaultCategoryDataset();
    final JFreeChart chart = ChartFactory.createBarChart(
            "",
            "Produk",
            "Penjualan",
            dataSet,
            PlotOrientation.VERTICAL,
            true,
            true,
            false
    );
    final ChartPanel chartPanel = new ChartPanel(chart);
    jPanel1.add(chartPanel);

    thread = new SwingWorker<>() {

        @Override
        protected Void doInBackground() throws Exception {
            publish(new DataPoint(34.0, "Penjualan/produk", "Kue Enak"));
            publish(new DataPoint(23.0, "Penjualan/produk", "Kue Enggak Enak"));
            publish(new DataPoint(54.0, "Penjualan/produk", "Kue Sultan"));
        }

        @Override
        protected void process(List<DataPoint> chunks) {
            for (DataPoint dp : chunks) {
                dataSet.addValue(dp.getValue(), dp.getRowKey(), dp.getColumnKey());
            }
        }
    };
    thread.execute();
}

Runnable example…

import java.awt.BorderLayout;
import java.awt.EventQueue;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.util.List;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.SwingWorker;
import org.jfree.chart.ChartFactory;
import org.jfree.chart.ChartPanel;
import org.jfree.chart.JFreeChart;
import org.jfree.chart.plot.PlotOrientation;
import org.jfree.data.category.DefaultCategoryDataset;

public class Test {

    public static void main(String[] args) {
        new Test();
    }

    public Test() {
        EventQueue.invokeLater(new Runnable() {
            @Override
            public void run() {
                JFrame frame = new JFrame();
                frame.add(new TestPane());
                frame.pack();
                frame.setLocationRelativeTo(null);
                frame.setVisible(true);
            }
        });
    }

    public class TestPane extends JPanel {

        public TestPane() {
            setLayout(new BorderLayout());
            DefaultCategoryDataset dataSet = new DefaultCategoryDataset();
            final JFreeChart chart = ChartFactory.createBarChart(
                    "",
                    "Produk",
                    "Penjualan",
                    dataSet,
                    PlotOrientation.VERTICAL,
                    true,
                    true,
                    false
            );
            final ChartPanel chartPanel = new ChartPanel(chart);
            add(chartPanel);

            ChartWorker worker = new ChartWorker(dataSet);
            worker.execute();
        }
    }

    public class ChartWorker extends SwingWorker<Void, ChartWorker.DataPoint> {

        public class DataPoint {

            private double value;
            private Comparable rowKey;
            private Comparable columnKey;

            public DataPoint(double value, Comparable rowKey, Comparable columnKey) {
                this.value = value;
                this.rowKey = rowKey;
                this.columnKey = columnKey;
            }

            public double getValue() {
                return value;
            }

            public Comparable getRowKey() {
                return rowKey;
            }

            public Comparable getColumnKey() {
                return columnKey;
            }
        }

        private DefaultCategoryDataset dataSet;

        public ChartWorker(DefaultCategoryDataset dataSet) {
            this.dataSet = dataSet;
        }

        @Override
        protected Void doInBackground() throws Exception {
            // The sleeps here are just for demonstration, 
            // you don't need them
            Thread.sleep(1000);
            publish(new DataPoint(34.0, "Penjualan/produk", "Kue Enak"));
            Thread.sleep(1000);
            publish(new DataPoint(23.0, "Penjualan/produk", "Kue Enggak Enak"));
            Thread.sleep(1000);
            publish(new DataPoint(54.0, "Penjualan/produk", "Kue Sultan"));
            return null;
        }

        @Override
        protected void process(List<DataPoint> chunks) {
            for (DataPoint dp : chunks) {
                dataSet.addValue(dp.getValue(), dp.getRowKey(), dp.getColumnKey());
            }
        }
    }
}

And waiting for stuff to be done…

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.EventQueue;
import java.beans.PropertyChangeEvent;
import java.beans.PropertyChangeListener;
import java.util.concurrent.ExecutionException;
import java.util.logging.Level;
import java.util.logging.Logger;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.SwingWorker;
import org.jfree.chart.ChartFactory;
import org.jfree.chart.ChartPanel;
import org.jfree.chart.JFreeChart;
import org.jfree.chart.plot.PlotOrientation;
import org.jfree.data.category.DefaultCategoryDataset;

public class Test {

    public static void main(String[] args) {
        new Test();
    }

    public Test() {
        EventQueue.invokeLater(new Runnable() {
            @Override
            public void run() {
                JFrame frame = new JFrame();
                frame.add(new TestPane());
                frame.pack();
                frame.setLocationRelativeTo(null);
                frame.setVisible(true);
            }
        });
    }

    public class TestPane extends JPanel {

        public TestPane() {
            setLayout(new BorderLayout());

            add(new JLabel("Loading stuff", JLabel.CENTER));

            ChartWorker worker = new ChartWorker();
            worker.addPropertyChangeListener(new PropertyChangeListener() {
                @Override
                public void propertyChange(PropertyChangeEvent evt) {
                    if ("state".equals(evt.getPropertyName())) {
                        if (worker.getState() == SwingWorker.StateValue.DONE) {
                            try {
                                DefaultCategoryDataset dataSet = worker.get();
                                setDataSet(dataSet);
                            } catch (InterruptedException ex) {
                                add(new JLabel("Loading fauked", JLabel.CENTER));
                                Logger.getLogger(Test.class.getName()).log(Level.SEVERE, null, ex);
                            } catch (ExecutionException ex) {
                                add(new JLabel("Loading fauked", JLabel.CENTER));
                                Logger.getLogger(Test.class.getName()).log(Level.SEVERE, null, ex);
                            }
                        }
                    }
                }
            });
            worker.execute();
        }

        @Override
        public Dimension getPreferredSize() {
            return new Dimension(400, 400);
        }

        protected void setDataSet(DefaultCategoryDataset dataSet) {
            final JFreeChart chart = ChartFactory.createBarChart(
                    "",
                    "Produk",
                    "Penjualan",
                    dataSet,
                    PlotOrientation.VERTICAL,
                    true,
                    true,
                    false
            );
            final ChartPanel chartPanel = new ChartPanel(chart);
            removeAll();
            add(chartPanel);
            revalidate();
            repaint();
        }

    }

    public class ChartWorker extends SwingWorker<DefaultCategoryDataset, Void> {

        public ChartWorker() {
        }

        @Override
        protected DefaultCategoryDataset doInBackground() throws Exception {
            DefaultCategoryDataset dataSet = new DefaultCategoryDataset();
            dataSet.addValue(34.0, "Penjualan/produk", "Kue Enak");
            dataSet.addValue(23.0, "Penjualan/produk", "Kue Enggak Enak");
            dataSet.addValue(54.0, "Penjualan/produk", "Kue Sultan");

            // Again, you don't need this
            Thread.sleep(5000);

            return dataSet;
        }
    }
}