.exe Datei stürtzt beim Aufruf ab

[Programmierer (Gast)]

Hallo Leute,
Ich habe ein Programm geschrieben, aber diese stürzt komischerweise beim 
aufrufen ab. Ich habe es heute in der Schule probiert, und dort hat es 
zuerst nicht funktioniert, dann habe ich einen neuen Ordner auf meinem 
Stick erstellt und es dort aufgerufen und dann ging es.

Jetzt zuhause geht es wieder nicht. Ich benutze Codeblocks.

Kurz zum Programm, es sollen mehrere Obejktnummern eingelesen werden und 
diese dann ausgegeben werden bzw ein gewünschtes gelöscht werden können.

Hier der Code:

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

struct haeuser
{
  int objektnummer;
   char strasse[20];
   int hausnummer;
   char haustyp;
   float wohnflaeche;
   int baujahr;
   float preis;
   struct haeuser *zg;
};


void hausanhaengen(struct haeuser *head, int Objekt);
void hausloeschen(struct haeuser *head, int objektnr);
void Ausgabe(struct haeuser *head);

void main()
{

   struct haeuser *head;
   head=(struct haeuser*)malloc(sizeof(struct haeuser));
   int Objekt=0,objektnr=0;


   char buchstabe;
   int flag=1;


      while(flag==1)
      {
      printf("Bitte geben Sie einen Buchstaben ein:\n");
        fflush(stdin);
       scanf("%c",&buchstabe);

       switch(buchstabe)
       {

            case'e':

              hausanhaengen(head,Objekt++);
               break;

            case'l':
              printf("Bitte geben Sie die zu loeschende Objektnummer ein:\n");
               scanf("%i",&objektnr);
              hausloeschen(head,objektnr);
          break;

            case'a':
              Ausgabe(head);
              break;

            case 'b':
             flag=0;
              break;
       }
     }

}


void hausanhaengen(struct haeuser *head,int Obj)
{
  struct haeuser *n;
   while((*head).zg!=NULL)
   head=(*head).zg;
   n=(struct haeuser*)malloc(sizeof(struct haeuser));

   if(n!=NULL)
   {
     (*n).zg=NULL;
      (*n).objektnummer=Obj;
      (*head).zg=n;
   }
   else
   printf("Fehler bei der Speicherreservierung\n");
}

void hausloeschen(struct haeuser *head,int objektnr)
{
  struct haeuser *suchhaus,*suchhausz;
   suchhaus=head;
   suchhausz=(*suchhaus).zg;
   while((*suchhausz).objektnummer!=objektnr)
   {
     suchhaus=suchhausz;
      suchhausz=(*suchhaus).zg;
   }
   (*suchhaus).zg=(*suchhausz).zg;
  free(suchhausz);  //Speicher freigeben
}

void Ausgabe(struct haeuser *head)
{
  while((*head).zg!=NULL)
  {
      head=(*head).zg;
      printf("Nummer: %i\n",(*head).objektnummer);
  }
  printf("\n");

}

Danke für die Hilfe!

[Rufus Τ. F. (rufus)]

Im Debugger laufenlassen. Dann kannst Du selbst herausfinden, was 
schiefläuft.
Ja: Genau dafür ist der Debugger da, nutze ihn!

[Karl H. (kbuchegg)]

void hausanhaengen(struct haeuser *head,int Obj)
{
  struct haeuser *n;
   while((*head).zg!=NULL)

...

wo kommt head her?
Von hier

   head=(struct haeuser*)malloc(sizeof(struct haeuser));

...
              hausanhaengen(head,Objekt++);

tja.
Aber nur weil du mittels malloc Speicher allokiert hast, bedeutet das 
nicht, dass da nur 0-en (und damit ein NULL Pointer) drinnen steht.

schreib dir eine Funktion, die ein neues Haus allokiert und dabei den 
Inhalt des neuen Hauses in einen vernünftigen Zustand bringt.
Und die Funktion verwendest du dann, wenn du ein neues Haus brauchst.

[Karl H. (kbuchegg)]

Ich weiß nicht, ob ihr das schon gelernt habt.

Aber das hier (beispielsweise), ist eine sehr mühsame Schreibweise

      (*head).zg=n;

Klammer auf, den Stern und wieder Klammer zu. Sie ist allerdings zu 100% 
richtig. Da kann man nicht meckern.

Und da das ziemlich mühsam zu tippen ist und dabei auch leicht Fehler 
entstehen, gibt es in C eine alternative Schreibweise.

   (*a).b

ist exakt völlig identisch zu

   a->b

Du kannst also anstelle von

      (*head).zg=n;

auch

      head->zg = n;

schreiben. Das ist exakt identisch dazu.
Letzteres ist die übliche Schreibweise. Der '->' (Pfeil) symbolisiert 
dabei das Dereferenzieren eines Pointers in einen Strukturmember.

[Programmierer (Gast)]

Die Schreibweise kennen wir nicht und das Programm ist ja richtig, einer 
aus meiner Klasse hat es auch getestet und es ging bei ihm von daher...

Und wie gesagt nachdem ich das Programm direkt auf den Stick kopiert 
habe ging es in der Schule ja auch.

Übrigens kommt das abstürzen nur wenn ich "e" drücken will bei b wird es 
beendet und bei l steht da dann "waehle zu loeschenden Objektnr aus:" 
natürlich crasht es dann wenn ich was eingebe.

[Karl H. (kbuchegg)]

Programmierer schrieb:
> Die Schreibweise kennen wir nicht

ok, dann will ich deinem Lehrer auch nicht vorgreifen.

> und das Programm ist ja richtig,

Nein, ist es nicht.
Das ist schwer fehlerhaft.

> einer
> aus meiner Klasse hat es auch getestet und es ging bei ihm von daher...

Durch testen kann man nur die Anwesenheit von Fehlern feststellen, nie 
die Abwesenheit. Wenn ein Programm fehlerhaft ist, heisst das noch lange 
nicht, dass es sofort und immer abstürzt. Je nachdem, in welchem Zustand 
der Speicher ist, denn das Betriebssystem auf die Anforderung hin 
rausrückt, können da 0-en drinnen stehen. Können - müssen aber nicht.

Es gibt keine Garantie dafür, welche Werte im Speicher stehen, der von 
malloc allokiert wird. Entweder du bringst dieses neue Objekt selbst in 
einen defnierten Zustand oder du benutzt calloc. Das nullt den Speicher 
aus, was dich zumindest in diesem Fall retten würde.