Functions in C# - How to get an integer value into a function?

Question

Basically I'm trying to get an integer value into a function, I'm making a simple program for a vending machine, where I have an option to "Inspect machine" which counts the number of crisps in the machine, Basically the problem I have is that when I come to see how many is actually in the machine, it says the amount is 0.. How would I fix this up so that it allows me to actually view how many crisps are currently in the machine?

Here is my code so far:

static void Main(string[] args)
{
   //Declare variables
   int iOption = 0; //used to store users menu option
   int iFillCrisps = 0; //used to store the amount of crisps the user wants to add to the machine
   int iBuyCrisps = 0; //used to store the amount of crisps the user wants to buy
   int iTotalNumCrisps = 0; //used to show the total number of crisps in the machine

   //Menu
   while (iOption != 4) //Program continously loops until user types "4" which is the exit key
   {
      GetMenuOption(iOption);
      iOption = Convert.ToInt32(Console.ReadLine());

      //Process menu
      if (iOption == 1)
      {
         FillCrisps(iFillCrisps, iTotalNumCrisps);
      }

      else if (iOption == 2)
      {
         BuyCrisps(iBuyCrisps, iTotalNumCrisps);
      }

      else if (iOption == 3)
      {
         InspectMachine(ref iTotalNumCrisps);
      }

      else if (iOption == 4)
      {
         Console.WriteLine("Exit");
      }
   }
}

static int GetMenuOption(int piOption)
{
   Console.WriteLine("Vending machine");
   Console.WriteLine();
   Console.WriteLine("Please choose an option");
   Console.WriteLine("1: Fill machine with crisps");
   Console.WriteLine("2: Buy Crisps");
   Console.WriteLine("3: Inspect Machine");
   Console.WriteLine("4: Exit");

   return piOption;
}

static int FillCrisps(int piFillCrisps, int piTotalNumCrisps)
{
   Console.WriteLine("Fill machine with crisps");
   Console.WriteLine("How many crisps would you like to add to the machine?");
   piFillCrisps = Convert.ToInt32(Console.ReadLine());
   piTotalNumCrisps = piFillCrisps + piTotalNumCrisps;
   Console.WriteLine("You are adding " + piFillCrisps + " packs of crisps to them machine");
   Console.WriteLine("There are now " + piTotalNumCrisps + " packs of crisps in the machine");

   return piTotalNumCrisps;
}

static int BuyCrisps(int piBuyCrisps, int piTotalNumCrisps)
{
   Console.WriteLine("Buy Crisps");
   Console.WriteLine("How many crisps would you like to buy?");
   piBuyCrisps = Convert.ToInt32(Console.ReadLine());
   piTotalNumCrisps = piTotalNumCrisps - piBuyCrisps;
   Console.WriteLine("You are buying " + piBuyCrisps + " crisps");
   Console.WriteLine("There are now " + piTotalNumCrisps + " packs of crisps in the machine");

   return piTotalNumCrisps;
}

static void InspectMachine(ref int piTotalNumCrisps) //Needs a way of retrieving the total value into it
{
   Console.WriteLine("Inspect Machine");
   Console.WriteLine("There are currently " + piTotalNumCrisps + " crisps in the machine.");
}

You're not doing anything with the results of your methods; e.g. the call to FillCrisps should probably be iTotalNumCrisps = FillCrisps(iFillCrisps, iTotalNumCrisps). — Jess
– Jess, Commented Dec 9, 2013 at 22:45

quetzalcoatl · Accepted Answer · 2013-12-09 22:51:54Z

2

Well, you see, usually, if you'd be a shop owner, yould like to first calculate how much change you need to give back to the client, and only then give him the change. Not the other way around.

So, change the code

    GetMenuOption(iOption);
    iOption = Convert.ToInt32(Console.ReadLine());

to

    iOption = Convert.ToInt32(Console.ReadLine());
    GetMenuOption(iOption);

and try again.

Also, the other function are just taking parameters from the outside. They are doing something, but they are not giving anything in return. For example, this function:

static int FillCrisps(int piFillCrisps, int piTotalNumCrisps)

tries to change the counters, but as it only "reads" the parameters from the outside, whatever it calculates is not visible at the outside variables. Try changing it to

static int FillCrisps(ref int piFillCrisps, ref int piTotalNumCrisps)

and adjust rest of the code. Now the changes will be visible in the outer variables too.

On the other hand, this function:

static void InspectMachine(ref int piTotalNumCrisps)
         //Needs a way of retrieving the total value into it

only READS and PRINTS the values. It does not change them. You do not need the ref here at all. Change it back to:

static void InspectMachine(int piTotalNumCrisps)

and it will be OK (as long as you correct the other functions too!).

So, your functions should have signatures of:

// functions that don't change the data
static int GetMenuOption(int piOption)
static void InspectMachine(int piTotalNumCrisps)

// functions that DO change the data
static int FillCrisps(ref int piFillCrisps, ref int piTotalNumCrisps)
static int BuyCrisps(ref int piBuyCrisps, ref int piTotalNumCrisps)

edited Dec 9, 2013 at 22:51

answered Dec 9, 2013 at 22:46

quetzalcoatl

33.8k9 gold badges75 silver badges117 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

crthompson Over a year ago

+.5 for spotting that answer so quickly, despite the crappy title and +.5 for being an aztec god. :)

Tom Over a year ago

Thank you! Exactly what I was looking for, Works 100%, I did know about this ref, but as you see I got them muddled up! Silly me, thanks for the help :)

user2945651 · Accepted Answer · 2013-12-09 23:01:42Z

2

There is a much simpler way to do this than the method mentioned above, you could simply just add "iTotalNumCrisps =" before the actual function that is about to happen.. for example:

            if (iOption == 1)
              {
                iTotalNumCrisps = FillCrisps(iFillCrisps, iTotalNumCrisps);
              }

answered Dec 9, 2013 at 23:01

user2945651

2 Comments

Tom Over a year ago

Thanks, tried that before the method above and works great, but I liked the idea more of using references in my code, thanks tho!

Tom Over a year ago

Yeah I guess your right, I guess with C# there are many different possible answers to problems, Both of these answers worked, only wish I could accept them both

Chris Catignani · Accepted Answer · 2024-05-01 15:57:08Z

-1

To make the ReadLine() function return an integer use this: System.Int32.Parse(Console.ReadLine());

edited May 1, 2024 at 15:57

Chris Catignani

5,30819 gold badges47 silver badges54 bronze badges

answered Apr 30, 2024 at 10:28

andrea bertacche

11 bronze badge

Collectives™ on Stack Overflow

Functions in C# - How to get an integer value into a function?

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related