Question

I have done a lot with Java but I am currently trying to learn c. I am trying to make a program to convert a number from decimal to binary.

Here is what I have:

#include <stdio.h>
#define LENGTH 33

int main( int argc, char*argv[ ] )

{
 unsigned int number, base, remainder, x, i;
 char result[LENGTH];

 puts( "Enter a decimal value, and a desired base: " );
 scanf( " %u", &number );
 scanf( " %u", &base );

 x = number;

 for( i=0; i < LENGTH; i++ )
 {
  remainder = x % base;
  x = x/base;
  result[i] = remainder;
 }

 printf( "%u equals  %s  (base-%u) \n", number, result, base );    

 //***result is probably backwards...


 return 0;



}

And here is what I get when I run it:

Enter a decimal value, and a desired base: 
5 2
5 equals    (base-2) 

What is the square and how to I get it to display as a string (char array)?

Answer 1

Last Edit: Your program might look like the following:

 unsigned int number, base, remainder, x, i;
 char result[LENGTH+1] = {0};

 puts( "Enter a decimal value, and a desired base: " );
 scanf( " %u", &number );
 scanf( " %u", &base );

 x = number;

 for( i=0; i < LENGTH; i++ )

 {
  // if base > 10, use alphabet!
  result[i] = remainder > 9 ? (remainder-10) + 'A' : remainder + '0';
  x = x/base;
  result[i] = remainder + '0';
 }

---

First, when writing the remainder, you should add the ASCII offset of the digit.

  result[i] = remainder + '0';

Also, you forgot to add'\0' at the end.

 result[i] = 0;
 printf( "%u equals  %s  (base-%u) \n", number, result, base );

---

EDIT: When writing to a c-string, I usually initialize the string to zeros:

char result[LENGTH] = {0};

This way you don’t need to write the null character at the end , it is written for you.

---

Thanks @mmyers for pointing out the overflow 🙂
I would declare the string to hold LENGTH+1:

char result[LENGTH+1] = {0};