да что она от меня хочет.

Вопрос по задаче

Решен

Я не понимаю, почему вывод - единица

1. Создай список чисел.
2. Добавь в список 10 чисел с клавиатуры.
3. Вывести на экран длину самой длинной последовательности повторяющихся чисел в списке.

Пример для списка 2, 4, 4, 4, 8, 8, 4, 12, 12, 14:
3

Искомое значение равно 3, т.к. самая длинная последовательность повторяющихся чисел состоит из трех четверок.

Требования:

Программа должна выводить число на экран.
Программа должна считывать значения с клавиатуры.
В методе main объяви переменную типа List с типом элементов Integer и сразу проинициализируй ee.
Программа должна добавлять в коллекцию 10 чисел, согласно условию.
Программа должна выводить на экран длину самой длинной последовательности повторяющихся чисел в списке.

package com.javarush.task.task08.task0812; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.ArrayList; import java.util.List; /* Cамая длинная последовательность */ public class Solution { public static void main(String[] args) throws IOException { int count = 1; int max = 1; List<Integer> list = new ArrayList<Integer>(); BufferedReader rd = new BufferedReader(new InputStreamReader(System.in)); for (int i = 0; i < 10; i++){ list.add(Integer.parseInt(rd.readLine())); } for (int i = 0; i < list.size() - i; i++){ if (list.get(i).equals(list.get(i+1))){ count++; } else if (!(list.get(i).equals(list.get(i+1)))) { max = count; count = 1; } } if (max >= count) System.out.println(max); else System.out.println(count); } }

Комментарии (6)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Александр

Уровень 41

Expert

16 августа 2019, 11:21

полезный

i < list.size() - i

Это что?

for (int i = 0; i < 10; i++){   // тут все верно.
            list.add(Integer.parseInt(rd.readLine()));
        }
        for (int i = 0; i < list.size() - i; i++){ // list.size = 10 при i = 5 цикл завершится.
            if (list.get(i).equals(list.get(i+1))){
                count++;
            }
            else if (!(list.get(i).equals(list.get(i+1)))) { // это условие выполнено уже раз ты попал в else
                max = count; // если взять из примера строку там три 4-ки в позиции 1, 2, 3, а на 4 ты
                                      //перезапишешь значение max
                count = 1;
            }
        }
        if (max >= count) System.out.println(max);
        else System.out.println(count);

Леонид

Уровень 15

16 августа 2019, 11:24

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

Александр

Уровень 41

Expert

16 августа 2019, 11:27

полезный

Надо писать иначе. Почему выход за границы происходит? Потому что i становится больше или равна размеру листа, но как это может быть если в условии i < list.size()? Это все из-за того что ты в коде прибавляешь единицу к i.

list.get(i+1)

Тут 2 выхода. Либо в условии написать i < list.size() - 1; либо i = 1 вместо 0 и использовать i - 1.

Леонид

Уровень 15

16 августа 2019, 11:33

public class Solution {
    public static void main(String[] args) throws IOException {
        int count = 1;
        int max = 1;
        List<Integer> list = new ArrayList<Integer>();
        BufferedReader rd = new BufferedReader(new InputStreamReader(System.in));
        for (int i = 0; i < 10; i++){
            list.add(Integer.parseInt(rd.readLine()));
        }
        for (int i = 0; i < list.size() - 1; i++){
            if (list.get(i).equals(list.get(i+1))){
                count++;
            }
            else  {
                max = count;
                count = 1;
            }
        }
        if (max >= count) System.out.println(max);
        else System.out.println(count);



    }

Леонид

Уровень 15

16 августа 2019, 11:33

не понимаю до конца, работает наполовину правильно

Александр

Уровень 41

Expert

16 августа 2019, 11:43

решение

То что у тебя происходит в коде. Тебе нужно сделать проверку на если count больше max то max = count.

2,   max = 1, count = 1;
 4,   max = 1, count = 2;
 4,   max = 1, count = 3;
 4,   max = 3, count = 1;
 8,   max = 3, count = 2;
 8,   max = 2, count = 1;
 4,   max = 1, count = 1;
 12,   max = 1, count = 2;
 12,   max = 2, count = 1;
 14   max = 1, count = 1;