Converting a number to binary java code

Question

I am writing a code to convert a number to binary representation.Here's my code.It doesn't give the correct answer but I can't figure out where I am making the mistake.If someone could point out where my mistake and how to correct it I would be grateful.

public class ConvertBinary{
    public static String convertBinary(int n){
        String s="";
        while(n>0){
            s+=(n%2);

            n=n/2;
        }


        int len=s.length();
        String[] binary=s.split("");
        String[] binaryCopy=new String[s.length()];

        for(int i=0;i<len;i++){
            binaryCopy[i]=binary[len-i-1];
            s+=binaryCopy[i];
        }
        return s;

    }

    public static void main (String args[]){
        System.out.println(convertBinary(19));
    }
}

Is there a reason not to use docs.oracle.com/javase/7/docs/api/java/lang/… ? — AntonH
– AntonH, Commented May 14, 2014 at 14:22
There’s even Integer.toBinaryString(…). But if you insist of doing it manually, why are you using s += … with that mind-twisting reverse operation, rather than just using s = … + s to prepend the digit right in the first place? — Holger
– Holger, Commented May 14, 2014 at 14:24

Syam S · Accepted Answer · 2014-05-14 14:56:52Z

1

Apart from all these answers the problem with your code was that you are not clearing the string before reversing it. So before the for loop just put s = "" and you code should work fine.. :)

Based on comment

public class ConvertBinary {
    public static String convertBinary(int n) {
        String s = "";
        while (n > 0) {
            s += (n % 2);

            n = n / 2;
        }

        int len = s.length();
        String[] binary = s.split("");
        String[] binaryCopy = new String[s.length()];

        s = "";

        for (int i = 0; i < len; i++) {
            binaryCopy[i] = binary[len - i - 1];
            s += binaryCopy[i];
        }
        return s;

    }

    public static void main(String args[]) {
        int num = 4;
        System.out.println(convertBinary(num));
        System.out.println(Integer.toBinaryString(num));
    }
}

edited May 14, 2014 at 14:56

answered May 14, 2014 at 14:35

Syam S

8,5071 gold badge29 silver badges37 bronze badges

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

10 Comments

sam_rox Over a year ago

Thanks a lot.Bur=t still one element doesn't get printed.Say I want to convert 4, what it gets printed out is 00, instead of 001

Syam S Over a year ago

Firstly binary of 4 is 100 and not 001. :) Beside that are you sure its printing wrong? I just copy pasted and executed the code and it works fine with 4. It does print 100.

sam_rox Over a year ago

I have to make in the for loop i<=len instead of i<len.Then it works fine.But why?

Syam S Over a year ago

I suppose you need not change anything. have a look at my answer update. Its your same code. I just put that s ="" and used Integer.toBinaryString to compare your result. Its works just fine with i<len

user2901535 Over a year ago

Check these links on split.. stackoverflow.com/questions/10233210/… stackoverflow.com/questions/145509/…

|

pratim_b · Accepted Answer · 2014-05-14 14:32:01Z

1

public static String convertBinary(int n){
        String s="";
        while(n>0){
            s+=(n%2);

            n=n/2;
        }



    return (new StringBuffer(s).reverse().toString());
}

answered May 14, 2014 at 14:32

pratim_b

1,22014 silver badges31 bronze badges

Comments

Dmitrii Bychenko · Accepted Answer · 2014-05-14 14:36:05Z

1

If you're looking for error in your implementation, you'd rather put:

  s = (n % 2) + s;

isntead of

  s+=(n%2);

so the code'll be

  // n should be positive
  public static String convertBinary(int n){
    if (n == 0)
      return "0";

    String s = "";

    // for is more compact than while here
    for (; n > 0; n /= 2)
      s = (n % 2) + s;

    return s;
  }

however in real life

  Integer.toString(n, 2);

is much more convenient

edited May 14, 2014 at 14:36

answered May 14, 2014 at 14:28

Dmitrii Bychenko

188k20 gold badges178 silver badges231 bronze badges

1 Comment

sam_rox Over a year ago

Thanks.I didn't think of prepending the digit

AntonH · Accepted Answer · 2014-05-14 14:24:34Z

0

Use Java standard library: http://docs.oracle.com/javase/7/docs/api/java/lang/Integer.html#toString%28int,%20int%29

public class ConvertBinary{
    public static String convertBinary(int n){
        return Integer.toString(n, 2);
}

    public static void main (String args[]){
        System.out.println(ConveryBinary.convertBinary(19));
    }
}

EDIT: As @Holger says, there's also a toBinaryString: http://docs.oracle.com/javase/7/docs/api/java/lang/Integer.html#toBinaryString%28int%29

public static String convertBinary(int n){
    return Integer.toBinaryString(n);
}

answered May 14, 2014 at 14:24

AntonH

6,4474 gold badges32 silver badges40 bronze badges

Collectives™ on Stack Overflow

Converting a number to binary java code

4 Answers 4

10 Comments

Comments

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

10 Comments

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related