Appending a char to a char*?

Hello, i really need help. I want to append a single char into a char* that i have..how do i do that? Is it even possible?

Code:

char temp[5] ="";

while(isalnum(*x))
{
	strcat(temp,*x);
	x++;
}
enqueue(&head,temp);

Seems that strcat isn't possible for this, i get an error when i run this.

The situation is i have a Queue which has a char* as a variable, because i want to store some string there. And then i have a char* to be read, and if the char it is pointing to is alphanumeric, i want it to append to temp and after all that, enqueue it.

Help guys?

Normally, something like this (assuming temp will always be large enough - if not, add some checks).

Code:

char temp[5] ="";
char *p = temp;
while(isalnum(*x))
{
    *p++ = *x++;
}
*p = '\0;
enqueue(&head,temp);

I'll be using this for small numbers, so i guess temp[5] will be large enough. Can you explain what happens in your code?

Originally Posted by mugiwara528

The situation is i have a Queue which has a char* as a variable, because i want to store some string there. And then i have a char* to be read, and if the char it is pointing to is alphanumeric, i want it to append to temp and after all that, enqueue it.
Help guys?

Since this isn't what I'd call a very clear description your best bet would be to rough out some code, even if it's not working and post it up and we'll see what we can do...

> Can you explain what happens in your code?
Maybe, but since you're already able to write enqueue() as well, it should be pretty easy for you to study.

Well at least longer than the maximum of 4 minutes it took from my reply to yours for you to resort to posting a question.

Originally Posted by CommonTater

Since this isn't what I'd call a very clear description your best bet would be to rough out some code, even if it's not working and post it up and we'll see what we can do...

Code:

struct node  //STACK
{
    char *x;
    struct node* next;
};

struct nd  //QUEUE
{
    char *x;
    struct nd* next;
};

void enqueue(QUEUE *head, char *c)
{    
    QUEUE p;
    QUEUE temp;
    
    temp = (queues*) malloc(sizeof(queues));

    if(*head==NULL)
    {
        temp->x = (char*)calloc(strlen(c),sizeof(char));
        temp->next = NULL;
        *head=temp;
    }
    
    else
    {
        p=*head;
        while(p->next!=NULL)
            p=p->next;

        temp->x =(char*)calloc(strlen(c),sizeof(char));
        temp->next = NULL;
        p->next = temp;    
    }
}

void toRPN(STACK *top, char* x, STACK2 *top2)
{
	setColor(2);

    while(*x!='\0')
    {
        if(*x=='(')
            push(&(*top),*x);

        else if(*x==')')
        {
            while(!isEmpty(*top) && strcmp((*top)->x,"(")!=0)
                pop(&(*top));
            
            if(strcmp((*top)->x,"(")==0)
                pop(&(*top));
        }

        else if(isalnum(*x))
		{
			char temp[5] ="";
			char *p = temp;

			while(isalnum(*x))
			{
				*p++ = *x++;
			}

			*p='\0';

            enqueue(&head,temp);
		}
...

Is that okay? I modified it with Salem's code already. I got rid of that error but now im getting another one -__-

Originally Posted by Salem

> Can you explain what happens in your code?
Maybe, but since you're already able to write enqueue() as well, it should be pretty easy for you to study.

Well at least longer than the maximum of 4 minutes it took from my reply to yours for you to resort to posting a question.

Thank you, i think i get it now. I'll try to see if this goes well with my whole program.

You can take strncat like

Code:

char temp[5] ="";

while(isalnum(*x))
{
	strncat(temp,x,1);
	x++;
}

Originally Posted by BillyTKid

You can take strncat like

Code:

char temp[5] ="";

while(isalnum(*x))
{
	strncat(temp,x,1);
	x++;
}

Now, BillyTKid's suggestion that I quoted above is very much based on the code in your post #1, and realistically with temp being a char[5] it is okay. However, in other situations, such a solution is needlessly inefficient. For a better understanding why, read Joel Spolsky's essay entitled Back to Basics.