[JAVA] Come correggo unreported exception

[neho34]

Sto studiando JAVA e ho provato a fare un piccolo esercizio. Ho un triangolo e fornisco in input le misure dei lati. Il programma deve darmi il perimetro del triangolo. Ho fatto così:

import java.io.*;

class LeggiEScrivi{
static double leggi(double val1){
System.out.println("Inserisci il valore richiesto: ");
InputStreamReader flusso = new InputStreamReader(System.in);
BufferedReader lex = new BufferedReader(flusso);
String input = lex.readLine();
val1 = Double.parseDouble(input);
return val1;
}
static double perimetro(double val1,double val2,double val3){
return val1+val2+val3;

}
public static void main(String[] args)throws IOException{
double a = 0, b = 0, c = 0;
leggi(a);
leggi(b);
leggi(c);

System.out.println(perimetro(a,b,c));
}
}
Ma mi dà questo errore:
LeggiEScrivi.java:8: unreported exception java.io.IOException; must be caught or declared to be thrown
String input = lex.readLine();
Che significa?

[unomichisiada]

Originariamente inviato da neho34
Sto studiando JAVA e ho provato a fare un piccolo esercizio. Ho un triangolo e fornisco in input le misure dei lati. Il programma deve darmi il perimetro del triangolo. Ho fatto così:

import java.io.*;

class LeggiEScrivi{
static double leggi(double val1){
System.out.println("Inserisci il valore richiesto: ");
InputStreamReader flusso = new InputStreamReader(System.in);
BufferedReader lex = new BufferedReader(flusso);
String input = lex.readLine();
val1 = Double.parseDouble(input);
return val1;
}
static double perimetro(double val1,double val2,double val3){
return val1+val2+val3;

}
public static void main(String[] args)throws IOException{
double a = 0, b = 0, c = 0;
leggi(a);
leggi(b);
leggi(c);

System.out.println(perimetro(a,b,c));
}
}
Ma mi dà questo errore:
LeggiEScrivi.java:8: unreported exception java.io.IOException; must be caught or declared to be thrown
String input = lex.readLine();
Che significa?

L'errore significa che non hai catturato l'eccezione IoException lanciata in fase si lettura dello stream,è sufficiente racchiudere la parte di codice che lancia l'eccezione (controlla la documentazione ma a occhi e croce è questa:

 InputStreamReader flusso = new InputStreamReader(System.in);
BufferedReader lex = new BufferedReader(flusso);)

in un blocco try{ ... }catch(IoEception e){} dove al posto dei puntini va la parte dio codice suddetta

[neho34]

Perfetto l'ho un pò cambiato, ma funziona. Dammi una tua opinione, tipo se si potevano evitare ripetizioni etc:
import java.io.*;

class LeggiEScrivi{
static double leggi(double val1, double val2, double val3){
try{
System.out.println("Inserisci il valore richiesto del lato1: ");
InputStreamReader flusso = new InputStreamReader(System.in);
BufferedReader lex = new BufferedReader(flusso);
String input = lex.readLine();
val1 = Double.parseDouble(input);
System.out.println("Inserisci il valore richiesto del lato2: ");
InputStreamReader flusso1 = new InputStreamReader(System.in);
BufferedReader lex1 = new BufferedReader(flusso1);
input = lex1.readLine();
val2 = Double.parseDouble(input);
System.out.println("Inserisci il valore richiesto del lato3: ");
InputStreamReader flusso2 = new InputStreamReader(System.in);
BufferedReader lex2 = new BufferedReader(flusso2);
input = lex2.readLine();
val3 = Double.parseDouble(input);

}
catch(IOException e){
System.out.println(e);
}
double risultato = val1+val2+val3;
return risultato;
}
public static void main(String[] args)throws IOException{
double a = 0, b = 0, c = 0;

System.out.println(leggi(a,b,c));
}
}

[floyd]

hai creato flusso, usalo su ogni richiesta

[anx721]

E' inutile pasare i valori a, b, c al metodo leggi in quanto resteranno immodificati all'uscita dal metodo: i parametri in java son opassati per valore, ovvero per copia. Io avrei fatto un metodo leggi() che restituisce il double letto da console, e poi avrei chiamato questo metodo tre volte per leggere i lati del triangolo.

[neho34]

In effetti all'inizio avevo fatto così, ma il risultato era 0.0.
import java.io.*;

class LeggiEScrivi{
static double leggi(double val1){
try{
System.out.println("Inserisci il valore richiesto del lato1: ");
InputStreamReader flusso = new InputStreamReader(System.in);
BufferedReader lex = new BufferedReader(flusso);
String input = lex.readLine();
val1 = Double.parseDouble(input);

}
catch(IOException e){
System.out.println(e);
}
return val1;
}
public static void main(String[] args)throws IOException{
double a = 0, b = 0, c = 0;
leggi(a);
leggi(b);
leggi(c);
double risultato = a+b+c;
System.out.println(risultato);
}
}
Sono sicuro che il mio sbaglio sta nel passaggio di valori. Ma non riesco a correggerlo.
Grazie comunque per l'aiuto.

[unomichisiada]

Ok ti hanno già fatto pressapoco i commenti ch eti avrei fatto anche io,solo una cosa METTI I TAG CODE QUANDO POSTI DEL CODICE!!! Non l'hai fatto nemmeno una benedetta volta.Ciao

[neho34]

Scusami, non l'ho fatto a posta. É stata solo una distrazione.

[anx721]

devi assegnare il valore di ritorno di leggi() alle variabili a, b e c che non è necessario passare come argomento:

class LeggiEScrivi{
static double leggi(){
try{
System.out.println("Inserisci il valore richiesto del lato1: ");
InputStreamReader flusso = new InputStreamReader(System.in);
BufferedReader lex = new BufferedReader(flusso);
String input = lex.readLine();
int val1 = Double.parseDouble(input);

}
catch(IOException e){
System.out.println(e);
}
return val1;
}
public static void main(String[] args)throws IOException{
double a = 0, b = 0, c = 0;
a = leggi();
b = leggi();
c = leggi();
double risultato = a+b+c;
System.out.println(risultato);
}
}

Inoltre dovresti catturare anche l'eccezione NumberFormatException che puo essere lanciata da parseDouble se la stringa non rappresenta un numero.