Why cant i send this to the cost class on button click?

Question

when the calculate button is clicked i want the program to check and see if a certain checkbox is selected and if set a variable in another class.
in the code below. why does it not recognize that i am trying to send the variable from the checkbox to the set method in the cost class?
public class ButtonHandler implements ActionListener{
          public void actionPerformed(ActionEvent ae){
               if(ae.getSource()==jbnCalculate)
                    String f = jtfFirst.getText();
                    String l = jtfLast.getText();
                    String id = jtfID.getText();
                    if(cbCream.isSelected()){
                         app1.setCream(ch);
package final_Kelm;
public class GMCCost extends GMCEmployee {
     double white;
     double wheat ;
     double creamCheese;
     double butter;
     double peachJelly;
     double blueberryJam;
     //double regularCoffee = 1.25;
     //double mocha = 2.75;
     //double latte = 3.05;
     double coffee;
     double total, subTotal;
     double TAX_RATE = 0.0825;
     public GMCCost()
          total = 0.0;
          subTotal = 0.0;
     public void setCoffee(double c)
          if(c==1)
               coffee = 1.25;
          else if(c==2)
               coffee = 2.75;
          else if(c==3)
               coffee = 3.05;
          else
               coffee = 0.0;
     public double getCoffee()
          return coffee;
     public void setCream(double ch)
          ch = 0.5;
          ch = creamCheese;
     public double getCream()
          return creamCheese;
     }i am having the same trouble with the coffee. i cannot add it to the output by "Coffee Price: " + app1.getCoffee() + . . . . .

Answer 1

why is this considered wrong or bad . . .
public class ButtonHandler implements ActionListener{
          public void actionPerformed(ActionEvent ae){
               if(ae.getSource()==jbnCalculate)
                    double butter, peach, blueberry, white, wheat;
                    double creamCheese, subTotal, total;
                    String f = jtfFirst.getText();
                    String l = jtfLast.getText();
                    String id = jtfID.getText();
                    double TAX_RATE = 0.0825;
                    if(cbCream.isSelected()){
                         creamCheese = 0.5;
                    else{
                         creamCheese = 0.0;
                    if(cbButter.isSelected()){
                         butter = 0.25;
                    else {
                         butter = 0.0;
                    if(cbPeach.isSelected()) {
                         peach = 0.75;
                    else {
                         peach = 0.0;
                    if(cbBlueberry.isSelected()) {
                         blueberry = 0.75;
                    else {
                         blueberry = 0.0;
                    if(jrbWhite.isSelected()) {
                         white = 1.25;
                    else{
                         white = 0.0;
                    if(jrbWheat.isSelected()) {
                         wheat = 1.50;
                    else {
                         wheat = 0.0;
                    double coffee = jcbCoffee.getSelectedIndex();
                    if(coffee==0) {
                         coffee = 0.0;
                    else if(coffee==1) {
                         coffee = 1.25;
                    else if(coffee==2) {
                         coffee = 2.75;
                    else {
                         coffee = 3.05;
                    subTotal = white + wheat + creamCheese + butter + peach + blueberry + coffee;
                    total = subTotal + (subTotal * TAX_RATE);
                    jtaOutput.setText("");
                    output = output+ "Employee Information: " +"\nStaff ID: " + id + "\nStaff First Name: " + f + "\nStaff Last Name: " + l +
                    "\nInvoice: " + "\nSubtotal: " + subTotal + "\nTotal: " + total;
               jtaOutput.append(output);
               }as compared to the code i first posted . . . . . this code works . . . the gui works . . . and im assuming this class of code can run all by itself and does have or need anything to do with the other 3 classes are in this program (see previous thread).
this program works but all the code is in the button handler class . . . i do not need to send anything anywhere, i do not need to use set get methods . . . so why would this code be frowned upon?