Please help: works when testing, but not validating 4th requirement.

Question about the task

Resolved

public class Solution { public static void main(String[] args) throws IOException { ArrayList<String> myList = new ArrayList<String>(); BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String s; int i=0; for (i=0;i<10;i++){ s = reader.readLine(); myList.add(s); } int longS = myList.get(0).length(); for (i=1;i<10;i++){ if (myList.get(i).length() < longS){ System.out.println(i); break; } } } }

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 { ArrayList<String> myList = new ArrayList<String>(); BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String s; int i=0; for (i=0;i<10;i++){ s = reader.readLine(); myList.add(s); } int longS = myList.get(0).length(); for (i=1;i<10;i++){ if (myList.get(i).length() < longS){ System.out.println(i); break; } } } }

Comments (3)

Popular
New
Old

You must be signed in to leave a comment

Elvis Lee

Level 41 , Aurora, United States

5 March 2020, 18:00

I think you should assign longS = myList.get(i) after the if loop at line 29, so to proceed the next comparison.

Johannes

Level 27 , Centurion, Pretoria, South-Africa

5 March 2020, 18:01

thank you, I figured it out LOL! Not sure why it's so petty in this specific instance, as it works either way, but I'll submit ;)

Johannes

Level 27 , Centurion, Pretoria, South-Africa

5 March 2020, 17:57

Nevermind. It works when you don't assign a value to longS BEFORE the for loop, but do it in the ELSE section. (longS is defined as int, and assigned a value where I defined it with MAX_VALUE.) for (i=0;i<10;i++){ if (myList.get(i).length() < longS){ System.out.println(i); break; } else{ longS = myList.get(i).length(); } } }