Beginner: C# input output program issues

Question

I am new to C# and I'm trying to get a basic I/O program to work asking for the users name and age. The program doesn't allow me to run because the "static void GetUserData()" part, giving this error:

Expected class, delegate, enum, interface, or struct. 
It does not like the void keyword.

Also the method GetUserData() gives the error Does not exist in current context. As far as I know this follows the rules in C#, I am declared a method to use later, and the declaration is made within a class so it should be ok?

Thanks all I got it sorted. I typed the program exactly how the book said, its a lesson learned anyway.

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

namespace BasicConsoleIO
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("basic input output");
            GetUserData();
            Console.ReadLine();
        }
    }

    static void GetUserData()
    {
    Console.Write("please enter your name: ");
    string userName = Console.ReadLine();
    Console.Write("please enter your age: ");
    string userAge = Console.ReadLine();

    //changes echo colour
    ConsoleColor prevColor = Console.ForegroundColor;
    Console.ForegroundColor = ConsoleColor.red;

    //echo to console
    Console.WriteLine("hello {0}! Your are {1} years old.", userName, userAge);

    //Restore previous color
    Console.ForegroundColor = prevColor;
    }   
}

Class methods must be defined inside er... classes.

Marcello Romani
– Marcello Romani

2013-07-01 11:30:25 +00:00
Commented Jul 1, 2013 at 11:30 — Marcello Romani
– Marcello Romani, Commented Jul 1, 2013 at 11:30

vrajs5 · Accepted Answer · 2014-05-20 08:15:53Z

Put GetUserData() in a class, for example the one you currently have;

class Program
{
    static void Main(string[] args)
    {
        Console.WriteLine("basic input output");
        GetUserData();
        Console.ReadLine();
    }

    static void GetUserData()
    {
        Console.Write("please enter your name: ");
        string userName = Console.ReadLine();
        Console.Write("please enter your age: ");
        string userAge = Console.ReadLine();

        //changes echo colour
        ConsoleColor prevColor = Console.ForegroundColor;
        Console.ForegroundColor = ConsoleColor.red;

        //echo to console
        Console.WriteLine("hello {0}! Your are {1} years old.", userName, userAge);

        //Restore previous color
        Console.ForegroundColor = prevColor;
    }   
}

Also, as they're in the same namespace and class, if you're only ever going to need them from there there's no need to have your method as static, it all depends on how you want to be able to access them, or more so, WHERE you want to access them from

Danny Beckett · Accepted Answer · 2013-07-01 11:29:45Z

0

Your method needs to be inside a class, like so:

class Program
{
    static void Main(string[] args)
    {
        Console.WriteLine("basic input output");
        GetUserData();
        Console.ReadLine();
    }

    static void GetUserData()
    {
        ...
    }
}

answered Jul 1, 2013 at 11:29

Danny Beckett

21k27 gold badges114 silver badges144 bronze badges

1 Comment

Charlie Over a year ago

oh...one misplaced bracket, because I closed the class and the method had to be inside it. the example in the book did not make this clear.

martavoi · Accepted Answer · 2013-07-01 11:30:48Z

0

GetUserData() method out of the scope of the class Program. Should be:

class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("basic input output");
            GetUserData();
            Console.ReadLine();
        }

        static void GetUserData() {/*code here*/}
    }

answered Jul 1, 2013 at 11:30

martavoi

7,1127 gold badges43 silver badges62 bronze badges

Comments

doctorlove · Accepted Answer · 2013-07-01 11:31:40Z

0

In C#, each method must be inside a class.

You have defined your program

class Program
{
    static void Main(string[] args)
    {
        Console.WriteLine("basic input output");
        GetUserData();
        Console.ReadLine();
    }
}

and then defined the GetUserData after the closing brace - move it inside the class definition.

answered Jul 1, 2013 at 11:31

doctorlove

19.4k3 gold badges49 silver badges65 bronze badges

Comments

vrajs5 · Accepted Answer · 2014-05-20 08:13:03Z

The method must be with in the class

namespace BasicConsoleIO
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("basic input output");
            GetUserData();
            Console.ReadLine();
        }

        static void GetUserData()
        {
            Console.Write("please enter your name: ");
            string userName = Console.ReadLine();
            Console.Write("please enter your age: ");
            string userAge = Console.ReadLine();

            //changes echo colour
            ConsoleColor prevColor = Console.ForegroundColor;
            Console.ForegroundColor = ConsoleColor.Red;

            //echo to console
            Console.WriteLine("hello {0}! Your are {1} years old.", userName, userAge);

            //Restore previous color
            Console.ForegroundColor = prevColor;
        }
    }
}

ajit · Accepted Answer · 2017-05-03 08:41:48Z

Basically in C# the code should be enclosed within a class.The error can be resolved by moving the GetUserData() to the program class

class Program
{
    static void Main(string[] args)
    {
     Console.WriteLine("basic input output");
     GetUserData();
     Console.ReadLine();
    }

 static void GetUserData()
  {
    Console.Write("please enter your name: ");
    string userName = Console.ReadLine();
    Console.Write("please enter your age: ");
    string userAge = Console.ReadLine();

    //changes echo colour
    ConsoleColor prevColor = Console.ForegroundColor;
    Console.ForegroundColor = ConsoleColor.red;

    //echo to console
    Console.WriteLine("hello {0}! Your are {1} years old.", userName, userAge);

    //Restore previous color
    Console.ForegroundColor = prevColor;
  }   
}

Collectives™ on Stack Overflow

Beginner: C# input output program issues

6 Answers 6

Comments

1 Comment

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

Comments

1 Comment

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related