-3

这是我的代码，输出的每个字母都被打印两次。它不是文本文件，它只在我插入我的putchar(tolower)语句时发生，但它的格式与它应该的完全一致。声明有什么问题？

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

#define MAX_STRING_SIZE 20
#define MAX_LIST_SIZE 50

int readFile(char *filename); /* function declaration for readFile, defined below */
void punct();

/* main function */
int main(int argc, char *argv[]) {
    if (argc < 2) {
        printf("%s: usage %s textFileName \n", argv[0], argv[0]);
        exit(1);
    }

    readFile(argv[1]);

    return 0;
}

int readFile(char *filename) {
    char ch;
    FILE *fPtr;

    fPtr = fopen(filename, "r"); /*open file filename, r is read only */
    if (!fPtr) {
        return 0;
    }

    while ((ch = fgetc(fPtr)) != EOF) {
        putchar(tolower(ch));
        printf("%c", ch);
    }

    fclose(fPtr);

    return 1; /* because success */
}
4

2 回答 2

5

输出的每个字母都被打印两次

这两行：

putchar(tolower(ch));
printf("%c", ch);

打印两次相同的字符。

putchar将一个字符放到标准输出中。
printf将格式化的文本放入标准输出。在您的情况下，格式化文本与打印在上面的行相同的字符。

仅当我在其中插入 putchar(tolower) 语句时才会发生这种情况

那是因为这样做实际上是使输出加倍。

于 2017-01-29T23:24:42.760 回答
0

您的代码中有多个问题：

  • ch必须用类型定义int以适应所有可能的返回值fgetc()。这些都是 typeunsigned char加上 special value 的值EOF。存储到char变量中会带来多个问题：

    • EOF无法再准确地测试该值。如果类型char是无符号的，(ch = fgetc(fp)) == EOF则始终为 false。相反，如果 typechar是有符号的，则对于有效的字节值(ch = fgetc(fp)) == EOF可能为 true 。ch = '\377

    • 如果char是有符号的，tolower(ch)则对于负值具有未定义的行为。

  • 从文件中读取的每个字节都会输出两次：第一次是 by putchar(tolower(ch));，第二次是 by printf("%c", ch);。

这是一个更正的版本：

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

char *progname;    

int readFile(const char *filename) {
    int ch;
    FILE *fPtr;

    fPtr = fopen(filename, "r"); /* open file filename for reading */
    if (!fPtr) {
        fprintf(stderr, "%s: cannot open %s: %s\n",
                progname, filename, strerror(errno));
        return 1;  /* failure */
    }

    while ((ch = fgetc(fPtr)) != EOF) {
        putchar(tolower(ch));
    }
    fclose(fPtr);

    return 0;  /* success */
}

int main(int argc, char *argv[]) {
    progname = argv[0];
    if (argc < 2) {
        fprintf(stderr, "usage: %s textFileName\n", progname);
        return 1;
    }
    return readFile(argv[1]);
}
于 2017-02-04T19:31:38.287 回答