[C]Musikplayer If-Abfrage Probleme

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.

Code:

---

#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

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

Greetings

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.

Zitat:

---

Zitat von QpL

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.

Zitat:

---

Zitat von QpL

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.

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

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

Code:

---

#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.

@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.

http://img3.imagebanana.com/img/cooz...t8.46.43PM.png

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.

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.
http://img3.imagebanana.com/img/9nxz...beEingabes.png
So sieht es bei mir aus.

Kannst du mal dein Projekt nochmal neu compilieren und es erneut versuchen.
Wenn es dann immernoch nicht klappt, dann poste bitte deinen Code nochmal genauso, wie er momentan bei dir da steht.

Das ist er.
Habe es nochmal versucht, jedoch wieder ohne Erfolg.

Code:

---

#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");
    system("pause");
    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");    //system("Pause");
return 0;

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

---

Also bei mir funktioniert alles ohne Probleme.
Ich kann nur nochmal wiederholen, dass du auf Rebuild (oder wie auch immer das in deiner IDE heißt) gehen solltest um sicherzustellen, dass auch der Code compiliert wird, den du momentan dastehen hast -- an diesem liegt es nämlich nicht.

Sonst würde ich empfehlen es noch mit einem anderen Compiler zu probieren.
Wie man schon an meinem Screenshot erkennen kann klappt es bei mir mit GCC (4.2.1).