我的C代码有什么问题？

当我调用

print\u token（）

时，我的应用程序崩溃。我花了一整天试图解决这个问题，但没有成功，我放弃了。救命啊

//#include "memory.h"
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

    struct TOKEN
    {
            int type;
            char *str;
    };

#define TOKEN_SIZE sizeof(struct TOKEN)

struct TOKEN *ini;
struct TOKEN *cur;
struct TOKEN *end;

int size = TOKEN_SIZE * 1;

void init_token_buffer(void);
void save_token(int type, const char *str);
void print_tokens(void);

void init_token_buffer(void)
{
    ini = (struct TOKEN*) malloc(size); // 8 bytes...
    cur = ini;
    end = ini + size;
}

void save_token(int type, const char *str)
{
    printf("cur == end : %d\n", cur == end);

    if (cur == end)
    {
        int lastSize = (end - ini);
        int  newSize = size + lastSize;
        ini = (struct TOKEN*) realloc(ini, newSize); // 16,24,32...
        end = ini +  newSize;
        cur = ini + lastSize;

        printf("l:%d, n:%d\n", lastSize, newSize);
    }

    int  strSize = strlen(str) * sizeof(char) + sizeof(char);
    char *newStr = (char*) malloc(strSize);

    cur->str  = (char*) memcpy(newStr, str, strSize);
    cur->type = type;

    printf("string:%s, type:%X\n", cur->str, cur->type);
    // here it prints right!
    cur += TOKEN_SIZE;
}

void print_tokens()
{
    int i;
    int total = (cur - ini);

    printf("\ntotal tokens:%d\n", total / TOKEN_SIZE);
    printf("total  bytes:%d\n", total);

    for (i = 0; i < total; i += TOKEN_SIZE)
    {
        struct TOKEN *t = &ini[i];
        printf("string:%s, type:%X\n", t->str, t->type);
        // here it crashes at the 2º loop! (ini[8])
    }
}

---

以下两点合在一起就是你的错误：

i+=令牌大小

&ini[i]

由于

ini

属于

TOKEN*

类型，因此其访问索引是

sizeof TOKEN

的倍数。你必须了解

我还建议您至少查看编译器警告：

$ gcc -Wall -pedantic -o test ./test.c 
./test.c: In function ‘init_token_buffer’:
./test.c:26:38: warning: C++ style comments are not allowed in ISO C90 [enabled by default]
./test.c:26:38: warning: (this will be reported only once per input file) [enabled by default]
./test.c: In function ‘save_token’:
./test.c:46:2: warning: ISO C90 forbids mixed declarations and code [-pedantic]
./test.c: In function ‘print_tokens’:
./test.c:62:40: warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘long unsigned int’ [-Wformat]
./test.c: In function ‘main’:
./test.c:81:1: warning: control reaches end of non-void function [-Wreturn-type]

。。。也可以使用valgrind或类似工具检查程序的运行时错误：

$ valgrind ./test 
==9732== Memcheck, a memory error detector
==9732== Copyright (C) 2002-2010, and GNU GPL'd, by Julian Seward et al.
==9732== Using Valgrind-3.6.1-Debian and LibVEX; rerun with -h for copyright info
==9732== Command: ./test
==9732== 
cur == end : 0
string:aaa, type:0
cur == end : 1
l:16, n:32
==9732== Invalid write of size 8
==9732==    at 0x400769: save_token (in /tmp/test)
==9732==    by 0x40089B: main (in /tmp/test)
==9732==  Address 0x51ce1e8 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid write of size 4
==9732==    at 0x400777: save_token (in /tmp/test)
==9732==    by 0x40089B: main (in /tmp/test)
==9732==  Address 0x51ce1e0 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 4
==9732==    at 0x400780: save_token (in /tmp/test)
==9732==    by 0x40089B: main (in /tmp/test)
==9732==  Address 0x51ce1e0 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 8
==9732==    at 0x400789: save_token (in /tmp/test)
==9732==    by 0x40089B: main (in /tmp/test)
==9732==  Address 0x51ce1e8 is not stack'd, malloc'd or (recently) free'd
==9732== 
string:bbb, type:0
cur == end : 1
l:32, n:48
==9732== Invalid write of size 8
==9732==    at 0x400769: save_token (in /tmp/test)
==9732==    by 0x4008AA: main (in /tmp/test)
==9732==  Address 0x51ce398 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid write of size 4
==9732==    at 0x400777: save_token (in /tmp/test)
==9732==    by 0x4008AA: main (in /tmp/test)
==9732==  Address 0x51ce390 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 4
==9732==    at 0x400780: save_token (in /tmp/test)
==9732==    by 0x4008AA: main (in /tmp/test)
==9732==  Address 0x51ce390 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 8
==9732==    at 0x400789: save_token (in /tmp/test)
==9732==    by 0x4008AA: main (in /tmp/test)
==9732==  Address 0x51ce398 is not stack'd, malloc'd or (recently) free'd
==9732== 
string:ccc, type:0
cur == end : 1
l:48, n:64
==9732== Invalid write of size 8
==9732==    at 0x400769: save_token (in /tmp/test)
==9732==    by 0x4008B9: main (in /tmp/test)
==9732==  Address 0x51ce558 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid write of size 4
==9732==    at 0x400777: save_token (in /tmp/test)
==9732==    by 0x4008B9: main (in /tmp/test)
==9732==  Address 0x51ce550 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 4
==9732==    at 0x400780: save_token (in /tmp/test)
==9732==    by 0x4008B9: main (in /tmp/test)
==9732==  Address 0x51ce550 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 8
==9732==    at 0x400789: save_token (in /tmp/test)
==9732==    by 0x4008B9: main (in /tmp/test)
==9732==  Address 0x51ce558 is not stack'd, malloc'd or (recently) free'd
==9732== 
string:ddd, type:0

total tokens:4
total  bytes:64
string:aaa, type:0
==9732== Invalid read of size 4
==9732==    at 0x400843: print_tokens (in /tmp/test)
==9732==    by 0x4008BE: main (in /tmp/test)
==9732==  Address 0x51ce350 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 8
==9732==    at 0x400849: print_tokens (in /tmp/test)
==9732==    by 0x4008BE: main (in /tmp/test)
==9732==  Address 0x51ce358 is not stack'd, malloc'd or (recently) free'd
==9732== 
string:(null), type:0
string:(null), type:0
string:ddd, type:0
==9732== 
==9732== HEAP SUMMARY:
==9732==     in use at exit: 80 bytes in 5 blocks
==9732==   total heap usage: 8 allocs, 3 frees, 176 bytes allocated
==9732== 
==9732== LEAK SUMMARY:
==9732==    definitely lost: 12 bytes in 3 blocks
==9732==    indirectly lost: 0 bytes in 0 blocks
==9732==      possibly lost: 0 bytes in 0 blocks
==9732==    still reachable: 68 bytes in 2 blocks
==9732==         suppressed: 0 bytes in 0 blocks
==9732== Rerun with --leak-check=full to see details of leaked memory
==9732== 
==9732== For counts of detected and suppressed errors, rerun with: -v
==9732== ERROR SUMMARY: 18 errors from 14 contexts (suppressed: 4 from 4)

以下是您的程序的外观：

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

struct TOKEN
{
    int type;
    char *str;
};

struct TOKEN *ini = NULL;
struct TOKEN *cur = NULL;
struct TOKEN *end = NULL;

static void save_token(int type, const char *str)
{
    struct TOKEN *p;
    size_t n;
    char *s;

    if (cur == end) {
        n = (size_t)(end - ini) + 1;
        p = realloc(ini, n * sizeof(struct TOKEN));
        if (!p) {
            /* Out of memory??? */
            abort();
        }
        cur = p + (cur - ini);
        ini = p;
        end = p + n;
    }

    if (!(s = strdup(str))) {
            /* SOS! */
            abort();
    }

    cur->str = s;
    cur->type = type;
    ++cur;
}

static void print_tokens()
{
    struct TOKEN *i = ini;
    struct TOKEN *e = end;

    printf("Total tokens: %ld\n", (e - i));
    printf("Total bytes: %ld\n", (e - i) * sizeof(struct TOKEN));

    while (i < e) {
        printf("string:%s, type:%X\n", i->str, i->type);
        ++i;
    }
}

static void clear_tokens()
{
    struct TOKEN *i = ini;
    struct TOKEN *e = end;

    if (!ini)
        return;

    while (i < e) {
        if (i->str)
            free(i->str);
        ++i;
    }

    free(ini);
    ini = cur = end = NULL;
}

int main() {
    save_token(0, "aaa");
    save_token(1, "bbb");
    save_token(2, "ccc");
    save_token(3, "ddd");
    print_tokens();
    clear_tokens(); /* Do not forget to clean up resources! */
    return 0;
}

#包括
#包括
#包括
结构令牌
{
int型；
char*str；
};
结构令牌*ini=NULL；
结构令牌*cur=NULL；
结构令牌*end=NULL；
静态void save_令牌（int类型，const char*str）
{
结构令牌*p；
尺寸；
char*s；
如果（cur==end）{
n=（大小_t）（结束-ini）+1；
p=realloc（ini，n*sizeof（结构令牌））；
如果（！p）{
/*内存不足*/
中止（）；
}
cur=p+（cur-ini）；
ini=p；
结束=p+n；
}
如果（！（s=strdup（str）））{
/*求救*/
中止（）；
}
cur->str=s；
cur->type=type；
++cur；
}
静态无效打印令牌（）
{
结构令牌*i=ini；
结构令牌*e=end；
printf（“令牌总数：%ld\n”，（e-i））；
printf（“总字节数：%ld\n”，（e-i）*sizeof（结构令牌））；
而（istr，i->type）；
++一,；
}
}
静态无效清除令牌（）
{
结构令牌*i=ini；
结构令牌*e=end；
如果（！ini）
返回；
而（istr）
自由（i->str）；
++一,；
}
免费（ini）；
ini=cur=end=NULL；
}
int main（）{
保存令牌（0，“aaa”）；
储蓄券（1，“bbb”）；
保存令牌（2，“ccc”）；
保存令牌（3，“ddd”）；
打印令牌（）；
清除令牌（）；/*不要忘记清理资源*/
返回0；
}

我很欣赏你的努力学习C。你的代码看起来部分C99，部分预ANSI C，部分C++：-D我建议你得到一些好的C书，并且首先摆脱IDE，它位于你和编译器之间。当然，您仍然可以将其用作编辑器，但请尝试使用命令行工具进行编译，并了解正在发生的事情

---

祝你好运

以下两点合在一起是您的错误：

i+=令牌大小

&ini[i]

由于

ini

属于

TOKEN*

类型，因此其访问索引是

sizeof TOKEN

的倍数。你必须了解

我还建议您至少查看编译器警告：

$ gcc -Wall -pedantic -o test ./test.c 
./test.c: In function ‘init_token_buffer’:
./test.c:26:38: warning: C++ style comments are not allowed in ISO C90 [enabled by default]
./test.c:26:38: warning: (this will be reported only once per input file) [enabled by default]
./test.c: In function ‘save_token’:
./test.c:46:2: warning: ISO C90 forbids mixed declarations and code [-pedantic]
./test.c: In function ‘print_tokens’:
./test.c:62:40: warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘long unsigned int’ [-Wformat]
./test.c: In function ‘main’:
./test.c:81:1: warning: control reaches end of non-void function [-Wreturn-type]

。。。也可以使用valgrind或类似工具检查程序的运行时错误：

$ valgrind ./test 
==9732== Memcheck, a memory error detector
==9732== Copyright (C) 2002-2010, and GNU GPL'd, by Julian Seward et al.
==9732== Using Valgrind-3.6.1-Debian and LibVEX; rerun with -h for copyright info
==9732== Command: ./test
==9732== 
cur == end : 0
string:aaa, type:0
cur == end : 1
l:16, n:32
==9732== Invalid write of size 8
==9732==    at 0x400769: save_token (in /tmp/test)
==9732==    by 0x40089B: main (in /tmp/test)
==9732==  Address 0x51ce1e8 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid write of size 4
==9732==    at 0x400777: save_token (in /tmp/test)
==9732==    by 0x40089B: main (in /tmp/test)
==9732==  Address 0x51ce1e0 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 4
==9732==    at 0x400780: save_token (in /tmp/test)
==9732==    by 0x40089B: main (in /tmp/test)
==9732==  Address 0x51ce1e0 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 8
==9732==    at 0x400789: save_token (in /tmp/test)
==9732==    by 0x40089B: main (in /tmp/test)
==9732==  Address 0x51ce1e8 is not stack'd, malloc'd or (recently) free'd
==9732== 
string:bbb, type:0
cur == end : 1
l:32, n:48
==9732== Invalid write of size 8
==9732==    at 0x400769: save_token (in /tmp/test)
==9732==    by 0x4008AA: main (in /tmp/test)
==9732==  Address 0x51ce398 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid write of size 4
==9732==    at 0x400777: save_token (in /tmp/test)
==9732==    by 0x4008AA: main (in /tmp/test)
==9732==  Address 0x51ce390 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 4
==9732==    at 0x400780: save_token (in /tmp/test)
==9732==    by 0x4008AA: main (in /tmp/test)
==9732==  Address 0x51ce390 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 8
==9732==    at 0x400789: save_token (in /tmp/test)
==9732==    by 0x4008AA: main (in /tmp/test)
==9732==  Address 0x51ce398 is not stack'd, malloc'd or (recently) free'd
==9732== 
string:ccc, type:0
cur == end : 1
l:48, n:64
==9732== Invalid write of size 8
==9732==    at 0x400769: save_token (in /tmp/test)
==9732==    by 0x4008B9: main (in /tmp/test)
==9732==  Address 0x51ce558 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid write of size 4
==9732==    at 0x400777: save_token (in /tmp/test)
==9732==    by 0x4008B9: main (in /tmp/test)
==9732==  Address 0x51ce550 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 4
==9732==    at 0x400780: save_token (in /tmp/test)
==9732==    by 0x4008B9: main (in /tmp/test)
==9732==  Address 0x51ce550 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 8
==9732==    at 0x400789: save_token (in /tmp/test)
==9732==    by 0x4008B9: main (in /tmp/test)
==9732==  Address 0x51ce558 is not stack'd, malloc'd or (recently) free'd
==9732== 
string:ddd, type:0

total tokens:4
total  bytes:64
string:aaa, type:0
==9732== Invalid read of size 4
==9732==    at 0x400843: print_tokens (in /tmp/test)
==9732==    by 0x4008BE: main (in /tmp/test)
==9732==  Address 0x51ce350 is not stack'd, malloc'd or (recently) free'd
==9732== 
==9732== Invalid read of size 8
==9732==    at 0x400849: print_tokens (in /tmp/test)
==9732==    by 0x4008BE: main (in /tmp/test)
==9732==  Address 0x51ce358 is not stack'd, malloc'd or (recently) free'd
==9732== 
string:(null), type:0
string:(null), type:0
string:ddd, type:0
==9732== 
==9732== HEAP SUMMARY:
==9732==     in use at exit: 80 bytes in 5 blocks
==9732==   total heap usage: 8 allocs, 3 frees, 176 bytes allocated
==9732== 
==9732== LEAK SUMMARY:
==9732==    definitely lost: 12 bytes in 3 blocks
==9732==    indirectly lost: 0 bytes in 0 blocks
==9732==      possibly lost: 0 bytes in 0 blocks
==9732==    still reachable: 68 bytes in 2 blocks
==9732==         suppressed: 0 bytes in 0 blocks
==9732== Rerun with --leak-check=full to see details of leaked memory
==9732== 
==9732== For counts of detected and suppressed errors, rerun with: -v
==9732== ERROR SUMMARY: 18 errors from 14 contexts (suppressed: 4 from 4)

以下是您的程序的外观：

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

struct TOKEN
{
    int type;
    char *str;
};

struct TOKEN *ini = NULL;
struct TOKEN *cur = NULL;
struct TOKEN *end = NULL;

static void save_token(int type, const char *str)
{
    struct TOKEN *p;
    size_t n;
    char *s;

    if (cur == end) {
        n = (size_t)(end - ini) + 1;
        p = realloc(ini, n * sizeof(struct TOKEN));
        if (!p) {
            /* Out of memory??? */
            abort();
        }
        cur = p + (cur - ini);
        ini = p;
        end = p + n;
    }

    if (!(s = strdup(str))) {
            /* SOS! */
            abort();
    }

    cur->str = s;
    cur->type = type;
    ++cur;
}

static void print_tokens()
{
    struct TOKEN *i = ini;
    struct TOKEN *e = end;

    printf("Total tokens: %ld\n", (e - i));
    printf("Total bytes: %ld\n", (e - i) * sizeof(struct TOKEN));

    while (i < e) {
        printf("string:%s, type:%X\n", i->str, i->type);
        ++i;
    }
}

static void clear_tokens()
{
    struct TOKEN *i = ini;
    struct TOKEN *e = end;

    if (!ini)
        return;

    while (i < e) {
        if (i->str)
            free(i->str);
        ++i;
    }

    free(ini);
    ini = cur = end = NULL;
}

int main() {
    save_token(0, "aaa");
    save_token(1, "bbb");
    save_token(2, "ccc");
    save_token(3, "ddd");
    print_tokens();
    clear_tokens(); /* Do not forget to clean up resources! */
    return 0;
}

#包括
#包括
#包括
结构令牌
{
int型；
char*str；
};
结构令牌*ini=NULL；
结构令牌*cur=NULL；
结构令牌*end=NULL；
静态void save_令牌（int类型，const char*str）
{
结构令牌*p；
尺寸；
char*s；
如果（cur==end）{
n=（大小_t）（结束-ini）+1；
p=realloc（ini，n*sizeof（结构令牌））；
如果（！p）{
/*内存不足*/
中止（）；
}
cur=p+（cur-ini）；
ini=p；
结束=p+n；
}
如果（！（s=strdup（str）））{
/*求救*/
中止（）；
}
cur->str=s；
cur->type=type；
++cur；
}
静态无效打印令牌（）
{
结构令牌*i=ini；
结构令牌*e=end；
printf（“令牌总数：%ld\n”，（e-i））；
printf（“总字节数：%ld\n”，（e-i）*sizeof（结构令牌））；
而（istr，i->type）；
++一,；
}
}
静态无效清除令牌（）
{
结构令牌*i=ini；
结构令牌*e=end；
如果（！ini）
返回；
而（istr）
自由（i->str）；
++一,；
}
免费（ini）；
ini=cur=end=NULL；
}
int main（）{
保存令牌（0，“aaa”）；
储蓄券（1，“bbb”）；
保存令牌（2，“ccc”）；
保存令牌（3，“ddd”）；
打印令牌（）；
清除令牌（）；/*不要忘记清理资源*/
返回0；
}

我很欣赏你的努力学习C。你的代码看起来部分C99，部分预ANSI C，部分C++：-D我建议你得到一些好的C书，并且首先摆脱IDE，它位于你和编译器之间。当然，您仍然可以将其用作编辑器，但请尝试使用命令行工具进行编译，并了解正在发生的事情

---

祝你好运

在你的源代码中，它用

s

表示

void print\u tokens（）

。考虑到你从未定义一个名为

print\u token（）

的函数，我不相信你发布的内容甚至编译了。对不起，这是一个打字错误。这并不是因为我的代码崩溃了。我没有收到任何编译器错误，我的应用程序在运行时崩溃。也许你应该使用调试器？它不会在我的机器上崩溃。是

memory.h

是定制的吗？在你的源代码中，它用

s

表示

void print\u tokens（）

。考虑到你从未定义一个名为

print\u token（）

的函数，我甚至不相信你发布的是编译的。对不起，这是一个打字错误。这并不是因为我的代码崩溃了。我没有收到任何编译器错误，我的应用程序在运行时崩溃。也许你应该使用调试器？它不会在我的机器上崩溃。是

内存。h

是定制的吗？@MarkSmith:谢谢。5年的钢琴课帮助很大！我还要感谢你提到编译器警告，但我已经提到了。编译器警告出现是有原因的。如果您收到编译器警告，我建议您修复它们，或彻底证明它们的合理性，尤其是当您的代码在运行时遇到问题时。@Fabricio:我已经用或多或少正确的代码为您更新了答案。希望它能对你有用