JavaRush/Помощь по Java задачам/Помогите найти ошибку(Или неправ валидатор?)

16 уровень

04.11.2019
438просмотров
17комментариев

Помогите найти ошибку(Или неправ валидатор?)

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

Обсуждается

Убедись, что программа работает правильно, если последовательность состоит из чисел, которые больше 127. ____ Такая ошибка выходит. Почему именно 127? Программу запускал, она делает все правильно

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; import java.util.Scanner; /* Cамая длинная последовательность */ public class Solution { public static void main(String[] args) throws IOException { Scanner sc = new Scanner(System.in); List <Integer> list = new ArrayList<>(); for( int i=0; i<10; i++){ list.add(sc.nextInt()); } List<Integer> maxCounts = new ArrayList<>(); int count = 1; for(int j = 0; j<9;j++){ if(list.get(j)==list.get(j+1) ){ count++; }else{ maxCounts.add(count); count = 1; } } for(int k = 0; k<maxCounts.size()-1;k++){ int max= maxCounts.get(0); if(max <= maxCounts.get(k+1) ){ max = maxCounts.get(k+1); count = max; } } System.out.println(count); } }

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

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

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

IceBerg

Уровень 41

Master

4 ноября 2019, 00:28

Как объекты нужно сравнивать?

Yan

Уровень 16

4 ноября 2019, 00:30

C помощью equals()?

IceBerg

Уровень 41

Master

4 ноября 2019, 00:33

Yes

Yan

Уровень 16

4 ноября 2019, 00:33

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

IceBerg

Уровень 41

Master

4 ноября 2019, 00:35

Так протестируй

А потом можешь погуглить про кеширование значений Integer

Yan

Уровень 16

4 ноября 2019, 00:37

Т.е, значения, которые я храню в Списках, коллекциях, массивах и прочем, это объекты и сравнивать их всегда нужно через equals?

IceBerg

Уровень 41

Master

4 ноября 2019, 00:41

Списки используют дженерики(то что в <>), значит они не могут хранить примитивы, так что здесь ответ однозначно - да, только equals(). Массивы же вполне могут хранить примитивы, так что можно смотреть по ситуации.

Yan

Уровень 16

4 ноября 2019, 00:47

public class Solution {
    public static void main(String[] args) throws IOException {
        Scanner sc = new Scanner(System.in);
        List <Integer> list = new ArrayList<>();
        for( int i=0; i<10; i++){
            list.add(sc.nextInt());
        }
         List<Integer> maxCounts = new ArrayList<>();
         int count = 1;

        for(int j = 0; j<9;j++){

            if(list.get(j).hashCode()==list.get(j+1).hashCode()){
                count++;

            }else{
                maxCounts.add(count);
                count = 1;

            }

        }
        for(int k = 0; k<maxCounts.size()-1;k++){
            int max= maxCounts.get(0);
            if(max <= maxCounts.get(k+1).hashCode()) {
                max = maxCounts.get(k+1);
                count = max;
            }
        }

        System.out.println(count);

    }
}

Исправил, добавил .hashCode(), ничего не изменилось, ошибка осталась той же, пробовал equals(), там где сравнивается на ==, тоже не помогло и ошибка сохранилась Как быть дальше?

IceBerg

Уровень 41

Master

4 ноября 2019, 00:57

Править последний цикл, там просто все элементы постоянно сравниваются с первым.

Yan

Уровень 16

4 ноября 2019, 01:01

Почему постоянно с первым? Я сравниваю с первым только до тех пор, пока не найдется элемент больше, затем присваиваю его max, и сравниваю следующий уже не с первым, а с максимальным if(max <= maxCounts.get(k+1).hashCode()) { max = maxCounts.get(k+1);

IceBerg

Уровень 41

Master

4 ноября 2019, 01:02

Какая у тебя первая строка на каждом шаге цикла?

Yan

Уровень 16

4 ноября 2019, 01:07

Ой, точно, у меня max в цикле, не заметил вынес ее объявление из цикла. Однако даже сейчас не проходит по последнему пункту

Yan

Уровень 16

4 ноября 2019, 01:08

public class Solution {
    public static void main(String[] args) throws IOException {
        Scanner sc = new Scanner(System.in);
        List <Integer> list = new ArrayList<>();
        for( int i=0; i<10; i++){
            list.add(sc.nextInt());
        }
         List<Integer> maxCounts = new ArrayList<>();
         int count = 1;

        for(int j = 0; j<9;j++){

            if(list.get(j).hashCode()==list.get(j+1).hashCode()){
                count++;

            }else{
                maxCounts.add(count);
                count = 1;

            }

        }
        int max= maxCounts.get(0);
        for(int k = 0; k<maxCounts.size()-1;k++){

            if(max <= maxCounts.get(k+1).hashCode()) {
                max = maxCounts.get(k+1);
                count = max;
            }
        }

        System.out.println(count);

    }
}

Такой код получился

IceBerg

Уровень 41

Master

4 ноября 2019, 01:10

Возьми пример выше, 5 повторений - первый элемент, а вот count чему будет равно? Зачем она вообще здесь нужна?

Yan

Уровень 16

4 ноября 2019, 01:17

Count cначала считывает количество повторений, в конце я просто присваваю count = max, это, конечно,бесполезно, можно вывести просто max, однако это ничего не меняет, все равно последний пункт не выполняется

Yan

Уровень 16

4 ноября 2019, 01:20

Еще значения count заполняют список maxCounts, в который я складываю длины всех повторяющихся последовательностей, затем нахожу и вывожу максимальное значение из этого списка

IceBerg

Уровень 41

Master

4 ноября 2019, 01:25

Я именно о её надобности в последнем цикле. Если считаешь, что это ничего не меняет, то лучше пройдись пошагово и подумай дважды. Рано хотеть пройти последний пункт, если у тебя этот пример по прежнему не отрабатывает. Всё я спать)