Help with errors again

[LeeAnne Murphy]

Errors in code are as follows:
C:\Users\LeeAnne\Documents\NetBeansProjects\week3mortcalcjp2\src\week3mortcalcjp2\Week3mortcalcjp2.java:274: invalid method declaration; return type required
C:\Users\LeeAnne\Documents\NetBeansProjects\week3mortcalcjp2\src\week3mortcalcjp2\Week3mortcalcjp2.java:274: invalid method declaration; return type required
public setSize() {
1 error
C:\Users\LeeAnne\Documents\NetBeansProjects\week3mortcalcjp2\nbproject\build-impl.xml:603: The following error occurred while executing this line:
C:\Users\LeeAnne\Documents\NetBeansProjects\week3mortcalcjp2\nbproject\build-impl.xml:245: Compile failed; see the compiler error output for details.

package week3mortcalcjp2;

        import java.awt.Component;
        import javax.swing.JButton;             // Code to Import Use of Buttons
	import javax.swing.JLabel;    		// Code to Import Use of Label
	import javax.swing.JPanel;     		// Code to Import Use of Panel
	import java.awt.BorderLayout;           // Code to Import and Provide Window Layout
        import java.awt.FlowLayout;
	import java.awt.GridLayout;             // Code to Import Grid Format Layout
	import java.awt.event.ActionEvent;      // Code to Import Action Event
	import java.awt.event.ActionListener;   // Code to Import Action Listener
	import javax.swing.JTextField;      	// Code to Import Use of Text Field
	import javax.swing.JTextArea;      	// Code to Import Use of Text Area
	import javax.swing.JScrollPane;         // Code to Import Scroll Pane
	import javax.swing.JComboBox;           // Code to Import JCombo Box
	import java.text.DecimalFormat;         // Code to Import Decimal Format    
        import javax.swing.JFrame;
/**
 *
 * Java Programming 2 week 3
 * August 22, 2011
 * Write the program in Java (with a graphical user interface) and have it calculate and
 * display the mortgage payment amount from user input of the amount of the mortgage and
 * the user's selection from a menu of available mortgage loans:
	 
	 * - 7 years at 5.35%
	 * - 15 years at 5.5%
	 * - 30 years at 5.75%
	 *
 * Use an array for the mortgage data for the different loans. Display the mortgage
 * payment amount followed by the loan balance and interest paid for each payment over
 * the term of the loan. Allow the user to loop back and enter a new mount and make a
 * new selection or quit. Please insert comments in the program to document the program.
	 *
	 */

public class Week3mortcalcjp2 extends JFrame implements ActionListener
{
    private static JLabel lMessages;

   // Create Mortgage Payment Class
    private static Component pLabels;
    private static Component pTextFields;
        
 
  DecimalFormat Dollar = new DecimalFormat("$###,###.00");
	 
	    // Beginning of Main
            
   Week3mortcalcjp2 newLoan = new Week3mortcalcjp2();  
		// Create Instance of Class and do all stuff to be done
    
   	
    // Declare Variables and Values
	    double principalBalance;       // Amount of Loan
	    double loanTerm;               // Term of Loan
	    double loanRate;               // Interest Rate of Loan
	    double interestMonthly;        // Amount of Monthly Interest
	    double monthlyPayment;         // Amount of Monthly Payment
	    double monthlyInterestRate;    // Amount of Monthly Interest Rate
	    double monthlyPrinciple;       // Amount of Monthly Principle
	    double loanBalance;            // Balance of Loan
	 
	    // GUI elements to declare input Panel
	    private JLabel labelPrincipal = new JLabel("Mortgage Principal:");
	    private JTextField txtPrincipal = new JTextField(10);
	    private JLabel labelPlan = new JLabel("Mortgage Plan:");
	    String loanPlan[] = {"Select Loan", "7 years at 5.35%", "15 years at 5.5%", "30 years at 5.75%"};
	    private JComboBox comboBox = new JComboBox(loanPlan);
	    private JTextArea result = new JTextArea(10, 20);
	    JScrollPane scrollResult = new JScrollPane(result);
            private boolean exp;
	 
    public static void main(String[] args) {
     
    
   //Declares button to trigger the calculation
    
	        final JButton buttonCalc = new JButton("Calculate");
	        buttonCalc.addActionListener(new ActionListener() {
	 
                @Override
	            public void actionPerformed(ActionEvent e) {
	                onButtonCalculate();
	            }
	        });
	 
	        //Declares button to clear (reset) the data
	        final JButton buttonClear = new JButton("Clear");
	        buttonClear.addActionListener(new ActionListener() {
	 
                @Override
	            public void actionPerformed(ActionEvent e) {
	                onButtonreset();
	            }
	        });
	 
	        //Separates panel used for messages to user
                
                JPanel pMessages = new JPanel();
	        pMessages.setLayout(new FlowLayout());
                pMessages.add(lMessages = new JLabel("Calculate Mortgage"));
                pMessages.setLayout(new GridLayout(6, 4, 6, 6));
	        pMessages.add(new JLabel("Enter Loan Amount"));
	        pMessages.add(new JLabel("Enter Interest Rate (percentage)"));
                pMessages.add(new JLabel("Enter Loan Length (years)"));
	        pMessages.add(new JLabel("Your Monthly Payment is:"));
	        	 
	        //Sets the layout
	        JPanel pData = new JPanel();
                pData.setLayout(new BorderLayout());
	        pData.add(pMessages, BorderLayout.CENTER);
	        pData.add(pTextFields, BorderLayout.SOUTH);
	 
	        }
	 
	// Calculate Information
	    public void onButtonCalculate() {
	        // Assign Amount of Loan
	        double loanAmt = Double.parseDouble(txtPrincipal.getText());
	        assignLoanAmount(loanAmt);
	 
	        //Select and assign loan rate and loan term
	        String str = (String) comboBox.getSelectedItem();
	        if (str.equals("7 years at 5.35%")) {
	            assignLoanRate(5.35);
	            assignLoanTerm(7 * 12);
	        } else if (str.equals("15 years at 5.5%")) {
	            assignLoanRate(5.5);
	            assignLoanTerm(15 * 12);
	        } else if (str.equals("30 years at 5.75%")) {
	            assignLoanRate(5.75);
	            assignLoanTerm(30 * 12);
	        }
	 
	        calcMonthlyPayment();               // Calculate Monthly Payment of Loan
	        calcMonthlyInterest();              // Calculate Monthly Interest on Loan
	        calcMonthlyPrinciple();             // Calculate Monthly Principle of Payment
	        calcLoanBalance();                  // Calculate Loan Balance
	        calculateNewBalance();              // Calculate New Balance Each Month
	        displayAmounts();                   // Display All Amounts
	        monthlyLoanAmount();
	        monthlyBalanceInterest(getAssignLoanAmount(), getAssignLoanTerm(), getAssignLoanRate());
	    }
	 
	// Information for Button
	    public void onButtonreset() {
	        txtPrincipal.setText("");
	        comboBox.setSelectedIndex(0);
	        result.setText("");
	    }
	 
	    // Display Methods to Print Values for Morgage Calculator
	    public void assignLoanAmount(double loanAmount) {
	        principalBalance = loanAmount;
	    }
 
		public void assignLoanRate(double interestAmount) {
	        loanRate = interestAmount;
	    }
	 
	    public void assignLoanTerm(double termAmount) {
	        loanTerm = termAmount;
	    }
	 
	    public double getAssignLoanAmount() {
	        return principalBalance;
	    }
	 
	    public double getAssignLoanRate() {
	        return loanRate;
	    }
	 
	    public double getAssignLoanTerm() {
	        return loanTerm;
	    }
	 
	    public void calcMonthlyPayment() {
	        interestMonthly = (loanRate / 100) / 12;
	        monthlyPayment = principalBalance * ((interestMonthly * (Math.pow((1 + interestMonthly), loanTerm)))
	                / (Math.pow((1 + interestMonthly), loanTerm) - 1));
	    }
	 
	    public void calcMonthlyInterest() {
	        monthlyInterestRate = (int) ((loanRate / 12) / 100);
	        interestMonthly = principalBalance * monthlyInterestRate;
	    }
	 
	    public void calcMonthlyPrinciple() {
	        monthlyPrinciple = monthlyPayment - interestMonthly;
		}
	 
	    public void calcLoanBalance() {
	        loanBalance = principalBalance - monthlyPrinciple;
		}
	 
	    public void calculateNewBalance() {
	        loanBalance -= monthlyPrinciple;
	    }
	 
	    public void monthlyLoanAmount() {
	        result.append("Month\tLoanBalance\tInterest Paid\n");
	        double newLoanTerm = loanTerm;
	        while (newLoanTerm > 0) {
	            double loanMonths = loanTerm - newLoanTerm;
	 
	            double newMonthlyBalance = principalBalance * ((Math.pow((1 + monthlyInterestRate), loanTerm))
	                    - (Math.pow((1 + monthlyInterestRate), loanMonths))) / (Math.pow((1 + monthlyInterestRate), loanTerm) - 1);
	 
	            double monthlyamountInterest = newMonthlyBalance * monthlyInterestRate;
	 
	            result.append("" + Math.round(newLoanTerm) + "\t" + Dollar.format(newMonthlyBalance)
                    + "\t" + Dollar.format(monthlyamountInterest) + "\n");
	 
	            newLoanTerm -= 1;
	        }
	    }
	 
    public void monthlyBalanceInterest(double loan, double term, double interestRate) {
	 
        // Format Decimal Point
        DecimalFormat df = new DecimalFormat("$###,###.00");
	 
      // Get Monthly Interest Rate
       double monthlyRate = (interestRate / (12 * 100));
     // Calculate Discount Factor
       double discountFactor = (Math.pow((1 + monthlyRate), term) - 1) / (monthlyRate * Math.pow((1 + monthlyRate), term));
     // Calculate Monthly Payment
        double payment1 = loan / discountFactor;
 
	        // Display Program Results
	        result.append("\nLoan Amount = " + df.format(loan));
	        result.append("\nInterest Rate = " + interestRate + "%");
	        result.append("\nLength of Loan = " + Math.round(term / 12) + " years");
	        result.append("\nMonthly Payment = " + df.format(payment1) + "\n");
	    }
	 
 public void displayAmounts() {
  System.out.println("Loan Amount = " + Dollar.format(principalBalance));            // Show Original Loan Amount
        String loan = null;
  System.out.println("Interest Rate = " + loan + "%");                           // Show Interest Rate
  System.out.println("Length of Loan = " + Math.round(loanTerm) + " months");        // Show Length of Loan
  System.out.println("Monthly Payment = " + Dollar.format(monthlyPayment));          // Show Monthly Payment
	    }

    private void setLayout(BorderLayout borderLayout) {
        throw new UnsupportedOperationException("Not yet implemented");
    }

    private void add(JPanel panel, String CENTER) {
        throw new UnsupportedOperationException("Not yet implemented");
    }

    private void add(JScrollPane scrollResult, String SOUTH) {
        throw new UnsupportedOperationException("Not yet implemented");
    }

   
      Week3mortcalcjp2 frame = new Week3mortcalcjp2();

    @Override
    public void actionPerformed(ActionEvent e) {
        throw new UnsupportedOperationException("Not supported yet.");
    }
    
    
    static class setSize {
    
    
   class frame {

     public setSize() {    
   //Set frame title
                frame.setSize(400,400); 
                frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	        frame.setResizable(false);
	        frame.setTitle("McBride Mortgage Calculator");
	        frame.setVisible(true);
   }

    @Override
    public void actionPerformed(ActionEvent e) {
        throw new UnsupportedOperationException("Not supported yet.");
    }
        }
   }
}

[Unnar Björnsson]

C:\Users\LeeAnne\Documents\NetBeansProjects\week3mortcalcjp2\src\week3mortcalcjp2\Week3mortcalcjp2.java:274: invalid method declaration; return type required
C:\Users\LeeAnne\Documents\NetBeansProjects\week3mortcalcjp2\src\week3mortcalcjp2\Week3mortcalcjp2.java:274: invalid method declaration; return type required
public setSize() {
1 error

You must specify a return type for every method declaration, if the method is not supposed to return anything you must put void instead.
So

public setSize()

becomes

public void setSize()

[Paul Clapham]

Layman's term for "return type": what kind of thing does is this method supposed to return?

Your code:

public setSize() { 

doesn't say what kind of thing the setSize() method is supposed to return when you call it. So there's something missing right here:

public XXXXXX setSize() { 

Now it's quite possible the method isn't supposed to return anything. Maybe it's just supposed to do something and that's all. In which case there's a special value you are supposed to put in there. So let me suggest you check back in your text book, or whatever resource you are using to learn Java, and see if you can find what that special value might be.

[Campbell Ritchie]

When you use the code button, you get a pair of tags like this [code][/code]. Put the code after the first ].

Is that the only error you are suffering? The message is quite clear; you ought to give it a return type. Not public setSize() but public void setSize().
Don't write so much code and compile it. You should be compiling after every 5 lines or so. That will make it so much easier for you to find errors.

[LeeAnne Murphy]

Ok changed that and now the lines below it are giving me errors.

     public void setSize() {    
   //Set frame title
                frame.setSize(400,400); 
                frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	        frame.setResizable(false);
	        frame.setTitle("McBride Mortgage Calculator");
	        frame.setVisible(true);
   }

Here is the error.

non-static variable frame cannot be referenced from a static context

[Campbell Ritchie]

Why have you got that static nested class in the first place?

[LeeAnne Murphy]

I'm really sorry...I don't understand what you are asking.

Why have you got that static nested class in the first place? ( I don't understand static statements)

[Matthew Brown]

I think the real cause of the error is getting obscured there. Looking at your original code, you've got a static class called setSize on line 268. Is that intended? It doesn't look right, and if it really is supposed to be a class it should start with a capital.

Then within that, you've got another class frame (again, should be Frame to follow standard conventions). Having an inner class inside an inner class is a bad idea even if it's possible (and I don't think it is). So maybe you need to rethink what that's supposed to be doing. Later on you try and treat frame like a variable rather than a class.

(I think. It's hard to read it accurately when your indentation is inconsistent).

Campbell's right - you need to try and keep your code simpler until you can get issues like this sorted out. Make sure you understand what each part of the code is supposed to be doing, rather than just trying to fix one error at a time.

[Unnar Björnsson]

I suggest you read this. The first line below the Static Nested Classes should answer your question why you are getting the error, but I'd study the whole article and familiarize yourself with the static keyword.
I also suspect you are a few steps ahead of yourself, which will only make things harder on you, good luck

[Campbell Ritchie]

I think you have inadvertently created a nested (static) class inside your class. I suspect you don't want that class at all; maybe you clicked on the wrong box in NetBeans and it slipped that nested class in while you weren't watching.

That is another reason why we often recommend you avoid IDEs when you are starting.

[Paul Clapham]

Matthew Brown wrote:Having an inner class inside an inner class is a bad idea even if it's possible (and I don't think it is).

It is actually possible and I know that because I have done it. On purpose, too. But yeah... not really recommended. Especially for the beginner.

And as Campbell said, if you find that you did it accidentally then your code is a bit out of control. Or more likely a lot out of control.