Question

目前，我的C程序alter.c从使用命令行参数调用的文本文件中成功创建了一个char数组a。

但是，我想改变它，以便它不会读取每一行，而只会读取不是连续重复前一行的行。如果一条线是重复但不遵循其相同的线，则不应删除它。

例如，假设testfile.txt的内容是：

你好

你好

你好

您好。

如果以＆＃39; alter testfile.txt＆＃39;调用，则创建的char数组应为：

你好。

您好。

这是我目前拥有的代码：

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

#define MAX_NAME_SZ 100
#define caseChange 32
#define MAXCHARS 79
int main (int argc, char *argv[])
 {
  char ch, *a;
  static char changed = 'f';

  /* if there is more than one argument */
  if (argc > 1)
    {
      int i = 1, j = 0;
      FILE *fp;
      while (i < argc)
    {
      a = malloc (MAX_NAME_SZ * sizeof (char));
      j = 0;
      fp = fopen (argv[i], "r");
      if (fp == NULL)
        {
          /*Error statement in case an error is encountered /*
             fprintf (stderr, "Error encountered for File %s : %s\n",
             argv[i], strerror (errno));
             return 3;            /* I chose to return 3 if an error was encountered */
        }
      else
        {
          while (((ch = fgetc (fp)) != EOF) && (j < 99))
        {
          a[j++] = ch;
        }
          a[j] = '\0';

          fclose (fp);
          changeCase (a, changed);
          noNums (a, changed);
          identLines (a, changed);
          spaces (a, changed);
          printf ("\n");
        }
      i++;

    }
      if (changed == 'f')
    return 1;
      else
    return 0;

    }
  else
    {
      a = malloc (MAX_NAME_SZ * sizeof (char));
      fgets (a, MAX_NAME_SZ, stdin);
      changeCase (a, changed);
      noNums (a, changed);
      identLines (a, changed);
      spaces (a, changed);
      printf ("\n");
    }
}

我认为在做

时我需要另一个while语句

while (((ch = fgetc (fp)) != EOF) && (j < 99))

但我不确定。任何帮助表示赞赏。

Answer 1

我没有阅读你的所有代码。（好吧，问题风格“这是我的代码有100行代码，请帮我查找错误”并且在本网站上不是很好的问题。）

但我想指出这段代码：

char ch;
((ch = fgetc (fp)) != EOF)

...是典型的初学者错误。 “fgetc”返回“int”。在将它与EOF进行比较之前，您不能将此“int”转换为“char”。将“fgetc”结果放入“int”变量，然后将其与EOF进行比较，然后将此“int”转换为“char”。然后用它做任何其他处理。

也许会有所帮助。也许不吧。您的代码可能包含其他问题。让你的代码更小，我们可能会阅读它。

另外，这个：

while (((ch = fgetc (fp)) != EOF) && (j < 99))

......风格不好。我的意思是在“同时”条件下放置很多东西的做法。这使得它难以阅读。假设while ((ch = fgetc (fp)) != EOF)为ch，int即可。此外，您的代码已破坏标识。如果您希望我们对其进行评论，您应该使代码尽可能易于阅读

我如何只使用命令行参数读取唯一的行？

1 个答案: