Бесконечный цикл там, где не предполагалось. Как исправить?

@Dika29 · Регистрация: 08.11.2017

Author24 — интернет-сервис помощи студентам

Пытаюсь сделать что-то вроде псевдоинтернет магазина в консоли. Есть менюшка, регистрация и вход. Когда пол-ль входит уже со своими данными, показывается вторая менюшка с выбором действии (купить, в корзину..). И тут при выборе купить товар, зацикливается. Хотя есть break. В чем причина?

C#

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
 
namespace Arbuz
{
    class Program
    {
        static int FirstPage()
        {
            Console.Clear();
            Console.WriteLine("1. Регистрация \n2. Вход");
 
            string menu = Console.ReadLine();
            return Convert.ToInt32(menu);
        }
 
        static int MainPage()
        {
            Console.Clear();
            Console.WriteLine("1. Купить товар \n2. Корзина \n3. Покупки \n0. Выход");
 
            string menu = Console.ReadLine();
            return Convert.ToInt32(menu);
        }
 
        static void Main(string[] args)
        {
            List<User> UserList = new List<User>();
            User user = new User();
 
            int firstPageInput = 0;
            int mainPageInput = 0;
            ConsoleKeyInfo input;
 
            do
            {
                firstPageInput = FirstPage();
                input = Console.ReadKey(true);
                switch (firstPageInput)
                {
                    case (1):
                        user.UserRegistration();
                        UserList.Add(user);
                        Console.ReadLine();
                        break;
                    case (2):
                        if (user.LogIn(UserList, user))
                        {
                            mainPageInput = MainPage();
                            MainPageSwitch();
                            break;
                        }
                        else
                        {
                            Console.WriteLine("Логин или пароль введены не правильно");
                            Console.ReadLine();             
                        }
 
                        break;
 
                }
            } while (input.Key != ConsoleKey.Escape);
 
            void MainPageSwitch()
            {
                Product product;
                List<Product> listOfSection = new List<Product>();
                listOfSection.Add(product = new Product {Section = "1. Подарки" });
                listOfSection.Add(product = new Product { Section = "2. Продукты" });
                listOfSection.Add(product = new Product { Section = "3. Детские товары" });
                listOfSection.Add(product = new Product { Section = "4. Напитки" });
                listOfSection.Add(product = new Product { Section = "5. Все для дома" });
                listOfSection.Add(product = new Product { Section = "6. Товары для животных" });
                //Product product;
 
 
                do
                {
                    switch (mainPageInput)
                    {
                        case (1):
                              foreach(var item in listOfSection)
                            {
                                Console.WriteLine(item.Section);
                            }
                            break;
 
                    }
 
                }
                while (mainPageInput != 9);
        }
 
        }
    }
}

@Young Vu · 04.10.2018, 16:54

Сообщение от Dika29

C#

                do
                {
                    switch (mainPageInput)
                    {
                        case (1):
                              foreach(var item in listOfSection)
                            {
                                Console.WriteLine(item.Section);
                            }
                            break;
 
                    }
 
                }
                while (mainPageInput != 9);

вот тут у вас проблема
вы написали, пока mainPageInput не равен 9 будет выполняться этот блок

@Элд Хасп · 05.10.2018, 00:52

Вы пишите об этом месте?

C#

                    switch (mainPageInput)
                    {
                        case (1):
                              foreach(var item in listOfSection)
                            {
                                Console.WriteLine(item.Section);
                            }
                            break;
 
                    }

Честно говоря, вообще, не пойму как это место у Вас работает. Переменная mainPageInput у Вас объявлена в другом методе как приватная, в этот метод она на передаётся. Вообще-то, этот код даже компилироваться не должен.
Прогоните в дебагере по значениям. Проверьте.

@Wolfdp · 05.10.2018, 03:58

могу предложить удочку вместо рыбы

https://msdn.microsoft.com/ru-... 5sadd.aspx

Сообщение от Элд Хасп

Честно говоря, вообще, не пойму как это место у Вас работает.

захват переменых. Метод MainPageSwitch обьявлен внутри метода Main (относительно новая фича C#), работает как и с лямдами.

@Dika29 · 05.10.2018, 08:13 **[ТС]**

Бесконечный цикл остановить получилось. А как теперь вернуться назад? Вот с этого места на MainPage. Около хожу, что-то не могу попасть

C#

do
                {
                    inputFirst = Console.ReadKey(true);
                    inputMain = Console.ReadKey(true);
                    switch (mainPageInput)
                    {
                        case (1):
                            Console.Clear();
                            foreach (var item in listOfSection)
                            {
                                Console.WriteLine(item.Section);         
                            }
                            Console.WriteLine("Назад (Q)");
                            if(inputMain.Key == ConsoleKey.Q)
                            {
                                
                            }
                            break;
 
                    }
 
                }
                while (inputFirst.Key != ConsoleKey.Escape);

@Wolfdp 1696 / 1325 / 316 Регистрация: 15.06.2012 Сообщений: 5,045 Записей в блоге: 3
	05.10.2018, 03:58	4
	могу предложить удочку вместо рыбы https://msdn.microsoft.com/ru-... 5sadd.aspx Сообщение от Элд Хасп Честно говоря, вообще, не пойму как это место у Вас работает. захват переменых. Метод MainPageSwitch обьявлен внутри метода Main (относительно новая фича C#), работает как и с лямдами. 0

	05.10.2018, 08:13

Бесконечный цикл там, где не предполагалось. Как исправить?

Решение