2017-12-11 5 views
0

Wenn ich mein Programm starte, scheint es irgendwo einen Segmentierungsfehler zu geben, aber ich habe Tests ausgeführt und es scheint, ich kann nicht finden, wo es ist. Ich verwende gcc -Wall -Werror -std = c99 -O zum Kompilieren.seg fehlerhafter Textpuffer newTB

Die Aufgabe besteht darin, einen Textpuffer in TB newTB zu erstellen, der durch "\ n" trennt und in eine verknüpfte Liste einfügt. Diese

test = newTB("hello\ngood bye\nworld\n"); 
[hello]->[good bye]->[world] 

ist derzeit die einzige Funktion im Moment, so dass es etwas sein, habe ich verpasst habe, dass es verursacht Fehler seg.

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

#include "textbuffer.h" 

struct textbuffer { 
    char *text; 
    int length; 
    TB next; 
}; 

void printBuffer(TB tb); 

TB newTB (char text[]){ 
    if (text == NULL) { 
     printf("buffer underflow\n"); 
     abort(); 
    } 

    TB new = calloc(1, sizeof(struct textbuffer)); 
    new->length = 0; 
    new->next = NULL; 
    TB current = new; 

    char *token; 
    int size; 

    //first token; intialise first. 
    token = strtok(text, "\n"); 
    size = strlen(token); 
    current->text = calloc(size + 1, sizeof(char)); 
    strcpy(current->text, token); 
    //use memset to add a NULL terminator at the end. 
    memset(current->text, '\0', size + 1); 
    new->length++; 
    current->next = NULL; 



    int count = 0; 
    while (token != NULL) { 
     //first linked list already done, do nothing for the first loop. 
     if (count == 0) { 

     } else { 
      //create next textbuffer and fill it with the token. 
      current->next = calloc(1, sizeof(struct textbuffer)); 
      current = current->next; 
      size = strlen(token); 
      current->text = calloc(size + 1, sizeof(char)); 
      strcpy(current->text, token); 
      memset(current->text, '\0', size + 1); 

      new->length++; 
     } 

     count++; 
     token = strtok(NULL, "\n"); 
    } 

    current->next = NULL; 


    return new; 
} 
+0

Verwenden 'malloc' statt' calloc' es, wie Sie es trotzdem zu initialisieren sind. –

+1

'memset (current-> text, '\ 0', size + 1);' setzt den gesamten String auf Null. Nicht was du willst. Sie können das entfernen, 'strcpy' tut es schon (ich hätte' strdup' verwendet) –

+1

Erstellen Sie eine Debug-Version (löschen Sie die '-O'-Flagge und fügen Sie die '-g' -Flagge beim Erstellen hinzu), und [lernen wie um Ihre Programme zu debuggen] (https://ericlippert.com/2014/03/05/how-to-debug-small-programs/). –

Antwort

0

Das Problem liegt in der STRTOK-Funktion. Wie in diesem Post beschrieben: C's strtok() and read only string literals
Die Strtok-Funktion funktioniert nicht für String-Literale, da es die Zeichenfolge ändert.
Versuche mal:

char str[] = "hello\ngood bye\nworld\n"; 
test = newTB(str);