linuxgaming.de

Hier ist mein Quellcode, ich bekomm das mit dem "if" vergleich nicht hin,
hat irgendwas mit variablentyp stezen zu tun soviel hab ich schon raus,
bitte um Hilfe

#include <stdio.h>
#include <string.h>
#define MAX 40

int main (void)
{

char eingegebenes [MAX+1];
printf ("################################\n");
printf ("# #\n");
printf ("# C #\n");
printf ("# #\n");
printf ("# Michael #\n");
printf ("# #\n");
printf ("################################\n");
printf ("\n\nBitte etwas eingeben\n");
gets (&eingegebenes);

if (eingegebenes == "hallo")
{
printf ("unmöglich\n");
}
else
{
printf ("tachchen psycho\n");
}

return 0;
}

Da strings in C eigentlich arrays von chars sind, kann man nicht einfach string1 == string2 machen, sondern muss eine entsprechende Funktion verwenden (strcmp um genau zu sein). Außerdem sollte man gets auch nicht verwenden (und wenn man es schon tut, dann richtig), sondern fgets(wohin_speichern, wieviele_zeichen, stdin).

Code: Alles auswählen

#include <stdio.h>
#include <string.h>
#define MAX 40

int main (void) {

  char eingegebenes[MAX+1];
 
  printf ("################################\n");
  printf ("#                             #\n");
  printf ("# C                           #\n");
  printf ("#                             #\n");
  printf ("# Michael                     #\n");
  printf ("#                             #\n");
  printf ("################################\n");
 
  printf ("\n\nBitte etwas eingeben\n");
  fgets(eingegebenes, MAX+1, stdin);

  if (strcmp(eingegebenes, "hallo"))
   printf ("unmöglich\n");
  else
   printf ("tachchen psycho\n");

  return 0;
}

[quote=Zappo,index.php?page=Thread&postID=31983#post31983]Da strings in C eigentlich arrays von chars sind, kann man nicht einfach string1 == string2 machen, sondern muss eine entsprechende Funktion verwenden (strcmp um genau zu sein). Außerdem sollte man gets auch nicht verwenden (und wenn man es schon tut, dann richtig), sondern fgets(wohin_speichern, wieviele_zeichen, stdin).

Code: Alles auswählen

#include <stdio.h>
#include <string.h>
#define MAX 40

int main (void) {

  char eingegebenes[MAX+1];
 
  printf ("################################\n");
  printf ("#                             #\n");
  printf ("# C                           #\n");
  printf ("#                             #\n");
  printf ("# Michael                     #\n");
  printf ("#                             #\n");
  printf ("################################\n");
 
  printf ("\n\nBitte etwas eingeben\n");
  fgets(eingegebenes, MAX+1, stdin);

  if (strcmp(eingegebenes, "hallo"))
   printf ("unmöglich\n");
  else
   printf ("tachchen psycho\n");

  return 0;
}

Danke, Danke, Danke, Danke, Danke
funktioniert, hab den ganzen letzten abend danach gesucht, und nichts gefunden.
Nochmal Danke!
[/quote]

[quote=michael,index.php?page=Thread&postID=31985#post31985]
[quote=Zappo,index.php?page=Thread&postID=31983#post31983]Da strings in C eigentlich arrays von chars sind, kann man nicht einfach string1 == string2 machen, sondern muss eine entsprechende Funktion verwenden (strcmp um genau zu sein). Außerdem sollte man gets auch nicht verwenden (und wenn man es schon tut, dann richtig), sondern fgets(wohin_speichern, wieviele_zeichen, stdin).

Code: Alles auswählen

#include <stdio.h>
#include <string.h>
#define MAX 40

int main (void) {

  char eingegebenes[MAX+1];
 
  printf ("################################\n");
  printf ("#                             #\n");
  printf ("# C                           #\n");
  printf ("#                             #\n");
  printf ("# Michael                     #\n");
  printf ("#                             #\n");
  printf ("################################\n");
 
  printf ("\n\nBitte etwas eingeben\n");
  fgets(eingegebenes, MAX+1, stdin);

  if (strcmp(eingegebenes, "hallo"))
   printf ("unmöglich\n");
  else
   printf ("tachchen psycho\n");

  return 0;
}

Danke, Danke, Danke, Danke, Danke
funktioniert, hab den ganzen letzten abend danach gesucht, und nichts gefunden.
Nochmal Danke!
[/quote][/quote]Ooops, zu foreilig
jetzt funktioniert das zweite nicht,
also das nach dem "else"

also noch offen, wie funktionierts?

@michael: Nutze doch bitte die code-Tags und zitiere nicht komplette Beiträge, wenn es nicht wirklich nötig ist, sonst wird das hier schnell unübersichtlich.

C ist nicht mein Gebiet aber wie rufst du denn "main (void)" auf muss da net irgendwo die subroutine aufgerufen werden???

@fiesling: main ist die Einstiegsroutine von jedem C-Programm. Die wird automatisch aufgerufen. Daher ist das schon korrekt so.

@michael: Nur ein kleiner Fehler (der dir hätte auch auffallen können). strcmp liefert bei Gleichheit der beiden Strings 0 zurück, was natürlich in diesem Fall als False ausgewertet wird. Entweder vertauschst du den if- und else-block oder du machst etwas in der Art

Code: Alles auswählen

if (strcmp(eingegebenes, "hallo\n") == 0) {
  printf("foo\n");
} else {
  printf("bar\n");
}

Man beachte, dass ich beim Vergleich "hallo\n" (also mit dem linebreak) schreibe, da fgets diesen beim Lesen von stdin nicht entfernt (im Gegensatz zum Fall, wenn es aus einer Datei liest).

[quote=Zappo,index.php?page=Thread&postID=31989#post31989]@fiesling: main ist die Einstiegsroutine von jedem C-Programm. Die wird automatisch aufgerufen. Daher ist das schon korrekt so.

@michael: Nur ein kleiner Fehler (der dir hätte auch auffallen können). strcmp liefert bei Gleichheit der beiden Strings 0 zurück, was natürlich in diesem Fall als False ausgewertet wird. Entweder vertauschst du den if- und else-block oder du machst etwas in der Art

Code: Alles auswählen

if (strcmp(eingegebenes, "hallo\n") == 0) {
  printf("foo\n");
} else {
  printf("bar\n");
}

Man beachte, dass ich beim Vergleich "hallo\n" (also mit dem linebreak) schreibe, da fgets diesen beim Lesen von stdin nicht entfernt (im Gegensatz zum Fall, wenn es aus einer Datei liest).[/quote]Ah ok jetzt läufts,
danke!
bin erst einsteiger daher fällt mir sowas eher nicht auf XD
trotzdem Danke!!!

Ich würde die Eingabe mit scanf machen. Das geht dann auch viel einfacher
Beispiel:

Code: Alles auswählen


scanf ("Eingabe %s", &eingabe)   


Zur Erklärung %s steht für String und &eingabe für die eingabevariable.(In Wirklichkeit ein Zeiger )

scanf ist eine schlechte Wahl, weil dabei nicht überprüft wird, ob die Eingabe auch in den string eingabe passt. Außerdem ist das Beispiel auch falsch, da bei der Eingabe von strings nicht die Adresse das Ziel ist (da strings schon pointer sind). Das macht man z.B. bei der Eingabe von irgendwelchen Zahlenwerten.