C#-ban mi indokolja az osztályon belül a "this" kulcsszó állandó használatát?

Egyértelműsíti a kódot. Nem kell találgatni, hogy az x változó az most függvény változó, vagy osztályváltozó, hanem egyértelmű: ha this.x, akkor osztály, egyébként függvény.

Persze lehet, hogy a józan észen kívül biztos van rá valami hangzatos elnevezés is.

Mindenhol az adott közösséghez kell igazodni. Ha ők kiírják, akkor te is kiírod, ha ők nem, akkor te sem. Ez a legfontosabb érv, függetlenül attól, hogy mennyire ésszerű.

Amúgy nem látok komoly érvet sem ellene, sem mellette. Persze, egyértelműsít, de milyen sűrűn lenne ebből probléma normális változóelnevezés mellett (tipikusan a memberek elé egy m vagy m_ kerül), és mennyire lesz tőle olvashatatlan a kód?

A lényeg az, hogy ránézésre egyértelmű legyen a kódban, hogy a this. mögött szereplő valami az egy példányszintű mezőre/függvényre való hivatkozás.

Az adott cégnél használatban lévő coding guideline írja elő, hogy ezt hogyan akarják elérni, van, ahol a this van előírva, van, ahol m_-el kell kezdődnie minden példányszintű változónak, van ahol csak simán _-al. Igazából teljesen mindegy, egyik sem jobb, vagy rosszabb a másiknál.

Én csak azért szavazok a this-re, mert ha letöltöd a stylecop-ot, az by default így működik, és nem kell átkonfigurálni.