Array di stringhe con dimensione sconosciuta a priori.

[Apicio]

Salve a tutti.
Avrai da fare un programma che gestisce un magazzino di Prodotti organizzati per Codice a barre, descrizione e prezzo.
Questo programma deve mettere a disposizione alcune funzionalità quali la carica di un magazzino da un file testuale organizzato come un CSV.

Questa è la parte di codice del carica file:

	public Magazzino leggiChar(String s){
		FileReader file;
		BufferedReader buff;
		Magazzino mag = new Magazzino();
		String temp ="";
		//ArrayList<String> vect = new ArrayList<String>();
                
                 try{
			file = new FileReader(s);
			buff = new BufferedReader(file);
			String[] vect;	
			while(temp!=null){
				
				Prodotto p = new Prodotto();
				Float f; //Questi due wrapper mi servono per forza perché la split() restituisce un vettore di stringhe, quindi mi servono per fare il "cast".
				Integer i;
				temp = buff.readLine();
				if(temp!=null){
					
					vect=temp.split(";");
					p.setCodBarre(vect[0]);
					p.setDescrizione(vect[1]);
					f = Float.valueOf(vect[2]);
					p.setPrezzo(f);
					i = Integer.valueOf(vect[3]);
					mag.inserimento(p, i);
				}
			}
				
			if(buff != null)
				buff.close();
		}catch(IOException e){}
		
		return mag;
	}

E questo è il file:

000;Latte;10.0;2; 
003;Caffè;2.2;10; 
001;Cioccolato;5;20;

Quindi è organizzato del tipo CodBarre;descrizione;prezzo;scorte;

Funziona tutto, tranne una cosa: la carica magazzino da un file già esistente sul computer. Praticamente mi rilancia l'eccezione ArrayIndexOutOfBoundException:1 .

Ecco cosa mi esce dalla console di Eclipse:

1. Carica un magazzino da file di Char
2. Carica un magazzino da un file di byte
3. Crea un nuovo magazzino
0. Esci dal programma
1
Inserisci il percorso del file da caricare
E:/prove/magazzino.txt
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 1
	at SerializzatoreMagazzino.leggiChar(SerializzatoreMagazzino.java:85)
	at UIMagazzino.main(UIMagazzino.java:29)

Ho pensato che il problema possa essere quel String[] vect; che mi alloca un vettore di una sola cella. Ho provato quindi a fare un ciclo while per contare quanti record ho, ma sarebbe terribilmente inefficiente. Qualcuno ha qualche altra soluzione?

Grazie a tutti.

[Alex'87]

vect è dichiarato ma mai inizializzato.

Ti manca un pezzo: vect = new String[dimensione];
Se non sai a priori la dimensione meglio se usi un ArrayList: in automatico si espande se necessario.

[Apicio]

La dimensione del mio magazzino non la conosco a priori. Quindi non saprei cosa metterci.

EDIT:Ho provato ad usare ArrayList, ma il metodo split() di String mi restituisce un array di String, e quindi il compilatore mi da errore quando faccio vect=temp.split() dicendo che non può convertire da String ad ArrayList<String>

[andbin]

Originariamente inviato da Apicio

Float f; //Questi due wrapper mi servono per forza perché la split() restituisce un vettore di stringhe, quindi mi servono per fare il "cast".
Integer i;

Il fatto che split fornisce un vettore di stringhe NON centra nulla con il fatto di dover usare un wrapper piuttosto che il primitivo.
E tra l'altro da Java 5, wrapper e primitivi sono "interscambiabili" (al costo di un boxing/unboxing).
Ma se setPrezzo riceve un float (primitivo) allora creare un Float è del tutto inutile.

Originariamente inviato da Apicio
Praticamente mi rilancia l'eccezione ArrayIndexOutOfBoundException:1 .

Ho pensato che il problema possa essere quel String[] vect; che mi alloca un vettore di una sola cella.

Scommetto che nel file hai o una riga "vuota" oppure una riga malformata (non in quel formato che ti aspetti). E quindi split fornisce meno "pezzi".

[Apicio]

Grazie! Avevi ragione sulla riga vuota!! Proverò a modificare il codice per fargli capire che se la riga è vuota deve fermarsi.

Peer quanto riguarda i wrapper: se io vado a fare p.setPrezzo(vect[2]); il compilatore mi da errore perché dice che non può convertire da String a float (setPrezzo riceve un float), per questo ho dovuto ricorrere ai wrapper.

Da un esempio visto sul tuo sito ho capito che potrei ovviare a questo inconventiente con un arraylist, o sbaglio?

[andbin]

Originariamente inviato da Apicio
Peer quanto riguarda i wrapper: se io vado a fare p.setPrezzo(vect[2]); il compilatore mi da errore perché dice che non può convertire da String a float

Questo è ovvio, da solo non fa queste conversioni.

Originariamente inviato da Apicio
(setPrezzo riceve un float), per questo ho dovuto ricorrere ai wrapper.

No.

Float.parseFloat(String)

restituisce il float primitivo. Se hai usato il valueOf che fornisce un Float (wrapper) è corretto (da Java 5) ma inutile perché comunque nella invocazione a setPrezzo viene unboxato e passato il float primitivo.

[Apicio]

Ho capito dove ho sbagliato.
Praticamente avrei potuto scrivere direttamente p.setPrezzo(Float.valueOf(vect[2])); ?

[andbin]

Originariamente inviato da Apicio
Ho capito dove ho sbagliato.
Praticamente avrei potuto scrivere direttamente p.setPrezzo(Float.valueOf(vect[2])); ?

Tecnicamente certo. Ma è perché da Java 5 si ha il boxing/unboxing automatico.
valueOf ritorna un java.lang.Float, che il compilatore "unboxa" a float. Ma sarebbe superfluo, un passaggio inutile.

Le classi wrapper (perlomeno per i tipi numerici) hanno due tipi di metodi statici per la conversione da un String: i parseXyz e i valueOf. Il primo ritorna il primitivo, il secondo il wrapper. In java.lang.Float ci sono:

public static float parseFloat(String s)
public static Float valueOf(String s)

Vedi la differenza?

Quindi:
p.setPrezzo(Float.parseFloat(vect[2]));

parseFloat ritorna un float primitivo che viene passato pari-pari al setPrezzo che riceve il float, senza passaggi intermedi/nascosti.