Controllo su Integer.parseInt()

[odranoel]

Ciao a tutti,
usando JOptionPane mi sono imbattuto in un problema. JOptionPane è un ottimo I/O ma in input purtroppo accetta solo String. Dato che mi sono incaponito e voglio fare in modo che accetti solo int, tramite Integer.parseInt() riesco a trasformare la stringa in un intero.

Ma adesso, come faccio a controllare se Integer.parseInt() mi restituisce un intero oppure no?

Ho tentato con la funzione isNaN() ma non mi funziona. Vi riporto il codice:

.....
String fraj = JOptionPane.showInputDialog("Inserisci Un Numero");
do {
	JOptionPane.showMessageDialog(null, "Non Hai Inserito Un Numero");
	fraj = JOptionPane.showInputDialog("Inserisci Un Numero");
	} while (isNaN(Integer.parseInt(fraj)));
j = Integer.parseInt(fraj);
.....

Ho tentato anche con un blocco Try/Catch ma non mi riesce a fare un ciclo.
Spero di essere stato chiaro e che mi possiate dare una mano.

Ciao

[andbin]

Originariamente inviato da odranoel
Ma adesso, come faccio a controllare se Integer.parseInt() mi restituisce un intero oppure no?

Ho tentato anche con un blocco Try/Catch ma non mi riesce a fare un ciclo.

Il modo corretto e appropriato è catturare NumberFormatException.

Per il fatto del ciclo, si potrebbe fare così. Innanzitutto fai un ciclo while(true) { ..... } e nel corpo come prima cosa fai la richiesta con il showInputDialog. Quindi fai un try-catch e nel blocco try fai il Integer.parseInt del String ricevuto. Se è ok non lancia nulla e puoi fare un break per uscire dal ciclo. Se passa nel catch puoi scegliere di mostrare un messaggio di errore o non fare nulla. Tanto il ciclo poi si ripete.

Meglio ancora se tutto questo fosse incapsulato in un metodo. Puoi fare direttamente un return del valore restituito da parseInt.

[odranoel]

Grazie bella idea!
adesso provo tutte e due le soluzioni poi ti faccio sapere

[odranoel]

Dunque ho fatto la modifica che mi hai suggerito, o almeno l'ho capita così:

private static int inserisciNum() {
	int j = 0;
	while (true) {
		String fraj = JOptionPane.showInputDialog("Inserisci Un Numero");
		try {
			j = Integer.parseInt(fraj);
		} catch (NumberFormatException e) {
			JOptionPane.showMessageDialog(null, "Non Hai Inserito Un Numero");
		}
		System.out.println(j);
		return j;
	}
}

Ancora non funziona, non so dove sbaglio. Nel senso se inserisco un numero ok bene, se inserisco un char mi ritorna zero e non cicla.

Riesci a darmi un suggerimento?
Grazie

[odranoel]

Ho risolto!

Posto il codice nel caso servisse a qualcuno.

private static int inserisciNum() {
	int j = 0;
	while (true) {
		String fraj = JOptionPane.showInputDialog("Inserisci Un Numero");
		try {
			j = Integer.parseInt(fraj);
			break;
		} catch (NumberFormatException e) {
			JOptionPane.showMessageDialog(null, "Non Hai Inserito Un Numero");
		}
	}
	System.out.println(j);
	return j;
}

Ciao grazie andbin.

[andbin]

Originariamente inviato da odranoel

private static int inserisciNum() {
	int j = 0;
	while (true) {
		String fraj = JOptionPane.showInputDialog("Inserisci Un Numero");
		try {
			j = Integer.parseInt(fraj);
		} catch (NumberFormatException e) {
			JOptionPane.showMessageDialog(null, "Non Hai Inserito Un Numero");
		}
		System.out.println(j);
		return j;
	}
}

Quando prima dicevo (se lo incapsuli in un metodo) "Puoi fare direttamente un return del valore restituito da parseInt." intendevo proprio dire che puoi fare nel blocco del try:

return Integer.parseInt(fraj);

E quindi senza dover usare alcuna variabile int. Se va bene ritorna subito, se non va bene, comunque cicla di nuovo.

[odranoel]

Mitico!

private static int inserisciNum() {
	while (true) {
		String fraj = JOptionPane.showInputDialog("Inserisci Un Numero");
		try {
			return Integer.parseInt(fraj);
			} catch (NumberFormatException e) {
			JOptionPane.showMessageDialog(null, "Non Hai Inserito Un Numero");
		}
	}
}

Grazie mille, ancora più pulito

[andbin]

Originariamente inviato da odranoel

private static int inserisciNum() {
....

Corretto.

Ti segnalo solo una piccola cosa in più. La dialog di input ha anche il pulsante di "Annulla". Se volessi gestirlo, si può fare ma devi cambiare le cose (valutando cosa fare nel caso di annullamento).

[odranoel]

Bene ci provo subito

[odranoel]

Per gestire il tasto annulla ho pensato a variare così:

private static int inserisciNum() {
	while (true) {
		String fraj = JOptionPane.showInputDialog("Inserisci Un Numero");
		if (fraj != null) {
			try {
				return Integer.parseInt(fraj);
			} catch (NumberFormatException e) {
				JOptionPane.showMessageDialog(null, "Non Hai Inserito Un Numero");
			}
		} else {
			JOptionPane.showMessageDialog(null, "Azione annullata");
			return -1;
		}
	}
}

Poi dato che il numero inserito mi serve per una somma nel main ho inserito anche

sum = Somma.calculateSum(sum);
if(sum != -1){
	System.out.println(sum);	
}else{
	System.out.println("Operazione annullata");
return;
}

Il codice gira correttamente.Ti sembra formalmente corretta? Ho hai in mente qualcosa di meglio?