Программа C для удаления пустых строк

Это небольшая модификация вашего кода. Одна из наиболее серьезных проблем, которые я обнаружил, заключается в том, что исходный код записывался во входной файл. Это может привести к серьезному неопределенному поведению и сбою программы.

Еще одно изменение заключается в том, что я ввел оператор switch(). Мне, switch() представляет идеальный метод допроса каждого прочитанного символа файл. По моему мнению, это делает понимание и установку флагов очень ясными. Ваш пробег может отличаться.

Также добавлена ​​проверка ошибок. Можно было бы добавить больше проверок на ошибки, и я выделил в комментариях как минимум одно такое место.

Пожалуйста, смотрите комментарии в коде для получения более подробной информации в целом. Этот код будет читать входной файл и по ходу исключать пустые строки из выходного файла. Код скомпилирован и протестирован мной всего несколько минут назад.

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

/* Note: some files have lines longer than 100 chars. Update this value if needed: */
#define BUFF_SIZE 100 

int main()
{
    FILE *fp_in = fopen("LINES.txt", "r");
    FILE *fp_out = fopen("TEMP.txt", "w");
    int i=0, ch, e_flag = 0;
    char c_buff[BUFF_SIZE+1];

    if (fp_in == NULL || fp_out == NULL)
    {
        printf("Cannot open file\n");
        if (fp_in)
            fclose(fp_in);
        if (fp_out)
            fclose(fp_out);
        exit(EXIT_FAILURE);
    }

    while((ch = fgetc(fp_in)) != EOF)
    {
        switch(ch)
        {
            case '\n': /* newline */
                if (e_flag == 1)
                {
                    /* Code comes here if we have a newline and it
                    * is Not the 1st char of the line.  Else, it 
                    * would be a blank line and we don't want 
                    * to write blank lines.*/
                    c_buff[i++] = (char)ch;
                    c_buff[i] = '\0';

                    /* This is one reason I changed the file pointer 
                    * naming  convention...
                    * ... to add clarity and ease trouble-shooting. 
                    * Original  code was  writing to the input file.  
                    * Pretty sure That wasn't desired. */
                    fputs(c_buff, fp_out); /* Write to OUTPUT file */
                    i = e_flag = 0;
                }
            break;

            /* Exclude these from buffer/output *unless* the line 
            * contains other char types too: */
            case '\r': /* carriage return */
            case '\t': /* tab */
            case ' ':  /* space */
                if (e_flag == 1) /* Not the only character type on the line? */ 
                    c_buff[i++] = (char)ch;  /* A: yes. OK, add to buffer. */
            break;

            default:  /* all other chars: */
                e_flag = 1; /* Line contains chars that are Not just a 
                            * newline, tab, space, etc...
                            * Flag this line as a 'go' for OUTPUT */
                c_buff[i++] = (char)ch;  /* ALWAYS add default chars, */
            break;
        }
        if (i > BUFF_SIZE-2)
        {
            printf("Need a larger buffer for this file!!\n");
            printf("Buffer has %d chars.\n", i);
            return EXIT_FAILURE;
        }
    }
    fclose(fp_in);
    fclose(fp_out);
    remove("LINES.txt");
    /* Should check to ensure remove() is successful before the rename().
    * Left as an exercise for OP...*/
    rename("TEMP.txt", "LINES.txt");
    printf("\nFile editing complete\n");

    return EXIT_SUCCESS;
}

Если вы хотите буферизовать каждую строку ввода, используйте getline() вместо fgetc(). Я написал вам альтернативную реализацию без ограничения строки, которая смотрит на 1 или 2 символа вперед. Он определил строку empty() как строку, которая начинается с:

• {EOF}
• {'\n'}
• {'\r', EOF}
• {'\r', '\n'}

Впереди может быть деталь реализации empty(), если вы используете ungetc() или fseek(). «гарантирован только один возврат» для ungetc() (хотя в моей системе работало два). fseek(input, -2, SEEK_CUR) не работает на трубах.

Я использовал потоки stdin и stdout для удобства тестирования. Включить уже имеющуюся у вас обработку файлов — это тривиально. Просто не забудьте использовать if-защиту для первых двух вызовов fclose(), как демонстрирует ответ @gregspears.

#include <stdint.h>
#include <stdio.h>

int empty(FILE *input, int lookahead[static 2], uint8_t *n) {
    lookahead[0] = fgetc(input);
    *n = 0;
    if (lookahead[0] == EOF)
        return 1;
    *n = 1;
    if (lookahead[0] == '\n')
        return 1;
    if (lookahead[0] != '\r')
        return 0;
    lookahead[1] = fgetc(input);
    if (lookahead[1] == EOF)
        return 1; // ?
    *n = 2;
    return lookahead[1] == '\n';
}

void print(int lookahead[static 2], uint8_t n, FILE *input, FILE *output) {
    for(int i = 0; i < n; i++)
        fputc(lookahead[i], output);
    for(;;) {
        int ch = fgetc(input);
        if (ch == EOF) break;
        fputc(ch, output);
        if (ch == '\n') break;
    }
}

int main() {
    FILE *input = stdin;
    FILE *output = stdout;
    if (!input || !output) {
        printf("Cannot open file\n");
        return 1;
    }
    for(;;) {
        int lookahead[2];
        uint8_t n;
        if (empty(input, lookahead, &n)) {
            if (!n) break;
            continue;
        }
        print(lookahead, n, input, output);
    }
}

и пример запуска (echo используется для обеспечения отображения строки с «7»):

$ cat input.txt && echo && ./a.out < input.txt && echo
# \n
1
2

3
# \r\n
4
5

6
# {\r, EOF}
7
# \n
1
2
3
# \r\n
4
5
6
# {\r, EOF}
7