Это правильный способ сортировки?

Question

Это правильный метод сортировки ArrayList? Проблема в том, что список не отсортирован.

out = new StringTokenizer(input.toString()); 
n = (out.countTokens()); 
for (int i = 0; i < n; i++) { 
    String[] words = { out.nextToken().toString() }; 
    final List<String> wordList = Arrays.asList(words); 
    Collections.sort(wordList); 
    System.out.println(wordList.toString()); 
} 

Answer 1

Каждых из ваших words[] массивов состоит из одной строки, полученный из следующих маркеров вашего StringTokenizer. И вы повторяете в точном порядке токенизацию. Так что да, ваш выход будет не быть отсортированным. Я полагаю, что вы хотели сделать что-то вроде этого:

out = new StringTokenizer(input.toString()); 
int count = out.countTokens(): 
List<String> wordList = new ArrayList<String>(count); 
for(int i = 0; i < count; i++) { 
    wordList.add(out.nextToken()); 
} 
Collections.sort(wordList); 

Но, не использовать класс Tokenizer, его наследие. Следующий код послужит вам лучше:

List<String> wordList = Arrays.asList(input.split("\\s")); 
Collections.sort(wordList); 

Answer 2

out.nextToken().toString() дает вам одну строку. Я предполагаю, что длина вашего массива должна быть 1. Даже если вы поместите это в цикл, вы сортируете в каждом цикле, вам придется сортировать вне цикла.

StringTokenizer out = new StringTokenizer(input.toString()); 
List<String> wordList = new ArrayList<String>(); 
while(out.hasMoreTokens()) { 
    wordList.add(out.nextToken()); 
} 
Collections.sort(wordList); 
System.out.println(wordList.toString());