qsort массива структуры не работает - PullRequest
Новая
       48

qsort массива структуры не работает

2 голосов
/ 02 декабря 2011

Я пытаюсь отсортировать массив struct run, называемый results, по символу, но когда я печатаю массив, ничего не сортируется.Взгляните на это: 

struct run {
  char name[20], weekday[4], month[10];
  (And some more...)
};
typedef struct run run;

int name_compare(const void *a, const void *b) 
{
    run *run1 = *(run **)a;
    run *run2 = *(run **)b;
    return strcmp(run1->name, run2->name);
}

int count_number_of_different_persons(run results[])
{
  int i = 0;


  qsort(results, sizeof(results) / sizeof(run), sizeof(run), name_compare);

  for(i = 0; i <= 999; i++)
  {
    printf("%s\n", results[i].name);
  }
  // not done with this function yet, just return 0
  return 0;
}

Вывод, приведенный выше, представляет собой просто список имен в порядке их первоначального размещения

Ответы [ 3 ]

5 голосов
/ 02 декабря 2011 
int count_number_of_different_persons(run results[])

Это на самом деле не позволяет использовать sizeof для массива, поскольку массив распадается на указатель.

Это 

run *run1 = *(run **)a;

тоже выглядит странно, разве это не должно быть 

run *run1 = (run*)a;

3 голосов
/ 02 декабря 2011

Проверьте следующий код:

Как уже упоминалось @michel, sizeof (массив) предоставляет размер указателя, а не размер самого массива, так как при передаче массива он рассматривается как указатель. Следовательно, либо отправьте количество элементов в функцию count_number_of_different_persons, либо определите MACRO количества элементов. Надеюсь это поможет. :). 

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define NOE 3

struct run
{
    char name[20];
};

typedef struct run run;

int name_compare (const void *a, const void *b )
{
    return strcmp (((run *)a)->name, ((run *)b)->name);
}

int count_number_of_different_persons(run results[], int noOfElements)
{
    int i=0;
    qsort(results, noOfElements, sizeof (run), name_compare);
    for (i=0; i<noOfElements; i++)
        printf ("%s\n",results[i].name);
}

int main ( int argc, char * argv[])
{
    run a, b, c;
    run  arg[NOE];

    strcpy (a.name, "love");
    strcpy (b.name, "you");
    strcpy (c.name, "i");
    arg[0] = a;
    arg[1] = b;
    arg[2] = c;

    count_number_of_different_persons(arg, sizeof(arg)/sizeof(run));
};
3 голосов
/ 02 декабря 2011

Одна проблема в name_compare. Попробуйте вместо этого: 

int name_compare(const void *a, const void *b) 
{
    run *run1 = (run *)a;
    run *run2 = (run *)b;
    return strcmp(run1->name, run2->name);
}
...