StringBuffer loop set up

Question

I want to loop through a StringBuffer and use the value of each char to calculate the output. Debug shows that when I pass in 10 that sb.length() is 4 even though it should be 2. Should I be using different methods?

    public double ConvertDecimal(double in) {

    double out = 0;

    sb = new StringBuffer(String.valueOf(Math.floor(in));

    sb.reverse();

    for (int i = 0; i < sb.length(); i++) {

        out += Double.valueOf(sb.charAt(i)) * Math.pow(3,i);

    }

    return Math.floor(out);

}

For passing in "10" the first iteration makes out=48 and I don't understand where it's going wrong when I want the following:

in = 10

sb.reverse = "01"

//first iteration of loop - char at first position of sb.reverse being 0 out += Double.valueOf(sb.chartAt(0)) * Math.pow(3,0);

//second iteration of loop - char at second position of sb.reverse being 1 out += Double.valueOf(sb.chartAt(1)) * Math.pow(3,1);

return out; //out should be 3

Scary Wombat · Accepted Answer · 2014-04-30 01:00:19Z

2

If you look at http://docs.oracle.com/javase/7/docs/api/java/lang/StringBuffer.html#reverse() you will see that reverse returns a new StringBuffer

so change your code to

sb = sb.reverse();

Also as you are using Intergers not Doubles, then why are you casting the char to a double?

try

int x = Character.getNumericValue(sb.charAt(i));
out +=  x * Math.pow(3,i);

edited Apr 30, 2014 at 1:00

answered Apr 30, 2014 at 0:53

Scary Wombat

44.9k7 gold badges38 silver badges67 bronze badges

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

1 Comment

user3152459 Over a year ago

This and then I realized using floor() made sb = "0.001" so the count was 4 (but not 5 with the "."?) Thanks

Dawood ibn Kareem · Accepted Answer · 2014-04-30 00:55:45Z

0

At

Double.valueOf(sb.charAt(i))

the character '0' is cast to a double, but its value is 48, not 0.

Also, you need to use the value returned by sb.reverse(), rather than the original sb.

answered Apr 30, 2014 at 0:55

Dawood ibn Kareem

80.2k15 gold badges103 silver badges112 bronze badges

Collectives™ on Stack Overflow

StringBuffer loop set up

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related