No errors...but no answers either

This message was edited 1 time. Last update was at 05/15/2007 13:14:45

I know where the problem is. I've tested it out a few times trying to get around it, but there is something wrong with my initialization of the variable S1block1Points, or I do believe so atleast.

It's a GPA finder that I've been trying to create for personal use, I'm only partially into it and I'm getting this one annoying problem. Maybe you can help me and tell me why when I enter A into the TextField (which should give an output of 8 ) why I get an answer of 0?

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

public class GPA extends GBFrame{
	//Declare window objects ----------------------------------------------------------
	
	private JButton submitButton, clearButton;
	//Semester#Quarter#Block#
	private JTextField S1Q1B1, S1Q1B2, S1Q1B3, S1Q1B4, S1Q2B1, 
			S1Q2B2, S1Q2B3, S1Q2B4, S1Q3B1, S1Q3B2, S1Q3B3, 
			S1Q3B4, S1Q4B1, S1Q4B2, S1Q4B3, S1Q4B4,
			S1MidTermB1, S1MidTermB2, S1MidTermB3, S1MidTermB4,
			S1FinalB1, S1FinalB2, S1FinalB3, S1FinalB4, 
			S2Q1B1, S2Q1B2, S2Q1B3, S2Q1B4, S2Q2B1, 
			S2Q2B2, S2Q2B3, S2Q2B4, S2Q3B1, S2Q3B2, S2Q3B3, 
			S2Q3B4, S2Q4B1, S2Q4B2, S2Q4B3, S2Q4B4,
			S2MidTermB1, S2MidTermB2, S2MidTermB3, S2MidTermB4,
			S2FinalB1, S2FinalB2, S2FinalB3, S2FinalB14;
	private DoubleField coreGPAField , weightedGPAField;
	private JLabel FirstSemesterLabel, SecondSemesterLabel,
			S1B1Label, S1B2Label, S1B3Label, S1B4Label,
			S2B1Label, S2B2Label, S2B3Label, S2B4Label,
			S1Q1Label, S1Q2Label, S1Q3Label, S4Q1Label, 
			S2Q1Label, S2Q2Label, S2Q3Label, S2Q4Label,
			S1MidtermLabel, S2MidtermLabel, S1FinalLabel, 
			S2FinalLabel, coreGPALabel, weightedGPALabel;
	private JCheckBox cbS1academicB1, cbS1academicB2, cbS1academicB3, 
			cbS1academicB4, cbS2academicB1, cbS2academicB2, cbS2academicB3,
			cbS2academicB4;
	private JRadioButton S1HB1, S1HB2, S1HB3, S1HB4, S2HB1, S2HB2, 
			S2HB3, S2HB4, S1APB1, S1APB2, S1APB3, S1APB4, S2APB1, S2APB2, 
			S2APB3, S2APB4; 
	private IntegerField checkerField;

// Other instance variables -------------------------------------------------------
	private int totalPoints;
	private int S1block2Points;
	private int S1block3Points;
	private int S1block4Points;
	private int S2block1Points;
	private int S2block2Points;
	private int S2block3Points;
	private int S2block4Points;
	private double coreGPA = 0;
	private double weightedGPA = 0;
	
	//Constructor ---------------------------------------------------------------------
	
	public GPA(){
		submitButton = addButton	("Submit"			,14,7,1,1);
		clearButton  = addButton	("Clear"			,14,8,1,1);
		S1Q1B1		 = addTextField (""					,3,6,1,1);
		S1Q1B2		 = addTextField (""					,4,6,1,1);
		S1Q1B3		 = addTextField (""					,5,6,1,1);
		S1Q1B4		 = addTextField (""					,6,6,1,1);
		S1Q2B1		 = addTextField (""					,3,7,1,1);
		S1Q2B2		 = addTextField (""					,4,7,1,1);
		S1Q2B3		 = addTextField (""					,5,7,1,1);
		S1Q2B4		 = addTextField (""					,6,7,1,1);
		S1Q3B1		 = addTextField (""					,3,9,1,1);
		S1Q3B2 		 = addTextField (""					,4,9,1,1);
		S1Q3B3 		 = addTextField (""					,5,9,1,1);
		S1Q3B4 		 = addTextField (""					,6,9,1,1);
		S1Q4B1		 = addTextField (""					,3,10,1,1);
		S1Q4B2		 = addTextField (""					,4,10,1,1);
		S1Q4B3		 = addTextField (""					,5,10,1,1);
		S1Q4B4		 = addTextField (""					,6,10,1,1);
		S1MidTermB1	 = addTextField ("" 				,3,8,1,1); 
		S1MidTermB2  = addTextField ("" 				,4,8,1,1);
		S1MidTermB3	 = addTextField ("" 				,5,8,1,1);
		S1MidTermB4	 = addTextField ("" 				,6,8,1,1);
		S1FinalB1	 = addTextField ("" 				,3,11,1,1); 
		S1FinalB2	 = addTextField ("" 				,4,11,1,1); 
		S1FinalB3	 = addTextField ("" 				,5,11,1,1); 
		S1FinalB4	 = addTextField ("" 				,6,11,1,1); 
		S2Q1B1		 = addTextField (""					,9,6,1,1); 
		S2Q1B2		 = addTextField (""				    ,10,6,1,1);
		S2Q1B3 		 = addTextField (""					,11,6,1,1);
		S2Q1B4 		 = addTextField (""					,12,6,1,1);
		S2Q2B1 		 = addTextField (""					,9,7,1,1);
		S2Q2B2 		 = addTextField (""					,10,7,1,1);
		S2Q2B3 		 = addTextField (""					,11,7,1,1);
		S2Q2B4 		 = addTextField (""					,12,7,1,1);
		S2Q3B1 		 = addTextField (""					,9,9,1,1);
		S2Q3B2 		 = addTextField (""					,10,9,1,1);
		S2Q3B3 		 = addTextField (""					,11,9,1,1);
		S2Q3B4 		 = addTextField (""					,12,9,1,1);
		S2Q4B1 		 = addTextField (""					,9,10,1,1);
		S2Q4B2 		 = addTextField (""					,10,10,1,1);
		S2Q4B3 		 = addTextField (""					,11,10,1,1);
		S2Q4B4		 = addTextField (""					,12,10,1,1);
		S2MidTermB1	 = addTextField (""					,9,8,1,1);
		S2MidTermB2  = addTextField (""					,10,8,1,1);
		S2MidTermB3  = addTextField (""					,11,8,1,1);
		S2MidTermB4	 = addTextField (""					,12,8,1,1);
		S2FinalB1	 = addTextField (""					,9,11,1,1);
		S2FinalB2 	 = addTextField (""					,10,11,1,1);
		S2FinalB3 	 = addTextField (""					,11,11,1,1);
		S2FinalB14	 = addTextField (""					,12,11,1,1);
		coreGPAField 	 	= addDoubleField 	(0						,13,6,1,1);
		weightedGPAField 	= addDoubleField	(0						,13,9,1,1);
		checkerField		= addIntegerField	(0						,13,11,1,1);	
		FirstSemesterLabel	= addLabel			("First Semester:"		,1,1,1,1);
		SecondSemesterLabel	= addLabel			("Second Semester:"		,7,1,1,1);	
		S1B1Label 			= addLabel			("Block 1: "			,3,5,1,1);
		S1B2Label 			= addLabel			("Block 2: "			,4,5,1,1); 
		S1B3Label  			= addLabel			("Block 3: "			,5,5,1,1);
		S1B4Label 			= addLabel			("Block 4: "			,6,5,1,1);
		S2B1Label  			= addLabel			("Block 1: "			,9,5,1,1);
		S2B2Label  			= addLabel			("Block 2: "			,10,5,1,1);
		S2B3Label  			= addLabel			("Block 3: "			,11,5,1,1);
		S2B4Label 			= addLabel			("Block 4: "			,12,5,1,1);
		S1Q1Label  			= addLabel			("Quarter 1"			,2,6,1,1);
		S1Q2Label 			= addLabel			("Quarter 2"			,2,7,1,1);
		S1Q3Label 			= addLabel			("Quarter 3"			,2,9,1,1);
		S4Q1Label 			= addLabel			("Quarter 4"			,2,10,1,1);
		S2Q1Label 			= addLabel			("Quarter 1"			,8,6,1,1);
		S2Q2Label 			= addLabel			("Quarter 2"			,8,7,1,1);
		S2Q3Label  			= addLabel			("Quarter 3"			,8,9,1,1);
		S2Q4Label 			= addLabel			("Quarter 4"			,8,10,1,1);
		S1MidtermLabel		= addLabel			("Mid-Term"				,2,8,1,1);
		S2MidtermLabel		= addLabel			("Mid-Term"				,8,8,1,1);
		S1FinalLabel		= addLabel			("Final"				,2,11,1,1); 
		S2FinalLabel		= addLabel			("Final"				,8,11,1,1);
		coreGPALabel		= addLabel			("Core GPA:"			,13,5,1,1);
		weightedGPALabel	= addLabel			("Weighted GPA:"		,13,8,1,1);
		cbS1academicB1 		= addCheckBox		("Academic"				,3,1,1,1);
		cbS1academicB2 		= addCheckBox		("Academic"				,4,1,1,1); 
		cbS1academicB3 		= addCheckBox		("Academic"				,5,1,1,1);
		cbS1academicB4 		= addCheckBox		("Academic"				,6,1,1,1);
		cbS2academicB1 		= addCheckBox		("Academic"				,9,1,1,1);
		cbS2academicB2 		= addCheckBox		("Academic"				,10,1,1,1);
		cbS2academicB3 		= addCheckBox		("Academic"				,11,1,1,1);
		cbS2academicB4 		= addCheckBox		("Academic"				,12,1,1,1);
		S1HB1				= addRadioButton	("Honors"				,3,2,1,1);
		S1HB2				= addRadioButton	("Honors"				,4,2,1,1); 
		S1HB3				= addRadioButton	("Honors"				,5,2,1,1); 
		S1HB4				= addRadioButton	("Honors"				,6,2,1,1);
		S2HB1 				= addRadioButton	("Honors"				,9,2,1,1);
		S2HB2 				= addRadioButton	("Honors"				,10,2,1,1);
		S2HB3				= addRadioButton	("Honors"				,11,2,1,1);
		S2HB4 				= addRadioButton	("Honors"				,12,2,1,1);
		S1APB1 				= addRadioButton	("AP"				,3,3,1,1);
		S1APB2  			= addRadioButton	("AP"				,4,3,1,1);
		S1APB3  			= addRadioButton	("AP"				,5,3,1,1);
		S1APB4  			= addRadioButton	("AP"				,6,3,1,1);
		S2APB1  			= addRadioButton	("AP"				,9,3,1,1);
		S2APB2 				= addRadioButton	("AP"				,10,3,1,1);
		S2APB3  			= addRadioButton	("AP"				,11,3,1,1);
		S2APB4  			= addRadioButton	("AP"				,12,3,1,1);
	}

public void buttonClicked(JButton buttonObj) {
		if (buttonObj == submitButton)		
			processInputs();
		
	}
	
	private void processInputs(){
			double coreGPA;
			double weightedGPA;
			int S1block1Points;
			
			S1block1Points = 0;

if (S1Q1B1.getText() == "A") 
				S1block1Points += 8;
			else if (S1Q1B1.getText() == "B")
				S1block1Points += 6;
			else if (S1Q1B1.getText() == "C")
				S1block1Points += 4;
			else if (S1Q1B1.getText() == "D")
				S1block1Points += 2;
			else if (S1Q1B1.getText() == "F")
				S1block1Points += 0;
			
			if (S1Q2B1.getText() == "A") 
				S1block1Points += 8;
			else if (S1Q2B1.getText() == "B")
				S1block1Points += 6;
			else if (S1Q2B1.getText() == "C")
				S1block1Points += 4;
			else if (S1Q2B1.getText() == "D")
				S1block1Points += 2;
			else if (S1Q2B1.getText() == "F")
				S1block1Points += 0;
			
			if (S1Q3B1.getText() == "A")
				S1block1Points += 8;
			else if (S1Q3B1.getText() == "B")
				S1block1Points += 6;
			else if (S1Q3B1.getText() == "C")
				S1block1Points += 4;
			else if (S1Q3B1.getText() == "D")
				S1block1Points += 2;
			else if (S1Q3B1.getText() == "F")
				S1block1Points += 0;
			
			if (S1Q4B1.getText() == "A")
				S1block1Points += 8;
			else if (S1Q4B1.getText() == "B")
				S1block1Points += 6;
			else if (S1Q4B1.getText() == "C")
				S1block1Points += 4;
			else if (S1Q4B1.getText() == "D")
				S1block1Points += 2;
			else if (S1Q4B1.getText() == "F")
				S1block1Points += 0;
			
			if (S1MidTermB1.getText() == "A")
				S1block1Points += 8;
			else if (S1MidTermB1.getText() == "B")
				S1block1Points += 3;
			else if (S1MidTermB1.getText() == "C")
				S1block1Points += 2;
			else if (S1MidTermB1.getText() == "D")
				S1block1Points += 1;
			else if (S1MidTermB1.getText() == "F")
				S1block1Points += 0;
			
			if (S1FinalB1.getText() == "A")
				S1block1Points += 8;
			else if (S1FinalB1.getText() == "B")
				S1block1Points += 3;
			else if (S1FinalB1.getText() == "C")
				S1block1Points += 2;
			else if (S1FinalB1.getText() == "D")
				S1block1Points += 1;
			else if (S1FinalB1.getText() == "F")
				S1block1Points += 0;
			
			
	}
		public static void main (String[] args) {
			GPA theGUI = new GPA();
			theGUI.setSize(1024, 741);
			theGUI.setVisible(true);
		
		
	}
	
}

This message was edited 4 times. Last update was at 05/15/2007 14:01:45

It looks like you've got two main issues. The first and most important is that you don't use == to see if two strings are equal. You'll want to use the equals() method on String to do that. So code like:

should instead be

and so on. Next, you do the work in processInputs() to calculate what S1block1Points is but then you never use that. Do you instead mean to use S1block1Points to set one of the text fields with a setNumber() call? For example, you may want to do:

as the last line in your processInputs() method.

Try that and see where you get.

Thanks alot! That worked. I did not remember the .equals thing, since it's been awhile since I've programmed.

and so on. Next, you do the work in processInputs() to calculate what S1block1Points is but then you never use that. Do you instead mean to use S1block1Points to set one of the text fields with a setNumber() call?

And yes, with all of my messing around trying to fix it, I forgot to re-add where S1block1Points outputs. Though it was in before.

Thanks again!

Yet another trip up in the code. When the radio buttons are selected they arn't adding the .04 to the totalGPA, and it's not displaying the weightedGPA anywhere. I'm not sure if I have my privates, and publics mixed up or what. Or maybe I'm just positioning things wrong.

The whole code:

private int totalPoints;
	private double S1block1Points;
	private double S1block2Points;
	private double S1block3Points;
	private double S1block4Points;
	private double S2block1Points;
	private double S2block2Points;
	private double S2block3Points;
	private double S2block4Points;
	private double coreGPA = 0;
	private double weightedGPA = 0;
	private double S1GPA;
	private double S2GPA;
	private double totalGPA;
	
	//Constructor ---------------------------------------------------------------------
	
	public GPA(){
			,12,1,1,1);
		rbS1HB1				= addRadioButton	("Honors"				,3,2,1,1);
		rbS1HB2				= addRadioButton	("Honors"				,4,2,1,1); 
		rbS1HB3				= addRadioButton	("Honors"				,5,2,1,1); 
		rbS1HB4				= addRadioButton	("Honors"				,6,2,1,1);
		rbS2HB1 			= addRadioButton	("Honors"				,9,2,1,1);
		rbS2HB2 			= addRadioButton	("Honors"				,10,2,1,1);
		rbS2HB3				= addRadioButton	("Honors"				,11,2,1,1);
		rbS2HB4 			= addRadioButton	("Honors"				,12,2,1,1);
		rbS1APB1 			= addRadioButton	("AP"				,3,3,1,1);
		rbS1APB2  			= addRadioButton	("AP"				,4,3,1,1);
		rbS1APB3  			= addRadioButton	("AP"				,5,3,1,1);
		rbS1APB4  			= addRadioButton	("AP"				,6,3,1,1);
		rbS2APB1  			= addRadioButton	("AP"				,9,3,1,1);
		rbS2APB2 			= addRadioButton	("AP"				,10,3,1,1);
		rbS2APB3  			= addRadioButton	("AP"				,11,3,1,1);
		rbS2APB4  			= addRadioButton	("AP"				,12,3,1,1);
	}
	
	public void buttonClicked(JButton buttonObj) {
		if (buttonObj == submitButton)		
			processInputs();
	}
	
	private void processInputs(){

//Block 1 Calculations
			if (S1Q1B1.getText().equals("A"))
				S1block1Points += 8;
			else if (S1Q1B1.getText().equals("B"))
				S1block1Points += 6;
			else if (S1Q1B1.getText().equals("C"))
				S1block1Points += 4;
			else if (S1Q1B1.getText().equals("D"))
				S1block1Points += 2;
			else if (S1Q1B1.getText().equals("F"))
				S1block1Points += 0;
			
			if (S1Q2B1.getText().equals("A"))
				S1block1Points += 8;
			else if (S1Q2B1.getText().equals("B"))
				S1block1Points += 6;
			else if (S1Q2B1.getText().equals("C"))
				S1block1Points += 4;
			else if (S1Q2B1.getText().equals("D"))
				S1block1Points += 2;
			else if (S1Q2B1.getText().equals("F"))
				S1block1Points += 0;
			
			if (S1Q3B1.getText().equals("A"))
				S1block1Points += 8;
			else if (S1Q3B1.getText().equals("B"))
				S1block1Points += 6;
			else if (S1Q3B1.getText().equals("C"))
				S1block1Points += 4;
			else if (S1Q3B1.getText().equals("D"))
				S1block1Points += 2;
			else if (S1Q3B1.getText().equals("F"))
				S1block1Points += 0;
			
			if (S1Q4B1.getText().equals("A"))
				S1block1Points += 8;
			else if (S1Q4B1.getText().equals("B"))
				S1block1Points += 6;
			else if (S1Q4B1.getText().equals("C"))
				S1block1Points += 4;
			else if (S1Q4B1.getText().equals("D"))
				S1block1Points += 2;
			else if (S1Q4B1.getText().equals("F"))
				S1block1Points += 0;
			
			if (S1MidTermB1.getText().equals("A"))
				S1block1Points += 4;
			else if (S1MidTermB1.getText().equals("B"))
				S1block1Points += 3;
			else if (S1MidTermB1.getText().equals("C"))
				S1block1Points += 2;
			else if (S1MidTermB1.getText().equals("D"))
				S1block1Points += 1;
			else if (S1MidTermB1.getText().equals("F"))
				S1block1Points += 0;
			
			if (S1FinalB1.getText().equals("A"))
				S1block1Points += 4;
			else if (S1FinalB1.getText().equals("B"))
				S1block1Points += 3;
			else if (S1FinalB1.getText().equals("C"))
				S1block1Points += 2;
			else if (S1FinalB1.getText().equals("D"))
				S1block1Points += 1;
			else if (S1FinalB1.getText().equals("F"))
				S1block1Points += 0;
			//End Block 1
			//-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-
			
all of the if-else contine several times, too much code to copy
			
			S1GPA = ((S1block1Points + S1block2Points + S1block3Points + S1block4Points) / 10);
			S2GPA = ((S2block1Points + S2block2Points + S2block3Points + S2block4Points) / 10);
			totalGPA = ((S1GPA + S2GPA) / 8);
			
			
			}
	public void weightedGPA() {
		//Calculate Honors weight
		if (rbS1HB1.isSelected())
			weightedGPA = totalGPA + .04;
		if (rbS1HB2.isSelected())
			weightedGPA = totalGPA + .04;
		if (rbS1HB3.isSelected())
			weightedGPA = totalGPA + .04;
		if (rbS1HB4.isSelected())
			weightedGPA = totalGPA + .04;
		if (rbS2HB1.isSelected())
			weightedGPA = totalGPA + .04;
		if (rbS2HB2.isSelected())
			weightedGPA = totalGPA + .04;
		if (rbS2HB3.isSelected())
			weightedGPA = totalGPA + .04;
		if (rbS2HB4.isSelected())
			weightedGPA = totalGPA + .04;
		
		checkerField.setNumber(weightedGPA);
	}
		public static void main (String[] args) {
			GPA theGUI = new GPA();
			theGUI.setSize(1024, 741);
			theGUI.setVisible(true);
		
		
	}
	
	}

I'm not sure what is going on now - it looks good. I'd encourage you to add some debugging statements to see if things are working the way you expect. Something like:

That would help you to be sure that the code you expect is getting called.

I found my problem and fixed it. I now have the whole GPAfinder program working.

Though, my last and final question is:

Is there a way just to open the GUI? Not open the whole project in Eclipse.

(The problem was..

You'll want to run the program on the command line. Your machine needs to have the Java JDK on your command line path. The instructions to do this vary based on your operating system. The Java SE 6 platform install release notes go into detail about this.

After it is installed, you can compile with

and run it with

assuming that you're in the same directory as your GPA.java file.

Take a look at the docs - they do a better job going into the details.