Dostałem dzisiaj na code review uwagę, że wywoływanie metod z użycie słówka 'this' wprowadza nadmiarowość i zaburza czytelność kodu np. this.foo(). Osobiście preferuję ten styl i według mnie to na pewno nie pogarsza czytelności kodu tylko ewentualnie poprawia. Co o tym sądzicie czy to faktycznie zła praktyka?
This przy wywołaniu metody
- Rejestracja: dni
- Ostatnio: dni
- Lokalizacja: Wrocław
- Postów: 524
1
Po co pisać więcej, jeśli można pisać mniej
- Rejestracja: dni
- Ostatnio: dni
- Lokalizacja: Kraków
- Postów: 834
1
Ja nie lubię. Przy metodach to już w ogóle wydaje mi się dziwne, bo nie wiem czemu miałoby służyć.
- Rejestracja: dni
- Ostatnio: dni
- Lokalizacja: UK
- Postów: 2235
2
Przez dłuższy czas używałem this do wywołania właściwości w C# (np. this.Name). Było to dyktowane tylko i wyłącznie gustami. Wywoływanie metod w ten sposób zawsze wydawało mi się dziwne, ale to również kwestia gustów. Z czasem przestałem używać this w ogóle.
Na logikę jednak patrząc, jeśli nie trzeba tego używać to po co to robić? Skoro wowolujesz metodę bez żadnego dopisku z przodu, to wiadomo że chodzi o tę konkretną instancję.
Tak czy inaczej, najważniejsze aby wszyscy trzymali się uzgodnionych konwencji. Jeśli ogólny konsensus jest taki aby nie używać this, to tego nie rób.