problema do while... primi passi

[degliantoni]

Salve, ho appena iniziato a studiare java, qualcuno può dirmi dove c'è un errore??
Se inserisco un dato errato, come una lettera, il ciclo non termina ma continua all'infinito.

//area cerchio con eccezioni e ciclo

import java.util.Scanner;

public class AreaCerchioEccezzioniCicloWhileDo {
public static void main (String[]args){

// area del cerchio = (raggio x raggio x 3.14)

Scanner in = new Scanner (System.in);
boolean errore = false;

do {
errore = false;

try {
System.out.print("Scrivi il raggio: ");
int r = in.nextInt();
double area = (double) (r*r*3.14);
System.out.println("L'area del cerchio misura: "+ area);

}

catch (Exception e) {
System.out.println("Valore errato, devi scrivere un numero");

errore = true; }
} while(errore == true);

}
}

se si scrive un numero tutto ok
se si scrive una lettera stampa: Valore errato.... Scrivi il raggio (all'infinito)
Thanks

Andrea

[andbin]

Originariamente inviato da degliantoni
Se inserisco un dato errato, come una lettera, il ciclo non termina ma continua all'infinito.

Il "problema" è di Scanner ma è ben noto: se usi i next di Scanner che forniscono qualcosa diverso da String (int, altri primitivi ....), se l'input è malformato/errato Scanner lancia InputMismatchException ma il token errato rimane nel suo buffer interno.
Se rifai nextInt .. idem, sei sempre sullo stesso token, stesso errore.
Basta che "butti" via il token errato con un next() .

P.S. ti conviene fattorizzare l'input di un int (con eventuale messaggio di prompt parametrabile) in un metodo apposito, invece di annegarlo nel resto della tua logica.

[francesco.muia]

Allora l'errore sta nel fatto che andando in errore lo scanner sul nextInt ovviamente gli rimane in "memoria" l'ultimo dato inserito.
Per ovviare a questo ti basta mettere nel catch "in.next()" in modo da far skippare il dato non valido.

[degliantoni]

Aggiunto in.next (). Il ciclo cosi finisce.... ma per farlo ripetere??
Grazie cmq x le risposte sopra elencate

[Alex'87]

Originariamente inviato da degliantoni
Aggiunto in.next (). Il ciclo cosi finisce.... ma per farlo ripetere??
Grazie cmq x le risposte sopra elencate

Posta (usando il tag code e un'indentazione leggibile) il nuovo codice. Dove lo hai aggiunto il next?

[degliantoni]

L' ho aggiunto all i terno del catch

[andbin]

Avevo fatto un esempio di metodo qui

[degliantoni]

Ok. Grazie mille per le tempestive risposte