C# Grocery List List, Loop, and Foreach

Question

0.00/5 (No votes)

See more: (untagged)

Hello Everyone,

This is my first time dealing with an array list along side a loop. I am thought that I understood it pretty well. So I started to make a simple Grocery List program. However, I cannot seem to get the foreach to allow me to use the item variable. Any pointers would be greatly appreciated. Thanks in advance!

C#

namespace VS_Console_Sand_Box
{
    class Program
    {
        static void Main(string[] args)
        {
            string item;

            List<string> GroceryList = new List<string>();

            do
            {
                Console.WriteLine("Enter new grocery item");
                Console.WriteLine("Enter Q to quit");
                item = Convert.ToString(Console.ReadLine());

                if (item == "Q")
                {
                    foreach (string item in GroceryList)
                    {
                        Console.WriteLine(GroceryList);
                    }
                    break;
                }
                else
                {
                    GroceryList.Add(item);
                }
            }
            while (item != "Q");
            Console.ReadLine();
        }
    }
}

Posted 7-Oct-14 11:19am

Member 11136294

Add a Solution

Comments

[no name] 7-Oct-14 17:28pm

Console.WriteLine(item);

1 solution

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)

Garth J Lancaster · Accepted Answer · 2014-10-07T11:32:00

in

C#

foreach (string item in GroceryList)
                    {
                        Console.WriteLine(GroceryList);
                    }
                    break;

wouldn't you be better off doing

C#

Console.WriteLine(item);

Also note that 'item' here has completely different 'scope' to the item declared at the top of your program - in fact, I prefer not to do this as all - I make the variable name in the foreach reflective of the list - so I might use

C#

foreach (string GroceryListitem in GroceryList)
                    {
                        Console.WriteLine(GroceryListitem);
                    }
                    break;

is that what you were looking for ?