Concatenating a char in a recursive function in C

Question

Even though I have more experience with higher level languages, I am having a lot of troubles understanding how memory allocation and how strings really work in C.

I am trying to implement a very simple base converter that works recursively. The only thing is that it should return a char* instead of an int

Here is my code. I already tested the recursive calls and it works if I use integers. So, the problem is definitely with the string part. It gives me an infinite loop.

char* baseConversion(int num, int baseIn, int baseOut){

    //convert num to base ten

    int quotient = num / baseOut;

    int remainder = num % baseOut;

    char rem = (char)(((int)'0') + remainder);

    char *result = malloc(strlen(output) + 1);

    strcpy(result, rem);

    if (quotient == 0)
        return result;
    else
        return strcat(result, baseConversion(quotient, baseIn, baseOut));
}

Many thanks

The second argument of strcpy must be a null-terminated string. rem is a char, not a string. — Barmar
– Barmar, Commented Oct 25, 2014 at 4:09
Sixth line states that there is an infinite loop super cool guy ;) — Luis Lavieri
– Luis Lavieri, Commented Oct 25, 2014 at 4:10
@CoolGuy: A question doesn't need to end with a question mark sign. — dotNET
– dotNET, Commented Oct 25, 2014 at 4:10

Barmar · Accepted Answer · 2014-10-25 04:12:44Z

1

Change:

strcpy(result, rem);

to:

result[0] = rem;
result[1] = 0;

This will create a single-character string containing the character in rem.

You also may need to fix:

malloc(strlen(output)+1)

as there's no variable named output in your function.

answered Oct 25, 2014 at 4:12

Barmar

789k57 gold badges555 silver badges669 bronze badges

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

Comments

Vlad from Moscow · Accepted Answer · 2014-10-25 05:01:19Z

0

If I have understood correctly what you are saying about then what you need is the following

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

char * baseConversion( unsigned int x, unsigned int base )
{
    unsigned int digit;
    char *p = NULL;
    size_t n = 2;

    if ( base > 10 || base < 2 ) base = 10;

    digit = x % base;

    if ( x / base ) p = baseConversion( x / base, base );

    if ( p ) n += strlen( p );

    p = realloc( p, n );

    *( p + n  - 2 ) = digit + '0';
    *( p + n  - 1 ) = '\0';

    return p;
}   


int main(void) 
{
    unsigned int x = 255;

    char *p = baseConversion( x, 10 );

    printf( "%s\n", p );

    free( p );

    p = baseConversion( x, 8 );

    printf( "%s\n", p );

    free( p );

    p = baseConversion( x, 2 );

    printf( "%s\n", p );

    free( p );

    return 0;
}

The output is

255
377
11111111

P.S. It is funny when one answer is marked as the best but the code will be used from other answer.:)

edited Oct 25, 2014 at 5:01

answered Oct 25, 2014 at 4:54

Vlad from Moscow

313k27 gold badges204 silver badges358 bronze badges

2 Comments

Luis Lavieri Over a year ago

My question was about how to correct the infinite loop. @Barmar helped me out with that. I was not asking for the proper algorithm. That is why you don't get the right answer.

Vlad from Moscow Over a year ago

@Luis Lavieri My answer is the most helpful because it shows how the program should be written correctly. It teaches how such programs should be written in C.

Collectives™ on Stack Overflow

Concatenating a char in a recursive function in C

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related