使用指针打印值

Question

我有一个由一些不确定数量的变量及其计数组成的排序数组。我需要像这样构建一个字符串：

Attribute[0]: p=2, e=8

我的问题是数组实际上是一个指针，并且我不想使用固定长度的循环，所以我看到的唯一解决方案是使用数组指针。

void printArray(Node_ptr node){
    int count=0; //character count
    char *temp= node->attributes; //duplicate attribute array

    char cur; //current char
    char *outputCat= emalloc(150); //concatenate counts to a single string

    strcpy(outputCat, "Attribute %d counts are: ");
    qsort(temp, lineCount, sizeof(char), compare); //sort the array

    while ((temp) != NULL){
        cur= *temp;
        ++temp;
        if (strcmp(&cur, temp)==0) //if characters are same
            count++; 
        else { //build reporting string
            strcat(outputCat, &cur);
            strcat(outputCat, "= ");
            sprintf(outputCat, "%d  ", count);
            count=0;
        }
    }
    free(outputCat);
}

这里的问题是 strcmp(&cur, ++temp)==0 每次都返回 false，即使我在调试器中看到它们的值也是如此。因此，else 条件会不断构建，并在多次迭代后抛出段错误。

两个问题：

1- 即使输入相同的值，什么才能使 strcmp 返回非零？ 2- 我可以做什么来修复代码？

Answer 1

在您的行中：

strcmp(&cur, temp)

cur 是本地声明的 char，因此，&cur 只是堆栈上的某个位置，在这个背景。

我相信您的意思是检查当前字符 cur 是否与下一个字符 *temp 相同。
这看起来像：

if (cur == *temp) //if characters are same
    count++; 

接下来，我会看看大规模简化你的输出部分：

sprintf(outputCat, "%c = %d", *cur, count);  // e.g.   "e = 8"
count=0;

最后，我怀疑你的循环是否会终止，因为它继续执行 temp++，而 temp != NULL。
我相信你打算检查存储在指针处的 VALUE 临时。
应根据“\0”而不是 NULL 正确检查 *temp。
（\0 和 NULL 碰巧具有相同的值，但它们在语义上不应该被视为相同）

while (*temp != '\0'){

PS 您的简单但出色的评论“// if strings are same “对我理解你的代码非常有帮助。这是一个简短而有意义的评论无价值的绝佳案例。谢谢。

---

（希望是最终编辑）
总的来说，我建议的改变是这样的：

void printArray(Node_ptr node){
    int count=0; //character count
    char *temp= node->attributes; //duplicate attribute array

    char cur; //current char
    char *outputCat= emalloc(150); //concatenate counts to a single string

    strcpy(outputCat, "Attribute %d counts are: ");
    qsort(temp, lineCount, sizeof(char), compare); //sort the array

    while (*temp != '\0'){
        cur= *temp;
        ++temp;
        if (cur == *temp) //if characters are same
            count++; 
        else { //build reporting string
            sprintf(outputCat, "%c = %d", *cur, count);  // e.g.   "e = 8"
            count=0;
        }
    }
    free(outputCat);
}

你觉得效果如何？