Question

我使用调试器来查找我的错误。我能够找到错误是什么。但我不知道它为什么会发生。错误是linep在1 push完成后添加了一些垃圾地址（添加了1个空格）。所以在第二个push它会在这一行上给我一个错误：tmp1 = tmp2 = *x;因为linep是一些垃圾地址（我猜）。我不知道为什么linep成为垃圾地址。我希望有人可以解释一下。这是代码：

注意：该程序的目的是用4个空格替换每个标签。

#include <stdio.h>

#define MAXLINESIZE 1000
#define TABSPACES 4

void push(char *x, char val, int index);
int mgetline(char *s, int lim);

int main(void) {
    char line[MAXLINESIZE];
    char *linep = line;
    while (mgetline(line, MAXLINESIZE) > 0) {
        while (*linep) {
            if (*linep == '\t') {
                *linep = ' ';
                for (int i = 1; i <= TABSPACES - 1; i++) {
                    push(line, ' ', linep - line + 1);
                    linep++;
                }
            }
            linep++;
        }
        printf("%s", line);
        linep = line;
    }
    return 0;
}

void push(char *x, char val, int index) {
    char tmp1, tmp2, cnt;
    char *ptc;
    x += index - 1;
    ptc = x;
    tmp1 = tmp2 = *x;
    cnt = 0;
    while (tmp2) {
        tmp2 = *x;
        *x = tmp1;
        tmp1 = *(x + 1);
        *(x + 1) = tmp2;
        x += 2;
        cnt++;
    }
    *ptc = val;
}

int mgetline(char *s, int lim)
{
    int c;
    char *t = s;

    while (--lim > 0 && (c = getchar()) != EOF && c != '\n')
        *s++ = c;
    if (c == '\n')
        *s++ = c;

    *s = '\0';

    return s - t;
}

注意：输入.等于tab（09十六进制ASCII CODE），*等于空格（20十六进制ASCII CODE）。

输入：

A.B.C.D * E.F **。克**** h.j * .K **升

Answer 1

您需要设置循环以将指针正确复制到新缓冲区中，该缓冲区允许将空格放入字符串而不覆盖实际输入。您需要确保额外的空格不会溢出线。您还需要确保已放入EOL。

假设push工作，那么你实际上是将线推到右边，这可能会超出线阵列的最大尺寸。您也不会返回新指针，这意味着您将测试额外的空间以及每次移动行数组的全部内容。最好将它作为输入和输出线单独处理。

如果你坚持使用你的推送代码，那么使线条足够长以允许插入额外的空格以及在循环内添加linep = line;。计算原始行大小（使用strlen（）），将其增加所添加的空格总数，并进行测试以确保它不会溢出数组（将设置为大于MAXLINESIZE）。从push返回新指针（在空格之后），这样就不会检查循环中添加的所有空格。

我建议不要试图“修复”代码，但要重做它以提高效率。在显示我的建议后，我将显示您的原始代码，并附有指出错误的评论。

int main(void) {
    char line[MAXLINESIZE];
    char *linep = line;
    char newline[MAXLINESIZE];
    char *newlinep = newline;
    while (mgetline(line, MAXLINESIZE) > 0) {
        linep = line;
        newlinep = newline;
        while (*linep) {
            if (*linep == '\t') {
                for (int i = 0; i <= TABSPACES - 1; i++) {
                    *newlinep = ' ';
                    newlinep++;
                }
                if ((newlinep - &newline) >= MAXLINESIZE) {
                    printf("Line filled, need to stop");
                    newlinep = &newline[MAXLINESIZE-1];
                    break;
            }
            else {
                *newlinep = *linep;
                newlinep++;
            }
            linep++;
        }
        *newlinep = '\0'
        printf("%s", newline);
    }
    return 0;
}

以下是您的原始代码以及我对需要更改的内容的评论

char *push(char *x, char val, int index);
int mgetline(char *s, int lim);

int main(void) {
    // Make line big enough to hold the extra spaces
    char line[3*MAXLINESIZE];
    char *linep = line;
    while (mgetline(line, MAXLINESIZE) > 0) {
        // You need to reset the line pointer here
        linep = line;
        while (*linep) {
            if (*linep == '\t') {
                *linep = ' ';
                for (int i = 1; i <= TABSPACES - 1; i++) {
                    // Return the pointer to put after the spaces
                    // You also need to test to avoid overrun
                    linep = push(line, ' ', linep - line + 1);
                }
            }
            linep++;
        }
        printf("%s", line);
        linep = line;
    }
    return 0;
}

char * push(char *x, char val, int index) {
    // You need to check to make sure you do not overrun x.
    char tmp1, tmp2, cnt;
    char *ptc;
    x += index - 1;
    ptc = x;
    tmp1 = tmp2 = *x;
    cnt = 0;
    while (tmp2) {
        tmp2 = *x;
        *x = tmp1;
        tmp1 = *(x + 1);
        *(x + 1) = tmp2;
        x += 2;
        cnt++;
    }
    *ptc = val;
    return ptc;
}

Entab程序意外错误（读取访问违规）

1 个答案: