Assign user's input to a variable

Question

I've cut the code down into this small piece so it would be easier to read. My problem is that the value that the user writes into "injtaille" doesn't transfer over to the switch a little lower down. How can I fix this?

if (inj == "oui")
{
    Console.WriteLine("Quel est la taille de l'injection? (30 - 50 - 60) ");
    injtaille = Convert.ToInt32(Console.ReadLine());
}
switch (client)
{
    case 1:
       consprix = 25;
       imgradprix = 55;
       analyzeprix = 28;
       switch (injtaille)
       {
           case 30
       }

The case 30 is missing a : and the code to be executed. What did you expect it to do? — user585968
– user585968, Commented Oct 9, 2015 at 4:18
Are you getting errors did you try a default case for the switch statement — EasyBB
– EasyBB, Commented Oct 9, 2015 at 4:18
@Roy Yes I am aware that there is no : after case 30. The problem is that when I put my cursor on switch (injtaille), it says that I'm using an unassigned value. — Dr.Roflcopter
– Dr.Roflcopter, Commented Oct 9, 2015 at 4:21
You should accept the answer of @MickyD. You may also see this question posted in 2011, with many other answers. — Massimiliano Kraus
– Massimiliano Kraus, Commented May 30, 2016 at 11:58

score 2 · Accepted Answer · 2015-10-09 04:29:48Z

2

The problem is that when I put my cursor on switch (injtaille), it says that I'm using an unassigned value

You need to initialise your variables before using them. Your original code (from what I can see) only sets it if the guard inj == "oui" holds true.

Try this:

int injtaille =0; // default value here

if (inj == "oui")
{
    Console.WriteLine("Quel est la taille de l'injection? (30 - 50 - 60) ");
    injtaille = Convert.ToInt32(Console.ReadLine());
}
switch (client)
{
    case 1:
       consprix = 25;
       imgradprix = 55;
       analyzeprix = 28;
       switch (injtaille)
       {
           case 30:
              ....
       }

Depending on the logic on your program, it's generally a good idea to include a default in your switch statements too.

edited Oct 9, 2015 at 4:29

answered Oct 9, 2015 at 4:22

user585968

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

4 Comments

Dr.Roflcopter Over a year ago

Alright so, the default starting value of injtaille will be 0. Let's say I write in 50 at the ReadLine line, will injtaille take on the value of whatever the user writes in the ReadLine?

DevT Over a year ago

@Dr.Roflcopter yes it is.

Sami Kuhmonen Over a year ago

@Dr.Roflcopter Yes, it will as you can easily see by running the code

user585968 Over a year ago

@Dr.Roflcopter You're missing a break; right at the end after the }

Collectives™ on Stack Overflow

Assign user's input to a variable

1 Answer 1

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related