Pie Chart is not visible

Question

I cannot get my pie chart to show in my JPanel. I am not sure why that is or how to get it to display. Everything compiles fine...I feel like an idiot. Anyway, could you look at this and let me know what I could be doing wrong. My piechart starts on line 302.
import javax.swing.*;
import javax.swing.event.*;
import java.awt.*;
import java.awt.event.*;
import java.io.*;
import java.text.*;
public class MortCalcWeek5p extends JFrame implements ActionListener
     DecimalFormat twoPlaces = new DecimalFormat("#,###.00");//number format
     int L[] = {7, 15, 30};
     double I[] = {5.35, 5.5, 5.75};
     double P, M, J, H, C, Q;
     boolean manual = true;
     JPanel panel = new JPanel ();
     JRadioButton opt1 = new JRadioButton ("Manual Input", true);
     JRadioButton opt2 = new JRadioButton ("Menu Selections", false);
     ButtonGroup radioSelect = new ButtonGroup();
     JLabel PLabel = new JLabel("Enter the mortgage amount: ");
     JTextField PField = new JTextField(10);//field for obtaining user input for mortgage amount
     JLabel LLabel1 = new JLabel("Enter the term in years: ");
     JTextField LField = new JTextField(3);//field for obtaining user input for term in years
     JLabel ILabel1 = new JLabel("Enter the interest rate: ");
     JTextField IField = new JTextField(5);//field for obtaining user input for interest rate
     JLabel choices = new JLabel ("Or Choose the Interest Rate and Term in Years");
     JButton calcButton = new JButton("Calculate");
     JButton clearButton = new JButton("Clear");
     JButton exitButton = new JButton("Exit");
     JTextField payment = new JTextField(10);
     JLabel ILabel2 = new JLabel("Interest Rate: choose one");
     String [] IChoice = {I[0] + "", I[1] + "", I[2] + ""};
     JComboBox IBox = new JComboBox(IChoice);
     JLabel LLabel2 = new JLabel("Term (in years): choose one");
     String [] LChoice = {L[0] + "", L[1] + "", L[2] + ""};
     JComboBox LBox = new JComboBox(LChoice);
     JLabel amortBox = new JLabel("Amortization Table");
     JTextArea ta = new JTextArea();//<----------------------added
     //JScrollPane amortScroll = new JScrollPane();//
     JScrollPane amortScroll = new JScrollPane(ta, ScrollPaneConstants.VERTICAL_SCROLLBAR_ALWAYS, ScrollPaneConstants.HORIZONTAL_SCROLLBAR_AS_NEEDED);
     public MortCalcWeek5p () //creates the GUI window
                    super("Mortgage Calculator Week 5");
                    setSize(500, 400);
                    panel.setBackground (Color.white);
                    panel.setLayout(null);
                    panel.setPreferredSize(new Dimension(900, 500));
                    setResizable(false) ;
                    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                    //Creates the container
                    Container contentPane = getContentPane();
                    FlowLayout fresh = new FlowLayout(FlowLayout.LEFT);
                    panel.setLayout(fresh);
                    //identifies trigger events
                    calcButton.addActionListener(this);
                    clearButton.addActionListener(this);
                  exitButton.addActionListener(this);
                    PField.addActionListener(this);
                    LField.addActionListener(this);
                    IField.addActionListener(this);
                    opt1.addActionListener(this);
                    opt2.addActionListener(this);
                    panel.setLayout(fresh);
                    //Options
                    radioSelect.add(opt1);
                    radioSelect.add(opt2);
                    panel.add(opt1);
                    opt1.setBackground (Color.white);
                    panel.add(opt2);
                    opt2.setBackground (Color.white);
                    //Manual Entries
                    panel.add(PLabel);
                    panel.add(PField);
                    panel.add(LLabel1);
                    panel.add(LField);
                    panel.add(ILabel1);
                    panel.add(IField);
                    //Pre-set Entries
                    panel.add(choices);
                    panel.add(ILabel2);
                    panel.add(IBox);
                    IBox.setBackground (Color.white);
                    panel.add(LLabel2);
                    panel.add(LBox);
                    LBox.setBackground (Color.white);
                    //Buttons
                    panel.add(calcButton);
                    calcButton.setBackground (Color.white);
                    panel.add(payment);
                    payment.setBackground (Color.white);
                    panel.add(clearButton);
                    clearButton.setBackground (Color.white);
                    panel.add(exitButton);
                    exitButton.setBackground (Color.white);
                    //Amortization Table
                    panel.add(amortBox);
                    payment.setEditable(false);
                    panel.add(amortScroll);
                    amortScroll.setPreferredSize(new Dimension(600,300));//<----------added
                    setContentPane(panel);
                    //Pie Chart
                    //panel.add(PieChart);
                    setVisible(true);
     }// end of GUI info
          public void actionPerformed(ActionEvent e)
               Object source = e.getSource();
               if (source == calcButton)
                    try
                    if (manual)
                         Calculations_manual();
                    else Calculations_menu();
               catch(NumberFormatException event)
                    JOptionPane.showMessageDialog(null, "You did not enter a number.\n\nYou're not too bright are you?\n\nTry again.", "ERROR", JOptionPane.ERROR_MESSAGE);
               if (source == clearButton)
               reset();
               if (source == exitButton)
               end();
               if (source == opt1)
               IBox.setEnabled(false);
               LBox.setEnabled(false);
               LField.setEnabled(true);
               LField.setEditable(true);
               IField.setEnabled(true);
               IField.setEditable(true);
               manual = true;
               if (source == opt2)
               IBox.setEnabled(true);
               LBox.setEnabled(true);
               LField.setEnabled(false);
               LField.setEditable(false);
               IField.setEnabled(false);
               IField.setEditable(false);
               manual = false;
          public void Calculations_menu() //performs the calculations from user input
               double P = Double.parseDouble(PField.getText());
               double I = Double.parseDouble((String) IBox.getSelectedItem());
               double L = Double.parseDouble((String) LBox.getSelectedItem());
               double J = (I  / (12 * 100));//monthly interest rate
               double N = (L * 12);//term in months
               double M = (P * J) / (1 - Math.pow(1 + J, - N));//Monthly Payment
             String showPayment = twoPlaces.format(M);
             payment.setText(showPayment);
               int month = 1;
                         while (month <= N)
                              //performs the calculations for the amortization
                              double H = P * J;//current monthly interest
                              double C = M - H;//monthly payment minus monthly interest
                              double Q = P - C;//new balance
                              P = Q;//sets loop
                              month++;
                              //String showAmort = twoPlaces.format(H + C + Q);
                              //amortScroll(showAmort);
                            //ta.append("Month " + month);
                            ta.append("Interest Paid: " + twoPlaces.format(H));
                            ta.append("\tPrincipal Paid: " + twoPlaces.format(C));
                            ta.append("\tNew Balance: " + twoPlaces.format(Q) + "\n");
          public void Calculations_manual() //performs the calculations from user input
               double P = Double.parseDouble(PField.getText());
               double I = Double.parseDouble(IField.getText());
               double L = Double.parseDouble(LField.getText());
               double J = (I  / (12 * 100));//monthly interest rate
               double N = (L * 12);//term in months
               double M = (P * J) / (1 - Math.pow(1 + J, - N));//Monthly Payment
             String showPayment = twoPlaces.format(M);
             payment.setText(showPayment);
               int month = 1;
                         while (month <= N)
                              //performs the calculations for the amortization
                              double H = P * J;//current monthly interest
                              double C = M - H;//monthly payment minus monthly interest
                              double Q = P - C;//new balance
                              P = Q;//sets loop
                              month++;
                              //String showAmort = twoPlaces.format(H + C + Q);
                              //amortScroll(showAmort);
                            //ta.append("Month " + month);
                            ta.append("Interest Paid: " + twoPlaces.format(H));
                            ta.append("\tPrincipal Paid: " + twoPlaces.format(C));
                            ta.append("\tNew Balance: " + twoPlaces.format(Q) + "\n");
          // resets GUI for another calculation
          public void reset ()
          PField.setText(null);
          payment.setText(null);
          ta.setText(null);
          LField.setText(null);
          IField.setText(null);
          // ends GUI and exits program
          public void end()
          System.exit(0);
public class PieChart extends JComponent {
    // Class to hold a value for a slice
    class PieSlice
        //private variables
        double value;
        Color color;
        public PieSlice(double value, Color color)
            this.value = value;
            this.color = color;
        }//end Constructor
       } //end class PieSlice
    //private variable slices are array of PieSlice
    PieSlice[] slices = new PieSlice[2];
    //constructor
    PieChart(double C, double H)
        slices[0] = new PieSlice(C, Color.red);
        slices[1] = new PieSlice(H, Color.green);
        setVisible(true);
    // This method is called whenever the contents needs to be painted
    public void paintComponent(Graphics g) {
        // Draw the pie
        this.drawPie((Graphics2D)g, getBounds(), slices);
     // slices is an array of values that represent the size of each slice.
     public void drawPie(Graphics2D g, Rectangle area, PieSlice[] slices)
             // Get total value of all slices
             double total = 0.0;
             for (int p=0; p<slices.length; p++) {
                 total += slices[p].value;
             // Draw each pie slice
             double curValue = 0.0;
             int startAngle = 0;
             for (int p=0; p<slices.length; p++)
                 // Compute the start and stop angles
                 startAngle = (int)(curValue * 360 / total);
                 int arcAngle = (int)(slices[p].value * 360 / total);
                 // Ensure that rounding errors do not leave a gap between the first and last slice
                 if (p == slices.length-1) {
                     arcAngle = 360 - startAngle;
             } //end if
             // Set the color and draw a filled arc
             g.setColor(slices[p].color);
             g.fillArc(area.x, area.y, 200, 200, startAngle, arcAngle);
             RenderingHints renderHints = new RenderingHints(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
               g.setRenderingHints(renderHints);
               //int border=10;
               //Ellipse2D.Double elb = new Ellipse2D.Double(area.x - border/2, area.y - border/2, pieWidth + border, pieHeight + border);
             //g.fillArc(area.x, area.y, area.width, area.height, startAngle, arcAngle);
             curValue += slices[p].value;
             } //end for
        }//end drawPie
        public void resetPieChart(double capital, double interest)
                         slices[0] = new PieSlice(capital, Color.red);
                         slices[1] = new PieSlice(interest, Color.green);
                         this.repaint();
          }//end resetPieChart
    }//end class PieChart
          public static void main(String[] args)
               MortCalcWeek5p app = new MortCalcWeek5p();
               app.pack();
          }//end main
}//end the programI am sorry for putting all this code here. I just wanted you to see the entire picture. I won't post all of this again unless requested.
Thanks,
Seawall

Answer 1

Did you write this? :)
Anyways, here you go. You never instantiated the pie chart or added it to gui. I created a separate JFrame and put pie chart in it. Also added PieChart.resetPieChart in actionHandler method. This should get you started.
import javax.swing.*;
import javax.swing.event.*;
import java.awt.*;
import java.awt.event.*;
import java.io.*;
import java.text.*;
public class MortCalcWeek5p extends JFrame implements ActionListener
     DecimalFormat twoPlaces = new DecimalFormat("#,###.00");//number format
     int L[] = {7, 15, 30};
     double I[] = {5.35, 5.5, 5.75};
     double P, M, J, H, C, Q;
     boolean manual = true;
        PieChart pc = null;
     JPanel panel = new JPanel ();
     JRadioButton opt1 = new JRadioButton ("Manual Input", true);
     JRadioButton opt2 = new JRadioButton ("Menu Selections", false);
     ButtonGroup radioSelect = new ButtonGroup();
     JLabel PLabel = new JLabel("Enter the mortgage amount: ");
     JTextField PField = new JTextField(10);//field for obtaining user input for mortgage amount
     JLabel LLabel1 = new JLabel("Enter the term in years: ");
     JTextField LField = new JTextField(3);//field for obtaining user input for term in years
     JLabel ILabel1 = new JLabel("Enter the interest rate: ");
     JTextField IField = new JTextField(5);//field for obtaining user input for interest rate
     JLabel choices = new JLabel ("Or Choose the Interest Rate and Term in Years");
     JButton calcButton = new JButton("Calculate");
     JButton clearButton = new JButton("Clear");
     JButton exitButton = new JButton("Exit");
     JTextField payment = new JTextField(10);
     JLabel ILabel2 = new JLabel("Interest Rate: choose one");
     String [] IChoice = {I[0] + "", I[1] + "", I[2] + ""};
     JComboBox IBox = new JComboBox(IChoice);
     JLabel LLabel2 = new JLabel("Term (in years): choose one");
     String [] LChoice = {L[0] + "", L[1] + "", L[2] + ""};
     JComboBox LBox = new JComboBox(LChoice);
     JLabel amortBox = new JLabel("Amortization Table");
     JTextArea ta = new JTextArea();//<----------------------added
     //JScrollPane amortScroll = new JScrollPane();//
     JScrollPane amortScroll = new JScrollPane(ta, ScrollPaneConstants.VERTICAL_SCROLLBAR_ALWAYS, ScrollPaneConstants.HORIZONTAL_SCROLLBAR_AS_NEEDED);
     public MortCalcWeek5p () //creates the GUI window
                    super("Mortgage Calculator Week 5");
                    setSize(500, 400);
                    panel.setBackground (Color.white);
                    //panel.setLayout(null);
                    panel.setPreferredSize(new Dimension(900, 500));
                    setResizable(false) ;
                    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                    //Creates the container
                    //Container contentPane = getContentPane();
                    FlowLayout fresh = new FlowLayout(FlowLayout.LEFT);
                                BorderLayout bl = new BorderLayout();
                    panel.setLayout(fresh);
                    //identifies trigger events
                    calcButton.addActionListener(this);
                    clearButton.addActionListener(this);
                                exitButton.addActionListener(this);
                    PField.addActionListener(this);
                    LField.addActionListener(this);
                    IField.addActionListener(this);
                    opt1.addActionListener(this);
                    opt2.addActionListener(this);
                    //panel.setLayout(fresh);
                    //Options
                    radioSelect.add(opt1);
                    radioSelect.add(opt2);
                    panel.add(opt1);
                    opt1.setBackground (Color.white);
                    panel.add(opt2);
                    opt2.setBackground (Color.white);
                    //Manual Entries
                    panel.add(PLabel);
                    panel.add(PField);
                    panel.add(LLabel1);
                    panel.add(LField);
                    panel.add(ILabel1);
                    panel.add(IField);
                    //Pre-set Entries
                    panel.add(choices);
                    panel.add(ILabel2);
                    panel.add(IBox);
                    IBox.setBackground (Color.white);
                    panel.add(LLabel2);
                    panel.add(LBox);
                    LBox.setBackground (Color.white);
                    //Buttons
                    panel.add(calcButton);
                    calcButton.setBackground (Color.white);
                    panel.add(payment);
                    payment.setBackground (Color.white);
                    panel.add(clearButton);
                    clearButton.setBackground (Color.white);
                    panel.add(exitButton);
                    exitButton.setBackground (Color.white);
                    //Amortization Table
                    panel.add(amortBox);
                    payment.setEditable(false);
                    panel.add(amortScroll);
                    amortScroll.setPreferredSize(new Dimension(600,300));//<----------added
                    setContentPane(panel);
                                setVisible(true);
                    //Pie Chart
                                pc = new PieChart(C,H);
                    JPanel piePanel = new JPanel();
                                piePanel.setLayout(new BorderLayout());
                                piePanel.add(pc);                               
                                JFrame pieFrame = new JFrame("Pie Chart");
                                pieFrame.setSize(210,230);
                                pieFrame.getContentPane().add(piePanel);
                                pieFrame.setVisible(true);
     }// end of GUI info
          public void actionPerformed(ActionEvent e)
               Object source = e.getSource();
               if (source == calcButton)
                                pc.resetPieChart(C, H);
                    try
                    if (manual)
                         Calculations_manual();
                    else Calculations_menu();
               catch(NumberFormatException event)
                    JOptionPane.showMessageDialog(null, "You did not enter a number.\n\nYou're not too bright are you?\n\nTry again.", "ERROR", JOptionPane.ERROR_MESSAGE);
               if (source == clearButton)
               reset();
               if (source == exitButton)
               end();
               if (source == opt1)
               IBox.setEnabled(false);
               LBox.setEnabled(false);
               LField.setEnabled(true);
               LField.setEditable(true);
               IField.setEnabled(true);
               IField.setEditable(true);
               manual = true;
               if (source == opt2)
               IBox.setEnabled(true);
               LBox.setEnabled(true);
               LField.setEnabled(false);
               LField.setEditable(false);
               IField.setEnabled(false);
               IField.setEditable(false);
               manual = false;
          public void Calculations_menu() //performs the calculations from user input
               double P = Double.parseDouble(PField.getText());
               double I = Double.parseDouble((String) IBox.getSelectedItem());
               double L = Double.parseDouble((String) LBox.getSelectedItem());
               double J = (I  / (12 * 100));//monthly interest rate
               double N = (L * 12);//term in months
               double M = (P * J) / (1 - Math.pow(1 + J, - N));//Monthly Payment
             String showPayment = twoPlaces.format(M);
             payment.setText(showPayment);
               int month = 1;
                         while (month <= N)
                              //performs the calculations for the amortization
                              double H = P * J;//current monthly interest
                              double C = M - H;//monthly payment minus monthly interest
                              double Q = P - C;//new balance
                              P = Q;//sets loop
                              month++;
                              //String showAmort = twoPlaces.format(H + C + Q);
                              //amortScroll(showAmort);
                            //ta.append("Month " + month);
                            ta.append("Interest Paid: " + twoPlaces.format(H));
                            ta.append("\tPrincipal Paid: " + twoPlaces.format(C));
                            ta.append("\tNew Balance: " + twoPlaces.format(Q) + "\n");
          public void Calculations_manual() //performs the calculations from user input
               double P = Double.parseDouble(PField.getText());
               double I = Double.parseDouble(IField.getText());
               double L = Double.parseDouble(LField.getText());
               double J = (I  / (12 * 100));//monthly interest rate
               double N = (L * 12);//term in months
               double M = (P * J) / (1 - Math.pow(1 + J, - N));//Monthly Payment
             String showPayment = twoPlaces.format(M);
             payment.setText(showPayment);
               int month = 1;
                         while (month <= N)
                              //performs the calculations for the amortization
                              double H = P * J;//current monthly interest
                              double C = M - H;//monthly payment minus monthly interest
                              double Q = P - C;//new balance
                              P = Q;//sets loop
                              month++;
                              //String showAmort = twoPlaces.format(H + C + Q);
                              //amortScroll(showAmort);
                            //ta.append("Month " + month);
                            ta.append("Interest Paid: " + twoPlaces.format(H));
                            ta.append("\tPrincipal Paid: " + twoPlaces.format(C));
                            ta.append("\tNew Balance: " + twoPlaces.format(Q) + "\n");
          // resets GUI for another calculation
          public void reset ()
          PField.setText(null);
          payment.setText(null);
          ta.setText(null);
          LField.setText(null);
          IField.setText(null);
          // ends GUI and exits program
          public void end()
          System.exit(0);
public class PieChart extends JComponent {
    // Class to hold a value for a slice
    class PieSlice
        //private variables
        double value;
        Color color;
        public PieSlice(double value, Color color)
            this.value = value;
            this.color = color;
        }//end Constructor
       } //end class PieSlice
    //private variable slices are array of PieSlice
    PieSlice[] slices = new PieSlice[2];
    //constructor
    PieChart(double C, double H)
        slices[0] = new PieSlice(C, Color.red);
        slices[1] = new PieSlice(H, Color.green);
        setVisible(true);
    // This method is called whenever the contents needs to be painted
    public void paintComponent(Graphics g) {
        // Draw the pie
        this.drawPie((Graphics2D)g, getBounds(), slices);
     // slices is an array of values that represent the size of each slice.
     public void drawPie(Graphics2D g, Rectangle area, PieSlice[] slices)
             // Get total value of all slices
             double total = 0.0;
             for (int p=0; p<slices.length; p++) {
                 total += slices[p].value;
             // Draw each pie slice
             double curValue = 0.0;
             int startAngle = 0;
             for (int p=0; p<slices.length; p++)
                 // Compute the start and stop angles
                 startAngle = (int)(curValue * 360 / total);
                 int arcAngle = (int)(slices[p].value * 360 / total);
                 // Ensure that rounding errors do not leave a gap between the first and last slice
                 if (p == slices.length-1) {
                     arcAngle = 360 - startAngle;
             } //end if
             // Set the color and draw a filled arc
             g.setColor(slices[p].color);
             g.fillArc(area.x, area.y, 200, 200, startAngle, arcAngle);
             RenderingHints renderHints = new RenderingHints(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
               g.setRenderingHints(renderHints);
               //int border=10;
               //Ellipse2D.Double elb = new Ellipse2D.Double(area.x - border/2, area.y - border/2, pieWidth + border, pieHeight + border);
             //g.fillArc(area.x, area.y, area.width, area.height, startAngle, arcAngle);
             curValue += slices[p].value;
             } //end for
        }//end drawPie
        public void resetPieChart(double capital, double interest)
                         slices[0] = new PieSlice(capital, Color.red);
                         slices[1] = new PieSlice(interest, Color.green);
                         this.repaint();
          }//end resetPieChart
    }//end class PieChart
          public static void main(String[] args)
               MortCalcWeek5p app = new MortCalcWeek5p();
               app.pack();
          }//end main
}//end the program