Odio la struttura switch!!!

[Ph0en1X]

Ciao, sn sempre io ke vi rompo le scatole con il C++ e questa volta lo faccio x kiedervi cosa c'è di male in questo esercizio. Praticamente devo calcolare il totale delle vendite di una azienda ke mi da i prezzi dei 5 prodotti ke vende, ecco il prog:

#include <iostream>

using namespace std;

int main()
{
double prod1 = 2.98, prod2 = 4.50, prod3 = 9.98, prod4 = 4.49, prod5 = 6.87,
totale = 0;
int num;

cout << "Inserisci il numero del prodotto compreso fra 1 e 5 "
<< "( -1 per terminare ): ";


while ( ( num = cin.get() ) != -1 ) {
switch ( num ) {

case '1':
totale += prod1;
break;

case '2':
totale += prod2;
break;

case '3':
totale += prod3;
break;

case '4':
totale += prod4;
break;

case '5':
totale += prod5;
break;

case '\n':
case '\t':
case ' ':
break;

default:
cout << "Hai inserito un carattere non valido."
<< " Inserisci un nuovo valore." << endl;
break;
}
}
cout << "\n\nIl totale delle vendite è: " << totale << endl;

return 0;
}

Potreste dirmi dove sta l'errore? Il problema è ke quando digito -1, ossia quello ke dovrebbe essere il valore sentinella, il prog nn termina. PLEASE HELP!!!

[Spillo]

forse le virgolette dei numeri dei case
prova

[TEC_MrHide]

Nel senso che rimane sempre nel while?

[Spillo]

si, perchè le virgolette si usano per tipi char (dovrebbe essere così, non mi ricordo molto bene)

[TEC_MrHide]

si, perchè le virgolette si usano per tipi char (dovrebbe essere così, non mi ricordo molto bene)

Sì ma non credo serva nel suo caso, in quanto il suo problema è che anche se il valore letto è -1, la condizione del while è sempre true.
Se ho capito bene...

[Spillo]

Sì ma non credo serva nel suo caso, in quanto il suo problema è che anche se il valore letto è -1, la condizione del while è sempre true.
Se ho capito bene...

è vero
domani ci penso adesso ho sonno e non ho + voglia di scrivere

[pentothal]

- togli le virgolette;
- se non funzia sostituisci il while con uno squallidissimo while(true) e usa un case anche per -1;

esempio:

while(true){
case 1:
<istruzioni>
break;
case 2:
<istruzioni>
break;
...
case -1:
break;
}

[TEC_MrHide]

Forse al posto di while(true) è meglio mettere un boolean dichiarato all'inizio true, poi dichiarato false nel case -1.
while(vai){
...
case -1:
vai=false;
break;
}

[hottar17]

a occhio basta mettere
while ( ( num = atoi(cin.get() )) != -1 )

dal momento che get lo tira dentro come char

nei case ti va di culo perchè anche '1' '2' e '3' etc sono char e non int (1 2 3...)

oppure il confronto del while lo fai con "-1", ma non è pulitissimo

[TEC_MrHide]

...
oppure il confronto del while lo fai con "-1", ma non è pulitissimo

Se usa i doppi apici viene interpretato come String.
Deve usare i singoli ('-1') anche se non so se esiste il char -1.
P.S.
Io conosco solo Java, magari mi sono sbagliato, nè.

[Yubei]

Ragazzi un consiglio, anzi un obbligo quendo si parla di queste cose.

Le sezioni di codice metteteli tra i tag (code) e (/code) , ovviamente usando le parentesi quadre al posto delle tonde.

[red.devil]

//ma nn puoi fare così??

int coso(){
	int totale=0;//prod1=...;prod2=...;...;...
	do{	
		scanf("%i",&asd);
		switch(asd){
			case 1: totale+=prod1; break;
			case 2: totale+=prod2; break;
			case 3: totale+=prod3; break;
			case 4: totale+=prod4; break;
			case 5: totale+=prod5; break;
			default:
					return totale;
		}
	}while(asd!=-1); //condizione inutile, perchè se in numero è <1 || >5
 killa la funzione ritornando il valore.
}

void main(){
	printf("totale: &i\n",coso());
}

[Ph0en1X]

Grazie x l'aiuto ragazzi, anke se credo di non averci capito niente, dopotutto odio questa struttura switch del ca**o.

[red.devil]

//così dovrebbe andare, ed è più pulito come codice ;)

void main(){
        int totale=0;//prod1=...;prod2=...;...;...
        do{     
                scanf("%i",&asd);
                switch(asd){
                        case 1: totale+=prod1; break;
                        case 2: totale+=prod2; break;
                        case 3: totale+=prod3; break;
                        case 4: totale+=prod4; break;
                        case 5: totale+=prod5; break;
			case -1:printf("totale: %i\n",totale);
                }
        }while(asd!=-1);  
}

cmq guarda che l'istruzione switch è utile e soprattutto molto semplice, se ci spieghi dove hai problemi ti aiutiamo noi

[Ph0en1X]

Il mio problema è ke nn riesco a uscire dal costrutto nemmeno digitando quello ke "dovrebbe" essere il valore sentinella, ossia -1! E poi un domanda a te Red.Devil: ke differenza c'è fra il dikiarare la variabile "main" come int o void? TNX