Local variable scope error - foreach loop

Question

I have the following code:

if (Current == false)
{
    foreach (var visual in visuals)
        visual.IsSelected = value;
}

Visual visual = visuals[currentIndex];

And when I compile I have this error:

A local variable named 'visual' cannot be declared in this scope because it would give a different meaning to 'visual', which is already used in a 'child' scope to denote something else

Also, if I don't declare the visual variable, that is I replace:

Visual visual = visuals[currentIndex];

with:

visual = visuals[currentIndex];

the error is the following:

The name 'visual' does not exist in the current context

Why this behavior?

which is already used in a child scope to denote something else Since in your foreach loop has a variable called visual, that's why you get this error. — Soner Gönül
– Soner Gönül, Commented Apr 5, 2014 at 11:08
@SonerGönül the scope of the "first" visual is not limited inside the if statement? — Nick
– Nick, Commented Apr 5, 2014 at 11:17
Check out Eric Lippert's answer.. stackoverflow.com/a/2050864/447156 — Soner Gönül
– Soner Gönül, Commented Apr 5, 2014 at 11:27
It is by design of C#. It reduces ambiguity. Similar [question][1] [1]: stackoverflow.com/questions/2059210/… — Balu
– Balu, Commented Apr 5, 2014 at 11:27

Amit Joki · Accepted Answer · 2014-04-05 11:09:21Z

1

In the first case there is ambiguity between the variables declared outside and inside.(global and local).

The compiler is confused as to what visual you are referring to. The outer one or inner one?

And in the second case, the compiler does not know what is visual.

Comments

Sudhakar Tillapudi · Accepted Answer · 2014-04-05 11:09:19Z

1

Why this behavior?

in your first case you have already declared the variable with name visual in your foreach loop.

in your second case you can not use the keyword visual because it does not exist. it is only available within your foreach loop.

Try This:

Visual visual1 = visuals[currentIndex];

answered Apr 5, 2014 at 11:09

Sudhakar Tillapudi

26.3k5 gold badges43 silver badges70 bronze badges

Comments

Octane · Accepted Answer · 2014-04-05 11:26:51Z

1

   if (Current == false)
    {
        foreach (var visual in visuals)
            visual.IsSelected = value;
    }

   // Parent and child scope have same variable name so it creates ambiguity.
    Visual visual = visuals[currentIndex];

and

if (Current == false)
    {
        foreach (var visual in visuals)
            visual.IsSelected = value;
    }

   // The variable visual in not defined outside the scope of if statement
    visual = visuals[currentIndex];

answered Apr 5, 2014 at 11:26

Octane

1,25012 silver badges11 bronze badges

Comments

Community · Accepted Answer · 2017-05-23 10:31:50Z

1

As Soner Gönül points out, the first construct:

if (Current == false)
{
    foreach (var visual in visuals)
        visual.IsSelected = value;
}

Visual visual = visuals[currentIndex];

is illegal by definition of the language. See the link that Soner provided:

https://stackoverflow.com/a/2050864/447156

Making this illegal reduces the chance of confusion when humans read the code. The compiler could treat this as legal, but the designers of the language felt (and I agree) that this was an opportunity to make C# easier to understand.

edited May 23, 2017 at 10:31

CommunityBot

11 silver badge

answered Apr 5, 2014 at 12:30

KevinS

2521 silver badge7 bronze badges

Collectives™ on Stack Overflow

Local variable scope error - foreach loop

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related