Creating substrings array from a string

Question

I have a string that has no spaces and I wanted to create an array that consists of the substrings of the word. For instance, let the string is stackoverflow The array should be like:

[sta, cko, ver, flo, w]

The code that I use is below and that does give me only the first item. Any help will be appreciated.

public static ArrayList<String> getWords(String s){
    ArrayList<String> words = new ArrayList<String>();
    for(int i=0;i<s.length(); i=i+3){
        words.add(s.substring(i, 3));
    }
    return words;
}

berko · Accepted Answer · 2017-05-28 20:35:01Z

3

There are two issues with your code. First, the "3" in

s.substring(i, 3)

means the 3rd index from the beginning of the string, not the 3rd index from i, so you'd want

s.substring(i, i + 3)

Second, if the string is shorter than i + 3, you'll get an exception. To solve this, you can use Math.min. It would look something like this:

public static ArrayList<String> getWords(String s){
ArrayList<String> words = new ArrayList<String>();
for(int i=0;i<s.length(); i=i+3){
    words.add(s.substring(i, Math.min(i + 3, i.length())));
}
return words;

}

answered May 28, 2017 at 20:35

berko

3832 silver badges4 bronze badges

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

Comments

Elliott Frisch · Accepted Answer · 2017-05-28 20:31:11Z

You need to pass i + 3 as the second parameter of the substring call (it takes begin and end indexes). Also, I would prefer to program to the List interface. You might use += instead of i = i + 3. And you need an else clause for when there aren't three letters in the String. Like,

public static List<String> getWords(String s) {
    List<String> words = new ArrayList<>();
    for (int i = 0; i < s.length(); i += 3) {
        if (i + 3 < s.length()) {
            words.add(s.substring(i, i + 3));
        } else {
            words.add(s.substring(i));
        }
    }
    return words;
}

Then, for completeness, I tested it with a basic main method like

public static void main(String[] args) {
    System.out.println(getWords("stackoverflow"));
}

Which outputs (as requested)

[sta, cko, ver, flo, w]

Mohd · Accepted Answer · 2017-05-28 20:31:20Z

1

You are on the right track, but your substring should be (i, i+3) as the following:

public static ArrayList<String> getWords(String s){
    ArrayList<String> words = new ArrayList<String>();
    for(int i=0;i<s.length(); i+=3){
        if (i+3 >= s.length())
            words.add(s.substring(i));
        else
            words.add(s.substring(i, i+3));
    }
    return words;

answered May 28, 2017 at 20:31

Mohd

5,6317 gold badges22 silver badges31 bronze badges

Comments

baao · Accepted Answer · 2017-05-28 20:36:52Z

1

You can ease it significantly by using guava's Splitter:

String f = "stackoverflow";
List<String> p = Splitter.fixedLength(3).splitToList(f);
System.out.println(p); // [sta, cko, ver, flo, w]

answered May 28, 2017 at 20:36

baao

73.5k18 gold badges152 silver badges209 bronze badges

Collectives™ on Stack Overflow

Creating substrings array from a string

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related