2024-09-09T14:47:52.437Z

1

정확히 이해한 것인지 모르겠지만 수정해보았습니다.

#define CRTSECURE_NO_WARNINGS

#include <stdio.h>

#include <stdlib.h>

int main(int argc, char* argv[])

{

int ch;

FILE* fr; // TODO: file pointer to write

FILE* fw;

const char* out_filename = "output.txt";

unsigned long count = 0;

if (argc < 2 || argc > 3)

{

printf("Usage: %s 읽을파일이름 [출력할파일이름]\n", argv[0]);

exit(EXIT_FAILURE);

}

if ((fr = fopen(argv[1], "r")) == NULL) // Open a text file for reading.

{

printf("Can't open %s\n", argv[1]);

exit(EXIT_FAILURE);

}

if (argc == 3)

{

out_filename = argv[2];

}

if ((fw = fopen(out_filename, "w")) == NULL)

{

printf("Can't open %s\n", out_filename);

exit(EXIT_FAILURE);

}

/*

r: reading

w: creating and writing or over writing

a: appending or creating and writing

r+: both reading and writing

w+: reading and writing, over writingor creaitng

a+: reading and writing, appending or creating

*/

while ((ch = fgetc(fr)) != EOF) // getc(fr)

{

fputc(ch, stdout);

fputc(ch, fw);

count++;

}

printf("\n");

fclose(fr);

fclose(fw);

printf("File %s has %lu characters\n", argv[1], count);

printf("Copied to %s", out_filename);

return 0;

}

0

네, 잘 이해하시고 작성하셨습니다.

강의의 내용에서 더 나아가 능동적으로 재밌게 학습하시는 것 같아, 저 또한 재미를 느끼네요. 감사합니다.

우선, 첨부해주신 코드를 컴파일 후 실행해보며 여러가지 경우를 체크해보았습니다.
이를 바탕으로, 문제가 발생할 '수' 도 있는 상황에 대해서 말씀드려봅니다.

const char* out_filename 에서의 자료형에 관한 문제입니다.
이렇게 const 로 한정한 포인터 변수에 대해서 추후 argv[2] 를 할당하는 것은 C언어에서 '문법적으로' 문제가 되지는 않지만, const 로 한정하는 것의 의미에 일치하지 않으며 추후 혼동을 일으킬 수 있으므로 권장되지 않는 방식입니다. (이 부분은 강의 10.5 포인터의 호환성 의 17:50 부분의 교수님 설명을 참고해보시면 이해에 도움이 많이 되실 것 같습니다.)

따라서, chsr* out_filename 을 선언하고, 문자열을 동적으로 할당해주는 것이 보다 더 좋은 코드라고 생각합니다.

이미 훌륭하게 작성하셨기에, 이해를 조금 도와드리기 위한 예시 코드를 첨부드려봅니다.

이해를 위해 답변드렸던 이전 답변의 내용의 일관성을 위하여, 로그 부분은 영어가 아닌 한글로 통일하여 작성하였습니다.

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define BUFFER_SIZE 4096
#define DEFAULT_OUT_FILENAME "output.txt"

int main(int argc, char* argv[])
{
    FILE* fr;
    FILE* fw;
    char buffer[BUFFER_SIZE];
    size_t bytes_read;
    char* out_filename;
    unsigned long count = 0;

    if (argc < 2 || argc > 3)
    {
        fprintf(stderr, "사용법: %s 읽을파일이름 [출력할파일이름]\n", argv[0]);
        exit(EXIT_FAILURE);
    }

    if ((fr = fopen(argv[1], "r")) == NULL)
    {
        fprintf(stderr, "%s 파일을 열 수 없습니다\n", argv[1]);
        exit(EXIT_FAILURE);
    }

    if (argc == 3)
    {
        out_filename = strdup(argv[2]);
    }
    else
    {
        out_filename = strdup(DEFAULT_OUTPUT);
    }

    if (out_filename == NULL)
    {
        fprintf(stderr, "문자열 메모리 할당 실패\n");
        fclose(fr);
        exit(EXIT_FAILURE);
    }

    if ((fw = fopen(out_filename, "w")) == NULL)
    {
        fprintf(stderr, "%s 파일을 열 수 없습니다\n", out_filename);
        free(out_filename);
        fclose(fr);
        exit(EXIT_FAILURE);
    }

    while ((bytes_read = fread(buffer, 1, BUFFER_SIZE, fr)) > 0)
    {
        fwrite(buffer, 1, bytes_read, stdout);
        fwrite(buffer, 1, bytes_read, fw);
        count += bytes_read;
    }

    printf("\n");
    fclose(fr);
    fclose(fw);

    printf("%s 파일은 %lu 바이트입니다\n", argv[1], count);
    printf("%s 파일로 복사되었습니다\n", out_filename);

    free(out_filename);
    return 0;
}