Help with C#??

[Guest]

Im trying to make a program where if you enter in a certain number, it outputs a result, then restarts itself so it can receive a different input to output a different result. This is my code so far. Can someone help me? Its for a project.

 

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading;

namespace Test
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.Write("NetScreen Diary Entry Log: Type in number of entry to access"); 
            do
            {
                int answer = Convert.ToInt16(Console.ReadLine());
                if (answer == 1)
                Console.Write("return blank space");
                Console.Write("NetScreen Diary Entry Log: Type in number of entry to access"); 
                answer = 0;
                if (answer == 2)
                Console.Write("return blank space 2");
                Console.Write("NetScreen Diary Entry Log: Type in number of entry to access"); 
                answer = 0;
            } while (true);
        }
    }
}

 

 

[dany_boy]

First off, please read and format with this in mind: https://linustechtips.com/main/announcement/12-please-use-code-tags/

Second: Use comments as often as possible, specially when sharing code with others

Third: check the use of curly braces after logic operators:

 using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading;

namespace Test
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.Write("NetScreen Diary Entry Log: Type in number of entry to access"); 
            do
            {
                int answer = Convert.ToInt16(Console.ReadLine());
				//if you dont use {} curly braces, then the if statement will only execute the immediatly next line, use this instead
                if (answer == 1)
				{
                	Console.Write("return blank space");
                	Console.Write("NetScreen Diary Entry Log: Type in number of entry to access"); 
                	answer = 0;
				}
                if (answer == 2)
				{
                Console.Write("return blank space 2");
                Console.Write("NetScreen Diary Entry Log: Type in number of entry to access"); 
                answer = 0;
				}
            } while (true);
        }
    }
}

  

Cheers!

[Guest]

1 minute ago, dany_boy said:

First off, please read and format with this in mind: https://linustechtips.com/main/announcement/12-please-use-code-tags/

Second: Use comments as often as possible, specially when sharing code with others

Third: check the use of curly braces after logic operators:

 using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading;

namespace Test
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.Write("NetScreen Diary Entry Log: Type in number of entry to access"); 
            do
            {
                int answer = Convert.ToInt16(Console.ReadLine());
				//if you dont use {} curly braces, then the if statement will only execute the immediatly next line, use this instead
                if (answer == 1)
				{
                	Console.Write("return blank space");
                	Console.Write("NetScreen Diary Entry Log: Type in number of entry to access"); 
                	answer = 0;
				}
                if (answer == 2)
				{
                Console.Write("return blank space 2");
                Console.Write("NetScreen Diary Entry Log: Type in number of entry to access"); 
                answer = 0;
				}
            } while (true);
        }
    }
}

  

Cheers!

Thank you so much! Ill use the code format from now on. 

[dany_boy]

1 minute ago, NovaMan01 said:

Thank you so much! Ill use the code format from now on. 

No worries man, if you need more help with C# or python, feel free to tag me in future posts.

[vorticalbox]

in this case i would suggest using a switch case rather than ifs, will give an over all better readability 

 

[HarryNyquist]

In addition to what's been said already: 

• You can put the prompt for a new choice at the top of your do-while loop, instead of outside the loop & after each choice. 
• You should use Int.TryParse() instead of Convert.ToInt16() and handle the situation if a user enters something that is not an integer. Right now, it'll crash if you enter something that isn't an integer.
• You don't need to set the answer to zero after a choice has been made. You should only set it to a default value if (from above) the user tries to choose something that isn't a valid choice (or isn't an integer, if that's how you choose to handle that situation).

vorticalbox is also correct, a switch statement should be used here for readability. Switches are pretty much always used for menus like this.