Probleme bei Programm! Verkettete Listen

[Fragant (Gast)]

hallo,

ich habe ein Programm für ein Wartezimmer, die Funktion um neue Personen 
aufzunehmen findet ihr im Bild.
Ich lege einen neuen Pointer (neu) an und reserviere Speicher für die 
nächste Struktur.
Dann gebe ich die Infos ein und will diese mit neu->... abspeichern.

Allerdings gibt es hier Probleme!

Bei neu-> age funktioniert das noch, aber bei den anderen beiden kommt 
die Fehlermeldung, dass der Ausdruck ein änderbarer ivalue sein muss.

Hoffe ihr könnt mir sagen, was ich ändern muss

[Peter II (Gast)]

Fragant schrieb:
> ich habe ein Programm für ein Wartezimmer, die Funktion um neue Personen
> aufzunehmen findet ihr im Bild.

das ist schon das Problem. Compiler und auch wir können keine Bilder 
parsen und den Inhalt verstehen.

[Tom (Gast)]

https://www.mikrocontroller.net/articles/String-Verarbeitung_in_C
>Arrays sind in C keine vollwertigen Datentypen, z. B. ist es nicht
>möglich, einem Array in einem Rutsch ein anderes Array zuzuweisen oder 2
>Arrays miteinander zu vergleichen. Genau das möchte man aber in der
>Stringverarbeitung häufig.

[Peter II (Gast)]

Fragant schrieb:
> Hoffe ihr könnt mir sagen, was ich ändern muss

Strings kann man in C nicht zuweisen, dafür gibt es Funktionen wie 
strcmp.

[Klaus (Gast)]

Ja, heilig's Blechle!

Wie doch immer wieder einer auf die Idee kommt, Programm/texte/ als JPG 
zu posten?

Ach, ich habs, das sind die ohne C-Buch. Klar. Stirnklatzsch.

Mach Deine Sch... hausaufgaben alleine.

[Walter S. (avatar)]

Peter II schrieb:
> Strings kann man in C nicht zuweisen, dafür gibt es Funktionen wie
> strcmp.

du meinst strcpy?

[Sebastian V. (sebi_s)]

Peter II schrieb:
> Compiler und auch wir können keine Bilder
> parsen und den Inhalt verstehen.

Also ich kann Text in Bildern schon "parsen". Wobei die Verwendung von 
JPG hier total ungeangebracht ist.

Peter II schrieb:
> Strings kann man in C nicht zuweisen, dafür gibt es Funktionen wie
> strcmp.

Ich denke du meinst strcpy. Mit strcmp kann man nichts kopieren oder 
zuweisen.

[Fragant (Gast)]

-------------------------------------------------------

struct patient{
  char name[20];
  char illness[30];
  int age;
  struct patient *next;
};

struct patient *start = NULL;
struct patient *ende = NULL;


void einlesen(){

  int alter_neu;
  char name_neu, krankheit_neu;
  struct patient *neu;

  neu = (struct patient*) malloc(sizeof(struct patient));

  printf("Name des Patienten: ");
  scanf("%c", &name_neu);
  printf("Krankheit des Patienten: ");
  scanf("%c", &krankheit_neu);
  printf("Alter des Patienten: ");
  scanf("%d", &alter_neu);

  neu->age = alter_neu;
  neu->name = name_neu;
  neu->illness = krankheit_neu;
  

}

---------------------------------------------------------------

Ok ich weiß jetzt, dass man bei strings nicht einfach -> benutzen kann, 
kann mir aber immer noch nicht vorstellen, wie man es dann machen soll.

[Klaus (Gast)]

> ... kann mir aber immer noch nicht vorstellen, wie man es dann machen soll.

Das gibt sich. Einfach immer weiter blättern. Das schwarze Gekräusel 
sind die Buchstaben.

[Klaus W. (mfgkw)]

Fragant schrieb:
> kann mir aber immer noch nicht vorstellen, wie man es dann machen soll.

Vielleicht ein gutes Buch lesen?

[Karl H. (kbuchegg)]

?

void einlesen(){

....
  char name_neu, krankheit_neu;
....

  printf("Name des Patienten: ");
  scanf("%c", &name_neu);

....

  printf("Krankheit des Patienten: ");
  scanf("%c", &krankheit_neu);
....

1 char für den Namen und 1 char für den Namen der Krankheit? Wie heissen 
deine Patienten? 'X' und 'Y'? Und die leiden an 'A' oder 'B'?

Weisst du was wirklich seltsam ist. Da versucht sich wer an dynamischen 
Datenstrukturen, konkret verketteten Listen. Das ist ein Thema, das man 
ca. nach frühestens einem Dreiviertel Jahr C Programmierung angeht und 
keineswegs trivial ist. Und dann scheitert er an Dingen, die er 
eigentlich nach 2 Monaten C-Programmierung alle schon beherrschen 
sollte.

[Karl H. (kbuchegg)]

Fragant schrieb:

> Ok ich weiß jetzt, dass man bei strings nicht einfach -> benutzen kann,

Doch, kann man.
Aber = kann man nicht benutzen!

Das eine hat mit dem anderen aber schon sowas von gar nichts zu tun.

[Fragant (Gast)]

Also so wie ich es verstanden habe, wird strcpy benutzt um Stellen von 
einem auf den anderen String zu übertragen.

Aber was ich gesucht habe, war wie ich einen string eingeben kann. Und 
wie schaffe ich es dann, diesen string in die struktur zu übertragen, 
auf die der zeiger neu zeigt.

[Karl H. (kbuchegg)]

Fragant schrieb:

> Aber was ich gesucht habe, war wie ich einen string eingeben kann. Und
> wie schaffe ich es dann, diesen string in die struktur zu übertragen,
> auf die der zeiger neu zeigt.

Erst mal musst du einen haben!

Bis jetzt hast du noch keinen String! Bis jetzt hast du nur eine 
Variable, die 1 (in Worten: ein) Character aufnehmen kann. 1 Character 
ist aber kein String!

  char name[20];

  scanf( "%s", name );

jetzt hast du einen String!

Und denn kann man kopieren. Zb in ein anderes char Array

   char tmp[20];

   strcpy( tmp, name );

oder in ein char-Array, welches in einem Struktur-Objekt steckt

struct person
{
  char name[20];
};

int main()
{
  struct person pers;

  ...

  strcpy( pers.name, name );
...
}

oder in ein char-Array, welches in einem Struktur-Objekt steckt, auf das 
man lediglich einen Pointer hat

  struct person* pers;

  pers = malloc( .....
  ...

  strcpy( pers->name, name );
...
}

Ehrlich. C-Buch. Durcharbeiten. Dringend!
String-Handling ist in C der erste Stolperstein von ca. 986 weiteren, 
die noch auf dich warten. Und es ist bei weitem der einfachste zu 
verstehen. Gegenüber allen anderen Stolpersteinen ist String-Handling 
nur ein müder Abklatsch.

[Matthias (Gast)]

Fragant schrieb:

> Aber was ich gesucht habe, war wie ich einen string eingeben kann.

Die Frage ist, wie sollen die Strings eingegeben werden. Bei Start des 
Programmes in der Shell oder zur Laufzeit über die Shell oder über eine 
Textbox in einer GUI?

Grundsätzlich würde ich Teilen meiner Vorredner zustimmen, dass Du Dir 
bezüglich des Themas 'Strings' einmal ein gutes C-Buch aneignen, 
ausleihen und dann ansehen solltest. Das ist alles machbar und nur halb 
so wild, vorausgesetzt Du hast die Lust und den Willen einige, wenige 
Seiten zu lesen.

http://openbook.rheinwerk-verlag.de/c_von_a_bis_z/

Das ist gut, umsonst und sollte alle Deine Problemstellungen in Kürze 
lösen können.

[Klaus (Gast)]

> Grundsätzlich würde ich Teilen meiner Vorredner zustimmen, ...

Welchen Teilen? Nicht meinem Arsch!

[Matthias (Gast)]

Klaus schrieb:

> Welchen Teilen? Nicht meinem Arsch!

Genau diesem Teil nicht. Man muss sich nicht wie ein Rotz am Ärmel 
benehmen.

[Klaus (Gast)]

Matthias schrieb:
> Klaus schrieb:
>
>> Welchen Teilen? Nicht meinem Arsch!
>
> Genau diesem Teil nicht. Man muss sich nicht wie ein Rotz am Ärmel
> benehmen.

Muss man nicht. Aber es macht Spaß und solange jigdo noch vor sich 
hinwerkelt ist diese Maschine ohnehin für alles andere unbrauchbar.

Aber ich nehme zur Kenntnis, dass Dir meine Art nicht gefällt.

[Fragant (Gast)]

Danke für eure Hilfe, hatten strings überhaupt nicht als thema und werde 
es mir deshalb demnächst mal in Ruhe anschauen.

Brauche aber nochmals eure Hilfe.

------------------------------------------------------------

struct patient{
  char name[20];
  char illness[30];
  int age;
  struct patient *next;
};

struct patient *start = NULL;
struct patient *ende = NULL;


void einlesen(){

  int alter_neu;
  char krankheit_neu[30], name_neu[20];
  
  struct patient *neu;

  neu = (struct patient*) malloc(sizeof(struct patient));

  printf("Name des Patienten: ");
  scanf("%s", &name_neu);

  printf("Krankheit des Patienten: ");
  scanf("%s", &krankheit_neu);

  printf("Alter des Patienten: ");
  scanf("%d", &alter_neu);

  neu->age = alter_neu;
  strcpy(neu->name, name_neu);
  strcpy(neu-> illness, krankheit_neu);


void ausgabe(){

  struct patient *ptr_anzeige = start;
  while (ptr_anzeige != NULL)
  {
    printf("%s\n", ptr_anzeige->name);
    printf("%s\n", ptr_anzeige->illness);
    printf("%d\n", ptr_anzeige->age);
    ptr_anzeige = ptr_anzeige->next;
  }
  printf("\n");

}

------------------------------------------------------------

Bei der Ausgabe kommt folgende
Fehlermeldung:
Zugriffsverletzung, Fehler beim Lesen der Zeichen der Zeichenfolge.

Könnt ihr einen Fehler erkennen?

[foo (Gast)]

Fragant schrieb:
> Könnt ihr einen Fehler erkennen?

Unvollständiger Copypaste. Von der Funktion "einlesen" fehlt die 
schließende Klammer.
Icb bin mir ziemlich sicher, dass das nicht der einzige Unterschied 
zwischen dem, was du testest, und dem, was du uns zeigst, ist.

> Bei der Ausgabe kommt folgende
> Fehlermeldung:
> Zugriffsverletzung, Fehler beim Lesen der Zeichen der Zeichenfolge.

Ansonsten: beim Einlesen initialisierst du "next" nicht. Das muss 
demnach nicht zwingend NULL sein. malloc garantiert dir keinen mit 
Nullen initialisierten Speicher.

[Matthias (Gast)]

> strcpy(neu-> illness, krankheit_neu);

Soll das so sein?

Ich würde Dir den Einsatz eines Debuggers, wahlweise GDB empfehlen.

Starte gdb mit dem Executable, setze mit 'break main' einen Breakpunkt, 
starte das Ganze mit 'run' und gehe Zeile für Zeile durch den Code.

Solltest Du hier Hilfe benötigen, findest Du eine kurze Referenz über 
'man gdb' in der Shell.

[Dirk B. (dirkb2)]

Beim einlesen von einem String, kommt kein & beim scanf hin.
Der Name vom Array ergibt schon die Adresse vom ersten Element.

name_neu ist gleich &name_neu[0]

Beide sind vom Typ char*.

Bedenke abei, dass scanf bei %s Whitespace als Trennzeichen nimmt und 
somit das Einlesen abbricht. Auch wäre eine Längenbegrenzung besser:

  printf("Name des Patienten: ");
  scanf("%19s", name_neu);

  printf("Krankheit des Patienten: ");
  scanf("%29s", krankheit_neu);

[Pandur S. (jetztnicht)]

Solch triviales Zeug sollte man Singlesteppen koennen. Und falls nicht, 
andere IDE, oder andere Sprache.