Advertisement
clockworksaint

Redundant modifiers

Feb 21st, 2013
137
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
C# 0.76 KB | None | 0 0
  1. // This is all fine:
  2.  
  3. public class ShopKeeper                              // "public" is useful here
  4. {
  5.     public string NameOnBadge { get; private set; }  // "public" is useful here
  6.     internal void SecretHandshake();                 // "internal" is useful here
  7. }
  8.  
  9. // This is needlessly verbose:
  10.  
  11. internal class ShelfStacker                          // "internal" is redundant at namespace level
  12. {
  13.     private Clothing Underpants { get; set; }        // "private is redundant at class level
  14. }
  15.  
  16. // I prefer:
  17.  
  18. class ShelfStacker                                   // defaults to "internal"
  19. {
  20.     Clothing Underpants { get; set; }                // defaults to "private"
  21. }
  22.  
  23. // ...especially when a class has a lot of private members. I don't like the clutter.
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement