views:

630

answers:

7

I'd like to create an internal auto-property:

internal bool IP { get; protected internal set; }

I thought it would be possible to make the setter protected or protected internal - but I always get the error accessibility modifier must be more restrictive than the property. Isn't that the case? Private does not help me, here.

EDIT:
The question is: How do I implement an auto-property with a internal getter and a protected setter?

+17  A: 

It's effectively protected or internal, not and. It's accessible both by derived classes and types in the same assembly. It's a common misconception to think protected internal means accessible only to derived classes in the same assembly.

Mehrdad Afshari
OK, how do I make the getter internal and the setter protected?
tanascius
You can't. Getter/setter access modifier should be a proper subset of the property access modifier.
Mehrdad Afshari
My problem was really the misunderstanding that protected internal means protected AND internal - thanks for pointing that out.
tanascius
I think your phrasing is VERY confusing. The use of AND and OR do not help clarify the issue. `protected internal` means that it is accessible by all that would be able to access `protected` AND all that would be able to access `internal`
@devinb: My first sentence might be. But the second sentence clarifies the issue: "It's accessible both by derived classes and types in the same assembly. "
Mehrdad Afshari
Yes, you're correct. I meant my critique only to your first sentence.
+2  A: 

protected internal is less restrictive than either protected or internal because it allows both its subclasses (protected) and anything in the same assembly (internal) to access something.

R. Bemrose
+1  A: 

protected internal means visible to classes in the same assembly, or to classes deriving from the containing class - in other words it is visible to those meeting the internal requirements OR the protected requirements, not AND. There is no access modifier meaning protected AND internal in this way.

David M
+1  A: 

protected internal means protected OR internal, not protected and internal. So scope is limited to the same assembly OR derived classes, not necessarily both.

Brian Rudolph
+11  A: 

At the .NET level, there are two similar but distinct access levels:

  • FamilyAndAssembly: more restrictive than either protected or internal
  • FamilyOrAssembly: less restrictive than either protected or internal

"protected internal" in C# means FamilyOrAssembly; there's no modifier for FamilyAndAssembly.

So, your protected internal setter is less restrictive than the internal overall property. What you could do is:

protected internal bool IP { internal get; set; }

But then your setter is less restricted than your getter, which is odd...

Another (somewhat equivalent) alternative is:

internal bool IP { get; set; }

protected void SetIP(bool ip)
{
    this.IP = ip;
}
Jon Skeet
OK, my solution could be a internal readonly property with a protected backing-field.
tanascius
Or your equivalent alternative with a private setter would do the job, too.
tanascius
It occurs to me, (months later of course) that the setter is still accessible "internally" which is in some ways less restrictive than protected. To fully make it an internal getter and a protected setter, you would have to have protected void SetIP(bool value) and internal bool GetIP() and then have a private bool _IP
A: 

accessibility modifier must be more restrictive than the property

Internal is more restrictive that protected: because protected things can be seen (by subclasses) outside the assembly.

The compiler is saying that there's no sense in saying that set is protected (i.e. visible to subclasses outside the assembly), when the whole IP property is internal (i.e. invisible outside the assembly).

ChrisW
+5  A: 

I would consider this cheating, since Eric Lippert is on SO himself, but he wrote an excellent blog post that considers this issue.

Why Can't I Access A Protected Member From A Derived Class, Part Three

Ultimately, his answer is largely the same as those given by the posters here, but he ads some interesting reasoning behind the desgin of the language and the implementation of these features.

It's not "cheating". By all means, post links. It saves me time.
Eric Lippert