CodeGym/Help with Java Tasks/Need some help

Level 17

Taipei

04.11.2018
1383views
10comments

Need some help

Question about the task

Resolved

I don't know how to set my condition in line 25, because it shows "java.lang.IndexOutOfBoundsException " Can anyone give me some advice? Thanks a lot~

1. Read 10 words from the keyboard and add them to a list of strings.
2. Determine whether the list is ordered by increasing string length.
3. If it is not, then display the index of the first element that violates this order.

Requirements:

Declare a string list variable and immediately initialize it.
Read 10 lines from the keyboard and add them to the list.
If the list is ordered by increasing string length, then you don't need to display anything.
If the list is not ordered by increasing string length, then display the index of the first element that violates this order.

package com.codegym.task.task07.task0718; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.ArrayList; /* Checking the order */ public class Solution { public static void main(String[] args) throws IOException { //write your code here BufferedReader br =new BufferedReader(new InputStreamReader(System.in)); ArrayList<String> list =new ArrayList<String>(); for(int i=0;i<10;i++){ list.add(br.readLine()); } for(int i=0; i<=list.size(); i++){ int strSizea=list.get(i).length(); int strSizeb=list.get(i+1).length(); if(strSizea>strSizeb) System.out.println(list.indexOf(list.get(i))); } } }

Comments (10)

Popular
New
Old

You must be signed in to leave a comment

Mohammed Amine EL AMRANI

Level 11 , Rabat, Morocco

22 December 2018, 16:48

Hello, change this :

System.out.println(list.indexOf(list.get(i)));

with

System.out.println(list.indexOf(list.get(i+1)));

Xubres

Level 16 , Uranus, Space

13 December 2018, 20:38

useful

In the last iteration when i = list.size + 1, it will exceed the size of the array so the program will crash. You need to add a way to exit the loop immediately after the condition is met. Also you might want to change the i <= list.size() to before the last element, for the same reason.

Level 17 , Taipei, Taiwan(is a country different from China)

28 December 2018, 12:08

Thank you, I've solved this question by your reply, thanks a lot!!

Janusz

Level 11 , Radomsko, Poland

12 December 2018, 21:46

Try in the line 23 instead of i<=list.size() insert i<=list.size()-1

Daniel Knox

Level 9 , Annfield Plain, United Kingdom

10 December 2018, 17:59

int strSizeb=list.get(i+1).length(); is fine until you get to last iteration of for loop then its out of bounds i < list.size() -1 should do it

kapil

Level 12 , delhi, India

4 November 2018, 08:55

try in for loop.....i < list.size()-1;

Level 17 , Taipei, Taiwan(is a country different from China)

4 November 2018, 16:04

for(int i=0; i<=list.size()-1; i++){
            int strSizea=list.get(i).length();
            int strSizeb=list.get(i+1).length();
            if(strSizea>strSizeb)
                System.out.println(list.indexOf(list.get(i)));
}

It's still not working~

kapil

Level 12 , delhi, India

5 November 2018, 09:36

try not using indexof.....just print(list.get(i))

Mohammed Amine EL AMRANI

Level 11 , Rabat, Morocco

22 December 2018, 16:49

Hello,

print(list.get(i))

will print the value, in the task condition we need the index of the first element.

Level 17 , Taipei, Taiwan(is a country different from China)

28 December 2018, 12:10

Thank you all, I've fixed it~