Builder

I've constructed a method that combines the use of a string tokeniser and string builder to translate words in a string to other words. This method though just seems to produce errors. Can someone take a look at this method and please explain what it is I'm doing wrong.

Code:

---

public void String replacePirate(String +pirate+, String +english+) {    (class, interface or enum expected)
    StringBuilder result = new StringBuilder(original.length());
    String delimiters = "+-*/(),. ";
    StringTokenizer st = new StringTokenizer(+pirate+, delimiters, true);
    while (st.hasMoreTokens()) {
        String w = st.nextToken();
        if (w.equals(find)) {
            result.append(+english+);
        } else {
            result.append(w);
        }
    }
    return result.toString();
}

---

Quote:

---

public void String replacePirate(String +pirate+, String +english+) {

---

I think you mean:

Code:

---

public String replacePirate(String pirate, String english) {

---

Quote:

---

Originally Posted by AgentApe

Code:

---

public void String replacePirate(String [COLOR="Red"]+[/COLOR]pirate[COLOR="#ff0000"]+[/COLOR], String [COLOR="#ff0000"]+[/COLOR]english[COLOR="#ff0000"]+[/COLOR]) {    (class, interface or enum expected)
    StringBuilder result = new StringBuilder(original.length());
    String delimiters = "+-*/(),. ";
    StringTokenizer st = new StringTokenizer([COLOR="Red"]+[/COLOR]pirate[COLOR="#ff0000"]+[/COLOR], delimiters, true);
    while (st.hasMoreTokens()) {
        String w = st.nextToken();
        if (w.equals(find)) {
            result.append([COLOR="#ff0000"]+[/COLOR]english[COLOR="#ff0000"]+[/COLOR]);
        } else {
            result.append(w);
        }
    }
    return result.toString();
}

---

---

Why do you have "+" sign around your parameter names? That could be causing one of the problems.

Another problem might be that the variables "original" and "find" are never declared. I would presume these are fields in your class, but just checking.

I have fixed what you have said I shoudl and still get the same error. Hmm.

OK, after actually reading your code, your method has two return types "void", and "String". Since you want to return a String, you should change the return type to String.

As for the error, I presume the method is inside a Class, as Java requires.

Code:

---

import java.util.StringTokenizer;

public class Test
{
        public String replacePirate(String pirate, String english)
        {
                StringBuilder result = new StringBuilder(original.length());
                String delimiters = "+-*/(),. ";
                StringTokenizer st = new StringTokenizer(pirate, delimiters, true);
                while (st.hasMoreTokens()) {
                        String w = st.nextToken();
                        if (w.equals(find)) {
                                result.append(english);
                        } else {
                                result.append(w);
                        }
                }
                return result.toString();
        }
}

---

For the above code, I only get two errors, which is that "original" and "find" are never declared.