0

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

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-我可以做些什么来修复代码？

4

1 回答 1

2

在您的行中：

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++while temp != NULL。
我相信您打算检查存储在指针处的 VALUE temp。
*temp应根据 '\0' 而不是 NULL 正确检查。
（\0 和 NULL 恰好具有相同的值，但它们在语义上不应被视为相同）

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

PS 您的简单但出色的评论“//如果字符相同”对我理解您的代码非常有帮助。这是INVALUABLE简短而有意义的评论的绝佳案例。谢谢你。

（希望是最终编辑）
总的来说，我推荐的更改如下：

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);
}

你觉得怎么样？

于 2011-09-12T00:34:51.787 回答