C - String-array dynamisch erweitern

OP #4712225
Hallo,
ich versuche gerade Strings aus einer Datei einzulesen und diese in ein 
String-Array zu schreiben. Dabei möchte ich nach jeder eingelesenen 
Zeile das Array um einen String erweitern.
Das Programm lässt sich auch kompilieren, ich bekomme aber nach dem 
Start einen Segmentation Fault.
Kann mir hier jemand Helfen?
lg
1
#include <stdio.h>
2
#include <stdlib.h>
3
#include <string.h>
4

5
int get_list();
6
void get_interprets();
7

8
char** interprets = 0;
9

10
int main()
11
{
12
  int songcount;
13

14
  songcount = get_list();
15
  //get_interprets();
16

17
  return 0;
18
}
19

20
int get_list()
21
{
22
  FILE *fp;
23
  char buf[255];
24
  int songcount = 0;
25

26
  fp = fopen("songlist.txt", "r");
27

28
  while(fgets(buf, 255, (FILE*)fp))
29
  {
30
    interprets = (char**) realloc(interprets, (songcount+1)*sizeof(char*));
31
    *interprets = (char*) malloc(255*sizeof(char));
32

33
    strcpy(interprets[songcount], buf);
34

35
    songcount++;
36
  }
37

38
  return songcount;
39
}
Gast #4712233
1
*interprets = (char*) malloc(255*sizeof(char));
2
 strcpy(interprets[songcount], buf);

ist falsch


müsste so sein:
1
 interprets[songcount] = (char*) malloc(255*sizeof(char));
2
 strcpy(interprets[songcount], buf);
1
while(fgets(buf, 255, (FILE*)fp))
wozu der cast?
OP #4712245
> ist falsch
> müsste so sein:
> interprets[songcount] = (char*) malloc(255*sizeof(char));
> strcpy(interprets[songcount], buf);
Oha.... Dummer Fehler :D Hätte ich selbst sehen sollen... Aber Danke!


> wozu der cast?
Hab die Zeile aus einem älteren Projekt kopiert weil ich mir nicht mehr 
sicher war (schon eine Zeit her dass ich C programmiert habe)... Hier 
ist er natürlich ohne Nutzen... Danke!

Vielen Danke, lg
Gast #4712264
Auf Mikrocontroller ist eine derart dynamische Speicherverwaltung 
allerdings ziemlich fehlerträchtig. Ich würde feste Puffergrößen 
bevorzugen, wann immer es geht.

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 10 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.