Aiuto su toUpperCase

[fermat]

ciao a tutti.
ho un problemino con toUpperCase.
vi riporto una parte di codice:

package Calcio;

public class Calciatore {
    private String nome;
    private String cognome;
    private String ruolo;
    private int eta;
    private boolean infortunato;
    int numero;

    public Calciatore(String n, String c,String r, int e , boolean i){
        nome=n;
        cognome=c;
        eta=e;
        ruolo=r;
        infortunato=i;
    }
        public String getNome(){
            return nome;
    }
        public void setNome(String nm){
            nome=nm;
            String nm1=nome.substring(0,1).toUpperCase();
            String nm2=nm.substring(1);
            String nm3=nm1.concat(nm2);
            
    }
ecc...

tutto ciò è poi richiamato in un'altra classe dal metodo main.
quello che vorrei ottenere è la prima lettera (in questo caso del nome) maiuscola.

ps: sappiate che sto solo alla quarta lezione quindi nn sono ancora molto pratico.

[andbin]

Originariamente inviato da fermat

        public void setNome(String nm){
            nome=nm;
            String nm1=nome.substring(0,1).toUpperCase();
            String nm2=nm.substring(1);
            String nm3=nm1.concat(nm2);

quello che vorrei ottenere è la prima lettera (in questo caso del nome) maiuscola.

E lo fa di certo perché la procedura che hai fatto è corretta. Chiaramente però il risultato del concat non volevi assegnarlo ad una variabile locale nm3 ma al campo di istanza nome. Fai la assegnazione, appunto.

E tieni presente i casi particolari. Se a setNome viene passata una stringa vuota "", la tua procedura così come è lancia una eccezione (indice "fuori" range).

[fermat]

intanto grazie per la risposta!

allora, ho modificato così:

        public void setNome(String nm){
            nome=nm;
            String nm1=nome.substring(0,1).toUpperCase();
            String nm2=nm.substring(1);
            String nome=nm1.concat(nm2);
            
    }

ma continua a darmi tutto in minuscolo.
la parola nome dell'ultima riga è sottolineata di giallo (uso netbeans) e mi dice:
Local variable hides a field.

però nn mi da nessun errore.

[Alex'87]

Originariamente inviato da fermat
intanto grazie per la risposta!

allora, ho modificato così:

        public void setNome(String nm){
            nome=nm;
            String nm1=nome.substring(0,1).toUpperCase();
            String nm2=nm.substring(1);
            String nome=nm1.concat(nm2);
            
    }

ma continua a darmi tutto in minuscolo.
la parola nome dell'ultima riga è sottolineata di giallo (uso netbeans) e mi dice:
Local variable hides a field.

però nn mi da nessun errore.

Stai ancora memorizzando il risultato in una variabile locale. Guarda bene cosa hai scritto

[fermat]

chiedo venia ma nn riesco a venirne a capo

[andbin]

Originariamente inviato da fermat
chiedo venia ma nn riesco a venirne a capo

public void setNome(String nome) {
    String nm1 = nome.substring(0,1).toUpperCase();
    String nm2 = nome.substring(1);
    
    this.nome = nm1.concat(nm2);
}

E generalmente almeno per convenzione, si usa mettere i nomi dei parametri uguali ai nomi dei campi di istanza (giusto anche solo per non inventarsi altri nomi come 'nm') e poi si usa this. per referenziare il campo (perché il parametro "nasconde" il campo).

[fermat]

ho apportato le modifiche che mi hai detto ma ancora nn funziona.

public void setNome(String nome){
            String nm1=nome.substring(0,1).toUpperCase();
            String nm2=nome.substring(1);
            this.nome=nm1.concat(nm2);
            
    }

ps: cmq this nn lo conoscevo perchè ancora nn lo abbiamo fatto al corso.

[andbin]

Originariamente inviato da fermat
ma ancora nn funziona.

Funziona di certo, non vedo nulla di sbagliato.
Se lo invochi tu, è un conto. Se invochi quel costruttore che hai scritto in Calciatore, no, perché non passa da lì.

[fermat]

tutto il codice (anche quello che nn ho postato perchè nn serviva) viene richiamato da provacalcio.java che è scritto così:

package Calcio;

public class provacalcio {

    public static void main(String[] args){

        Calcio.Calciatore calciatore1=new Calcio.Calciatore("roberto", "baggio","Centrocampista", 35,false);
        Calcio.Calciatore calciatore2=new Calcio.Calciatore("paolo", "maldini","Difensore", 35,true);

        calciatore1.setInfortunato(true);
        calciatore2.setInfortunato(false);
        
        System.out.println(calciatore1.getDescrizione());
        System.out.println(calciatore2.getDescrizione());

    }
}

il codice funziona, tranne appunto la lettera maiuscola nel nome.
continua a stampare i nomi con la prima lettera minuscola.

[andbin]

Originariamente inviato da fermat

        Calcio.Calciatore calciatore1=new Calcio.Calciatore("roberto", "baggio","Centrocampista", 35,false);
        Calcio.Calciatore calciatore2=new Calcio.Calciatore("paolo", "maldini","Difensore", 35,true);

Stai usando i costruttori e per come hai scritto il costruttore (vedi primo post della discussione), non hai affatto invocato setNome ma hai assegnato direttamente con nome=n;