Ваши данные добавляются с помощью обратного вызова addValueEventListener
В начале вашей функции вы очищаете свой список:
Common.questionList.clear();
Так что Collections.suffle
перетасовывается на вашем пустомтолько список
То, что вы хотите, это поместить эту функцию перемешивания в ваш обратный вызов:
@Override
public void onDataChange(DataSnapshot dataSnapshot) {
for(DataSnapshot postSnapshot : dataSnapshot.getChildren())
{
Question ques = postSnapshot.getValue(Question.class);
Common.questionList.add(ques);
}
// Shuffle it
Collections.shuffle(Common.questionList);
}