2013-02-04 57 views
3

我們已經開始實施checkstyle作爲在開發人員中提供一致的代碼風格的一種方式。其中一項檢查要求所有實例變量都是使用「this」的引用。我沒有習慣使用「this」,所以有成千上萬個需要修復的實例。自動添加「this」。到實例變量

是否有工具或IntelliJ插件,可以通過並添加適當的代碼?

+2

爲什麼?這是多餘的。不要執行冗餘工作。沒有人願意爲此付錢,爲什麼他們呢?修改您的checkstyle配置,以免它需要它。不要讓工具告訴你如何花錢,也不要被愚蠢地說,跨開發人員的「一致的代碼風格」對於交付工作產品是必不可少的。它不是,只要代碼清晰易讀並且可維護。我看到在這個問題上浪費的時間和金錢比我想象的要多。 – EJP

+1

這是checkstyle中比較缺乏的檢查之一。每個人都使用語法突出顯示的IDE。 – millimoose

+1

這個答案正確地解釋你需要什麼:http://stackoverflow.com/a/19865423/1149414 – bluesman80

回答

5

其中一項檢查要求使用「this」引用所有實例變量。

所以'checkstyle'是錯誤的。

我一直沒有習慣使用「this」。

很對。這是多餘的。對你有好處。

因此有數千個需要修復的實例。

所以是錯的。

您沒有任何需要修復的實例。

不要這樣做。這是多餘的。不要執行冗餘工作。沒有人願意爲此付費,爲什麼他們呢?修改您的checkstyle配置,以免它需要它。不要讓工具告訴你如何花錢,也不要被愚蠢地說,跨開發人員的「一致的代碼風格」對於交付工作產品是必不可少的。它不是,只要代碼清晰易讀並且可維護。我看到在這個問題上浪費的時間和金錢比我想象的要多。

+2

+1爲理智的聲音。可以自動檢測到的迂腐垃圾幾乎不會顯着影響代碼庫的理解程度。如果一個實習生曾經寫過一個循環嵌套七層,沒有人敢觸摸的JSF,那麼任何收益都會被這個班級所淹沒。如果您進行自動代碼質量檢查,請關注像cyclomatic /數據流複雜性之類的事情。測試覆蓋率也可能是「可讀性」的更好代理。 – millimoose

+1

我會補充一點,我在1​​971年開始編程,當時存在各種編碼風格,其中大多數是個人的,其中大多數是除作者外難以辨認的。 Fortran程序沒有空格;程序全部寫在一行上......朝向編碼標準的轉變旨在解決這個問題,*而且確實如此。自1982年以來,我認爲這是一個徹底解決的問題,而且我從不讓軟件工具決定我如何花費我的時間或金錢。 – EJP

5

在Eclipse: 窗口 - 首選項 - 爪哇 - 編輯 - 保存動作 - 配置... - 成員訪問 -

在這裏,您選中 「使用此」 複選框。

每次保存代碼時都會進行修正。

我剛剛發現this頁面。

對於那些想要高效地使用eclipse的人來說,閱讀是永不嫌晚的。我想會有類似Netbeans和其他的東西。

+1

順便說一句,你可以應用像刪除未使用的導入等規則...我真的很喜歡這個功能。 – CsBalazsHungary

1

其中一項檢查要求所有實例變量都是使用「this」的引用。

IMO,最好的解決辦法是開發團隊的其他成員,以提高這一點,並得到同意把這一愚蠢檢查關。如果添加多餘的關鍵字this可提高可讀性,那麼您必須懷疑閱讀代碼的人的Java技能。

如果他們推後,我會誘惑來登記正是如此厭惡我:

public class SomeClass { 
    private String thisName; 

    public String getName() { 
     return this.thisName; 
    } 

    public void setName(String notThisName) { 
     this.thisName = notThisName; 
    } 

    // and so on 
}