Array of ArrayLists?

Question

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!

You want to execute each case? If not write break after each case. — Achintya Jha
– Achintya Jha, Commented Mar 21, 2013 at 16:27
Why not List<List<String>>? You'd have to do a listlist.get(i).add(temp)... — ppeterka
– ppeterka, Commented Mar 21, 2013 at 16:27
@CPerkins: A MultiMap<Integer, String> would be easier to use. — jlordo
– jlordo, Commented Mar 21, 2013 at 16:29

Zutty · Accepted Answer · 2013-03-21 16:27:32Z

3

Don't bother with the switch, just do

wordsByLen[temp.length() - 1].add(temp)

answered Mar 21, 2013 at 16:27

Zutty

5,37729 silver badges32 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

scharette · Accepted Answer · 2013-03-21 16:28:00Z

3

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

Ajinkya · Accepted Answer · 2013-03-21 16:29:03Z

3

wordsByLen[temp.length()].add(temp); --- this should do I guess.

edited Mar 21, 2013 at 16:29

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

Ankur Shanbhag · Accepted Answer · 2013-03-21 16:29:35Z

2

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

Olaf Dietsche · Accepted Answer · 2013-03-21 16:27:56Z

2

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

Vishal K · Accepted Answer · 2013-03-21 16:28:35Z

1

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

Collectives™ on Stack Overflow

Array of ArrayLists?

6 Answers 6

Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related