Najlepsza praktyka dla właściwości Auto-implementacji C# i zmiennej lokalnej różniących się tylko przypadkiem?

Question

Podam przykład:

public class MyClass 
{ 
    public string MyProperty { get; set; } 

    public MyClass(string myProperty) 
    { 
     MyProperty = myProperty; // bad? 
     this.MyProperty = myProperty; // good? 
    } 
} 

Wziąłem do korzystania this w tym scenariuszu, ponieważ mam drobne paranoję, że opierając się na razie w spokoju może być mylące lub gorzej w rzeczywistości może prowadzić do błędów.

Co to jest "najlepsza praktyka" tutaj?

EDIT:

tej pory, to brzmi jak to jest dużo bardziej subiektywne niż myślałem. Domyśliłem się, że ludzie zejdą mocno z jednej lub drugiej strony.

Answer 1

Użycie "this." jest zbędne w dowolnej klasie. Od twojego sklepu deweloperskiego zależy wyznaczenie standardu jego używania.

Zaletą używania "this." jest to, że niektórym programistom łatwiej jest skojarzyć je w umyśle z instancją klasy, gdy czytają kod, a jak wspomniałeś, spraw, aby były jaśniejsze podczas pracy z podobnie nazwanymi przedmiotami.

Wady są takie, że niektórzy ludzie postrzegają to jako zagracenie twojego pliku kodu i jeśli używasz narzędzi takich jak ReSharper, domyślnie oznaczyli go jako kod nadmiarowy.

Answer 2

Obie opcje zależą wyłącznie od przypadku. Nie ma różnicy między tymi wersjami.

Answer 3

Jak powiedział womp. "to" jest zbędne, ale czyni kod łatwiejszym do odczytania. Lub raczej trudniejsze do odczytania.

Answer 4

Moim zdaniem "najlepsza praktyka" to "nie rób tego". Jeśli natknę się na to w recenzowanym przeze mnie kodzie, natychmiast go podbiję. Posiadanie dwóch zmiennych różniących się tylko przypadkiem to nieporozumienie, które tylko czeka. To zbyt proste, aby programista obsługi technicznej pojawił się kilka miesięcy lub lat później i nieumyślnie dokonał przypisania do myThing zamiast MyThing.

Dodano później:

commenter poprosił o moją propozycją zastąpienia górna/dolna przypadku konwencji nazewnictwa. Do tego potrzebuję konkretnego przykładu. Załóżmy, że masz prosty Book klasę, która ma tylko jedną właściwość: Title:

public class Book 
{ 
    public string Title { get; private set; } 
} 

teraz trzeba konstruktora. Wspólna konwencja jest użycie małą wersję właściwości:

public Book(string title) 
{ 
    Title = title; 
} 

Albo, jeśli chcesz, aby upewnić się, że nie ma dwuznaczności: this.Title = title.

Można argumentować, że jest to w porządku w konstruktorach. I może tak być, gdyby wszyscy konstruktorzy byli tak prości. Ale moje doświadczenie było takie, że gdy konstruktor wykracza poza zaledwie kilka linii, rozróżnienie pomiędzy Title i title zostaje utracone. Problem staje się gorszy, gdy mówimy o metodach innych niż konstruktorzy. Tak czy inaczej, potrzebujesz innej konwencji.

Czego używać? Różnie używałem i widziałem używane skróty w parametrach: ttl, na przykład.Lub coś w rodzaju bookTitle, które jest bardziej opisowe podczas używania IntelliSense. Moim zdaniem albo jest lepsze od konwencji używania nazwy, która różni się tylko przypadkiem.

Answer 5

C# jest zdecydowanie wielkość liter, więc nie ma żadnego ryzyka w użyciu ...

myProperty = myProperty;

Tak więc szukałbym innych najlepszych praktyk, takich jak pisanie najmniejszej ilości kodu potrzebnego do osiągnięcia celu (podczas samodzielnego dokumentowania). Prawda jest taka, że ​​nie jest wymagana, minimalistyczni mogą powiedzieć, że ją opuścili.

Answer 6

W moim miejscu pracy standardy kodowania nakazują zapisywanie właściwości LikeThis, podczas gdy zmienne lokalne są napisane tak. This is this. Ponieważ w języku C# rozróżniana jest wielkość liter, jest to dobre narzędzie do rozróżniania zmiennych. Jeśli jednak znajdziesz się z właściwością i zmienną lokalną o dokładnie takiej samej nazwie, użycie tego słowa kluczowego zdecydowanie rozróżni użycie.

Answer 7

Oto jak ja obecnie zainicjować właściwości za swój przykład (zarówno auto-wdrożone i nie)

 public class MyClass 
    { 
     public string MyProperty { get; set; } 

     public string AnotherProperty 
     { 
      get { return _anotherProperty; } 
      set { _anotherProperty = value; } 
     } 
     private string _anotherProperty; 

     public MyClass(string myProperty, string anotherProperty) 
     { 
      MyProperty = myProperty; // auto-implemented property initialization 
      _anotherProperty = anotherProperty; //property with member variable initialization     
     } 
    } 

rozsianych w użyciu „to” jest nad specyfikacją do mnie. Wiem, że jest to lokalna właściwość, ponieważ jest pisana wielką literą. Wszystkie właściwości powinny być ograniczone do siebie. Wiem, że zmienna "_anotherProperty" ma zasięg klasy ze względu na podkreślenie. Wcześniej pomijałem podkreślenia ze zmiennych na poziomie klasy. Kod jest dla mnie łatwiejszy do odczytania, gdy pojawi się podkreślenie, ponieważ od razu poznałem zakres bez konieczności przesuwania kursora myszy nad zmienną, aby zobaczyć deklarację w etykiecie narzędzi z VS. Ponadto uzyskuję korzyść z używania tej samej nazwy dla zmiennych lokalnych, po prostu pomijając podkreślenie. To sprawia, że ​​twoje początki wyglądają na czyste. Inną korzyścią podkreślenia jest to, że możesz wpisać podkreślenie i nacisnąć ctrl + spacja, a wszystkie zmienne z zakresu klasy są pogrupowane.