❔ Quitting to main menu in console app

Krzak · 2023-03-14T18:46:43.473Z

The app is used to monitor a habit with saving the data to a database. The user has to insert a date and one number (for example how many glasses of water they drank that day) and I wanted to make it so writing 0 in the console would result in quitting to main menu, but with the way I did it first, it kind of "opens" another menu in that menu? I read that usually these kinds of apps are done with a "while" loop but they don't mention going back, only closing. How to untangle this mess? (I'll post code snippets in a second)

P

phaseshift•3/14/23, 6:54 PM

All you have to do to go back is... Nothing. When a function ends control automatically goes back to where it was

K

KrzakOP•3/14/23, 6:54 PM

so I overcomplicated this probably

P

phaseshift•3/14/23, 6:56 PM

Beginners typically keep calling 'menu()' thinking they're going back to the beginning, but in reality they're going deeper and deeper into a maze

K

KrzakOP•3/14/23, 6:56 PM

haha that's what I did lol

K

KrzakOP•3/14/23, 6:58 PM

but with this I can't just do nothing?

case "2": //add new
                    Console.WriteLine("How many glasses of water did you drink?");
                    GetQuantity();
                    Console.Clear();
                    GetDate();
                    var newDocument = new HabitModel(VerifiedDate, VerifiedQuantity);
                    var bsonDocument = newDocument.ToBsonDocument();
                    dbOperations.CreateRecord(MongoCollection, bsonDocument);
                    Console.WriteLine("New record created, press any key to return to menu.");
                    Console.ReadKey();
                    break;

case "2": //add new
                    Console.WriteLine("How many glasses of water did you drink?");
                    GetQuantity();
                    Console.Clear();
                    GetDate();
                    var newDocument = new HabitModel(VerifiedDate, VerifiedQuantity);
                    var bsonDocument = newDocument.ToBsonDocument();
                    dbOperations.CreateRecord(MongoCollection, bsonDocument);
                    Console.WriteLine("New record created, press any key to return to menu.");
                    Console.ReadKey();
                    break;

case "2": //add new
                    Console.WriteLine("How many glasses of water did you drink?");
                    GetQuantity();
                    Console.Clear();
                    GetDate();
                    var newDocument = new HabitModel(VerifiedDate, VerifiedQuantity);
                    var bsonDocument = newDocument.ToBsonDocument();
                    dbOperations.CreateRecord(MongoCollection, bsonDocument);
                    Console.WriteLine("New record created, press any key to return to menu.");
                    Console.ReadKey();
                    break;

case "2": //add new
                    Console.WriteLine("How many glasses of water did you drink?");
                    GetQuantity();
                    Console.Clear();
                    GetDate();
                    var newDocument = new HabitModel(VerifiedDate, VerifiedQuantity);
                    var bsonDocument = newDocument.ToBsonDocument();
                    dbOperations.CreateRecord(MongoCollection, bsonDocument);
                    Console.WriteLine("New record created, press any key to return to menu.");
                    Console.ReadKey();
                    break;

which probably means I have to kind-of-start-over

K

KrzakOP•3/14/23, 6:58 PM

I don't know how much to paste here so it makes sense

P

phaseshift•3/14/23, 7:00 PM

Depends what is around this code, I guess

P

phaseshift•3/14/23, 7:01 PM

Basically all you need is

While not quitting
Show menu
Do action

P

phaseshift•3/14/23, 7:02 PM

And you can use the same pattern for sub-menus etc as well.

K

KrzakOP•3/14/23, 7:03 PM

this is above

public void MainMenu()
        {
            Console.WriteLine("1 - view all records");
            Console.WriteLine("2 - add new record");
            Console.WriteLine("3 - edit a record");
            Console.WriteLine("4 - delete a record");
            Console.WriteLine("0 - quit");


            string chosenOption = Console.ReadLine();
            Console.Clear();
            switch (chosenOption)
            {
                case "0": //quit
                    IsRunning = false;
                    break;

                case "1": //view all
                    dbOperations.ReadAllRecords(MongoCollection);
                    Console.WriteLine("\nAll documents in the collection are listed. Press any key to return to main menu.");
                    Console.ReadLine();
                    break;

                case "2": //add new...

public void MainMenu()
        {
            Console.WriteLine("1 - view all records");
            Console.WriteLine("2 - add new record");
            Console.WriteLine("3 - edit a record");
            Console.WriteLine("4 - delete a record");
            Console.WriteLine("0 - quit");


            string chosenOption = Console.ReadLine();
            Console.Clear();
            switch (chosenOption)
            {
                case "0": //quit
                    IsRunning = false;
                    break;

                case "1": //view all
                    dbOperations.ReadAllRecords(MongoCollection);
                    Console.WriteLine("\nAll documents in the collection are listed. Press any key to return to main menu.");
                    Console.ReadLine();
                    break;

                case "2": //add new...

public void MainMenu()
        {
            Console.WriteLine("1 - view all records");
            Console.WriteLine("2 - add new record");
            Console.WriteLine("3 - edit a record");
            Console.WriteLine("4 - delete a record");
            Console.WriteLine("0 - quit");


            string chosenOption = Console.ReadLine();
            Console.Clear();
            switch (chosenOption)
            {
                case "0": //quit
                    IsRunning = false;
                    break;

                case "1": //view all
                    dbOperations.ReadAllRecords(MongoCollection);
                    Console.WriteLine("\nAll documents in the collection are listed. Press any key to return to main menu.");
                    Console.ReadLine();
                    break;

                case "2": //add new...

public void MainMenu()
        {
            Console.WriteLine("1 - view all records");
            Console.WriteLine("2 - add new record");
            Console.WriteLine("3 - edit a record");
            Console.WriteLine("4 - delete a record");
            Console.WriteLine("0 - quit");


            string chosenOption = Console.ReadLine();
            Console.Clear();
            switch (chosenOption)
            {
                case "0": //quit
                    IsRunning = false;
                    break;

                case "1": //view all
                    dbOperations.ReadAllRecords(MongoCollection);
                    Console.WriteLine("\nAll documents in the collection are listed. Press any key to return to main menu.");
                    Console.ReadLine();
                    break;

                case "2": //add new...

K

KrzakOP•3/14/23, 7:03 PM

and this in Program.cs

while (habitsApp.IsRunning)
{
    habitsApp.MainMenu();
}

while (habitsApp.IsRunning)
{
    habitsApp.MainMenu();
}

while (habitsApp.IsRunning)
{
    habitsApp.MainMenu();
}

while (habitsApp.IsRunning)
{
    habitsApp.MainMenu();
}

K

KrzakOP•3/14/23, 7:04 PM

told you, tangled mess XD

P

phaseshift•3/14/23, 7:04 PM

It's not too bad. Does 0 option set IsRunning to false?

K

KrzakOP•3/14/23, 7:05 PM

yes and no

K

KrzakOP•3/14/23, 7:05 PM

it returns false when 0 is written in the main menu

K

KrzakOP•3/14/23, 7:05 PM

but not in the "submenus"/"options"

P

phaseshift•3/14/23, 7:07 PM

Ok. So is your problem specifically when you're in sub menus?

P

phaseshift•3/14/23, 7:08 PM

From your earlier question asking about doing nothing in option 2, yes, doing nothing should be fine afaict

K

KrzakOP•3/14/23, 7:08 PM

yeah and it's because of this:

 private int GetQuantity()
        {
            Console.WriteLine("Use only integers. Write 0 to return to main menu.");
            string userQuantity = Console.ReadLine();
            while (true)
            {
                if (IsInputZero(userQuantity))
                {
                    QuitToMainMenu();
                }
                else if (!IsQuantityInteger(userQuantity))
                {
                    Console.WriteLine("The value wasn't an integer, try again. Write 0 to return to main menu.");
                    userQuantity = Console.ReadLine();
                }
                else
                {
                    break;
                }
            }
            return VerifiedQuantity;
        }
        

        private bool IsInputZero(string consoleInput)
        {
            if (consoleInput == "0")
            {
                QuitToMainMenu();
            }
            return false;
        }
        private void QuitToMainMenu()
        {
            Console.WriteLine("Returning to main menu...");
            Console.Clear();
            MainMenu();
        }

 private int GetQuantity()
        {
            Console.WriteLine("Use only integers. Write 0 to return to main menu.");
            string userQuantity = Console.ReadLine();
            while (true)
            {
                if (IsInputZero(userQuantity))
                {
                    QuitToMainMenu();
                }
                else if (!IsQuantityInteger(userQuantity))
                {
                    Console.WriteLine("The value wasn't an integer, try again. Write 0 to return to main menu.");
                    userQuantity = Console.ReadLine();
                }
                else
                {
                    break;
                }
            }
            return VerifiedQuantity;
        }
        

        private bool IsInputZero(string consoleInput)
        {
            if (consoleInput == "0")
            {
                QuitToMainMenu();
            }
            return false;
        }
        private void QuitToMainMenu()
        {
            Console.WriteLine("Returning to main menu...");
            Console.Clear();
            MainMenu();
        }

 private int GetQuantity()
        {
            Console.WriteLine("Use only integers. Write 0 to return to main menu.");
            string userQuantity = Console.ReadLine();
            while (true)
            {
                if (IsInputZero(userQuantity))
                {
                    QuitToMainMenu();
                }
                else if (!IsQuantityInteger(userQuantity))
                {
                    Console.WriteLine("The value wasn't an integer, try again. Write 0 to return to main menu.");
                    userQuantity = Console.ReadLine();
                }
                else
                {
                    break;
                }
            }
            return VerifiedQuantity;
        }
        

        private bool IsInputZero(string consoleInput)
        {
            if (consoleInput == "0")
            {
                QuitToMainMenu();
            }
            return false;
        }
        private void QuitToMainMenu()
        {
            Console.WriteLine("Returning to main menu...");
            Console.Clear();
            MainMenu();
        }

 private int GetQuantity()
        {
            Console.WriteLine("Use only integers. Write 0 to return to main menu.");
            string userQuantity = Console.ReadLine();
            while (true)
            {
                if (IsInputZero(userQuantity))
                {
                    QuitToMainMenu();
                }
                else if (!IsQuantityInteger(userQuantity))
                {
                    Console.WriteLine("The value wasn't an integer, try again. Write 0 to return to main menu.");
                    userQuantity = Console.ReadLine();
                }
                else
                {
                    break;
                }
            }
            return VerifiedQuantity;
        }
        

        private bool IsInputZero(string consoleInput)
        {
            if (consoleInput == "0")
            {
                QuitToMainMenu();
            }
            return false;
        }
        private void QuitToMainMenu()
        {
            Console.WriteLine("Returning to main menu...");
            Console.Clear();
            MainMenu();
        }

K

KrzakOP•3/14/23, 7:08 PM

quitting runs a menu again hahaaa

P

phaseshift•3/14/23, 7:08 PM

Yeah ok. That's the problem

K

KrzakOP•3/14/23, 7:09 PM

I don't know what to do about it so it probably means there's quite a bit to do again

P

phaseshift•3/14/23, 7:11 PM

Two tips. Try to make functionality do just one thing.

E.g why is getting a quantity mixed with quitting to main menu.

P

phaseshift•3/14/23, 7:12 PM

The other is you can have a function indicate whether to keep going or not by returning a bool

P

phaseshift•3/14/23, 7:13 PM

Like

bool TryGetInt(out int number)

bool TryGetInt(out int number)

bool TryGetInt(out int number)

bool TryGetInt(out int number)

K

KrzakOP•3/14/23, 7:13 PM

How to define "one thing"? It's always an issue for me

P

phaseshift•3/14/23, 7:13 PM

Yeah it can be ambiguous

P

phaseshift•3/14/23, 7:14 PM

But getting a number is a different thing to a decision about menu navigation

K

KrzakOP•3/14/23, 7:15 PM

hmm okay makes sense

P

phaseshift•3/14/23, 7:15 PM

Like, how should a small utility to ask for a number have the power to move through menus

P

phaseshift•3/14/23, 7:16 PM

No, they're completely separate concepts

K

KrzakOP•3/14/23, 7:26 PM

So for the 0 to work as exit I should construct the procedures in a way that would lead to leaving the method (for example getting a date)? A sort of shortcut to "break;"?

P

phaseshift•3/14/23, 7:28 PM

Yes. For utilities getting data, they should always return something.

If you want the option for it to fail, or be cancelled, then it still needs to return, but you use the "try get" pattern

P

phaseshift•3/14/23, 7:29 PM

So

int getint()

or

bool TryGetInt(out int number)

K

KrzakOP•3/14/23, 7:30 PM

ooh alright

K

KrzakOP•3/14/23, 7:30 PM

that's so clever

K

KrzakOP•3/14/23, 9:12 PM

This proved to be more tricky than I anticipated >.> for the 0 to work all that procedure for adding has to, in the end, become 1 function? I'd think so if it has to react immediately for that 0 on any stage

P

phaseshift•3/14/23, 9:16 PM

I don't really understand what youre saying

K

KrzakOP•3/14/23, 9:17 PM

I'm not entirely sure myself

K

KrzakOP•3/14/23, 9:17 PM

okay so um

K

KrzakOP•3/14/23, 9:19 PM

if the only thing that changed was changing GetQuantity/GetDate to returning bool then it wouldn't stop the rest from executing even if it returned false

K

KrzakOP•3/14/23, 9:19 PM

unless I got it wrong?

P

phaseshift•3/14/23, 9:20 PM

well something reacts according to the bool.

P

phaseshift•3/14/23, 9:20 PM

if true, then do something with data, if false then... whatever you want

P

phaseshift•3/14/23, 9:21 PM

and 'whatever you want' includes returning from the method, ie not doing anything else

K

KrzakOP•3/15/23, 3:53 PM

that didn't work out :x whoops

K

KrzakOP•3/15/23, 4:02 PM

I came up with this but it doesn't work for the date part because 0 isn't a valid dd-mm-yyyy format

K

KrzakOP•3/15/23, 4:02 PM

    case "2": //add new                   

                    while(true)
                    {
                        Console.WriteLine("How many glasses of water did you drink?");                                              
                        
                        
                        if (GetQuantity() != 0)
                        {
                            

                            if(GetDate() != "0")
                            {
                                var newDocument = new HabitModel(VerifiedDate, VerifiedQuantity);
                                var bsonDocument = newDocument.ToBsonDocument();
                                dbOperations.CreateRecord(MongoCollection, bsonDocument);
                                Console.WriteLine("New record created, press any key to return to menu.");
                                Console.ReadKey();
                            }
                            
                        }                        
                        break;
                    }
                    break;

    case "2": //add new                   

                    while(true)
                    {
                        Console.WriteLine("How many glasses of water did you drink?");                                              
                        
                        
                        if (GetQuantity() != 0)
                        {
                            

                            if(GetDate() != "0")
                            {
                                var newDocument = new HabitModel(VerifiedDate, VerifiedQuantity);
                                var bsonDocument = newDocument.ToBsonDocument();
                                dbOperations.CreateRecord(MongoCollection, bsonDocument);
                                Console.WriteLine("New record created, press any key to return to menu.");
                                Console.ReadKey();
                            }
                            
                        }                        
                        break;
                    }
                    break;

    case "2": //add new                   

                    while(true)
                    {
                        Console.WriteLine("How many glasses of water did you drink?");                                              
                        
                        
                        if (GetQuantity() != 0)
                        {
                            

                            if(GetDate() != "0")
                            {
                                var newDocument = new HabitModel(VerifiedDate, VerifiedQuantity);
                                var bsonDocument = newDocument.ToBsonDocument();
                                dbOperations.CreateRecord(MongoCollection, bsonDocument);
                                Console.WriteLine("New record created, press any key to return to menu.");
                                Console.ReadKey();
                            }
                            
                        }                        
                        break;
                    }
                    break;

    case "2": //add new                   

                    while(true)
                    {
                        Console.WriteLine("How many glasses of water did you drink?");                                              
                        
                        
                        if (GetQuantity() != 0)
                        {
                            

                            if(GetDate() != "0")
                            {
                                var newDocument = new HabitModel(VerifiedDate, VerifiedQuantity);
                                var bsonDocument = newDocument.ToBsonDocument();
                                dbOperations.CreateRecord(MongoCollection, bsonDocument);
                                Console.WriteLine("New record created, press any key to return to menu.");
                                Console.ReadKey();
                            }
                            
                        }                        
                        break;
                    }
                    break;

K

KrzakOP•3/15/23, 4:02 PM

it looks weird

P

phaseshift•3/15/23, 10:35 PM

why are they even returning a value that you dont use for anything

❔ Quitting to main menu in console app

Similar Threads

Similar Threads

Similar Threads