我一直在学习C编程语言书(K& R),并正在编写一个从输入中删除尾随空白的练习。我知道分段错误在某种程度上是一个与访问无法访问的内存有关的问题,但我已经多次读过这段代码而无法找到错误。我非常希望有人可以帮助找到这个错误并告诉我如何在将来发现这样的错误。
#include <stdio.h>
#define MAXLINE 1000
#define CHAR 0 /*character definition*/
#define TRAIL 1 /*determines whether program is in a trailing blank*/
int getinput(char input[], int max);
int trailrem(char input[], char copyto[]);
int len;
int main() {
char line[MAXLINE]; /*current line*/
char newline[MAXLINE];
int i, c, newreturn; /*integer counter, character holder, current line length, and trailrem return value*/
int len;
while((len = getinput(line, MAXLINE)) > 0) {
newreturn = trailrem(line, newline);
for(i = 0; i <= newreturn; ++i)
printf("\n%c\n", newline[i]);
}
}
int getinput(char input[],int max) {
int i, c, line;
for(i = 0; (c = getchar()) != EOF && c != '\n' && c < (max-1); ++i)
input[i] = c;
if(c == '\n') {
input[i] = c;
++i;
}
input[i] = '\0';
return i;
}
int trailrem(char input[], char copy[]) {
int i, j, minusin, state, r;
for(i = len; input[i] != EOF && i >= 0; --i) {
if(input[i] =='\n')
state = TRAIL;
else if((input[i] == ' ' && state == TRAIL) ||( input[i] == '\t' && state == TRAIL))
++minusin;
else if(state == TRAIL && (input[i] != ' ' || input[i] != '\t'))
state = CHAR;
for(j = (r = len-minusin); state == CHAR; --j){
copy[j-2] = input[i];
}
}
copy[r] = '\0';
copy[r-1] = '\n';
return r;
}
答案 0 :(得分:4)
代码中存在很多问题。但主要问题是,您有global
len
int len;
len
函数中的本地main
。
您正在len
函数中初始化main
,如下所示:
while((len = getinput(line, MAXLINE)) > 0)
因此更新了本地len
。但全球len
仍为0
。
您期望这样,您将在len
方法中获得trailrem
的更新值,但您不会。在trailrem()
中,您将len
等于0
!
for(i = len; input[i] != EOF && i >= 0; --i)
所以i
也是0
。因此,copy[r-1] = '\n';
会崩溃,因为r-1
可能是否定的。
其他问题:(评论中提到的BLUEPIXY和WhozCraig)。
for(i = 0; (c = getchar()) != EOF && c != '\n' && c < (max-1); ++i)
此处,c < (max-1)
应为i < (max-1)
。
++minusin;
函数中的{p> trailrem
minusin
未初始化。