strano errore c

ho creato un piccolo programma (piccolo e stupido :mrgreen:) in c da linux, ma c'è un errore:
se provate a sbagliare due volte la stessa tabellina, e poi a digitare 1, e quindi rincominciare, non partira più dalla prima tabellina, ma da quella successiva a quella in cui siete arrivati.. ma non riesco a capire il perchè!

ecco il codice:

Codice:

---

#include <stdio.h>
#include <stdlib.h>
main () {
int numero, numero2, risultato, ok, azione;
inizio:
printf("Scrivi il risultato della tabellina\n");
calcola:
numero = rand() %10;
numero2 = rand() %10;
ok = numero*numero2;
printf("%d x %d = ", numero,numero2);
scanf("%d", &risultato);
if (risultato == ok) {
goto calcola;
} else {
printf("Errore! Riprova\n");
printf("%d x %d = ", numero,numero2);
scanf("%d", &risultato);
if (risultato == ok) {
goto calcola;
} else {
printf("Hai perso! Digita:\n0) Uscita\n1) Riprova\n");
printf("Cosa scegli? ");
scanf("%d", &azione);
if (azione == 1) {
printf("\n\n");
goto inizio;
} else {
}
}
}
}

---

ciao!!
giaco

Sembra un programma assemby :P
Mi svugge il perchè salti, ma è concettualmente fatto male!
1) Evita di usare le etichette, usale solo per la gestione degli errori
2) Devi inizializzare il seme random o ad ogni esecuzione la funzione random ti darà sempre gli stessi numeri!

no, non da gli stessi numeri, cambiano ogni volta che il risultato è corretto...
perchè non dovrei usare le etichette? cosa uso come alternativa? gli switch-case?

ciao!!
giaco

wrestlingforum:
-in quel programma qualunque cosa tu faccia , ogni volta le variabili numero e numero2 vengono reinizializzate

-usare i goto in un programma di alto livello è controproducente , crea confusione ed è quindi bandito dal mondo della programmazione (tranne che in assembler ovviamente)

-binarysun intendeva che se non inizializzi il seme , ogni volta che avvi il programma verra generata sempre la stessa sequenza casuale (che non sarà quindi più casuale)

-quell'else vuoto in fondo è inquietante :mrgreen:

Citazione:

---

Originalmente inviato da xDestroyer

wrestlingforum:
-in quel programma qualunque cosa tu faccia , ogni volta le variabili numero e numero2 vengono reinizializzate

-usare i goto in un programma di alto livello è controproducente , crea confusione ed è quindi bandito dal mondo della programmazione (tranne che in assembler ovviamente)

-binarysun intendeva che se non inizializzi il seme , ogni volta che avvi il programma verra generata sempre la stessa sequenza casuale (che non sarà quindi più casuale)

-quell'else vuoto in fondo è inquietante :mrgreen:

---

azz...:
- in che senso reindirizzate? :?????:
- beh, potrei risolvere con gli switch.. no?
- e come faccio? :lol:
- gia, l'ho fatto apposta per spaventarvi
:mrgreen: :mrgreen: :twisted:! dai skerzo è perchè cosi su linux, tanto si usa da shell, se fai 0 si chiude automaticamente.. :razz:

ciao!!

Puoi farla con un ciclo until.

questa non l'ho capita.. :?????: a che cosa ti riferisci?

Già, until non esiste in C :P
Usa il ciclo while!
Esce dal ciclo quando ha fatto 2 tentativi O ha dato la risposta esatta O vuole uscire

Citazione:

---

Originalmente inviato da wrestlingforum

azz...:
- in che senso reindirizzate? :?????:
- beh, potrei risolvere con gli switch.. no?
- e come faccio? :lol:
- gia, l'ho fatto apposta per spaventarvi
:mrgreen: :mrgreen: :twisted:! dai skerzo è perchè cosi su linux, tanto si usa da shell, se fai 0 si chiude automaticamente.. :razz:

ciao!!

---

se fai 0 si chiude automaticamente? eh? intendi return 0? spiegati meglio

gli switch si, ma passa prima per gli if

no non reindirizzate, reinizializzate! nel caso premi 1 il programma non deve passare due nuovi valori alle due variabili

no lo zero era a riguardo di:
printf("Cosa scegli? ");
scanf("%d", &azione);
if (azione == 1) {
printf("\n\n");
goto inizio;
} else {
}
questo!
ora cercherò di trovare qualche esempio di while..
grazie a tutti!

continuo a non capire il perchè di quel blocco else{} vuoto

perchè chiude automaticamente il programma! se vien premuto 1 va all'inizio, altrimenti niente :lol:

ciao!

ma guarda che l'else vuoto è inutile

funzionerà uguale se lo levi

ahhhhhhhh ok!
thank's! :mrgreen:

che nostalgia :cry: mi sto ricordando il basic del commodore.

comunque ti consiglio di riscrivere il programma dall'inizio, bandendo i goto