Switch Case Causing Trouble In Going To A particular Function

Question

     FunWithScheduling fun = new FunWithScheduling();
     Console.WriteLine("This Is Your Scheduler");
     Console.WriteLine("What Do You Wish To Do");
     Console.WriteLine("Enter 1 To Add, 2 To Edit, 3 To Search And 4 To Exit");
     int Choice = Convert.ToInt32(Console.ReadLine());
     switch (Choice)
     {
              case 1:
                  goto fun.Add();
                  break;
              case 2:
                  goto fun.Edit();
                  break;
              case 3:
                  goto fun.Search();
                  break;
              case 4:
                  goto fun.Exit();
                  break;
             Default:
                   Console.WriteLine("Enter a Valid Number");   
                   return;
      }
 }

I got 4 functions that would help me do the following Add Edit Search Exit

I want to use switch case to go to the function. Is it possible? It asked for an object reference and then a label.

you shouldn't place the _goto_ word there. Even if a method needed to be prefix, it would be _call_. But that isn't needed here. — Jeroen van Langen, Aug 10 '13 at 22:55

score 2 · Answer 1 · answered Aug 10 '13 at 22:55

2

Why don't you just call the method without the goto ? and for me it's not the proper way to use a goto, cfr MSDN Reference

This should be ok for me :

FunWithScheduling fun = new FunWithScheduling();
 Console.WriteLine("This Is Your Scheduler");
 Console.WriteLine("What Do You Wish To Do");
 Console.WriteLine("Enter 1 To Add, 2 To Edit, 3 To Search And 4 To Exit");
 int Choice = Convert.ToInt32(Console.ReadLine());
 switch (Choice)
 {
          case 1:
              fun.Add();
              break;
          case 2:
              fun.Edit();
              break;
          case 3:
              fun.Search();
              break;
          case 4:
              fun.Exit();
              break;
         Default:
               Console.WriteLine("Enter a Valid Number");   
               return;
  }

}

answered Aug 10 '13 at 22:55

Emmanuel Istace

1,209
2
14
32

unreachable code detected, This label has not been referenced. – Joanah Jocks Aug 10 '13 at 23:10
Can you post the content of FunWithScheduling class ? @JoanahJocks – Emmanuel Istace Aug 10 '13 at 23:12
Sorry Sir, Its too huge. I am not able format it here in this blog – Joanah Jocks Aug 10 '13 at 23:24
Ok, from what I read, you're defining some goto label in this class. It's a bit out of scope but I think you should read this http://stackoverflow.com/questions/11906056/goto-is-this-bad and start using methods instead :) – Emmanuel Istace Aug 10 '13 at 23:32

score 0 · Answer 2 · answered Aug 10 '13 at 22:55

0

Remove the goto. Those are only used if you are using labels. Simply call fun.Add() or fun.Edit()., etc.

answered Aug 10 '13 at 22:55

Inisheer

20,376
9
50
82

Switch Case Causing Trouble In Going To A particular Function

2 Answers2