Программа подсчета бутылок, переключение запросов, поиск максимального числа

Question

моя программа записывает количество бутылок, которое собрало 4 комнаты. Я новичок в C #, но делал Java в прошлом.

Я не буду использовать LINQ, я не буду использовать массивы. Только оператор Switch (извините, я знаю, что он неэффективен)

Моя программа должна записывать количество бутылок, введенных пользователем, и когда пользователь нажимает «выйти», программа предполагает выплевать количество бутылок во всех комнатах и ​​определять комнату с наибольшим количеством бутылок в качестве победителя.

Я застрял в этом операторе switch, который не может найти способ инициировать мои комнаты (room1, room2, room3, room4), он говорит, что переменная room1-4 не назначена. Я должен быть в состоянии постоянно добавлять бутылку в комнату с помощью переключателя.

Когда я набираю quit, программа может выплюнуть все бутылки, собранные комнатами, и найти комнату с наибольшим количеством бутылок.

Спасибо за ваше время, я ценю, насколько это сообщество помогло мне.

 namespace BottleDrive1
 {
    class Program
    {
    static void Main(string[] args)
    {//Initialize 4 rooms. 
        int room1 = 0;

        int room2 = 0;

        int room3 = 0;

        int room4 = 0;
        //Start of while loop to ask what room your adding into. 
        while (true)
        {
            Console.Write("Enter the room you're in: ");
            //If user enters quit at anytime, the code will jump out of while statement and enter for loop below
            string quit = Console.ReadLine();
            if (quit == "quit")
                //Break statement allows quit to jump out of loop
                break;}}
            private void SetRoom(int room, int value)
            {
            switch (room)
            {
                case 1:
                    room1 = value;
                    break;
                case 2:
                    room2 = value;
                    break;
                case 3:
                    room3 = value;
                    break;
                case 4:
                    room4 = value;
                    break;
            }
            }
            public int GetRoom(int room)
            {
                int count = int.Parse(Console.ReadLine());
                switch (room)
                {
                    case 1:
                        room1 += count;
                        break;
                    case 2:
                        room2 += count;
                    case 3:
                        room3 += count;
                        break;
                    case 4:
                        room4 += count;
                        break;
                }

            }
        }
        //This for statement lists the 4 rooms and their bottle count when the user has entered quit. An alternative to below
        /*for (int i = 0; i < rooms.Length; ++i)
            Console.WriteLine("Bottles collected in room {0} = {1}", i + 1, rooms[i]);*/

      /*  int maxValue = 0;//initiates the winner, contructor starts at 0
        int maxRoomNumber = 0;//initiates the room number that wins
        for (int i = 0; i < room[i].Length; ++i)//This loop goes through the array of rooms (4)
        {
            if (room[i] > maxValue)//Makes sure that the maxValue is picked in the array
            {//Looking for room number for the  
                maxValue = room[i];
                maxRoomNumber = i + 1;
            }//Writes the bottles collected by the different rooms
            Console.WriteLine("Bottles collected in room {0} = {1}", i + 1, rooms[i]);
        }
        //Outputs winner
        Console.WriteLine("And the Winner is room " + maxRoomNumber + "!!!");
        */
    }

Последняя часть программы была моей попыткой найти максимальное число, так как я использовал массив для запуска. Я не должен использовать массив.

Answer 1

Просто переместите объявления int над методом Main следующим образом:

class Program
{
     int room1 = 0;
     ....
    static void Main(string[] args)
    {
    ....
    }
}

Answer 2

Вот пример, который использует класс для хранения информации для каждой комнаты.Причина использования класса заключается в том, что если в будущем вашей программе понадобятся изменения для сбора дополнительной информации, вам не придется отслеживать еще один массив, вы можете просто добавить свойства в класс.

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

Вот новый класс комнат:

public class Room
{
    public int Number { get; set; }
    public int BottleCount { get; set; }

    public Room(int wNumber)
    {
        Number = wNumber;
    }
}

А вот и новая версия программы,Обратите внимание, что добавлена ​​дополнительная проверка значений, введенных конечным пользователем, чтобы исключить исключения при попытке получить текущую комнату или проанализировать int значение, введенное пользователем:

    static void Main(string[] args)
    {
        const int MAX_ROOMS = 4;
        var cRooms = new System.Collections.Generic.List<Room>();

        for (int nI = 0; nI < MAX_ROOMS; nI++)
        {
            // The room number is 1 to 4
            cRooms.Add(new Room(nI + 1));
        }

        // Initializes the room that wins
        //Start of while loop to ask what room your adding into. 
        while (true)
        {
            Console.Write("Enter the room you're in: ");
            //If user enters quit at anytime, the code will jump out of while statement and enter for loop below
            string roomNumber = Console.ReadLine();
            if (roomNumber == "quit")
            {
                //Break statement allows quit to jump out of loop
                break;
            }
            int room = 0;
            if (int.TryParse(roomNumber, out room) && (room < MAX_ROOMS) && (room >= 0)) {
                Room currentRoom;

                currentRoom = cRooms[room];

                Console.Write("Bottles collected in room {0}: ", currentRoom.Number);

                int wBottleCount = 0;

                if (int.TryParse(Console.ReadLine(), out wBottleCount) && (wBottleCount >= 0))
                {
                    // This line adds the count of bottles and records it so you can continuously count the bottles collected.
                    currentRoom.BottleCount += wBottleCount;
                }
                else
                {
                    Console.WriteLine("Invalid bottle count; value must be greater than 0");
                }
            }
            else
            {
                Console.WriteLine("Invalid room number; value must be between 1 and " + MAX_ROOMS.ToString());
            }
        }

        Room maxRoom = null;

        foreach (Room currentRoom in cRooms) //This loop goes through the array of rooms (4)
        {
            // This assumes that the bottle count can never be decreased in a room
            if ((maxRoom == null) || (maxRoom.BottleCount < currentRoom.BottleCount))
            {
                maxRoom = currentRoom;
            }
            Console.WriteLine("Bottles collected in room {0} = {1}", currentRoom.Number, currentRoom.BottleCount);
        }
        //Outputs winner
        Console.WriteLine("And the Winner is room " + maxRoom.Number + "!!!");
    }