Поиск уникальных элементов в строковом массиве в C - PullRequest
5 голосов
/ 10 мая 2010

C беспокоит меня обработкой строк. У меня в голове есть такой псевдокод:

char *data[20]; 

char *tmp; int i,j;

for(i=0;i<20;i++) {
  tmp = data[i]; 
  for(j=1;j<20;j++) 
  {
    if(strcmp(tmp,data[j]))
      //then except the uniqueness, store them in elsewhere
  }
}

Но когда я закодировал это, результаты были плохими. (Я занимался всеми вещами памяти, мелочами и т. Д.) Проблема, очевидно, во втором цикле: D. Но я не могу придумать никакого решения. Как найти уникальные строки в массиве.

Пример ввода: введено abc def abe abc def deg уникальные: нужно найти abc def abe deg.

Ответы [ 5 ]

6 голосов
/ 10 мая 2010

Вы можете использовать qsort , чтобы поместить дубликаты рядом друг с другом. После сортировки вам нужно только сравнить соседние записи, чтобы найти дубликаты. Результатом является O (N log N), а не (я думаю) O (N ^ 2).

Вот 15-минутная обеденная версия без проверки ошибок:

  typedef struct {
     int origpos;
     char *value;
  } SORT;

  int qcmp(const void *x, const void *y) {
     int res = strcmp( ((SORT*)x)->value, ((SORT*)y)->value );
     if ( res != 0 )
        return res;
     else
        // they are equal - use original position as tie breaker
        return ( ((SORT*)x)->origpos - ((SORT*)y)->origpos );
  }

  int main( int argc, char* argv[] )
  {
     SORT *sorted;
     char **orig;
     int i;
     int num = argc - 1;

     orig = malloc( sizeof( char* ) * ( num ));
     sorted = malloc( sizeof( SORT ) * ( num ));

     for ( i = 0; i < num; i++ ) {
        orig[i] = argv[i + 1];
        sorted[i].value = argv[i + 1];
        sorted[i].origpos = i;
        }

     qsort( sorted, num, sizeof( SORT ), qcmp );

     // remove the dups (sorting left relative position same for dups)
     for ( i = 0; i < num - 1; i++ ) {
        if ( !strcmp( sorted[i].value, sorted[i+1].value ))
           // clear the duplicate entry however you see fit
           orig[sorted[i+1].origpos] = NULL;  // or free it if dynamic mem
        }

     // print them without dups in original order
     for ( i = 0; i < num; i++ )
        if ( orig[i] )
           printf( "%s ", orig[i] );

     free( orig );
     free( sorted );
  }
5 голосов
/ 10 мая 2010
char *data[20];
int i, j, n, unique[20];

n = 0;
for (i = 0; i < 20; ++i)
{
    for (j = 0; j < n; ++j)
    {
        if (!strcmp(data[i], data[unique[j]]))
           break;
    }

    if (j == n)
        unique[n++] = i;
}

Индексы первого вхождения каждой уникальной строки должны быть уникальными [0..n-1], если я сделал это правильно.

2 голосов
/ 10 мая 2010

Почему вы начинаете второй цикл с 1?

Вы должны начать это с я + 1. т.е.

for(j=i+1;j<20;j++) 

Как если бы список был

abc
def
abc
abc
lop

тогда

когда я == 4

TMP = "обкорнать"

но затем начинается второй цикл, который составляет от 1 до 19. Это означает, что он также получит значение 4 на одной стадии, а затем

data [4], который является "lop", будет таким же, как tmp. Таким образом, хотя «lop» уникален, но он будет помечен как повторенный.

Надеюсь, это было полезно.

1 голос
/ 15 мая 2010

Подумайте немного больше о своей проблеме - то, что вы действительно хотите сделать, это посмотреть на ПРЕДЫДУЩИЕ строки, чтобы увидеть, видели ли вы это уже. Таким образом, для каждой строки n сравните ее со строками 0 до n-1.

print element 0 (it is unique)
for i = 1 to n
  unique = 1
  for j = 0 to i-1 (compare this element to the ones preceding it)
    if element[i] == element[j]
       unique = 0
       break from loop
  if unique, print element i
0 голосов
/ 10 мая 2010

Может ли быть так, что ваш тест if (strcmp (this, that)), который будет успешным, если оба они различны? ! strcmp, вероятно, то, что вы хотите там.

...