Cannot get String.replace() to replace the only certain letters in string [duplicate]

Question

I need this program to replace all r's with h's if they follow a vowel. This is just a test program, my actual assignment is to replace all the r's in the "Jaws" script with h's that follow a vowel, and do other various task to that string.

    public static void main(String[] args) {
        String s = "Hey, I'm from boston. harbor, fotter, slobber, murder.";
        System.out.println(replace(s));

    }

    //this method should replace r with h if it follows a vowel.
    public static String replace(String s) {
        String newS = "";
        String vowels ="aeiouAEIOU";
        for (int i = 1; i < s.length(); i++) {
            if (s.charAt(i) == 'r' && isVowel(s.charAt(i-1))) {
                newS = s.replace("r", "h");
            }   
        }
        return newS;
    }
    //this method will check if a character is a vowel or not.
    public static Boolean isVowel(char s) {
        String vowels="aeiouAEIOU";
        if (vowels.contains("" + s)) {
            return true;
        }
        return false;
    }
}

What does your code produce instead of what expected output, have you debugged your code using breakpoints and / or System.out.println-Statements? — luk2302
– luk2302, Commented Oct 23, 2017 at 13:14
replace will replace all occurrence instead of particular one — Pavneet_Singh
– Pavneet_Singh, Commented Oct 23, 2017 at 13:15
What @Pavneet_Singh means is that newS = s.replace("r", "h") will replace all occurrences of "r" by "h" as explained here. — Bernat
– Bernat, Commented Oct 23, 2017 at 13:18
Sorry for forgetting to mention. My code currently replaces all r's with h's regardless of what character comes before them. I will try debugger. Is there a String.replace function that will only replace particular occurrences rather than all? — John Antonio Mancini
– John Antonio Mancini, Commented Oct 23, 2017 at 13:19

shakeel · Accepted Answer · 2017-10-23 13:29:02Z

0

Please use String builder to replace at specific index as said Replace a character at a specific index in a string? Below how your replace method should look like

public static String replace(String s) {
StringBuilder myName = new StringBuilder(s);

for (int i = 1; i < s.length(); i++) {
  if (s.charAt(i) == 'r' && isVowel(s.charAt(i - 1))) {
    myName.setCharAt(i, 'h');

  }
}
return myName.toString();

}

edited Oct 23, 2017 at 13:29

answered Oct 23, 2017 at 13:23

shakeel

1,6952 gold badges14 silver badges14 bronze badges

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

2 Comments

John Antonio Mancini Over a year ago

This worked, thanks!

shakeel Over a year ago

@John Antonio please accept it as answer. Thank you

Collectives™ on Stack Overflow

Cannot get String.replace() to replace the only certain letters in string [duplicate]

1 Answer 1

2 Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Linked

Related