我试图分解我的功能，但我不知道如果我做对了

Question

最初，这个函数被嵌入到主函数中，创建一个非常混乱的主函数。该程序将使用空格数替换制表符。我仍然对我的函数的参数列表内部以及如何将main中的argc/argv传递给这些函数感到困惑。我做对了吗？

有在文件的顶部部分定义的变量：

#define OUTFILE_NAME "detabbed" 
#define TAB_STOP_SIZE 8 
#define NUM_ARGS 2 
#define FILE_ARG_IDX 1 

这里是我的第二次尝试吧：

void open_file(FILE *inf, FILE *outf, char *in[]) /*I feel like the arguments aren't right 
{             and this function is just opening 
                and reading files*/ 
    inf = fopen(in[1], "r"); 
    outf = fopen(OUTFILE_NAME, "w"); 

    if (inf == NULL) 
    { 
     perror(in[1]); 
     exit(1); 
    } 

    else if (outf == NULL) 
    { 
     perror(OUTFILE_NAME); 
     exit(1); 
    } 

    fclose(inf); 
    fclose(outf); 
} 

void detab(FILE *infile, FILE *outfile, char *argument[]) /* Confused about argument list 
{               and this function actually 
    char c;             does the detabbing */ 
    int character_count = 0, i, num_spaces; 

    open_file(infile, outfile, argument);     /* I want to call the previous 
                  function but again, confused 
    while (fscanf(infile, "%c", &c) != EOF)    about the argument list */ 
    { 
     if (c == '\t') 
     { 
     num_spaces = TAB_STOP_SIZE - (character_count % TAB_STOP_SIZE); 

     for (i = 0; i < num_spaces; i++) 
     { 
      fprintf(outfile, " "); 
     } 

     character_count += num_spaces; 
     } 
     else if (c == '\n') 
     { 

     fprintf(outfile, "\n"); 
     character_count = 0; 
     } 
     else 
     { 
     fprintf(outfile, "%c", c); 
     character_count++; 
     } 
    } 

} 

int main(int argc, char *argv[]) 
{ 
    if (argc < 1) 
    { 
     fprintf(stderr, "usage: prog file\n"); 
     exit(1); 
    } 

    else if (argc < NUM_ARGS) 
    { 
     fprintf(stderr, "usage: %s file\n", argv[0]); 
     exit(1); 
    } 

    detab(argc, argv); /* I want to pass argc and argv to the detab function, but I'm 
          having trouble with the argument list */ 
    return 0; 
} 

我需要帮助，是找出在发生的事情函数的参数列表。我认为让我困惑的是如何让我的参数类型匹配，以便我可以将变量从一个函数传递给另一个函数。

Answer 1

分解不是你最大的问题。相当不小心的错误检查，使用旧的超过fscanf()和fprintf()和全局变量。此外，输入文件名中缺少const正确性，过长和冗长的变量名称以及您对运营商的不了解只是红利。我想这就是为什么你的代码看起来像臃肿（事实上，它是）。

我重写这样的功能：

void detab(const char *in, const char *out, int tabstop) 
{ 
    FILE *inf = fopen(in, "r"); 
    if (!inf) return; 

    FILE *outf = fopen(out, "w"); 
    if (!outf) { 
     fclose(inf); 
     return; 
    } 

    int n = 0; 
    int c; 
    while ((c = fgetc(inf)) != EOF) { 
     if (c == '\t') { 
      int pad = tabstop - n % tabstop; 

      for (int i = 0; i < pad; i++) 
       fputc(' ', outf); 

      n += pad; 
     } else if (c == '\n') { 
      fputc('\n', outf); 
      n = 0; 
     } else { 
      fputc(c, outf); 
      n++; 
     } 
    } 

    fclose(inf); 
    fclose(outf); 
} 

如果您想进一步分解这一点，那么你可以编写一个函数接受两个FILE *和制表位作为参数，并须仅包含while循环 - 这样做仅供练习使用。

Answer 2

_{注：这个答案是考虑到问题的早期编辑。它在此期间发生了变化，所以这个答案可能不再相关。}

从面向对象的背景来看，我将关注一个叫做Single Responsibility Principle (SRP)的问题。我认为detab（和其他所有函数）应该只做一件特定的事情，但做得很好。正如它的名字所暗示的那样，它并不仅仅是“被剥夺”的;它也有通过main在命令行变量argc和argv，将其强加于它提取其实际参数：

detab(argc, argv); 

main做之前，一些验证，但因为命令行是然后简单地传递给函数，你显然觉得继续验证在detab（我也做一些补充说明有关违反以下SRP）：

void detab(int arg_list, char *array[]) // why ask for arg_list if you don't need it? 
{ 
    … 
    infile = fopen(array[1], "r"); // not obvious to caller that only array[1] is relevant 
    … 
    if (infile == NULL) 
    { 
     perror(array[1]); 
     exit(1); // should your function really have the power to terminate the program? 
    } 
    … 
} 

这似乎更reaso将所有的命令行验证和价值提取逻辑集中在一个地方，并将其分解在另一个地方;即明确责任界限。不要让一个函数的逻辑溢出到另一个函数中！

对我而言，detab的签名应该看起来更像这个：

void detab(FILE *in, FILE *out);