do while loop

Question

5.00/5 (1 vote)

See more:

I am trying to figure out how to do a do while loop for my project in my c# class. We have not covered any of the loops yet, but he asked us to try. I have no idea what I am doing wrong. I am wanting it to ask the user if they would like to continue if they answer no then the program should terminate.

C#

string stateAbbrev;

            Console.WriteLine("Enter the state abbreviation. ");
            Console.WriteLine("Its full name will" + " be displayed");
            Console.WriteLine();
            stateAbbrev = Console.ReadLine();
            stateAbbrev = stateAbbrev.ToUpper();    //Converts the characters to uppercase

            switch (stateAbbrev)
            {

                case "AL": Console.WriteLine("Alabama");
                    break;

                case "AK": Console.WriteLine("Alaska");
                    break;

                case "AZ": Console.WriteLine("Arizona");
                    break;

                case "AR": Console.WriteLine("Arkansas");
                    break;

                case "CA": Console.WriteLine("California");
                    break;

                case "CO": Console.WriteLine("Colorado");
                    break;

                case "CT": Console.WriteLine("Connecticut");
                    break;

                case "DE": Console.WriteLine("Delaware");
                    break;

                case "FL": Console.WriteLine("Florida");
                    break;

                case "GA": Console.WriteLine("Georgia");
                    break;

                case "HI": Console.WriteLine("Hawaii");
                    break;

                case "ID": Console.WriteLine("Idaho");
                    break;

                case "IL": Console.WriteLine("Illinois");
                    break;

                case "IN": Console.WriteLine("Indiana");
                    break;

                case "IA": Console.WriteLine("Iowa");
                    break;

                case "KS": Console.WriteLine("Kansas");
                    break;

                case "KY": Console.WriteLine("Kentucky");
                    break;

                case "LA": Console.WriteLine("Louisiana");
                    break;

                case "ME": Console.WriteLine("Maine");
                    break;

                case "MD": Console.WriteLine("Maryland");
                    break;

                case "MA": Console.WriteLine("Massachusetts");
                    break;

                case "MI": Console.WriteLine("Michigan");
                    break;

                case "MN": Console.WriteLine("Minnesota");
                    break;

                case "MS": Console.WriteLine("Mississippi");
                    break;

                case "MO": Console.WriteLine("Missouri");
                    break;

                case "MT": Console.WriteLine("Montana");
                    break;

                case "NE": Console.WriteLine("Nebraska");
                    break;

                case "NV": Console.WriteLine("Nevada");
                    break;

                case "NH": Console.WriteLine("New Hampshire");
                    break;

                case "NJ": Console.WriteLine("New Jersey");
                    break;

                case "NM": Console.WriteLine("New Mexico");
                    break;

                case "NY": Console.WriteLine("New York");
                    break;

                case "NC": Console.WriteLine("North Carolina");
                    break;

                case "ND": Console.WriteLine("North Dakota");
                    break;

                case "OH": Console.WriteLine("Ohio");
                    break;

                case "OK": Console.WriteLine("Oklahoma");
                    break;

                case "OR": Console.WriteLine("Oregon");
                    break;

                case "PA": Console.WriteLine("Pennsylvania");
                    break;

                case "RI": Console.WriteLine("Rhode Island");
                    break;

                case "TN": Console.WriteLine("Tennessee");
                    break;

                case "TX": Console.WriteLine("Texas");
                    break;

                case "UT": Console.WriteLine("Utah");
                    break;

                case "VT": Console.WriteLine("Vermont");
                    break;

                case "VA": Console.WriteLine("Virginia");
                    break;

                case "WA": Console.WriteLine("Washington");
                    break;

                case "WV": Console.WriteLine("West Virginia");
                    break;

                case "WI": Console.WriteLine("Wisconsin");
                    break;

                case "WY": Console.WriteLine("Wyoming");
                    break;

                default: Console.WriteLine("No match");
                    break;


                    do
                    {
                        Console.WriteLine("Would you like to try again? ", answer);
                        counter = Convert.ToString(Console.ReadLine);
                    }
                    while (anwer > yes);


            }
            Console.WriteLine("\n");
            Console.WriteLine("\n");
            Console.Write("press -1 to Exit:");
            Int32.Parse(Console.ReadLine());
        }
    }
}

Posted 21-May-10 14:52pm

sloshedmr_bones

Updated 24-May-10 3:31am

Ankur^\m/

v2

Add a Solution

4 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Christian Graus · Answer 1 · 2010-05-21T14:56:00

sloshedmr_bones wrote:
do
{
Console.WriteLine("Would you like to try again? ", answer);
counter = Convert.ToString(Console.ReadLine);
}
while (anwer > yes);

The braces define your loop. The while goes back to where it says do. Notice that the rest of your code, the stuff you want to run to 'try again' is outside the do/while loop and so will not be run. Also note that > is not an acceptable test for strings. != is the test for if answer does not equal yes. If yes is a constant, not a variable, it needs to be in quotes. If you want a do while loop to ever terminate, it needs to be checking a variable that is changed within the loop ( in this case, counter, not answer ). Console.ReadLine is a method, it needs () after it. It returns a string. Convert.ToString is not needed.

This code should not even compile, you should learn to read error messages and diagnose them, and post them here along with the line that generates them, if you want us to fix them for you.

Johnny J. · Answer 2 · 2010-05-24T03:27:00

You need to have basically all your code inside the loop. Something like this:

C#

string stateAbbrev;
            string answer;

            do
            {
                Console.WriteLine("Enter the state abbreviation. ");
                Console.WriteLine("Its full name will" + " be displayed");
                Console.WriteLine();

                stateAbbrev = Console.ReadLine();

                stateAbbrev = stateAbbrev.ToUpper();    //Converts the characters to uppercase

                switch (stateAbbrev)
                {
                    case "AL": Console.WriteLine("Alabama");
                        break;
                    case "AK": Console.WriteLine("Alaska");
                        break;
                    case "AZ": Console.WriteLine("Arizona");
                        break;
                    case "AR": Console.WriteLine("Arkansas");
                        break;
                    case "CA": Console.WriteLine("California");
                        break;
                    case "CO": Console.WriteLine("Colorado");
                        break;
                    case "CT": Console.WriteLine("Connecticut");
                        break;
                    case "DE": Console.WriteLine("Delaware");
                        break;
                    case "FL": Console.WriteLine("Florida");
                        break;
                    case "GA": Console.WriteLine("Georgia");
                        break;
                    case "HI": Console.WriteLine("Hawaii");
                        break;
                    case "ID": Console.WriteLine("Idaho");
                        break;
                    case "IL": Console.WriteLine("Illinois");
                        break;
                    case "IN": Console.WriteLine("Indiana");
                        break;
                    case "IA": Console.WriteLine("Iowa");
                        break;
                    case "KS": Console.WriteLine("Kansas");
                        break;
                    case "KY": Console.WriteLine("Kentucky");
                        break;
                    case "LA": Console.WriteLine("Louisiana");
                        break;
                    case "ME": Console.WriteLine("Maine");
                        break;
                    case "MD": Console.WriteLine("Maryland");
                        break;
                    case "MA": Console.WriteLine("Massachusetts");
                        break;
                    case "MI": Console.WriteLine("Michigan");
                        break;
                    case "MN": Console.WriteLine("Minnesota");
                        break;
                    case "MS": Console.WriteLine("Mississippi");
                        break;
                    case "MO": Console.WriteLine("Missouri");
                        break;
                    case "MT": Console.WriteLine("Montana");
                        break;
                    case "NE": Console.WriteLine("Nebraska");
                        break;
                    case "NV": Console.WriteLine("Nevada");
                        break;
                    case "NH": Console.WriteLine("New Hampshire");
                        break;
                    case "NJ": Console.WriteLine("New Jersey");
                        break;
                    case "NM": Console.WriteLine("New Mexico");
                        break;
                    case "NY": Console.WriteLine("New York");
                        break;
                    case "NC": Console.WriteLine("North Carolina");
                        break;
                    case "ND": Console.WriteLine("North Dakota");
                        break;
                    case "OH": Console.WriteLine("Ohio");
                        break;
                    case "OK": Console.WriteLine("Oklahoma");
                        break;
                    case "OR": Console.WriteLine("Oregon");
                        break;
                    case "PA": Console.WriteLine("Pennsylvania");
                        break;
                    case "RI": Console.WriteLine("Rhode Island");
                        break;
                    case "TN": Console.WriteLine("Tennessee");
                        break;
                    case "TX": Console.WriteLine("Texas");
                        break;
                    case "UT": Console.WriteLine("Utah");
                        break;
                    case "VT": Console.WriteLine("Vermont");
                        break;
                    case "VA": Console.WriteLine("Virginia");
                        break;
                    case "WA": Console.WriteLine("Washington");
                        break;
                    case "WV": Console.WriteLine("West Virginia");
                        break;
                    case "WI": Console.WriteLine("Wisconsin");
                        break;
                    case "WY": Console.WriteLine("Wyoming");
                        break;
                    default: Console.WriteLine("No match");
                        break;
                }

                Console.WriteLine();
                Console.WriteLine("Would you like to try again (yes/no)?");

                answer = Console.ReadLine();

                Console.WriteLine();
            }
            while (answer.ToUpper() == "YES");

bany39 · Answer 3 · 2010-05-28T10:50:00

Solution 3

ok if u sure u need to run porgram at least one time , u need do while
do
{statement
}while(condition)

Posted 28-May-10 10:50am

bany39

Comments

William Winner 28-May-10 17:25pm

Reason for my vote of 2
I'm sorry..how was this supposed to answer the question?

Donsw · Answer 4 · 2010-06-02T17:20:00

Solution 4

All of these answers look good. they are all right.

Posted 2-Jun-10 17:20pm

Donsw

do while loop

4 solutions

Solution 1

Solution 2

Solution 3

Solution 4

Add your solution here

Preview 0