[java] problem with return statement and placement of {}'s - Tilted Forum Project Discussion Community

merkerguitars · 11-15-2004, 12:01 PM

Ok here is what I have in my Test class

Code:

package a4; 

/** 
* <p>Title: Assignment 4</p> 
* <p>Description: CS 161 Assignment 4 </p> 
* <p>Copyright: Copyright (c) 2004</p> 
* <p>Company: UW-River Falls</p> 
* @author Eric Merker 
* @email ericmerker@centurytel.net 
* @date started: Novemeber 9th 2004 finished: ???? 
* @team prometheusfree.org crew visit us at irc.shadowfire.org at #prometheus 
* @version 1.0 
 */ 


import javax.swing.*; 

public class Test { 
  public Test() { 
  } 

      public static void main(String[] args) { 
      Test test1 = new Test(); 

      int x = Integer.parseInt(JOptionPane.showInputDialog(null, "Enter a number")); 

      } 
  }

And this is what I have in my convert class

Code:

/** 
* <p>Title: Assignment 4</p> 
* <p>Description: CS 161 Assignment 4 </p> 
* <p>Copyright: Copyright (c) 2004</p> 
* <p>Company: UW-River Falls</p> 
* @author Eric Merker 
* @email ericmerker@centurytel.net 
* @date started: Novemeber 9th 2004 finished: ???? 
* @team prometheusfree.org crew visit us at irc.shadowfire.org at #prometheus 
* @version 1.0 
*/ 

import javax.swing.*; 

public class Convert { 
public void print(int x); 
int printPower = x; {{
//missing an open {, and a return type on this line 
if (paraMeter > 12) System.out.println("Does not compute");
 switch (x) { 

case 1: System.out.println("Ten"); 
break; 

case 2: System.out.println("Hundred"); 
break; 

case 3: System.out.println("Thousand"); 
break; 

case 4: System.out.println("Ten Thousand"); 
break; 

case 5: System.out.println("Hundred Thousand"); 
break; 

case 6: System.out.println("Million"); 
break; 

case 7: System.out.println("Ten Million"); 
break; 

case 8: System.out.println("Hundred Million"); 
break; 

case 9: System.out.println("Billion"); 
break; 

case 10: System.out.println("Ten Billion"); 
break; 

case 11: System.out.println("Hundred Billion"); 
break; 

case 12: System.out.println("Trillion"); 
break; }}}

the convert class is the one i'm having problems with, basically I just can't get the return statement correct and the placement of the {}s what the problem does is it takes an input x adn then puts that input into the convert class which matches it up with the proper case statemnet.

welshbyte · 11-15-2004, 03:52 PM

I dont usually do people's homework for them, especially when the attempt looks pretty appauling like this. But, i'm a softy and i need the practice so... does this help:

Code:

import javax.swing.*; 

public class Convert {
	private int x;

	public Convert(int x) {
		this.x = x;
	}

	public void print {
		if (x > 12) {
			System.out.println("Does not compute");
		} else {
			switch (x) {
				case 1: System.out.println("Ten");
					break;
				case 2: System.out.println("Hundred");
					break;
				case 3: System.out.println("Thousand");
					break;
				case 4: System.out.println("Ten Thousand");
					break;
				case 5: System.out.println("Hundred Thousand");
					break;
				case 6: System.out.println("Million");
					break;
				case 7: System.out.println("Ten Million");
					break;
				case 8: System.out.println("Hundred Million");
					break;
				case 9: System.out.println("Billion");
					break;
				case 10: System.out.println("Ten Billion");
					break;
				case 11: System.out.println("Hundred Billion");
					break;
				case 12: System.out.println("Trillion");
					break;
			}
		}
	}
}

BTW, please note i havent bothered testing it.

oblar · 11-15-2004, 08:37 PM

Get in the habit of indenting properly ;p Proper indention makes it so easy to spot missing brackets or too many brackets.

pretty much, if you ever have more than one { or } on a line you should work on your formatting. Otherwise you will hit a snag and it will be harder to spot. This comment is mainly for you at the beginning, until you get used to matching up your { } without thinking

11-15-2004, 08:37 PM	#3 (permalink)
oblar Über-Rookie Location: No longer, D.C	Get in the habit of indenting properly ;p Proper indention makes it so easy to spot missing brackets or too many brackets. pretty much, if you ever have more than one { or } on a line you should work on your formatting. Otherwise you will hit a snag and it will be harder to spot. This comment is mainly for you at the beginning, until you get used to matching up your { } without thinking