Possible error in String's split method

Question

I have de next code:

String test = "A,B,,,,,";

String[] result = test.split(",");

for(String s : result){
    System.out.println("$"+s+"$");
}

The output is:

$A$
$B$

and y I expected:

$A$
$B$
$$
$$
$$
$$
$$

but, I modified the code as follows:

String test = "A,B,,,,,C";

String[] result = test.split(",");

for(String s : result){
    System.out.println("$"+s+"$");
}

and the result is:

$A$
$B$
$$
$$
$$
$$
$C$

other variation:

String test = "A,B,,,C,,";

String[] result = test.split(",");

for(String s : result){
    System.out.println("$"+s+"$");
}

the result:

$A$
$B$
$$
$$
$C$

any idea?

I need this for convert csv file to java objects, but when they don't send me the last column the code not work correctly

You can try String[] result = test.split(",",test.length());, and you'll see that you get your expected results. — Alexis C.
– Alexis C., Commented Nov 28, 2013 at 14:51

Jeroen Vannevel · Accepted Answer · 2013-11-28 14:50:26Z

2

From the docs:

This method works as if by invoking the two-argument split method with the given expression and a limit argument of zero. Trailing empty strings are therefore not included in the resulting array.

answered Nov 28, 2013 at 14:50

Jeroen Vannevel

44.6k25 gold badges113 silver badges174 bronze badges

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

2 Comments

victorpacheco3107 Over a year ago

yes, but, if the empty string are not included, the result with the entry String test = "A,B,,,C,,"; should be {A,B,C} ?

Jesper Over a year ago

@victorpacheco3107 No. Read the docs more carefully, it says "trailing empty strings". Not "empty strings anywhere".

Felipe Antonio · Accepted Answer · 2013-11-28 15:14:51Z

1

i did this code right now, it worked fine for me, try this:

String test = "A,B,,,,,";
    int i;
    int countOfCommas = 0;
    int countOfLetters = 0;
    String[] testArray = test.split("");
    String[] result = test.split(",");

    for(i=0;i<=test.length();i++)
        if(testArray[i].equals(","))
            countOfCommas++;

    for(String s : result){
        System.out.println("$"+s+"$");
    }

    if(test.length() > result.length)
        countOfLetters = test.length()-countOfCommas;

    for(i=0;i<(test.length()-countOfLetters)-result.length;i++)
        System.out.println("$$");

edited Nov 28, 2013 at 15:14

answered Nov 28, 2013 at 15:07

Felipe Antonio

2910 bronze badges

Comments

piet.t · Accepted Answer · 2013-11-28 15:14:30Z

0

As Jeroen Vannevel stated this is documented behaviour of String.split(). If you need to kepp all empty Strings just use test.split(",",-1)

answered Nov 28, 2013 at 15:14

piet.t

11.9k21 gold badges45 silver badges56 bronze badges

Comments

victorpacheco3107 · Accepted Answer · 2013-11-28 15:22:48Z

0

I have the solution thanks to @ZouZou, is the next:

String test = "A,B,,,C,,";

String[] result = test.split(",",test.length()); // or the number of elements you expect in the result

for(String s : result){
    System.out.println("$"+s+"$");
}

answered Nov 28, 2013 at 15:22

victorpacheco3107

8623 gold badges11 silver badges35 bronze badges

Collectives™ on Stack Overflow

Possible error in String's split method

4 Answers 4

2 Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related