struct "undeclared (first use in this function)"

[crocodilechris]

Hallo,

ich muss bis Donnerstag einen Stack implementieren für die Uni und ich habe ein Problem, welches ich mir nicht erklären kann:
Ich habe 2 structs:

struct stack_element{
/* Wert des Elements */
long value;
/* Zeiger auf (naechstes oder vorheriges) Element im Stack */
struct stack_element *element;
};

/*
* Das struct zur Verwaltung des Stacks könnte zum Beispiel wie folgt aussehen:
*/
struct stack_data{
/* Zeiger auf das unterste/erste Element im Stack */
struct stack_element *first;
/* Zeiger auf das oberste Element im Stack */
struct stack_element *top;
/* Anzahl der Elemente im Stack */
long count;
};
zum Testen ist meine Main-Methode erstmal nur folgendes:

int main() {
struct stack_data * Stack;
//long Zahl;
// (a):
stack_initialize(&Stack);
return 0;
}
Die Funktion sieht wie folgt aus:

int stack_initialize (struct stack_data ** Stack) {
//Speicherplatz allockieren
struct stack_data *tmp_d = /*(stack_data *)*/calloc(1, sizeof stack_data);
if (tmp_d == NULL) {
return -1;
} else {
Stack = &tmp_d;
return 0;
}
};
Und die mir völlig unerklärliche Compilermeldung ist:

chris@Terra-Lap:Blatt6$ gcc stack_2.c
stack_2.c: In function ‘stack_initialize’:
stack_2.c:109: error: ‘stack_data’ undeclared (first use in this function)
stack_2.c:109: error: (Each undeclared identifier is reported only once
stack_2.c:109: error: for each function it appears in.)
Warum nun ist "stack_data" undeclared? Das ist doch das definierte struct. :(

[Yonibear]

struct stack_data *tmp_d = /*(stack_data *)*/calloc(1, sizeof stack_data);


Hier müsste es sizeof(struct stack_data) heißen.

[sommerfee]

und hier



Stack = &tmp_d;


muß es stattdessen


*Stack = tmp_d;

heißen.

Liebe Grüße,
Axel

[crocodilechris]

struct stack_data *tmp_d = /*(stack_data *)*/calloc(1, sizeof stack_data);
Hier müsste es sizeof(struct stack_data) heißen.
Ah da lag der Fehler, vielen Dank!
Find das nicht ganz intuitiv, dass man bei nem Verbund immer wieder angeben muss, dass es sich um einen Verbund handelt.

@sommerfee:
THX, hatte ich beim Neuschreiben verdreht. :o

Jetzt hab ich nurnoch nen Fehler beim pop, dass sieht bisher noch so aus, schmeißt aber nen "segmentation fault":

int stack_pop(struct stack_data * Stack, long * Wert) {
*Wert = Stack->top->value;
--(Stack->count);
*Stack->top = Stack->first[Stack->count];
//free(Stack->top);
return 0;
};Hab den Fehler schon auf die Zeile mit dem Stack->top lokalisiert.
Und wenn ich das free(Stack->top) versuche, schaltet sich "glibc" ein und gibt mir ne Memory map ...

[anda_skoa]

Hab den Fehler schon auf die Zeile mit dem Stack->top lokalisiert.

Dann ist entweder Stack, Stack->top oder Wert kein gültiger Pointer.
Setzt deine stack_initialize auch alle Werte auf sinnvolle Defaults? Z.B NULL für die enthaltenen Pointer?

Ciao,
_

[crocodilechris]

das wars gewesen, stack->first war nicht gesetzt, hab vergessen die Aktualisierung zu posten.