2010-05-06 94 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