Question

因此，由于某种原因，此函数似乎将所有相同的值推送到堆栈上（或者只是打印相同的值;我想它实际上可能与我也包含的printAll函数有关。）？问题是，对于元素数组（它是一个整数数组），printAll会适当地循环遍历这些值。但是有关charElement数组的任何内容，printAll函数只打印charElements函数中每个值的最新值。知道为什么会这样吗？

void pushString(Stack *S, char element[])
{        
    /* If the stack is full, we cannot push an element into it as there is no space for it.*/        
    if(S->size == S->capacity)        
    {                
        printf("Stack is Full\n");        
    }        
    else        
    {                
        /* Push an element on the top of it and increase its size by one*/ 
        S->charElements[S->size++] = element;
        S->elements[S->size-1] = '\n'; 
    }        
    return;
}



void printAll(Stack *S)
// NOTE:  Called w/ user input 'f'
// PRE:   Stack S is initialized
// POST:  Print all values from the stack
//    Do NOT alter anything
{

    int i; 
    for(i = 0; i < S->size; i++)
    {
        printf("%d \n", i);
        if(S->charElements[i] == "\n")
        {
            printf("%.2f \n", (float)S->elements[i]);   
        }
        else if(S->elements[i] == '\n')
        { 
            printf("%s \n", S->charElements[i]); 
        }
    }
}

Answer 1

S->charElements[S->size++] = element;

只是将指针复制到传入的char* element。

如果您使用相同的缓冲区来读取输入，然后重复将其传递给pushString函数，则所有堆栈元素最终都指向同一个缓冲区，其内容是最后输入的值。

您需要使用strdup或malloc和strcpy复制内容。

S->charElements[S->size++] = strdup(element);

RESP。

S->charElements[S->size] = malloc(strlen(element) + 1);
// check for NULL
strcpy(S->charElements[S->size++], element);

Answer 2

代码很奇怪。元素在推送功能中由char []表示，但使用%f中的printAll()打印。这对我来说没有多大意义。

如果没有Stack结构的声明，就很难遵循代码的意图。

c - 我的push函数总是将最新元素推送到堆栈的每个值

2 个答案: