clean code wise it would be nicer to have that method return true or false and then go there with    String separator = listConsistsOfSingleCharacters(userListStringValues) ? "" : " ";    because method name implies to me that it would return true or false, not a separator.