Проблемы с перетасовкой символов в строке

Question

Я хочу перетасовать определенную строку для задания. Когда я пытаюсь это сделать, он бросает NullPointerException.

for (char d : masterCode.toCharArray()) { 
    possibleLetters.add(d); 
} 
Collections.shuffle(possibleLetters); 
for (char c : possibleLetters) { // arraylist of characters 
    sb.append(c); //stringbuilder 
    masterCode = sb.toString(); 
} 

Так что я попытался это по-другому:

masterCode = masterCode.substring(0, 4); 
for (char d : masterCode.toCharArray()) { 
    possibleLetters.add(d); 
} 
Collections.shuffle(possibleLetters); 
for (char c : possibleLetters) { // arraylist of characters 
    //sb.append(c); //stringbuilder 
    masterCode = masterCode.concat(Character.toString(c)); // string 
} 

, но это не дает ожидаемого результата либо. masterCode - ECAB сначала, затем для следующего вывода CCBA.

masterCode - это строка из произвольно сгенерированных символов, не допускающая дубликатов, в другом методе.

Вопрос: что это хороший способ исправить это?

Answer 1

В примере один, вы уверены, что sb инициализирован с использованием StringBuilder sb = new StringBuilder();? Проверьте трассировку стека за ошибку, она, вероятно, показывает, что sb имеет значение NULL.

В примере две строки masterCode = masterCode.concat(Character.toString(c)); добавляет ваших персонажей. Сначала вам нужно установить masterCode = "". Согласно the docs, этот метод:

Объединение указанной строки в конец этой строки.