Почему не изменяется значение переменной?

Question

[BitNeBolt]

В главной активити, в методе OnCreate() загружается файл .txt в ArrayList. Этот массив содержит в себе Предложения из текста.

Я создал отдельный класс с методом CreateWords(), цель которого - выбрать одно предложение из текста, 3 слова, которых нет в нем, и 1 слово, которое есть.

Но при запуске приложения может быть и такое, что из 4 слов 2, а то и 3 могут быть в нужном предложении.

Вот моя попытка, но она не работает. Не могли бы вы подсказать, что не так?

public static class WordInText
    {
        public static int right_sent_index; //индекс нужного предложения в массиве предложений
        public static ArrayList<String> right_array; //нужный массив, в котором предложение разбито на слова
        public static int right_word_index; //нидекс нужного слова в нужном массиве

        private static int random_sent_index; //случайное предложение
        private static int random_word_index; //индекс случайного слова в случ. предлодении
        private static ArrayList<String> random_array; //массив, аналогичный roght_array

        //flag - индекс в массиве out[] слова, которое должно быть в предложении
        public static String[] CreateWords(int flag)
        {
            right_sent_index = TaskGenerator.random.nextInt(MainActivity.sentences.size());
            right_array = new ArrayList<String>(Arrays.asList(MainActivity.sentences.get(right_sent_index).toLowerCase().split(" ")));
            right_word_index = TaskGenerator.random.nextInt(right_array.size());

            String[] out = new String[4];

            for (int i = 0; i < 4; i++)
            {
                if (i == flag)
                {
                    out[i] = (right_array.get(right_word_index));
                }

                else
                {
                    random_sent_index = TaskGenerator.random.nextInt(MainActivity.sentences.size());

                    //чтобы случайное предложение не было нужным
                    while (random_sent_index == right_sent_index)
                    {
                        random_sent_index = TaskGenerator.random.nextInt(MainActivity.sentences.size());
                    }

                    random_array = new ArrayList<String>(Arrays.asList(MainActivity.sentences.get(random_sent_index).toLowerCase().split(" ")));
                    random_word_index = TaskGenerator.random.nextInt(random_array.size());
                    String random_word = random_array.get(random_word_index);

                    //одна из попыток
                    for (int j = 0; j < 5; j++)
                    {
                        //if (right_array.contains(random_array.get(WordInText.random_word_index)))// & Arrays.asList(out).contains(random_array.get(random_word_index).toLowerCase()))
                        //if (TaskGenerator.ArrayListContains(WordInText.right_array, random_word))
                        if (right_array.contains(random_word))
                        {
                            //Toast.makeText(context, "AGAIN", Toast.LENGTH_SHORT).show();
                            WordInText.random_word_index = TaskGenerator.random.nextInt(random_array.size());
                        }
                        else
                        {
                            break;
                        }
                    }

                    /*while (right_array.contains(random_word))
                    {
                        WordInText.random_word_index = TaskGenerator.random.nextInt(random_array.size());
                    }*/

                    out[i] = random_array.get(random_word_index);
                }
            }

            return out;
        }
    }

P.S. версия с циклом for выдает такой же результат, как и while. Почему же значение переменной индекса не изменяется?

Comments

[Alex Avaj]

Не мог бы ты подробнее описать что именно тебе нужно. Я не понимаю задачу. Что подразумевается под "выбрать одно предложение из текста, 3 слова, которых нет в нем, и 1 слово, которое есть"?
Если можешь просто скопипастить текст задачи кидай сюда
Не совсем понимаю почему не написать out[flag] = (right_array.get(right_word_index)); вместо цикла?
В общем я бы помог, но просто не могу понять что именно требуется чтобы было в out по итогу

[BitNeBolt]

Да, конечно.

Мне нужно понять, почему массив out[] иногда включает в себя строки (слова), которые уже есть в массиве right_array. То есть мне нужно выбрать из массива строк sentences (который принадлежит MainActivity) нужное мне предложение, затем из того же массива выбрать 4 слова: 3 из них нету в нужном предложении, а 1 есть.

Я сделал проверку правильности условия с помощью Toast, и, вроде бы, оно правильно определяет ситуации, когда выбранное слово есть в right_array. Поэтому, мне кажется, что не изменяется значение right_word_index. Также, когда использовался цикл while, то программа зависала в таких ситуациях.

Массив sentences содержит в себе строки предложений, а разбиваются они на массивы слов ужев методе CreateWords().

[Alex Avaj]

for (int j = 0; j<4;j++){
String wordToAdd = random_array.get(j);
if(!right_array.contains(wordToAdd)){
out[i] = wordToAdd
if (i==3){
break;
}
}
}
Не знаю правильно понял или нет, но вот на скорую руку сделал, попробуй, в много чего удалить можно

[BitNeBolt]

Спасибо!

Ваше решение мне помогло, я добился нужного результата!

[Alex Avaj]

NeMalenKihren, не за что.
Вот набросал пример, так как всех условий задачи не знаю, т.е. мне не известно что гарантируется в данной задаче.
Ниже реализация для наиболее общего случая. То есть получаем массив всех слов из всех предложений:
List sentence = new ArrayList<>();
sentence.add("Раз два три четыре");
sentence.add("Четыре Пять шесть семь");
sentence.add("Восемь деввять десять одинадцать");
sentence.add("Раз два три четыре");
sentence.add("Раз два три четыре");

List allWords = new ArrayList<>();

for (String split: sentence) {
String [] s = split.toLowerCase().split(" ");
allWords.addAll(Arrays.asList(s));
}
далее простым циклом добавляем в out те слова которые не встречаются в нужном предложении

Answer 1

[Alex Avaj]

То есть все что после else удалить нужно, оставить только return out. Все или почти все индексы не нужны по моему