ArrayList e popolamento

[Dj_Mark]

Ciao a tutti ragazzi...ho un problema con il popolamento di un ArrayList

protected String titolo,issn,isbn; 
protected final int dataPub; 
public List authors=new ArrayList<RepoAuthors>(); 
public List citazioni = new ArrayList(); 

public ScientificPublication(String unTitolo, Object RepoAuthors, int unAnno,String unIssn,String unIsbn, int unCit){ 
if (unTitolo!="")this.titolo=unTitolo; 
if (RepoAuthors!=null)this.authors.add(RepoAuthors); 
this.dataPub = unAnno; 
if (unIssn=="")this.issn=null;else this.issn=unIssn; 
if (unIsbn=="")this.isbn=null;else this.isbn=unIsbn; 
this.citazioni.add(new int[2]); }

Praticamente ho quell'array citazioni dove devo inserire le citazioni per anno quindi l'utente mi inserisce un anno che equivale a dataPub mentre citazioni è un intero....

this.citazioni.add(new int[2]); in questa riga teoricamente dentro l'arraylist citazioni dovrei creare un record (se così si può chiamare) chi alla prima posizione libera dell'arraylist mi inserisca dataPub nello spazio 0 e nello spazio 1 il numero di citazioni....mettendo quel new int[2] mi crea i 2 spazi ma poi come inserisco i dati in quei due?????


grazie mille a tutti!!!

[LeleFT]

Questa riga di codice non è sbagliata, ma genera un forte warning:

public List citazioni = new ArrayList();

Stai, infatti, dichiarando un oggetto istanziandolo come raw-type di ArrayList(), mentre, da Java 5 in avanti, gli ArrayList dovrebbero essere tipizzati (come hai fatto poco sopra).
Essendo che tale ArrayList dovrebbe contenere array di interi, dovresti istanziarlo in questo modo:

public List<int[]> citazioni = new ArrayList<int[]>();

Anche la riga precedente, antrebbe sistemata. L'interfaccia List, infatti, va tipizzata:

public List<RepoAuthors> authors=new ArrayList<RepoAuthors>();

Per il resto, mi sa che stai facendo troppi passaggi in un sol colpo: crea prima l'array di interi, poi popolalo e, alla fine, aggiungilo all'ArrayList:

int[] record = new int[2];
record[0] = dataPub;
record[1] = numero di citazioni;
citazioni.add( record );

Si può anche fare tutto in un passaggio, ma perdi enormemente in leggibilità.

PS: Non so quale sia il valore "numero di citazioni", tant'è che l'ho messo in corsivo nel codice, ma questo lo sai tu.

Ciao.

[Alex'87]

Inoltre le stringhe non si confrontano con == ma con equals.