2010-05-06 95 views
0

我已經在我的的java項目剛剛發現這個代碼片段:代碼格式化初始化列表

List<IssueType> selectedIssueTypes = new ArrayList<IssueType>(); 
    for (Object item : selectedItems) 
     selectedIssueTypes.add((IssueType) item); 

你怎麼想,可這種風格可以用嗎?

回答

2

Imho,這種風格似乎顯示不存在的層次結構。此外,我會主張始終使用大括號,我喜歡用一個空行分開聲明和代碼,所以我會用:

List<IssueType> selectedIssueTypes = new ArrayList<IssueType>(); 

for (Object item : selectedItems) { 
    selectedIssueTypes.add((IssueType) item); 
} 
2

我認爲這將是清晰不容易出現問題時,另一家開發商看着它。

List<IssueType> selectedIssueTypes = new ArrayList<IssueType>(); 
for (Object item : selectedItems) { 
    selectedIssueTypes.add((IssueType) item); 
} 

總是使用括號來闡明循環。不要縮進for語句,因爲它違背了標準慣例。

+0

括號內循環不只是澄清,就可以防止錯誤。我試圖總是讓循環和ifs明確。 – 2010-05-06 20:04:32

1

爲什麼不乾脆:

List<IssueType> selectedIssueType = Arrays.asList(selectedItems); 
+0

感謝您的建議!但這個問題通常是關於風格。我正在審查項目代碼並嘗試使其更好。我已經發現了很多'臭'的地方,但是這個讓我想到了。 – Roman 2010-05-06 20:17:11

+0

啊,我現在看到了。我想我並沒有真正理解這個問題。 – tylermac 2010-05-07 14:05:20