为什么这段代码不能正确地对包含数字的行进行排序？答案

【问题标题】：Why doesn't this code sort lines containing numbers correctly?为什么这段代码不能正确地对包含数字的行进行排序？
【发布时间】：2010-01-05 18:41:45
【问题描述】：

我正在研究这个程序，它从 K&R 读取输入行并对其进行排序。

如果我输入示例，我无法弄清楚为什么它不能正确排序

1234532 first line
abc second line

它不会按升序对它们进行排序。如果输入行包含数字或除字母以外的其他内容，我认为基本上它不起作用。

但这适用于带有字母的行：

abc
abcsda

将得到正确排序。

这是代码：

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

#define MAXLINES 5000

char *lineptr[MAXLINES];

int readlines(char *lineptr[], int nlines);
void writelines(char *lineptr[], int nlines);

void qsort(char *lineptr[], int left, int right);

int main(int argc, char *argv[])
{
     int nlines;

     if((nlines = readlines(lineptr, MAXLINES)) >= 0) {
          qsort(lineptr, 0, nlines-1);
          writelines(lineptr, nlines);
          system("PAUSE");
          return 0;
     } 
     else {
          printf("error: input too big to sort\n");
          return 1;
     }


  system("PAUSE");  
  return 0;
}


#define MAXLEN 1000
int getline(char *, int);
char *alloc(int);

int readlines(char *lineptr[], int maxlines)
{
    int len, nlines;
    char *p, line[MAXLEN];

    nlines = 0;
    while((len = getline(line, MAXLEN)) > 0)
       if(nlines >= maxlines || (p = alloc(len)) == NULL)
          return -1;
       else {
            line[len-1] = '\0';
            strcpy(p, line);
            lineptr[nlines++] = p;
       }
    return nlines;
}

void writelines(char *lineptr[], int nlines)
{
     while(nlines -- > 0)
         printf("%s\n", *lineptr++);
}

int getline(char s[], int lim)
{
  int c, i;

  for (i = 0; i < lim - 1 && (c = getchar()) != EOF && c != '\n'; i++)
    s[i] = c;                                                         
  if (c == '\n') {
    s[i++] = c;   
  }
  s[i] = '\0';
  return i;
}

#define ALLOCSIZE 10000

static char allocbuf[ALLOCSIZE];
static char *allocp = allocbuf;

char *alloc(int n)
{
     if(allocbuf + ALLOCSIZE - allocp >= n) {
          allocp +=n;
          return allocp - n;
     }
     else 
          return 0;
}

void swap(char *v[], int i, int j)
{
     char *temp;

     temp = v[i];
     v[i] = v[j];
     v[j] = temp;
}

void qsort(char *v[], int left, int right) {

    int i, last;

    if(left >= right) 
       return;

    swap(v, left, (left+right)/2);
    last = left;

    for(i = left + 1; i <= right; i++)
      if(strcmp(v[i], v[left]) < 0)
         swap(v, ++last, i);

    swap(v, left, last);
    qsort(v, left, last-1);
    qsort(v, last+1, right);
}

【问题讨论】：

您希望第一个示例的顺序是什么？你应该先得到数字线，然后是字母线。
qsort() 在stdlib.h 中被声明（正确） - 你为什么在你的程序中再次（并且错误地）声明它？
我认为它是根据大小对它们进行分类； 1234123 比 abc 大，所以我想应该先打印 abc，而不是数字行。
不，除非行等于较短行的末尾，否则它不会按行长度排序。在你的情况下'1'
@Chris：K&R 使用他们自己的 qsort 实现：来自勘误表：119-121(§5.11)：qsort 讨论需要以多种方式重新定义。首先，qsort 是 ANSI/ISO C 中的标准例程，所以这里的再现应该被赋予一个不同的名称，特别是因为标准 qsort 的参数有点不同：标准接受一个基指针和一个计数，而这个例子使用一个基指针和两个偏移量。 (cm.bell-labs.com/cm/cs/cbook/2ediffs.html)

标签： c pointers function

【解决方案1】：

它正在正确地对它们进行排序。在 ASCII 中，数字排在字母之前。你期待什么输出？

【讨论】：

【解决方案2】：

你认为正确的顺序是什么？您可能对字符串上的 lexicographic ordering 到底是什么感到困惑。从这个角度来看，正确的排序

1234532 first line
abc second line

是

1234532 first line
abc second line

因为1 在 ASCII 中位于 a 之前。我以前有一个关于这个主题的answer。

【讨论】：

哦，好吧，我认为它是根据长度而不是 ASCII 值来比较它们的。我想这可以解释它。
不。它就像一本字典，数字在字母之前。查看 Wikipedia 文章和我链接到的先前答案。如果您还有其他问题，请随时在此处提问。
如果要按长度排序，请将上面qsort()中的if(strcmp(v[i], v[left]) < 0)更改为if (strlen(v[i]) < strlen(v[left]))。

【解决方案3】：

它与strcmp() 进行比较，因此它比较单个字符。如果要比较数字，则需要添加一个选项以（尝试）将尽可能多的行首转换为数字，并在进行排序时比较转换后的数字（并决定如何处理以非数字开头的行）。

【讨论】：

【解决方案4】：

如果您想严格按照行长排序，您需要创建一个比较函数来执行此操作并将其传递给 qsort()。

【讨论】：