is there a better way?

Question about the task

Archived

This seems to work, but it also seems quite clumsy running line 23 and 24 over and over again? Is there a better way of doing this?

1. Create a list of strings.
2. Read 5 strings from the keyboard and add them to the list.
3. Using a loop, find the shortest string in the list.
4. Display the string.
5. If there is more than one, display each on a new line.

Requirements:

Declare a string list variable and immediately initialize it.
The program should read 5 strings from the keyboard and write them to the list.
The program should display the shortest string.
If several strings are the shortest, then you need to display each of them on a new line.

package com.codegym.task.task07.task0709; import java.io.BufferedReader; import java.io.InputStreamReader; import java.util.ArrayList; /* Expressing ourselves more concisely */ public class Solution { public static void main(String[] args) throws Exception { //write your code here BufferedReader nancy =new BufferedReader(new InputStreamReader(System.in)); ArrayList<String> arr =new ArrayList<String>(); int shortCat=0; for(int i=0;i<5;i++){ String inString =nancy.readLine(); arr.add(inString); if(shortCat==0) shortCat=inString.length(); if(inString.length()<shortCat) shortCat=inString.length(); } for(int i=0;i<arr.size();i++){ if(arr.get(i).length()==shortCat) System.out.println(arr.get(i)); } } }

Comments (1)

Popular
New
Old

You must be signed in to leave a comment

Nouser

Level 36 , Germany

30 July 2020, 09:39

to get rid of these two lines initialize shortCat with Integer.MAX_VALUE

int shortCat=Integer.MAX_VALUE;