Converting string array to list: double brackets

Question

I use this part of the code to get the select items from a jlist into a string list. I get the following result:

[[String1, String2, String3,...]]

How can I avoid the double []? Thanks

static List<String> strlist = new ArrayList<String>();

public class tog {

        List<String> strlisttemp = new ArrayList<String>();

        final JList list = new JList(strlisttemp.toArray());
        list.setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);

        final ListSelectionListener listSelectionListener = new ListSelectionListener() {
            public void valueChanged(ListSelectionEvent e) {

                String lstr = list.getSelectedValuesList().toString();
            System.out.println(lstr);
                strlist = Arrays.asList(lstr.split("\\s*,\\s*"));

            }
        };

        list.addListSelectionListener(listSelectionListener);
        JOptionPane.showMessageDialog(null, list, "Select", JOptionPane.QUESTION_MESSAGE);

        System.out.println(strlist);

}

This is the part that has problem: When I print lstr it works properly [...]. When I use this:

strlist = Arrays.asList(lstr.split("\\s*,\\s*"));

Then System.out.println(strlist); prints double brackets

I think your missing the point list.getSelectedValuesList().toString(). You are not using the generic interface for JList and you are parsing the String when you can get a List<E> from getSelectedValuesList(). — Boris the Spider
– Boris the Spider, Commented Oct 3, 2013 at 12:05
Works fine otherwise: String str = "abc,def"; System.err.println(Arrays.asList(str.split("\\s*,\\s*"))); — Optional
– Optional, Commented Oct 3, 2013 at 12:24
What do you mean it works fine? Strlist shows the results like this [[aaa,bbb]]. I need [aaa,bbb]. @Optional — user2598911
– user2598911, Commented Oct 3, 2013 at 15:17
I believe the code that is vital in solving the problem is clearly readeable. @tom — user2598911
– user2598911, Commented Oct 3, 2013 at 15:19

rob · Accepted Answer · 2013-10-04 16:38:41Z

2

You're unnecessarily converting the returned List to its raw String representation, splitting that String into a String[], then converting the String[] to List.

Instead, just work directly with the List which is returned.

package com.example.list;

import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collections;
import java.util.List;

import javax.swing.JList;
import javax.swing.JOptionPane;
import javax.swing.ListSelectionModel;
import javax.swing.event.ListSelectionEvent;
import javax.swing.event.ListSelectionListener;

public class ListExample {
    static List<String> strlist = new ArrayList<String>();

    public static void main(String[] args) {
        List<String> strlisttemp = new ArrayList<String>();
        strlisttemp.add("a");
        strlisttemp.add("b");
        strlisttemp.add("c");

        final JList list = new JList(strlisttemp.toArray());
        list.setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);

        final ListSelectionListener listSelectionListener = new ListSelectionListener() {
            public void valueChanged(ListSelectionEvent e) {

                strlist = list.getSelectedValuesList(); // use this instead

//              String lstr = list.getSelectedValuesList().toString();
//              strlist = Arrays.asList(lstr.split("\\s*,\\s*"));

                System.out.println(strlist);
            }
        };

        list.addListSelectionListener(listSelectionListener);
        JOptionPane.showMessageDialog(null, list, "Select", JOptionPane.QUESTION_MESSAGE);


    }
}

edited Oct 4, 2013 at 16:38

answered Oct 4, 2013 at 16:33

rob

6,2873 gold badges39 silver badges57 bronze badges

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

2 Comments

user2598911 Over a year ago

Yes but I need the info to end up a str array list. Thats why I use: strlist = Arrays.asList(lstr.split("\\s*,\\s*")); @rob

rob Over a year ago

I'm not following. Do you need it in a String[] or a List<String>?

dcsohl · Accepted Answer · 2013-10-04 16:36:30Z

2

When you call List.toString it puts brackets around the result.

So you call list.getSelectedValuesList() and you are retrieving a List<String>. You call toString() on this which gives you "[a, b, c, d]". You then split this string on the comma-space sequence and get "[a", "b", "c", "d]". You put this into another list and call toString() on it and get another bracket around it.

It's double-bracketed because in this second list the first entry is "[a" and the last entry is "d]"!

answered Oct 4, 2013 at 16:36

dcsohl

7,4261 gold badge30 silver badges45 bronze badges

Comments

StormeHawke · Accepted Answer · 2013-10-04 16:39:02Z

This is to answer your question on why you're getting the double square brackets. For a better way to code this, see rob's answer.

String lstr = list.getSelectedValuesList().toString();

At this point, lstr == "[aaa,bbb,ccc]"

strlist = Arrays.asList(lstr.split("\\s*,\\s*"));

Now at this point, the first element of strlist will start with '[' because that is the first character of lstr, and the last element will end with ']' because that is the last character of lstr.

Now, when you go to print strlist

System.out.println(strlist);

Java will implicitly be calling list.toString(), which appends a set of square brackets when printing the list. Since the first element in your list starts with a square bracket, and the last element ends with one, what you end up with is a double bracket at the beginning and the end

Collectives™ on Stack Overflow

Converting string array to list: double brackets

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related