assegnare valori ad array (java)

[dik1992]

ho fatto questo semplice programma :

import javax.swing.JOptionPane;
public class borsamain {

	public static void main(String[] args) {
	int borsasize=0;
	int i =0;
	int prov = 0;
	int stop= 2;
	String [] borsa = new String [5];
	
	
	while( stop > prov){
		String m = JOptionPane.showInputDialog("inserire il tipo di moneta da mettere nella borssa");
		stop = Integer.parseInt(JOptionPane.showInputDialog("inserire 0 se è l'ultima moneta da mettere nella borsa altrimenti qualsiasi altro numero positivo"));
		borsa[i] = m;
		i++;
		borsasize++;
	}
	for(int j=0; j<=borsasize; j++){
	System.out.println( borsa[i]);
	j++;
	}
	}

}

assegno "m" all'array (o almeno vorrei fare questo), quando eseguo il programma come risultati però mi da "null" non capisco se l'errore sia nell'assegnazione del valore o nel println e in cosa consiste

[Roby.zip]

Ciao,
for(int j=0; j<=borsasize; j++){
System.out.println( borsa[i]);
j++;
}

borsa[j] non borsa[i]

[c0der]

Ci sono diversi problemi, quello in particolare che non ti stampa è dovuto probabilmente a:
System.out.println(borsa[i]); => vedi che hai scritto "i" mentre il ciclo for è basato su "j"

A parte questo il ciclo while è fatto male e non controlli che non venga superato il limite di 5, cioè quanto è grande l'array.

In pratica non ti conviene usare un "while" ma un for che va da 0 a 5 e nel caso uno immetta 0 esci prima.

[LeleFT]

Aggiungo anche questa

for(int j=0; j<=borsasize; j++){

Se hai inserito "borsasize" elementi nell'array, l'indice andrà da 0 a (borsasize-1). Quindi dovresti modificare la condizione di uscita dal ciclo for, altrimenti incorrerai in una ArrayIndexOutOfBoundsException.

for(int j=0; j<borsasize; j++){

Ciao.

[dik1992]

Originariamente inviato da c0der
Ci sono diversi problemi, quello in particolare che non ti stampa è dovuto probabilmente a:
System.out.println(borsa[i]); => vedi che hai scritto "i" mentre il ciclo for è basato su "j"

A parte questo il ciclo while è fatto male e non controlli che non venga superato il limite di 5, cioè quanto è grande l'array.

In pratica non ti conviene usare un "while" ma un for che va da 0 a 5 e nel caso uno immetta 0 esci prima.

ciao forse non ho capito bene quello che vuoi dire ma usando il for uno dovrebbe contare prima quanti dati deve inserire poi usare quel dato per il for giusto??? ed io qesto vorrei evitarlo vorrei che facesso tutto in automatico

[Nikopol]

Ciao,
non hai bisogno di contare niente perchè sai già che non potrai inserire più di 5 elementi dato che hai creato il vettore borsa lungo 5. Il tuo programma però non ha nessuna condizione che impedisca all'utente di inserire più di 5 elementi.

Se il tuo programma dovesse prendere in input al massimo 5 monete e terminare nel caso l'utente inserisca lo 0, dovresti scrivere il tuo ciclo così:

while (stop != prov && i < borsa.length){
     .........
}

Un piccolo consiglio: prendi l'abitudine di non dichiarare mai le variabili all'interno di un ciclo come hai fatto tu con la m; in questo modo continui a creare e distruggere la variabile ad ogni iterazione. Creala sempre all'esterno. Inoltre non ne hai bisogno, potresti semplicemente scrivere così:

       int i = 0;
       while (stop != prov && i < borsa.length){
       borsa[i] = JOptionPane.showInputDialog(".....");
       i++;
       .........
       }

A proposito di borsasize:
-la convenzione java dice di usare caratteri maiuscoli per separare un nome formato da piu parole; in questo caso borsaSize. So che sembra una cavolata ma aiuta a rendere più leggibile il codice.
-non ti serve a molto in questo caso. Avresti potuto semplicemente mettere nel for che usi per stampare, una condizione che escluda gli elementi del vettore che non hai riempito (nulli).

      for (int j = 0; j<borsa.length; j++){
            if (borsa[j] != null){
                 System.out.println(borsa[j]);
            }
      }

Nel caso non sappia a priori quante monete l'utente andrà ad inserire, dovresti usare un ArrayList invece che un vettore.
Te lo scrivo usando gli ArrayList ma se ancora non li conosci dai un occhiata alla documentazione ufficiale o in alternativa usa il buon vecchio google.

      int prov = 0;
      int stop = 2;
      ArrayList<String> borsa = new ArrayList<>();
      while (stop != prov){
             borsa.add(JOption.showInputDialog("..."));
             stop = Integer.parseInt(JOption.showInputDialog("..."));
      }
      for(int j = 0; j < borsa.size(); j++){
           System.out.println(borsa.get(j));
      }

Comunque anche io non sono un esperto quindi prendi tutto quello che ho detto come un consiglio

[Roby.zip]

ArrayList<String> borsa = new ArrayList<>();

Se definisci l'ArrayList coi generics devi istanziarla coi generics

 ArrayList<String> borsa = new ArrayList<String>();

[Nikopol]

Se definisci l'ArrayList coi generics devi istanziarla coi generics

è proprio quello che ho fatto. Ho solo abbreviato la sintassi usando la diamond form introdotta in java 1.7

[neroux]

Originariamente inviato da Nikopol
è proprio quello che ho fatto. Ho solo abbreviato la sintassi usando la diamond form introdotta in java 1.7

Giusto, da 1.7 quella sintassi è correttissima