Thread: Displaying an hourglass 2

Hello,

I am working on the same code to produce the Hour Glass figure.

My code is a little bit different because I am using a CONSTANT to change the size of the Hour Glass. I have my CONSTANT set to 4. The problem I am having is in my method for the BOTTOM HALF.
I am getting an error message that says this.

HourGlass.java:56: error: class, interface, or enum expected
	public static void drawBottomHalf()

What I don't understand is that to me it look like the same as the Top Half but with reverse order so I don't understand why I am getting this error.

Here is my code.

public class HourGlass
{
	public static final int HEIGHT = 4;
 
	public static void main(String[] args)
	{
		drawTop();
		drawTopHalf();
		drawVertical();
		drawBottomHalf();
		drawTop();
	}
 
	// Produces the top part of the hour glass
	public static void drawTop()
	{
		System.out.print("|");
		for (int i = 1; 1 <= (2 * HEIGHT + 1); i++)
		{
			System.out.print("\"");
		}
		System.out.println("|");
	}
 
	// Produces the top half of the hour glass
	public static void drawTopHalf()
	{
		for (int line = 1; line <= HEIGHT; line++)
		{
			for (int i = 1; i <= (line-1); i++)
			{
			System.out.print(" ");
			}
			System.out.print("\\");
			int dots = 2 * HEIGHT - 2 * line;
			for (int i = 1; i <= dots; i++)
			{
				System.out.print(":");
			}
			System.out.print("/");
			}
	}
	// Produces the Vertical two line in the center
	public static void drawVertical()
	{
		System.out.print("|");
		System.out.print(" ");
		system.out.print("|");
		}
	}
	// Produces the bottom half of the hour glass
	public static void drawBottomHalf()
	{
		for (int line = 1; line <= HEIGHT; line++)
		{
			for (int i = 1; i <= (HEIGHT - line); i++)
			{
				System.out.print(" ");
			}
			System.out.print("/")
			for (int i = 1; i <= 2 * (line-1); i++)
			{
				System.out.print(":");
			}
			System.out.print("\\");
			for (int i = 1; i <= (HEIGHT - line);
			{
				System.out.print(" ");
			}
		}
	}
}

Hello Everyone,

I am trying to print out an Hour Glass figure using CONSTANTS so that I can change sizes from 4 to 10. The problem I am having is that I am getting a error message for my method called for on the bottom half.
Here is the error message.

HourGlass.java:56: error: class, interface, or enum expected
	public static void drawBottomHalf()
	              ^
HourGlass.java:58: error: class, interface, or enum expected
		for (int line = 1; line <= HEIGHT; line++)

Here is my code for the hour glass figure.

public class HourGlass
{
	public static final int HEIGHT = 4;
 
	public static void main(String[] args)
	{
		drawTop();
		drawTopHalf();
		drawVertical();
		drawBottomHalf();
		drawTop();
	}
 
	// Produces the top part of the hour glass
	public static void drawTop()
	{
		System.out.print("|");
		for (int i = 1; 1 <= (2 * HEIGHT + 1); i++)
		{
			System.out.print("\"");
		}
		System.out.println("|");
	}
 
	// Produces the top half of the hour glass
	public static void drawTopHalf()
	{
		for (int line = 1; line <= HEIGHT; line++)
		{
			for (int i = 1; i <= (line-1); i++)
			{
			System.out.print(" ");
			}
			System.out.print("\\");
			int dots = 2 * HEIGHT - 2 * line;
			for (int i = 1; i <= dots; i++)
			{
				System.out.print(":");
			}
			System.out.print("/");
			}
	}
	// Produces the Vertical two line in the center
	public static void drawVertical()
	{
		System.out.print("|");
		System.out.print(" ");
		system.out.print("|");
		}
	}
	// Produces the bottom half of the hour glass
	public static void drawBottomHalf()
	{
		for (int line = 1; line <= HEIGHT; line++)
		{
			for (int i = 1; i <= (HEIGHT - line); i++)
			{
				System.out.print(" ");
			}
			System.out.print("/")
			for (int i = 1; i <= 2 * (line-1); i++)
			{
				System.out.print(":");
			}
			System.out.print("\\");
			for (int i = 1; i <= (HEIGHT - line);
			{
				System.out.print(" ");
			}
		}
	}

I can't seem to see how this bottom half of the method is different from the top have of the method. So I don't understand what the error is or why it is giving me this error.

Any suggestions on how to fix this would be great.

Thanks.

Methods should be inside classes but NOT inside other methods. Check that the '{' and '}' match up correctly.

I think Norm helped, but please don't make multiple posts for the same topic. Threads merged into one.

Ok that worked. I had one too many } brackets above it.

Thanks

--- Update ---

Well,

Once I fix the bracket issue for my Bottom Half method. I tried to finish the code by adding the top part of the method again to the bottom. Now I am getting an error that says

HourGlass.java:79: error: illegal start of expression
	public static void drawTop()
	^
HourGlass.java:79: error: illegal start of expression
	public static void drawTop()

I have move and added brackets above the expression and to me it seems that all my methods are lined up under class and that each method is closed off. Unless I am not see it. Here is my code with the added Top half to the bottom of it.

public class HourGlass
{
	public static final int HEIGHT = 4;
 
	public static void main(String[] args)
	{
		drawTop();
		drawTopHalf();
		drawVertical();
		drawBottomHalf();
		drawTop();
	}
 
	// Produces the top part of the hour glass
	public static void drawTop()
	{
		System.out.print("|");
		for (int i = 1; 1 <= (2 * HEIGHT + 1); i++)
		{
			System.out.print("\"");
		}
		System.out.println("|");
	}
 
	// Produces the top half of the hour glass
	public static void drawTopHalf()
	{
		for (int line = 1; line <= HEIGHT; line++)
		{
			for (int i = 1; i <= (line-1); i++)
			{
			System.out.print(" ");
			}
			System.out.print("\\");
			int dots = 2 * HEIGHT - 2 * line;
			for (int i = 1; i <= dots; i++)
			{
				System.out.print(":");
			}
			System.out.print("/");
			}
	}
 
	// Produces the Vertical two line in the center
	public static void drawVertical()
	{
		System.out.print("|");
		System.out.print(" ");
		system.out.print("|");
 
	}
 
	// Produces the bottom half of the hour glass
	public static void drawBottomHalf()
	{
		for (int line = 1; line <= HEIGHT; line++)
		{
			for (int i = 1; i <= (HEIGHT - line); i++)
			{
				System.out.print(" ");
			}
			System.out.print("/");
			for (int i = 1; i <= 2 * (line-1); i++)
			{
				System.out.print(":");
			}
			System.out.print("\\");
			for (int i = 1; i <= (HEIGHT - line); i++)
			{
			System.out.print(" ");
 
	}
 
	// Produces the top part of the hour glass
	public static void drawTop()
	{
		System.out.print("|");
		{
		for (int i = 1; 1 <= (2 * HEIGHT + 1); i++)
		{
			System.out.print("\"");
		}
		System.out.println("|");
	}
}

yeah I have tried moving adding and deleting {} and I can not get it to work. I don's understand this simple process of {}. I know it is used to group together related bits of code. I open and closed off my other method so why I am getting this error.

There are one too many '{'s in the last drawTop() method, AND there are too many drawTop() methods. A single method called drawTop() can be called multiple times, but multiple methods called drawTop() is not allowed.

There are TWO too few '}'s in drawBottom().

There is also one 'system' that should be "System".

What editor are you using?

jGRASP

I don't know it, but I found this tip that might be helpful:

Cleaning-Up Indenting: jGrasp does not have a command that re-indents code, but you can trick it into doing so.
Click on View+Generate CSD (or the button).
Click on View+Remove CSD (or the button).

Since the graphics are lost, the tip is here.