Question

我最近加入了Stackoverflow社区，因为我不得不问这个问题。我一直在网站上寻找可能的解释和解决方案，但到目前为止，没有任何东西能像我想的那样启发。我的错误可能是由一行特定的代码引起的。我正在尝试创建一个读取struct投票数组的函数，（struct包含整数成员编号，char * category，char * nominee）并将包含相同编号和类别的所有投票复制到另一个struct数组。基本上是为了显示所有重复的投票。

typedef struct
{ 
  int member;
  char *categ;
  char *nom;
}Vote

Vote vote(int member, char *categ, char *nom)
{
  Vote result;
  result.member = member;
  result.categ = categ;
  result.nom = nom;
  return result;
}

int votes_count(Vote *v, int n, Vote *v1) 
{
    int result = 0;
    int *index = malloc(sizeof(int) * 1000);
    int a = 0;
    for (int i = 0; i < n; ++i)
    {
     for (int j = 0; j < n; ++j)
     {
       if (a == 0 && v[i].member == v[j].member && strcmp(v[i].categ, v[j].categ) == 0)
       {
          v1[result++] = vote(v[j].member, str_dup(v[j].categ), str_dup(v[j].nom)); 
          index[a++] = j;
       }
        for (int b = 0; b < a; ++b)
        {
          if( a > 0 && v[i].member == v[j].member && strcmp(v[i].categ, v[j].categ) == 0 && j != index[b])
          {
            v1[result++] = voto(v[j].member, str_dup(v[j].categ), str_dup(v[j].nom));
            index[a++] = j; 
          }
        }       
      }
    }
    return result;
 }

Afterwads，它返回包含所有重复的新数组的元素数。我想使用一个int数组来保存所有行索引，以便该函数不会读取和复制它已经占用的行。很抱歉，如果代码很难理解，如果需要我可以编辑以使其更容易理解。谢谢你的回复。 P.S：我是葡萄牙人，提前抱歉语法错误

Answer 1

如果您的唯一目的是收集重复项，则只需要与元素之前的元素进行比较

您不需要index[]数组

为简单起见，我使用了两个整数数组，您应该将它们更改为struct数组，同时更改compare函数。

unsigned fetchdups(int orig[], int dups[], unsigned count) { unsigned this, that, ndup=0; for (this=1; this<count; this++){ for (that=0; that<this; that++){ /* change this to your compare() */ if(orig[that] == orig[this]) break; } if (this == that) continue; /* no duplicate */ dups[ndup++] = this; } return ndup; }

返回整数时的分段错误

1 个答案: