Упражнение K&R 1.19 (обратная функция)

У вашей функции reverse есть проблемы:

• Вы не должны сохранять новую строку в массиве s, так как вы не хотите, чтобы она участвовала в обратной операции.

• Вы должны остановить последующий цикл for, когда дойдете до конца строки в s, а не до конца буфера.

• Вы должны обнулить массив rev.

• Вам не нужно выводить массив rev по одному символу за раз, используйте его как строку.

Вот исправленная и упрощенная версия:

#include <stdio.h>

#define LIM 40

int reverse(char line[], int size);

int main(void) {
    char line[LIM];
    int len;

    while (reverse(line, LIM) > 0) {
        continue;
    }
    printf("\n      END OF THE PROGRAM     \n");
    return 0;
}

/* THE REVERSE FUNCTION */
int reverse(char s[], int size) {
    char rev[size];
    int i, r, c, len;
    for (i = 0; i < size - 1 && (c = getchar()) != EOF && c != '\n'; i++) {
        s[i] = c;
    }
    len = i;
    s[i] = '\0';

    for (i = 0; i < len; i++) {
        rev[len - i - 1] = s[i];
    }
    rev[i] = '\0';

    printf("%s\n", rev);
    return len;
}

Немного более модульное решение. Я не был точно уверен, что K&R имел в виду под «по одной строке за раз». Но это изменит строку до тех пор, пока она не найдет новую строку. Затем дождитесь пользователя и повторите.

#include <stdio.h>

#define MAXLINE 1000

int get_line(char s[], int limit);
int reverse(char to[], char from[], int l);

int main() {
    int size;
    char line[MAXLINE];
    while ((size = get_line(line, MAXLINE)) > 0) {
        char revline[size];
        int len = reverse(revline, line, size);
        printf("%s\n", revline);
    }

    return 0;
}

int reverse(char to[], char from[], int l) {
    int i;
    int j = l - 2;
    for (i = 0; i < l; i++, j--) {
        to[i] = from[j];
    }
    to[i] = '\0';
    return i;
}

// read a line into s until limit
// return length of line
int get_line(char s[], int limit) {
    int c = 0;
    int i = 0;

    for (i = 0; i < limit-1 && (c = getchar()) != '\n'; ++i) {
        s[i] = c;
    }
    if (c == '\n') {
        s[i] = c;
        ++i;
    }
    s[i] = '\0';
    return i;
}

Выход:

testing one two
owt eno gnitset
three four
ruof eerht
five six
xis evif