[C]Musikplayer If-Abfrage Probleme

[KnechtRuprecht]

Hey,

ich wollte ein Programm schreiben, das wie folgt läuft:
Kurze Beschreibung:
Das Programm soll aufgebaut sein, wie der Musikplayer im Handy, PC etc.
Man bekommt eine Anfrage, wie man es abspielen will : Zufall oder Normal.
Wenn man Zufall wählt, bekommt man ein Lied ausgegeben, bei Normal werden beide Lieder der Reihe nach ausgegeben.

Problem:
Nach scanf() wird die Eingabe überprüft, wie das funktioniert etc. lernt man ganz am Anfang, trotzdem will es bei mir nicht so funktionieren, wie ich es will.
Die If-Abfrage soll überprüfen, ob es sich bei der Eingabe um 1 oder 2 handelt, wenn dies nicht der Fall ist, dann soll sich das Programm beenden.
Ich habe die if-Abfrage schon mit <= und >= geschrieben, aber ich will auch ausschließen, dass man Wörter eingibt und damit "weiterkommt".

Gegooglet, nachgefragt etc. wurde schon.

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
void main()
{    
     
     int d;
     int x;
     int z=0;
     time_t t;
     printf("1. Zufallswiedergabe\n");
     printf("2. Normale Wiedergabe\n");
     printf("Was wollen Sie : ");
     scanf("%i",&d);
     if( (d=1) || (d=2)  ) 
     goto ALLESOK
     else 
     printf("Falsche Eingabe\n");
     return 0;
     ALLESOK :
     printf("Sie haben %i genommen\n",d);
     switch(d) {
      case 1: goto Zufall;
              break;
      case 2: goto Normal;
              break;
               }
Zufall : 
srand(3*time(&t));
x=rand()%2;
x++;
switch(x) {
      case 1: printf("Linkin Park\n");
              break;
      case 2: printf("Billy Talent\n");
              break;
      }
system("Pause");
return 0;

Normal :
printf("1. Linkin Park\n");
printf("2. Billy Talent\n");
system("Pause");
}

KnechtRuprecht

[QpL]

Diese goto sprünge sind da wohl das Problem - verpack das ganze doch in eigene Methoden

Greetings

[krypt0n]

Ihhhhhh GOTOs gehören KEINESFALLS in modernen Code. Ausserdem:
(d=1) wird IMMER zu true ausgewertet, du musst schon (d==1) nehmen. Sonst hab ich mir den Code nicht weiter angeschaut, gewöhn dir bitte an von Anfang an einzurücken, leserlich Leerzeichen zu setzen und solchen Stuff.

[KnechtRuprecht]

Diese goto sprünge sind da wohl das Problem - verpack das ganze doch in eigene Methoden -
Außerdem würde ich einfach in die If-Abfrage für die Variable "d" folgendes machen:

Ja, das hatte ich auch schonmal und habe es gerade wieder versucht,
jetzt bricht es bei jeder Eingabe ab, egal ob es 2 oder a ist.

[blackberry]

if((d!==1)||(d!==2))

Wohl eher:
if (d != 1 && d != 2)

Ansonsten: egal was man eingibt -- beispielsweise 2, dann wäre es immernoch != 1 und damit die Bedingung im if wahr.

btw: das mit dem !== ist noch nicht mal C.
btw #2: ihhhhh void main... /me führt den Code hinter den Schuppen und erschießt ihn.

[KnechtRuprecht]

Zu btw #2, ich wollte kurz den Code schreiben und fertig ist, es sollte kein großes Programm werden.

Zu if (d != 1 && d != 2)Klappt nicht. Ich versteh nicht, wieso.

[blackberry]

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

int main()
{    
     
     int d;
     int x;
     int z=0;
     time_t t;
     printf("1. Zufallswiedergabe\n");
     printf("2. Normale Wiedergabe\n");
     printf("Was wollen Sie : ");
     scanf("%i",&d);
     if( (d==1) || (d==2)  ) 
     goto ALLESOK;
     else 
     printf("Falsche Eingabe\n");
     return 0;
     ALLESOK :
     printf("Sie haben %i genommen\n",d);
     switch(d) {
      case 1: goto Zufall;
              break;
      case 2: goto Normal;
              break;
               }
Zufall : 
srand(3*time(&t));
x=rand()%2;
x++;
switch(x) {
      case 1: printf("Linkin Park\n");
              break;
      case 2: printf("Billy Talent\n");
              break;
      }
//system("Pause");
return 0;

Normal :
printf("1. Linkin Park\n");
printf("2. Billy Talent\n");
system("Pause");
}

Funktioniert bei mir... Außerdem wäre eine entsprechende Beschreibung was denn bitte nicht klappt hilfreich.
Klar sollte sein, dass auf dieses if die falsche Eingabe Meldung folgen sollte und nicht das GOTO (ungeachtet dessen, das krypt0n bereits sagte, dass GOTO prinzipiell nirgendwo stehen sollte...)

Und um auf das void main zurück zu kommen: int main wäre sogar um ein Zeichen kürzer und ein return am Ende muss man auch nicht schreiben, wenn man nicht will (obwohl es ohne Zweifel ein besserer Stil wäre...).

btw #3: das else zu dem in fetter Schrift markierten if ist unnötig.

[KnechtRuprecht]

@BlackBerry

Bei mir sieht es, nachdem ich deinen Code eingefügt habe so aus.
Wenn ich 2 eingebe, klappt alles.
Wenn ich 1 eingebe, beendet sich das Programm.
Wenn ich s eingebe, kommt das selbe wie bei 2 raus.

[blackberry]

Und das mit dem Schließen, wenn du 1 eingibst führt daher, dass ich das system("Pause") auskommentiert habe, weil ich momentan nicht auf Windows arbeite und das nur zu einer dämlichen Meldung "couldn't find blah" führt.

Das mit dem s macht keinen Sinn. Bei mir funktioniert es -- siehe Screenshot.

[KnechtRuprecht]

Ja, das mit 1 geht, ich habe es völlig überlesen, da mein Firefox besser gesagt die Free-Hack Seite im Moment nicht so läuft.

So sieht es bei mir aus.