解析章节时的Coredump

Question

我正在做一个阅读书籍的家庭作业。首先读入一行，并在该行指向一个指针。然后一个段落函数读取行并将它们的地址存储到一个指针数组中。现在，我正在阅读一章（下一行被识破的一段被破坏）。它应该调用get_paragraph()并存储段落的地址，直到出现新章节。

新的篇章是在本书，在该行的第一个字符是不是空格唯一的一次。我认为这是我的代码有问题。到此为止的所有功能都能正常工作。我希望我已经提供了足够的信息。代码在启动时编译但核心转储。

我是一名学生，学习的，所以请善待。谢谢。

char*** get_chapter(FILE * infile){ 

    int i=0; 

    char **chapter[10000];//an array of pointers 
    // Populate the array 
    while(chapter[i]=get_paragraph(infile)) { //get address store into array 
     if(!isspace(**chapter[0])){ //check to see if it is a new chapter<---problem line? 
     // save paragraph not used in chapter using static to put into next chapter 
     break; 
     } 
     i++;//increment array 
    } 
    //add the null 
    chapter[++i]='\0';//put a null at the end to signify end of array 
    //Malloc the pointer 
    char**(*chap) = malloc(i * sizeof(*chap));//malloc space 
    //Copy the array to the pointer 
    i=0;//reset address 
    while(chapter[i]){//while there are addresses in chapter 
     chap[i] = chapter[i++];//change addresses into chap 
    } 
    chap[i]='\0';//null to signify end of chapter 
    //Return the pointer 
    return(chap);//return pointer to array 
    } 

对于那些谁宁愿看到没有评论：

char*** get_chapter(FILE * infile){ 

    int i=0; 

    char **chapter[10000]; 
    while(chapter[i]=get_paragraph(infile)) { 
     if(!isspace(**chapter[0])){ 
     break; 
     } 
     i++; 
    } 
    chapter[++i]='\0'; 
    char**(*chap) = malloc(i * sizeof(*chap));//malloc space 
    i=0; 
    while(chapter[i]){ 
     chap[i] = chapter[i++]; 
    } 
    chap[i]='\0'; 
    return(chap); 
    } 

Answer 1

我可以建议你使用for循环，而不是while S'如果空间不足，你需要停下来，所以你不妨使用适当的结构。

我怀疑你有这个代码中的错误：

while(chapter[i]=get_paragraph(infile)) { 
    if(!isspace(**chapter[0])){ 
    break; 
    } 
    i++; 
} 
chapter[++i]='\0'; 

首先，它不应该是chapter[i]，而不是chapter[0]？您想知道chapter[i]处的指针是否指向空格，而不是chapter中的第一个指针。所以这可能会无限循环 - 因此需要一个for循环，所以你不会意外地永远循环。

其次，你在chapter[++i]分配在同时段结束递增i，然后再次。 i在while条件中断之前已经被最终循环执行递增，所以它已经是正确的使用位置。++i递增之前产生的价值，所以大概你打算在这里有i++，所以它会增加后产生当前值i。无论哪种方式，我们中的一个人对你的意思感到困惑，所以也许只是为了清晰起见将增量放在单独的行上。编译器将整理出任何可用的优化。

最后（我可能在这里错了）为什么你将值设置为'\0'？这是一个空字符，不是吗？但是你的数组是指针。我认为，空指针将是0，而不是'\0'。如果我是对的，如果'\0'与空指针产生相同的一组零，那么你可能仍然会摆脱它。

Answer 2

它不应该是：

if(!isspace(**chapter[i])){ 

每个chapter[i]是一个指针的指针到char，这char是每章中的第一个字。所以**chapter[i]代表i章中的第一个字符。使用chapter[0]只会看第一章。

Answer 3

您是否尝试过单在gdb步进虽然，偶尔倾销局部变量查看当前的状态？这是学习的好方法。您可能需要添加一些额外的中间变量“信息机”会自动转储以及（指向当前XXX，XXX是层次结构中的各种物品）

我假设一个GNU环境：

% gcc -g homework.c -o hw 
% gdb hw 
(gdb) b 10 
(gdb) r 
(gdb) info locals 
(gdb) n 
(gdb) info locals 
... 

替换“10”与附近的功能的开始的合适的行数。

Answer 4

评论在线。

char*** get_chapter(FILE * infile) { 
    int i=0; 

    // This is a zero length array! 
    // (The comma operator returns its right-hand value.) 
    // Trying to modify any element here can cause havoc. 
    char **chapter[10,000]; 

    while(chapter[i]=get_paragraph(infile)) { 
    // Do I read this right? I translate it as "if the first character of 
    // the first line of the first paragraph is not whitespace, we're done." 
    // Not the paragraph just read in -- the first paragraph. So this will exit 
    // immediately or else loop forever and walk off the end of the array 
    // of paragraphs. I think you mean **chapter[i] here. 
    if(!isspace(**chapter[0])){ 
     break; 
    } 
    i++; 
    } 

    // Using pre-increment here means you leave one item in the array uninitialized 
    // which can also cause a fault later on. Use post-increment instead. 
    // Also '\0' here is the wrong sort of zero; I think you need NULL instead. 
    chapter[++i]='\0'; 

    char**(*chap) = malloc(i * sizeof(*chap)); 
    i=0; 
    while(chapter[i]) { 
    // This statement looks ambiguous to me. Referencing a variable twice and 
    // incrementing it in the same statement? You may end up with an off-by-one error. 
    chap[i] = chapter[i++]; 
    } 

    // Wrong flavor of zero again. 
    chap[i]='\0'; 
    return(chap); 
}