The name XXXX does not exist in the current context

Question

I have the following code:

public Form1()
{
    InitializeComponent();


    string strRadio = Utils.ReadFile(strTemp + @"\rstations.txt");
    string[] aRadio = strRadio.Split(new string[] { "#" }, StringSplitOptions.RemoveEmptyEntries);    
    for (int i = 0; i < aRadio.Length; i += 2)
    {
       listBox.Items.Add(aRadio[i]);
    }

}

private void listBox_SelectedIndexChanged(object sender, EventArgs e)
{
    int index = listBox.SelectedIndex;
    MessageBox.Show(aRadio[(index+1)]);
}

Now the error is The name 'aRadio' does not exist in the current context. Which comes from MessageBox.Show(aRadio[(index+1)]);. Do I need to declare the aRadio as public or something? If so, how would this be done?

Jon Skeet · Accepted Answer · 2012-09-28T07:38:25.867

21

You're declaring aRadio as a local variable within your constructor. You need to declare it as an instance variable, and just assign it a value within your constructor:

// TODO: Give this a better name
private readonly string[] aRadio;

// TODO: Give your form a better name too
public Form1()
{
    InitializeComponent();

    // TODO: You might want to reconsider reading files in a GUI constructor, too
    // TODO: Use Path.Combine(strTemp, "rstations.txt" instead of concatenation
    string strRadio = Utils.ReadFile(strTemp + @"\rstations.txt");
    aRadio = strRadio.Split(new string[] { "#" },
                            StringSplitOptions.RemoveEmptyEntries);

    for (int i = 0; i < aRadio.Length; i += 2)
    {
       listBox.Items.Add(aRadio[i]);
    }
}

I wouldn't be surprised if you could do rather better than this approach though, by adding a custom object (or just a KeyValuePair<string, string>) to the list box and binding the display part through properties. That way you could get the selected item rather than the selected index... there's no need to keep text/value pairs like this.

edited Sep 28 '12 at 07:38

answered Apr 23 '12 at 13:27

Jon Skeet

1,421,763
867
9,128
9,194

Could you elaborate yourself on the additional comments you posted? Where would you want to read files? On another thread? – Devator Apr 23 '12 at 13:41
@Devator: Probably on start-up in some entirely separate code, while a splash-screen is showing. It really depends on the rest of the application. – Jon Skeet Apr 23 '12 at 14:01
You should use `Path.Combine` – user Sep 28 '12 at 07:10
@user: Have added a "TODO" - I wanted to keep the original code close in general. – Jon Skeet Sep 28 '12 at 07:38

score 2 · Answer 2 · answered Apr 23 '12 at 13:31

   private string[] aRadio;

    public Form1() { 
      InitializeComponent();       
      string strRadio = Utils.ReadFile(strTemp + @"\rstations.txt");
      this.aRadio = strRadio.Split(new string[] { "#" }, StringSplitOptions.RemoveEmptyEntries);
      for (int i = 0; i < aRadio.Length; i += 2)
      {
        listBox.Items.Add(aRadio[i]);
      }
    }

    private void listBox_SelectedIndexChanged(object sender, EventArgs e) {
      int index = listBox.SelectedIndex;
      MessageBox.Show(this.aRadio[(index+1)]);
    }

Jorge · Answer 3 · 2012-04-23T13:40:59.607

you're to access to a variable defined in the domain of your contructor form1 so you could do this

//Define the variable as an attribute of class

private string[] strRadio;
public Form1()
{
        InitializeComponent();
        string strRadio = Utils.ReadFile(strTemp + @"\rstations.txt");
        aRadio = strRadio.Split(new string[] { "#" }, StringSplitOptions.RemoveEmptyEntries);

        for (int i = 0; i < aRadio.Length; i += 2)
        {
           listBox.Items.Add(aRadio[i]);
        }

    }

    private void listBox_SelectedIndexChanged(object sender, EventArgs e)
    {
        int index = listBox.SelectedIndex;
        MessageBox.Show(aRadio[(index+1)]);
    }
}

Please don't recommend public fields like this :( – Jon Skeet Apr 23 '12 at 13:31 — Jon Skeet, Apr 23 '12 at 13:31

score -3 · Answer 4 · answered Dec 10 '14 at 04:40

-3

System.StringSplitOptions.RemoveEmptyEntries : typed out in this manner will solve your problem.

Don't ask me why I just know it works this way.

answered Dec 10 '14 at 04:40

Ricahrd Rindom

1

The name XXXX does not exist in the current context

4 Answers4

Linked

Related