Reversing a string java

Question

Working on a problem to reverse a string and I am getting [C@659e0bfd when I try to use the following method to reverse a string. Anyone know what's wrong with this code?

    public static String reverseString(String string) {
    if ((string == null) || (string.length() <= 1)){
        return string;
    }
    char[] charArray = string.toCharArray();
    char[] newString = new char[charArray.length];

    for (int i = 0; i < charArray.length; i++ ){
        newString[i] = charArray[(charArray.length -1)-i];
        }       
    return newString.toString();
}

Answer 1

you are returning array.toString() . if you want to know why it gives C@659e0bfd read this question .

to fix this you can use

return new String(newString);//create string from adding all chars together

public String(char[] chars) is a overloaded constructor of String class which create a new String with array of character

Answer 2

A char[] 's toString() method is not overridden. Which means, it will give back the value provided by Object.toString() method.

You should instead use return new String(newString)

Answer 3

Firstly, all your code, except first if clause can be replaced by only one string:

return new StringBuilder(string).reverse().toString();

Secondly, you don't need nested parentheses in your if clause. So, the method will be the following:

public static String reverseString(String string) {
        if (string == null || string.length() <= 1){
            return string;
        }
        return new StringBuilder(string).reverse().toString();
    }