cicciozza ha scritto:
public class Squadra {
String Nome;
String Numero;
public Squadra(String nom, String numer){
this.Nome=nom;
this.Numero=numer;
}
In casi di classi "bean" come queste, tieni sempre le variabili di istanza come
private, a meno che ci siano altre necessità che devono essere ben IMPORTANTI e precise.
Inoltre cerca di seguire almeno un minimo delle convenzioni standard. Nomi di variabili che iniziano con la
minuscola. E per i parametri si usa tipicamente lo stesso nome dei campi, usando
this per qualificare il campo.
cicciozza ha scritto:
Main:
for(int z=0;z<primoNumero;z++){
while(rigaLetta!=null){
No, questo ha abbastanza poco senso
Questo invece ha più senso.
BufferedReader br = new BufferedReader(fr);
String primaRiga = br.readLine();
int numeroRecord = Integer.parseInt(primaRiga);
for (int i = 0; i < numeroRecord; i++) {
String lineaRecord = br.readLine();
if (lineaRecord != null) {
// elabora linea
} else {
// qui è un errore in quanto mancano record
}
}
Inoltre tutta la lettura del file sarebbe stato meglio almeno farla in un metodo separato in SorteggioSquadre.