Почему цикл foreach в бесконечном цикле продолжается вечно? - PullRequest
Новая
       35

Почему цикл foreach в бесконечном цикле продолжается вечно?

0 голосов
/ 29 апреля 2018

Я пытаюсь реализовать менеджер рецептов, где пользователь может выбирать способ сортировки каждого рецепта. Основываясь на вводе пользователя, у меня есть цикл foreach, проходящий по каждому элементу в списке Recipes, который должен перебирать каждый из них и сортировать их при необходимости. Прямо сейчас у меня есть бесконечный цикл, который позволял бы мне позволять пользователю вводить ввод, если это необходимо, но когда я пытаюсь просмотреть список, он продолжает выводить результаты снова и снова. 

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace recipeManager
{
    class Program
    {
        static List<recipes> Recipes = new List<recipes>()
        {
            new recipes() { recipeName = "Meatloaf", recipeType = "Dinner", listOfIngredients = new List<string> { "Ground beef", "Egg", "Onion", "Milk" } },
            new recipes() { recipeName = "PBnJ Sandwich", recipeType = "Lunch", listOfIngredients = new List<string> { "Peanut Butter", "Jelly", "Bread" } },
            new recipes() { recipeName = "Cake", recipeType = "Dessert", listOfIngredients = new List<string> { "Flour", "Sugar", "Baking Powder" } },
            new recipes() { recipeName = "Key lime pie", recipeType = "Dessert", listOfIngredients = new List<string> { "Key Lime Juice", "Egg Yolk", "Milk" } },
            new recipes() { recipeName = "Jambalaya", recipeType = "dinner", listOfIngredients = new List<string> { "Crawfish", "Egg Yolk", "Milk" } }
        };

        static void Main(string[] args)
        {

            Console.WriteLine("Hi there, how would you like to view our recipes? Enter View All for the");
            Console.WriteLine("entire recipe book, Sort Name to sort the recipes by name, Sort Type to sort");
            Console.WriteLine("the recipes by type, Sort Ingredients to sort by ingredients, break to break");
            Console.WriteLine("the loop");
            var input = Console.ReadLine();

            while (true)
            {
                switch (input)
                {
                    case "View All":
                        printAll();
                        break;
                    case "Sort Name":
                        sortName();
                        break;
                    case "Sort Ingredients":
                        sortIngredients();
                        break;
                    case "break":
                        return;
                    default:
                        Console.WriteLine("Not a valid command, please try again");
                        break;
                }
            }
        }

        //print entire list
        static void printAll()
        {
            foreach (recipes recipeProp in Recipes)
            {
                Console.WriteLine(recipeProp.recipeName + ", " + recipeProp.recipeType + ", " + "[" + String.Join(", ", recipeProp.listOfIngredients) + "]");
            }
        }

        //sort by name
        static void sortName()
        {
            var orderedNames = Recipes.OrderBy(l => l.recipeName);

            foreach (recipes recipeNames in orderedNames)
            {
                Console.WriteLine(recipeNames.recipeName + ", " + recipeNames.recipeType + ", " + "[" + String.Join(", ", recipeNames.listOfIngredients) + "]");
            }
        }

        //sort by type
        static void sortType()
        {
            var orderedType = Recipes.OrderBy(t => t.recipeType);

            foreach (recipes recipeTypes in orderedType)
            {
                Console.WriteLine(recipeTypes.recipeName + ", " + recipeTypes.recipeType + ", " + "[" + String.Join(", ", recipeTypes.listOfIngredients) + "]");
            }
        }

        static void sortIngredients()
        {

        }
    }
}

Удаление while сработало, но я не понимаю, почему оно будет зацикливаться вечно, потому что foreach не продолжается вечно, и у меня даже было выражение break, которое должно было прерваться, когда foreach был завершено.

Ответы [ 2 ]

0 голосов
/ 29 апреля 2018

@ Запрашиваемая,

Вы должны поместить readLine внутри цикла, так что будет остановка для ожидания ввода пользователя. 

   while (true)
   {
      Console.WriteLine(...);
      var input = Console.ReadLine();
      ...
   }

При этом программа всегда будет выводить сообщение и запрашивать ввод, прежде чем печатать что-либо еще.

0 голосов
/ 29 апреля 2018 
//var input = Console.ReadLine();

while (true)
{
    var input = Console.ReadLine();  // put it here

    switch (input)
    {
...