questo metodo non parte!

[N'zù]

ho provato a fare un programma chi simuli in maniera molto superficiale la calcolatrice......ma nella classe main, quando chiedo se vuole procedere con un'altra operazione, il metodo che dovrebbe essere eseguito in base alla decisione presa non lo avvia e anzi termina il programma......mi potete spiegare dov'è che sbaglio??? il codice del main è questo....

import java.util.*;

public class CalcolatriceTest
{
public static void main (String args[])
{
Scanner input = new Scanner(System.in);
Calcolatrice oggetto = new Calcolatrice();

System.out.println("Benvenuto! Questo programma simula una calcolatrice...");
System.out.println("Lista Operazioni: \n1. Addizione \n2. Sottrazione \n3. Moltiplicazione \n4. Divisione \nDigiti l'operazione");

int operazione = input.nextInt();
oggetto.setScelta(operazione);

System.out.println();
System.out.println("Inserire il primo numero: ");
int num1 = input.nextInt();
oggetto.setPrimoNum(num1);

System.out.println("Inserire il secondo numero: ");
int num2 = input.nextInt();
oggetto.setSecondoNum(num2);
oggetto.SceltaOperazione();

System.out.println("\nVuole continuare con una prossima operazione?: Y o N");
String decisione = input.nextLine();
oggetto.setDecisione(decisione);
oggetto.Decisione();
}
}

funziona tutto tranne il metodo Decisione.....questo metodo non parte....

[alka]

Originariamente inviato da N'zù
funziona tutto tranne il metodo Decisione.....questo metodo non parte....

Cosa vuol dire non parte? Da errore? Da risultati diversi da quelli attesi? E se sì, quali?

[N'zù]

è come se il metodo Decisione non esistesse........Dopo che stampa a video se vuole procedere con un'altra operazione, il programma termina! Invece dovrebbe continuare perché il metodo Decisione permette di fare o meno un'altra operazione......spero di essere stato più chiaro..

[N'zù]

Benvenuto! Questo programma simula una calcolatrice...
Lista Operazioni:
1. Addizione
2. Sottrazione
3. Moltiplicazione
4. Divisione
Digiti l'operazione
3

Inserire il primo numero:
1
Inserire il secondo numero:
6
Il risultato e': 6
Vuole continuare con una prossima operazione?: Y o N

Process completed.

Ecco cosa mi spunta! Il processo non dovrebbe essere completato.....non riesco a digitare la mia decisione se voglio o meno continuare con un'altra operazione..

[LeleFT]

E noi come possiamo sapere cosa dovrebbe fare il metodo "Decisione" della classe Calcolatrice di cui non conosciamo assolutamente nulla? Se non ne posti il codice, non troveremo mai l'errore che sicuramente stà lì. Quello che ti posso assicurare con assoluta certezza è che quel metodo viene invocato eccome (a meno che il metodo setDecisione() non sollevi un'eccezione, di cui nessuno sa nulla perchè, evidentemente, non è stata trattata nel modo corretto).


PS: quando posti del codice, usa i tag CODE, altrimenti non si capisce nulla.


Ciao.

[N'zù]

public void SceltaOperazione()
	{
		if (scelta == 1)
		{
			Addizione();
		}
		if (scelta == 2)
		{
			Sottrazione();
		}
		if (scelta == 3)
		{
			Moltiplicazione();
		}
		if (scelta == 4)
		{
			Divisione();
		}
	}
	
	public void Decisione()
	{
		
		if (decisione.equals("y"))
		{
			SceltaOperazione();
		}
		if (decisione.equals("n"))
		{
			System.out.println("Arrivederci per una prossima operazione!");
		}
	}
}

come puoi notare i metodi delle operazioni funzionano........questo è il metodo Decisione della classe Calcolatrice.

[LeleFT]

Da quello che posso vedere, quel metodo fa qualcosa se e solo se la variabile "decisione" (che non si vede come viene inizializzata) è "y" o "n" esattamente in minuscolo, senza spazi superflui, ecc... In qualunque altro caso, non fa proprio nulla.

Io proverei a farmi stampare il valore della variabile "decisione", giusto per capire cosa c'è dentro...

E non hai usato i tag CODE, come richiesto.... li ho aggiunti io.

[N'zù]

ahahahah scusami, dammi tempo di imparare....mi sono registrato ieri!

[N'zù]

il problema è che non faccio in tempo di digitare un "y" o un "n".....perché il programma termina subito dopo! è questo che non capisco.....perché termina senza considerare il metodo Decisione() ???

[LeleFT]

Il problema è la riga evidenziata:

System.out.println("Inserire il secondo numero: ");
int num2 = input.nextInt();

nextInt() legge un intero... ma tu, per dare in pasto l'input al programma, scrivi '6' e poi premi INVIO. Ebbene... nextInt() prende il 6 e lo converte in intero, ma nel buffer di input c'è ancora l'INVIO... che viene letto dall'istruzione successiva, ovvero:

System.out.println("\nVuole continuare con una prossima operazione?: Y o N");
String decisione = input.nextLine();

Per questo non fai a tempo a dargli l'input... La gestione dell'input va fatta in modo oculato. Ci sono diversi modi per farlo. Quello che serve a te in questo momento è una "pulitura" del buffer: ovvero, dopo che hai letto l'intero con nextInt() devi effettuare una lettura "a vuoto" per pulire il buffer dal carattere di INVIO che ancora è lì.

System.out.println("Inserire il secondo numero: ");
int num2 = input.nextInt();
input.nextLine();

Io preferirei, al contrario, gestire l'input in modo diverso: leggere sempre e solo delle stringhe (quindi usare sempre nextLine()) ed effettuare la validazione manuale (quando necessito di un intero, faccio il parsing della stringa come intero, quando mi serve un double faccio il parsing della stringa come double, ecc). In questo modo non corro il rischio di lasciare nel buffer sporcizia che poi mi genera questo tipo di fastidi.


Ciao.