Passaggio di una stringa da una classe

[wFree]

Non riesco a capire come mai non funziona:

  /**
   * Restituisce una stringa con l'elenco delle voci
   * della rubrica separate da ", ".
   * L'elenco inizia con "(" e termina con ")"
   */
  public String elenco() {
      int size=arrayPersona.length;
      String s="(";
      String []voce=new String[size];
              for(int i=0;i<size;i++){
                  s=s.concat(arrayPersona[i].getInfo())+" ";
              }
              s=s+")";
      return s;
  }

Il metodo chiamato è

    public String getInfo(){
        String info;
        info=nome+" "+cognome+" "+numero;
        return info;

Ho anche provato ad usare una stringa d'appoggio ma non è quello. Il problema è qua:

s=s.concat(arrayPersona[i].getInfo())+" ";

[minomic]

Cosa significa "non funziona"? Dà qualche errore o non fa quello che dovrebbe? E in quest'ultimo caso... cosa fa?

[andbin]

  public String elenco() {
      int size=arrayPersona.length;
      String s="(";
      String []voce=new String[size];
              for(int i=0;i<size;i++){
                  s=s.concat(arrayPersona[i].getInfo())+" ";
              }
              s=s+")";
      return s;
  }

Sintatticamente e come concetto a me pare corretto. (l'array voce non lo usi ... non so che ti serve).
È un po' un "macello" perché fai un mix di concat, +, assegnamenti che sarebbero evitabili (ad esempio direttamente con l'uso di StringBuffer/Builder).

Le uniche cose che materialmente possono far fallire qualcosa sono che arrayPersona sia null oppure arrayPersona[i] sia un null.


P.S. prima della parentesi di chiusura hai comunque sempre uno spazio. Anche questo sarebbe evitabile.