If statement not working for dropmenu calculations

[Angel eyes]

I am having trouble with my if statements being overlooked or something. In the code below for case 0, case 1 and case 2. I have a menu the user selects and it is to calculate depending on what the user selects. It calculates the same thing no matter which options is chosen. Could someone please let me know where I am going wrong. I am new to this and have spent many hours trying to adjust my code to calculate correctly. Thank you in advance for helping me with this.

 import java.awt.*;
import java.awt.event.*;

import java.util.*;

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

import java.awt.event.ActionListener;
import java.text.DecimalFormat;

import java.text.NumberFormat;
import java.math.*;

import java.awt.BorderLayout;
import java.awt.Color;

import java.awt.Container;
import java.awt.FlowLayout;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

//declared area for user inputs
import javax.swing.JButton;
import javax.swing.JComboBox;

import javax.swing.JCheckBox;
import javax.swing.JFrame;

import javax.swing.JPanel;
import javax.swing.JTextArea;

import javax.swing.JTextField;

import javax.swing.JComboBox;
import javax.swing.JFrame;




//declared public class
public class MortgageCalculatorCR62 {

// declared number formaats
double amount, interest, payments;
double term;
double amortize;
double interestRateMonths;
double monthlyInterestPayment;
double monthlyPrincipalPayment;
int termMonths;
double balance;
int Labels;
int years, months;



//declared Variables
JLabel amountLbl, termLbl, IntrateLbl, paymentsLbl, amortizeLb1;
JTextField amountTf, termTf, intrateTf, paymentsTf, amortizeTf;
JButton calcBtn;
JButton clearButton;
JButton exitButton;
JFrame frame;
JComboBox cOption;
JTextArea AmortizeTable;




public MortgageCalculatorCR62() {
  System.out.println("Mortgage Calculator Change Request #6.");

     String Labels[] = { "7 Years at 5.35%", "15 years at 5.5%", "30 years at 5.75%",};




amountLbl = new JLabel("  Loan Amount In Whole Dollars");
termLbl = new JLabel("  Terms In Years");
IntrateLbl = new JLabel("  Interest Rate - Decimal Format (0.000)");
paymentsLbl = new JLabel("  Monthly Payment Amount");

    amountTf = new JTextField();
      JComboBox comboBox1 = new JComboBox(Labels);

    cOption = new JComboBox(Labels);



//this is for the menu screen and user input interface
termTf = new JTextField();
intrateTf = new JTextField();

  AmortizeTable = new JTextArea (10, 10);

  amortizeTf = new JTextField();
amortizeLb1 = new JLabel("Amortization Table");
paymentsTf = new JTextField();
calcBtn = new JButton("Calculate");
clearButton = new JButton("Clear");
exitButton = new JButton("Exit");


frame = new JFrame("Mortgage Calculator CR#6");
frame.setLayout(new GridLayout(25,25));
frame.add(amountLbl);
frame.add(amountTf);
frame.add(comboBox1);

frame.add(paymentsLbl);
frame.add(paymentsTf);


//declared Added textfield/label
frame.add(amortizeLb1);
// frame.add(amortizeTf);

frame.add(AmortizeTable);

frame.add(calcBtn);
frame.add(clearButton);
frame.add(exitButton);

frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
clearButton.addActionListener(new Handler());
calcBtn.addActionListener(new Handler());
exitButton.addActionListener(new Handler());
frame.setLocation(10, 10);
frame.pack();

frame.setVisible(true);
}
private class Handler implements ActionListener
{
public void actionPerformed(ActionEvent event)
{


Object command = event.getSource();
  if(command == calcBtn)
        {
          calculate();
          }
if(command == clearButton)
        {

          amountTf.setText(null);
          termTf.setText(null);
          intrateTf.setText(null);
          paymentsTf.setText(null);
          amortizeTf.setText(null);
          AmortizeTable.setText(null);
            }

            if(command == exitButton)
            {
          System.exit(0);


}
}
}

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


//formula for calculations with in the program
  private void calculate() {
DecimalFormat decimalPlaces=new DecimalFormat("0,000.00");

    String Labels = (String) cOption.getSelectedItem();

int selected = cOption.getSelectedIndex();
switch (selected)  {
case 0: // 7 yr at 5.35

if (Labels.equalsIgnoreCase("7 years at 5.35%"))

                {
            double amount = Double.parseDouble(amountTf.getText());
                int    years = 7;
                int    months = (years * 12);
                double interest    = .0535;
                double termMonths = months;
                double term = 7;

            double payments = (amount * ((interest/12) / (1-Math.pow((1+(interest/12)),-(term*12)))));

                paymentsTf.setText("" + decimalPlaces.format(payments));
                      AmortizeTable.setText("" + decimalPlaces.format(amount-payments));


}
break;
case 1: //15 yr at 5.5

if (Labels.equalsIgnoreCase("15 years at 5.5%"))
            {
           double amount = Double.parseDouble(amountTf.getText());
                int    Years = 15;
                int    Months = (Years * 12);
                double    Interest    = .055;
                     double TermMonths = months;
                     double Term = 15;

        double Payments = (amount * ((interest/12) / (1-Math.pow((1+(interest/12)),-(Term*12)))));

            paymentsTf.setText("" + decimalPlaces.format(Payments));
                 AmortizeTable.setText("" + decimalPlaces.format(amount-Payments));

}

break;
case 2: // 30 yr at 5.75

if   (Labels.equalsIgnoreCase("30 years at 5.75%"))
            {
         double Amount = Double.parseDouble(amountTf.getText());
                int    Years = 30;
                int   Months = (years * 12);
                double    Interest    = .0575;
                     double TermMonths = months;
                     double Term = 15;

        double Payments = (amount * ((Interest/12) / (1-Math.pow((1+(Interest/12)),-(Term*12)))));
         paymentsTf.setText("" + decimalPlaces.format(Payments));
             AmortizeTable.setText("" + decimalPlaces.format(amount-Payments));


       }

break;
}


     // start while loop
     while  (termMonths > 0)


   AmortizeTable.setText("" + decimalPlaces.format(amount-payments));

          months--;


    }}   

[Rob Camick]

Well I don't know what a "drop menu" is. Where in the API or documentation do you see that term?

I am new to this and have spent many hours trying to adjust my code to calculate correctly.

Well, then learn to simplify the problem.

I assume you are talking about a combo box, so create a simple JFrame with a single combo box. Then learn how to write simple test code for only that one component that simply displays a different message for each item select.

Then once you understand the basics of using a combo box you apply the knowledge to your real program. Its called a SSCCE.

If you need further help then you need to create a SSCCE (Short, Self Contained, Compilable and Executable, Example Program), that demonstrates the incorrect behaviour.

[Maneesh Godbole]

Angel eyes wrote:

Please check your private messages for an important administrative matter

[Michael Dunn]

> I am having trouble with my if statements being overlooked or something.

are you sure?

debugging 101, add 1 or more System.out.println()'s to see what values the program is working with.

first up would be
System.out.println("Combo selected index = "+selected);

run the program, select the first item, 'calculate' and see what prints
select the second item, 'calculate' and see what prints

if the same value is printed, backtrack the comboBox and you should find the problem