求教一下，这段C代码有什么问题

lsof

已知 file1 file2 file3 .... 是已经使用 sort 按行排序的文本文件（约 10 到 100 个，至少 10M 以上），请编写程序合并这些文件。

文件名列表请从 argv 读取，如：

./sort-merge *.txt > result.txt

这里是仿 sort -m 方式排序文件，我这里写了一个非常简单的程序，但是用起来得core dump，麻烦哪个老师指导一下？
并且是当文件特别大的时候就崩溃了，我找不出什么原因， 555~~~

1. #include <stdio.h>
   
2. #include <string.h>
   
3. #include <stdlib.h>
   
4. 
   
5. int readandwrite(FILE **fp, FILE *op, char line[][8192], char *max, int num, int m);
   
6. int compare_x (FILE **fp, FILE *op, char line[][8192], char *max, int num);
   
7. 
   
8. int main(int argc, char *argv[])
   
9. {
   
10.         FILE *fp[argc];
    
11.         FILE *op;
    
12.         char line[argc][8192];
    
13.         int i;
    
14.         char *max = NULL;
    
15.         if (argc < 2)
    
16.         {
    
17.                 printf("Usage: %s <ARGV>\n", argv[0]);
    
18.                 exit(EXIT_FAILURE);
    
19.         }
    
20.         op = fopen("all.txt", "w");
    
21.         for (i = 0; i < argc - 1; i ++)
    
22.         {
    
23.                 fp[i] = fopen(argv[i+1], "r");
    
24.         }
    
25.         compare_x (fp, op, line, max, argc - 1);
    
26.         for (i = 0; i < argc - 1; i ++)
    
27.         {
    
28.                 fclose(fp[i]);
    
29.         }
    
30.         fclose(op);
    
31.         return 0;
    
32. }
    
33. 
    
34. int readandwrite(FILE **fp, FILE *op, char line[][8192], char *max, int num, int m)
    
35. {
    
36.         int i, j = 0;
    
37.         for (i = 0; i < num; i ++)
    
38.         {
    
39.                 if (!max || strcmp(max, line[i]) < 0)
    
40.                 {
    
41.                         max = line[i];
    
42.                         j = i;
    
43.                 }
    
44.         }
    
45.         fputs(max, op);
    
46.         if (fgets(line[j], 8192, fp[j]) == NULL)
    
47.         {
    
48.                 m ++;
    
49.                 memset (line[j], '\0', 8192);
    
50.                 if (m == num) return 0;
    
51.         }
    
52.         max = NULL;
    
53.         readandwrite(fp, op, line, max, num, m);
    
54.         return 0;
    
55. }
    
56. 
    
57. int compare_x (FILE **fp, FILE *op, char line[][8192], char *max, int num)
    
58. {
    
59.         int i, j = 0, m = 0;
    
60.         for (i = 0; i < num; i ++)
    
61.         {
    
62.                 fgets(line[i], 8192, fp[i]);
    
63.                 if (!max || strcmp(max, line[i]) < 0)
    
64.                 {
    
65.                         max = line[i];
    
66.                         j = i;
    
67.                 }
    
68.         }
    
69.         fputs(max, op);
    
70.         if (fgets(line[j], 8192, fp[j]) == NULL)
    
71.         {
    
72.                 memset (line[j], '\0', 8192);
    
73.         }
    
74.         max = NULL;
    
75.         readandwrite(fp, op, line, max, num, m);
    
76.         return 0;
    
77. }

复制代码

ismelong

你一下子打开那么多的文件，恐怕不是很好吧，应该尽量操作完一个就关掉，然后再开一个，文件的句柄应该是有限的

lsof

文件的句柄？ 同志你还没有搞清楚是什么东西吧？

另外，解释性语言的句柄也不是有限的吧？ 谢谢你的回答，但是你的回答也太离谱，你还是多看看书本吧。

有老师帮忙解决么？我想知道什么原因导致错误的？

[ 本帖最后由 lsof 于 2008-1-9 20:40 编辑 ]

猪知猪之道

这程序写的 很好很强大

ilex

基本上没看懂，但是：
        FILE *fp[argc];
        FILE *op;
        char line[argc][8192];
数组可不可以用 fp[argc]的方式来定义？

francybright

好像加const 就可以！

lsof

原帖由 francybright 于 2008-1-9 22:12 发表
好像加const 就可以！

什么意思？

lsof

原帖由 ilex 于 2008-1-9 21:57 发表
基本上没看懂，但是：
        FILE *fp[argc];
        FILE *op;
        char line[argc][8192];
数组可不可以用 fp[argc]的方式来定义？

你说的文件指针fp不能用数组？
指针是指内存里的一个地址，肯定可以啊

zszyj

原帖由 lsof 于 2008-1-9 10:55 发表
已知 file1 file2 file3 .... 是已经使用 sort 按行排序的文本文件（约 10 到 100 个，至少 10M 以上），请编写程序合并这些文件。

文件名列表请从 argv 读取，如：

./sort-merge *.txt > result.txt

...

看似这个程序问题比较多, 都有可能是异常的原因:
1. 如果楼主不是为了发贴而刻意改过的话, 我认为是不可能通过编译的,因为C语言里面,局部变量是不支持动态数组的,因此以下的两个局部动态数组的定义是不允许的:
        FILE *fp[argc];
        char line[argc][8192];

       要实现动态数组,在C语言里用的是malloc函数实现,在C++里则是用new操作符实现的.

2. 程序中使用 char line[argc][8192] 的方式来定义数组, 排除语法上的错误,假设语法通过了. 但这种定义, 是在栈上分配内存的, 而大部分平台上的栈空间只有64K, 可以想象,一个数组就占用了大部分的栈空间,而且程序中使用了递归算法, 也是非常耗栈内存的, 因此很容易因为产生栈溢出的异常.

3. if (!max || strcmp(max, line) < 0) , 某些编译器可能会在!max (即max==NULL)的情况下,仍然执行strcmp(max, line) , 这种情况下,CORE DUMP就有可能了.

当然,最好的出错原因定位, 就是使用调试工具, 例如gdb.

猪知猪之道

fputs(max, op);  这句出错的