2

本来这个函数是嵌入到main函数中的，造成了一个很杂乱的main函数。该程序用空格数替换制表符。我仍然对我的函数的参数列表中的内容以及如何将 argc/argv 从 main 传递到这些函数感到困惑。我做对了吗？

文件顶部有一些已定义的变量：

#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;
}

我需要帮助的是弄清楚函数的参数列表中的内容。我认为让我感到困惑的是如何让我的参数类型匹配，以便我可以将变量从一个函数传递给另一个函数。

4

2 回答 2

3

分解不是你最大的问题。错误检查比较粗心，使用旧的超重fscanf()和fprintf()全局变量都是。此外，输入文件名缺乏 const 正确性、变量名过长和冗长以及您不知道+=and++运算符只是额外的好处。我想这就是为什么你的代码看起来很臃肿（事实上确实如此）。

我会像这样重写函数：

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只包含循环 - 这样做留给您作为练习。

于 2013-06-01T07:31:40.733 回答
3

注意：此答案是对问题的早期编辑给出的。与此同时，它已经发生了变化，所以这个答案可能不再相关。

来自 OOP 背景，我将专注于一个单一的问题，即单一职责原则 (SRP)。我认为detab（和其他所有功能）应该只做一件特定的事情，但要把那件事做好。

但它不仅仅是“detab”，正如它的名字所暗示的那样；它还必须从命令行变量和中提取其实际参数，这些变量是由以下命令强加给它的：argcargvmain

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?
   }
   …
}

将所有命令行验证和值提取逻辑集中在一个地方，而将 detabbing 集中在另一个地方似乎更合理；也就是划清责任界限。不要让一个功能的逻辑溢出到另一个功能！

对我来说，detab的签名应该更像这样：

void detab(FILE *in, FILE *out);
于 2013-06-01T08:46:01.213 回答