1 do while loop gives me 34 errors? ):

[dragone227]

Hey guys! I'm working on an assignment for school, and I need a little bit of help.
This is my current code: [Java] import java.io.*; import java.util.*; public class Assignment{ - Pastebin.com

I want to make it that if the user enters something that is greater than 20 for shape then it outputs "invalid choice"..and it gives the user another try. I have the invalid choice part down, but I can't get it to give the user another try. I tried the do while loop in the "horizontal line" part of the section, it gave me 30 errors, if I take out the loop everything works perfectly fine. Any help appreciated, thanks! :)

[Reeling]

Apart from the horrific code formatting, you start the do while loop before you properly start the if statement.

The reason the forum has code tags is so that you can post your code in the forum.

[dragone227]

Sorry, this is how we were taught in school...and first time using this forum sorry again!
Can you suggest how I could fix this?

[Reeling]

dont start the do while loop inside the conditions of the if statement.

[penguinCoder]

You really need to choose a type of formatting and just stick with it throughout the code. It seems like you change styles about 5 different times.
You do this for part of it:

for(int c = 0; c<i;c++)
{
  System.out.print("*");
}

And then you switch it to something like this:

for (int j = 1; j<=spaces;j++ ){System.out.print(" ");}

And then to this:

for(int c=1; c<= length; c++){
  System.out.println("*");}

And I don't even know what this is:

if (shape == 1)
  do{
    {

And lastly, to this:

if (length > 20)
  {System.out.println("Invalid dimension! The dimension must be between 1 and 20.");}

I mean, do you get what I am saying?.. I feel like I am going crosseyed trying to make sense of this. You really got to go through this code and fix some of this...

Stick to a one of these two formatting techniques.
One:

if(condition){
  ...code...
}

Two:

if(condition)
{
  ...code...
}

It seems like you are trying to save space in all the wrong areas.. I am not trying to be mean, but this is like someone typing English like:
Ho w iS IT g oING toD aY??

It is VERY hard to read.