K&R Упражнение 1-19 неожиданный и непоследовательный вывод

Работал над упражнением K&R 1-19:

Напишите программу, которая переворачивает ввод построчно.

Написал следующую программу:

#include <stdio.h>
#define MAXLINE 1000

main () {
    int c, x, y, z;
    char ip[MAXLINE];
    char ln[MAXLINE];
    char rv[MAXLINE];

    for (x = 0;(c=getchar()) != EOF; ++x)       
        ip[x] = c;

    for (x = 0; ip[x] != '\0'; ++x) {               
        for (y = 0; ip[x] != '\n'; ++y) {           
            ln[y] = ip[x];
            ++x;
        }

        for (z = 0; y != -1; ++z) {                 
            rv[z] = ln[y];
            --y;
        }
        printf("%s\n", rv);                         
    }
}

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

Кто-нибудь сталкивался с такой волатильностью раньше, не меняя свой код? Как это исправить?


c runtime-error garbage kernighan-and-ritchie
person Nathaniel Lindsey    schedule 24.02.2016    source источник
comment
В прошлый раз вы разместили довольно много связанных вопросов. Пожалуйста, рассмотрите возможность использования другой (и более поздней - C сильно изменился с момента последнего обновления K&R) книги и/или учебника. переполнение стека не является учебным сайтом.   -  person too honest for this site    schedule 25.02.2016
comment
Учтите, что во втором цикле вы проверяете нулевые символы в ip. Вы установили нулевой символ, когда читали это?   -  person Jeff Mercado    schedule 25.02.2016
comment
Вы читаете весь ввод в 1000-байтовый буфер, не проверяя, превышает ли ввод 1000 байт. Вы не '\0' завершили строку ip, но цикл x ищет ip[x] != '\0'. И вы не поставили '\0' в конце строки rv, а это значит, что printf будет идти дальше конца строки.   -  person user3386109    schedule 25.02.2016
comment
настоятельно рекомендуем читать только одну строку за раз для обработки. Это означает, что первый цикл for() также должен проверять \n, чтобы найти конец строки. И этот же цикл должен проверять количество символов в строке, чтобы входной буфер не переполнялся.   -  person user3629249    schedule 25.02.2016
comment
в C существует очень мало допустимых подписей для main() Все они имеют возвращаемый тип и (если только вы не работаете на «голом железе» без ОС, этот возвращаемый тип будет int   -  person user3629249    schedule 25.02.2016
comment
при написании кода: 1) следуйте аксиоме: только один оператор в строке и (максимум) одно объявление переменной на оператор. 2) используйте осмысленные имена переменных. т.е. имена, которые указывают на использование или содержание (или лучше) и то, и другое.   -  person user3629249    schedule 25.02.2016

Ответы (1)


arrow_upward
0
arrow_downward

Если вы делаете K & R, я бы порекомендовал вам купить «Книгу ответов C», в которой есть ответы на все упражнения.

Попробуй это:

#include <stdio.h>

#define MAX_LINE 1024

void discardnewline(char s[])
{
  int i;
  for(i = 0; s[i] != '\0'; i++)
  {
    if(s[i] == '\n')
      s[i] = '\0';
  }
}

int reverse(char s[])
{
  char ch;
  int i, j;

  for(j = 0; s[j] != '\0'; j++)
  {
  }

  --j;

  for(i = 0; i < j; i++)
  {
    ch   = s[i];
    s[i] = s[j];
    s[j] = ch;
    --j;
  }

  return 0;
}

int getline(char s[], int lim)
{
  int c, i;

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

  if(c == '\n')
  {
    s[i++] = c;
  }

  s[i] = '\0';

  return i;

}

int main(void)
{
  char line[MAX_LINE];

  while(getline(line, sizeof line) > 0)
  {
    discardnewline(line);
    reverse(line);
    printf("%s\n", line);
  }
  return 0;
}

Некоторая часть из книги, но я даже сделал свои собственные модификации.

person Ashish Ahuja    schedule 25.02.2016