Shortening my questions code

Question

Is there any other way to shorten this code without sacrificing readability? I am fairly new to C#, let alone programming, and I wanted to know if this is as short as this code can possibly become. Feedback would be greatly welcomed.

Side note: was Questions.Questask used properly here, or could there have been a better way?

 class Program
 {
    static void Main(string[] args)
    {
        string name = "";
        while (name.ToLower() != "none")
        {
            Console.WriteLine("Are you Kydd,Leo,Jay,Sha, or Zigg");
            name = Console.ReadLine();

            if (name.ToLower() == "kydd")
                Question.Questask("Input question here", "Kquestion","Link");
            else if (name.ToLower() == "leo")
                Question.Questask("Input question here", "Lquestion", "Link");
            else if (name.ToLower() == "jay")
                Question.Questask("Input question here", "Jquestion", "Link");
            else if (name.ToLower() == "sha")
                Question.Questask("Input question here", "Squestion", "Link");
            if (name.ToLower() == "zigg")
                Question.Questask("Input question here", "Zquestion", "Link");
       }
    }

    class Question
    {
        public static void Questask(string question, string questAnswer, string url)
        {
            Console.WriteLine(question);
            questAnswer = Console.ReadLine();
            if (questAnswer.ToLower() == "yes")
            {
                Console.WriteLine("You are right!");
                Console.ReadKey();
            }
            else
            {
                System.Diagnostics.Process.Start(url);
            }
        }
    }
}

Answer 1

I rewrote this for you, making it easier to understand and making it a little more efficient (although, the size is just a tad bigger then your original). Hopefully it makes a little more sense and you can ask any questions if you need to:

class Program
{
    static void Main(string[] args)
    {
        var names = new List<NameClass>
        {
            new NameClass {Name = "Kydd", Answer = "Kquestion"},
            new NameClass {Name = "Leo", Answer = "Lquestion"},
            new NameClass {Name = "Jay", Answer = "Jquestion"},
            new NameClass {Name = "Sha", Answer = "Squestion"},
            new NameClass {Name = "Zigg", Answer = "Zquestion"}
        };

        string name = "";
        while (name.ToLower() != "none")
        {
            Console.WriteLine("Are you Kydd,Leo,Jay,Sha, or Zigg");
            name = Console.ReadLine().ToLower();

            var a = names.FirstOrDefault(z => z.Name.ToLower() == name);
            if (a != null)
            {
                Questask(a.Question, a.Answer, a.Url);
            }
        }
    }

    private static void Questask(string question, string questAnswer, string url)
    {
        Console.WriteLine(question);
        questAnswer = Console.ReadLine();
        if (questAnswer.ToLower() == "yes")
        {
            Console.WriteLine("You are right!");
            Console.ReadKey();
        }
        else
        {
            System.Diagnostics.Process.Start(url);
        }
    }

    public class NameClass
    {
        public string Name { get; set; }
        public string Question { get; set; }
        public string Answer { get; set; }
        public string Url { get; set; }

        public NameClass()
        {
            Question = "Input question here";
            Url = "Link";
        }
    }
}

The beauty of this code is that you don't need a bunch of if statements. If you need to add more, simply add one more line of code in the names definition.

Answer 2

When I need a simple strategy pattern I'll use a Dictionary. This can drastically reduce code by cutting repetition.

Note the use of StringComparer.InvariantCultureIgnoreCase for comparer to do away with the String.ToLower calls.

private static readonly IDictionary<string, Tuple<string, string>> NamesDictionary =
    new Dictionary<string, Tuple<string, string>>(StringComparer.InvariantCultureIgnoreCase)
    {
        { "kydd", Tuple.Create("question 1", "link1") },
        { "zydd", Tuple.Create("question 2", "link2") },
        ...
    };

private static void DoSomething(string name)
{
    Tuple<string, string> questionAndLinkTuple;

    if (!NamesDictionary.TryGetValue(name, out questionAndLinkTuple))
        return;

    var question = questionAndLinkTuple.Item1;
    var link = questionAndLinkTuple.Item2;

    Question.AskQuestion("Input answer here", question, link);
}

Answer 3

I like icemanind's answer, but to give an alternate point of view -- sometimes you just need to get it done.

string[] names = {"kydd","leo","jay","sha","zigg");
if (Array.IndexOf(names, name.ToLower())>-1){
   Question.Questask("Input question here", 
       string.Format("{0}question",name[0].ToUpper()),"Link");
}