I am using an array of arraylists to store words from a string. I am storing them in the array of array lists by the length of the word. I am using a switch statement right now, but I have to have 45 cases the way I am doing it, and I was wondering if anyone knew of an easier and shorter way that I could carry out the same operation. Here's some code:
String temp;
Scanner sc = new Scanner(str);
while(sc.hasNext())
{
temp = sc.next();
switch(temp.length()){
case 1:
wordsByLen[0].add(temp);
case 2:
wordsByLen[1].add(temp);
case 3:
wordsByLen[2].add(temp);
I have cases 1-45 and a default. I would just like to shorten this up, if possible. Thanks!
asked Mar 21, 2013 at 16:25
mkjo0617
331 gold badge3 silver badges9 bronze badges
6 Answers 6
Don't bother with the switch, just do
wordsByLen[temp.length() - 1].add(temp)
Sign up to request clarification or add additional context in comments.
Comments
String temp;
Scanner sc = new Scanner(str);
while(sc.hasNext())
{
temp = sc.next();
wordsByLen[temp.length()-1].add(temp);
}
answered Mar 21, 2013 at 16:28
scharette
6351 gold badge9 silver badges26 bronze badges
Comments
wordsByLen[temp.length()].add(temp); --- this should do I guess.
Ajinkya
22.7k33 gold badges113 silver badges164 bronze badges
answered Mar 21, 2013 at 16:27
Sudhanshu Umalkar
4,2021 gold badge26 silver badges33 bronze badges
Comments
Why do you need to use switch case over here. try this out :
String temp;
Scanner sc = new Scanner(str);
while(sc.hasNext())
{
temp = sc.next();
if(temp.length > 0) {
wordsByLen[temp.length()-1].add(temp)
}
answered Mar 21, 2013 at 16:29
Ankur Shanbhag
7,8042 gold badges31 silver badges38 bronze badges
Comments
Just use one shorter than the length
int len = temp.length();
wordsByLen[len - 1].add(temp);
answered Mar 21, 2013 at 16:27
Olaf Dietsche
74.4k9 gold badges113 silver badges214 bronze badges
Comments
Instead of switch you can simply use the following:
wordsByLen[temp.length()-1].add(temp);
answered Mar 21, 2013 at 16:28
Vishal K
13.1k2 gold badges30 silver badges38 bronze badges
Comments
lang-java
List<List<String>>? You'd have to do alistlist.get(i).add(temp)...MultiMap<Integer, String>would be easier to use.