Форматирование кода для инициализации списков - PullRequest
0 голосов
/ 06 мая 2010

Я только что нашел в моем java проекте этот фрагмент кода:

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

Как вы думаете, можно ли использовать этот стиль?

Ответы [ 3 ]

2 голосов
/ 06 мая 2010

Я думаю, что это будет намного яснее и менее подвержено проблемам, когда другой разработчик смотрит на это.

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

Всегда используйте скобки для уточнения петель. Не делайте отступ для оператора for, так как это противоречит стандартной практике.

2 голосов
/ 06 мая 2010

Имхо, этот стиль, кажется, показывает иерархию, которой нет. Также я бы рекомендовал всегда использовать фигурные скобки, и я хотел бы разделить объявления и код пустой строкой, поэтому я бы использовал:

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

for (Object item : selectedItems) {
    selectedIssueTypes.add((IssueType) item);
}
1 голос
/ 07 мая 2010

Почему бы просто:

List<IssueType> selectedIssueType = Arrays.asList(selectedItems);
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...