Using List in C# Interface, How to return and set the List Value

Question

I am trying to practice OOP. I have a class called Cinema and interface called location. Location Interface has List type and when I am trying to add the location name, It is giving error. Any direction or suggestion will be great -

The error is: "The name 'cinema_location' does not exist in the current context."

namespace MOSS.Interfaces
{
    interface loc
    {
        List<string> location { get; set;}
    }
}


namespace MOSS
{
    class Cinema : Interfaces.loc
    {

        List<string> cinema_location = new List<string>();

        cinema_location.Add("location1");
        cinema_location.Add("location2");
        public List<string> location
        {
            get
            {
            return cinema_location;
            }

            set
            {
                cinema_location = value;
            }
        }

        public void DisplayCinema()
        {
            string loc;
            string Session;
            for(int i = 0; i < cinema_location.Count; i++)
            {
                loc = cinema_location[i];
                Console.WriteLine("Cinema Location: {0}", loc);

            }
        }
    }
}

Your method calls aren't within a method. cinema_location.Add("location1"); can't just be in the body of a class. — ProgrammingLlama
– ProgrammingLlama, Commented May 30, 2017 at 5:48

Wiktor Zychla · Accepted Answer · 2017-05-30 05:55:03Z

2

A code like this one

cinema_location.Add("location1");
cinema_location.Add("location2");

can't be put wherever you like. In particular, it has to be put in a method, could be a class constructor.

Try

public class Cinema
{
  List<string> cinema_location = new List<string>();

  public Cinema()
  {
     cinema_location.Add("location1");
     cinema_location.Add("location2");
  }

  ...

answered May 30, 2017 at 5:55

Wiktor Zychla

48.5k6 gold badges83 silver badges114 bronze badges

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

Comments

Sebastian Kaupper · Accepted Answer · 2017-05-30 05:53:35Z

1

Your c cinema_location.Add(...)s have to be inside a method (you cannot execute code directly in class scope).

It seems like you want these names added at object creation, right? In that case you need a constructor and add thr method calls there:

class Cinema
{
    List<string> cinema_locations = new List<string>();
    public Cinema()
    {
         cinema_locations.Add("location1");
    }
}

answered May 30, 2017 at 5:53

Sebastian Kaupper

1639 bronze badges

2 Comments

Ovic Over a year ago

What if I put it in a new method called addLocation instead of constructor and also define that method in the loc interface as well.Is that will be a good way to structure this!!!!

Sebastian Kaupper Over a year ago

of course you could implement a addLocation method. either you force it in the interface directly or you give the 'user' of the interface the choice of how to fill the list.

Daniel Tshuva · Accepted Answer · 2017-05-30 06:10:01Z

1

You need a constructor (https://learn.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/constructors):

Public Cinema()
{
    cinema_location.Add("location1");
    cinema_location.Add("location2");
}

edited May 30, 2017 at 6:10

answered May 30, 2017 at 5:53

Daniel Tshuva

5035 silver badges12 bronze badges

1 Comment

Sebastian Kaupper Over a year ago

actually it is called a constructor

Collectives™ on Stack Overflow

Using List in C# Interface, How to return and set the List Value

3 Answers 3

Comments

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related