Spent two days on this! Anyone care to help a fellow code lifter?

Question about the task

Under discussion

This is the recommendation I get from the mentor Be sure that the length of the longest sequence is calculated correctly when it is located at the end of the list of entered numbers.f

1. Create a list of numbers.
2. Use the keyboard to add 10 numbers to the list.
3. Display the length of the longest sequence of repeating numbers in the list.

Example for the list 2, 4, 4, 4, 8, 8, 4, 12, 12, 14:
3

The value is 3, because the longest sequence of repeating numbers is three fours.

Requirements:

The program must display a number on the screen.
The program should read values from the keyboard.
In the main method, declare an ArrayList variable with Integer elements and immediately initialize it.
The program should add 10 numbers to the collection in accordance with the conditions.
The program should display the length of the longest sequence of repeating numbers in the list.

package com.codegym.task.task08.task0812; import java.io.*; import java.util.ArrayList; import java.util.Collections; /* Longest sequence */ public class Solution { public static void main(String[] args) throws IOException { BufferedReader br = new BufferedReader(new InputStreamReader(System.in)); ArrayList<Integer> list = new ArrayList<>(); for(int i = 0; i <10; i++){ list.add(Integer.parseInt(br.readLine())); } int count1 = 1; int count = 0; for(int i = 1; i < 10; i++){ if(list.get(i).equals(list.get(i-1))) count1++; if(count1 > count) count = count1; else count1 = 1; } System.out.println(count); } }

Comments (7)

Popular
New
Old

You must be signed in to leave a comment

Guadalupe Gagnon

Level 37 , Tampa, United States

31 January 2019, 14:32

The code keeps resetting count1 to one. Copy and paste this code into your IDE. It is your same exact code with a modification at the top so you don't have to manually enter numbers each time you hit 'Run', and sys.out statements through to check status.

public class Solution {
    public static void main(String[] args) throws IOException {
        //BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
        int[] temp = {1,1,2,2,2,3,3,3,3,3}; // <- added
        ArrayList<Integer> list = new ArrayList<>();

        for(int i = 0; i <10; i++){
            list.add(temp[i]);
            //list.add(Integer.parseInt(br.readLine()));
        }

        int count1 = 1;
        int count = 0;

        for(int i = 1; i < 10; i++){
            System.out.print("Loop cycle " + i + ": ");
            System.out.print("checking if " + list.get(i) +" equals " + list.get(i-1)+"? ");
            System.out.print(list.get(i).equals(list.get(i-1)));

            if(list.get(i).equals(list.get(i-1))) {
                System.out.print(": count1 increased by 1");
                count1++;
            }
            if(count1 > count) {
                System.out.println(": count now equals count1");
                count = count1;
            }else {
                System.out.println(": count1 reset to 1");
                count1 = 1;
            }
            System.out.println("End of loop cycle: "+i+": count1 value = "+ count1+" - count value = "+count);
        }
        System.out.println(count);
    }
}

Guadalupe Gagnon

Level 37 , Tampa, United States

31 January 2019, 14:33

The correct output, which is the last line output, would be 5

Darko Jakimovski

Level 18 , Kriva Palanka, Macedonia, The Former Yugoslav Republic of

31 January 2019, 18:07

Thanks for the effort man, as always! Keep it up! But I really don't get it why during the first loop count1 is not set to 1 :/ Even more why does it execute the else statement when the if returns true Sigh...

Darko Jakimovski

Level 18 , Kriva Palanka, Macedonia, The Former Yugoslav Republic of

31 January 2019, 18:14

I solved it. OMG I'm always overthinking it. Thanks ALOT MAN! Captain Guadalupe! Long live!

Guadalupe Gagnon

Level 37 , Tampa, United States

31 January 2019, 18:48

Congrats! Overthinking is always a problem, I even get caught up in it still. My friends, who are professional coders, say that I should focus on just getting the code working before I focus on making it clean and efficient. But ya, I understand the frustration when you are first learning to code. There are a lot of blank spots where you just are not sure how things work, and using google would be easy if only you knew what to ask! As you practice, though, a lot of these concepts start coming together and they make sense on why they are implemented the way they are. Hang in there and keep chugging along. I am sure if you look back at the first couple of levels all the tasks seem really easy now; the same thing will happen when you look back at these levels when you are in you're 6 levels ahead.

Guadalupe Gagnon

Level 37 , Tampa, United States

31 January 2019, 18:51

I can recommend taking a little time to understand the code that i shared, with the sys.out statements above. This is a good way to troubleshoot code, and understand what the code is doing. Start small with practicing this in your future code, when you start implementing it successfully it will be a BIG help.

Darko Jakimovski

Level 18 , Kriva Palanka, Macedonia, The Former Yugoslav Republic of

31 January 2019, 19:00

I'm always trying to figure out what's happening with my code by printing out some of the lines in the code, but never every little thing like you did, which was a big mistake on my side