Modificare valore memorizzato in un metodo get

[n1o]

mmm buona idea.

Nel metodo ci piazzo tutti gli if di conversione?

 
if(var.equalsIgnoreCase("E")){
        label ="Ernesto";
        } else if(var.equalsIgnoreCase("K")) {
        label ="Kat";
        } else if(var.equalsIgnoreCase("T")) {
        label="Tonto";
        }

o sbaglio?

perciò il metodo dovrà essere una cosa del genere:

 
String getDescrizioneStatus() {
 char var;
 String label;

if(var.equalsIgnoreCase("E")){
        label ="Ernesto";
        } else if(var.equalsIgnoreCase("K")) {
        label ="Kat";
        } else if(var.equalsIgnoreCase("T")) {
        label="Tonto";
        }
}

Una cosa del genere o sbaglio?
e poi nella jsp che faccio?

Grazie mille And sei un grande, mi stai aiutando tantissimo

[andbin]

Originariamente inviato da n1o
Nel metodo ci piazzo tutti gli if di conversione?

Sì certo, però ovviamente devi fare un return della descrizione. Se getStatus() usa e restituisce il campo 'status', allora in getDescrizioneStatus() usi ancora 'status' ma per testare i vari casi e restituire una descrizione.

Originariamente inviato da n1o
e poi nella jsp che faccio?

<%= codMazz.getDescrizioneStatus() %>

[n1o]

allora il mio status è:

 
public char getStatus() {
        return status;
    }

    public void setStatus(char status) {
        this.status = status;
    }

perciò getDescrizioneStatus dovrà essere:

public String getDescrizoneStatus(){
        char status; 
        String descrizioneStatus;
        if(status.equalsIgnoreCase("E")) {   //QUA MI DA ERRORE: CHAR CANNOT BE REFERENCED
             descrizioneStatus = "Electonical Image" ;
                        return descrizioneStatus;
         }
    }

dvoe sbaglio?
ma così non mi

[andbin]

Originariamente inviato da n1o

public String getDescrizoneStatus(){
        char status; 
        String descrizioneStatus;
        if(status.equalsIgnoreCase("E")) {   //QUA MI DA ERRORE: CHAR CANNOT BE REFERENCED
             descrizioneStatus = "Electonical Image" ;
                        return descrizioneStatus;
         }
    }

dvoe sbaglio?

Devi usare il campo 'status' .... non una variabile locale (oltretutto non è nemmeno inizializzata, quindi sarebbe comunque un errore di compilazione).
E puoi anche non usare alcuna variabile, nel senso che testi 'status' e fai direttamente un return "..." per ogni if.

[n1o]

scusami che significa che devo usare il campo status???
Non capisco
questo è il mio entity bean

 
package it.poste.tool.persistence.entities;

import java.io.Serializable;
import java.util.Date;
import javax.persistence.Basic;
import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.Id;
import javax.persistence.NamedQueries;
import javax.persistence.NamedQuery;
import javax.persistence.Table;
import javax.persistence.Temporal;
import javax.persistence.TemporalType;

@Entity
@Table(name = "V_RICERCA_AGGR_COD_MAZZETTO")
@NamedQueries({
@NamedQuery(name = "VRicercaAggrCodMazzetto.findAll", query = "SELECT v FROM VRicercaAggrCodMazzetto v"),
@NamedQuery(name = "VRicercaAggrCodMazzetto.findByCode", query = "SELECT v FROM VRicercaAggrCodMazzetto v WHERE v.code = :code"),
@NamedQuery(name = "VRicercaAggrCodMazzetto.findBySiglavideo", query = "SELECT v FROM VRicercaAggrCodMazzetto v WHERE v.siglavideo = :siglavideo"),
@NamedQuery(name = "VRicercaAggrCodMazzetto.findByFrazionario", query = "SELECT v FROM VRicercaAggrCodMazzetto v WHERE v.frazionario = :frazionario"),
@NamedQuery(name = "VRicercaAggrCodMazzetto.findByStatus", query = "SELECT v FROM VRicercaAggrCodMazzetto v WHERE v.status = :status"),
@NamedQuery(name = "VRicercaAggrCodMazzetto.findByCreationDate", query = "SELECT v FROM VRicercaAggrCodMazzetto v WHERE v.creationDate = :creationDate"),
@NamedQuery(name = "VRicercaAggrCodMazzetto.findByScanDate", query = "SELECT v FROM VRicercaAggrCodMazzetto v WHERE v.scanDate = :scanDate"),
@NamedQuery(name = "VRicercaAggrCodMazzetto.findByCheckDate", query = "SELECT v FROM VRicercaAggrCodMazzetto v WHERE v.checkDate = :checkDate"),
@NamedQuery(name = "VRicercaAggrCodMazzetto.findByName", query = "SELECT v FROM VRicercaAggrCodMazzetto v WHERE v.name = :name"),
@NamedQuery(name = "VRicercaAggrCodMazzetto.findById", query = "SELECT v FROM VRicercaAggrCodMazzetto v WHERE v.id = :id")})
    public class VRicercaAggrCodMazzetto implements Serializable {
    private static final long serialVersionUID=1L;
    @Basic(optional = false)
    @Id
    @Column(name = "CODE")
    private String code;
    @Basic(optional = false)
    @Column(name = "SIGLAVIDEO")
    private String siglavideo;
    @Column(name = "FRAZIONARIO")
    private String frazionario;
    @Basic(optional = false)
    @Column(name = "STATUS")
    private char status;
    @Basic(optional = false)
    @Column(name = "CREATION_DATE")
    @Temporal(TemporalType.TIMESTAMP)
    private Date creationDate;
    @Column(name = "SCAN_DATE")
    @Temporal(TemporalType.TIMESTAMP)
    private Date scanDate;
    @Column(name = "CHECK_DATE")
    @Temporal(TemporalType.TIMESTAMP)
    private Date checkDate;
    @Basic(optional = false)
    @Column(name = "NAME")
    private String name;
    @Basic(optional = false)
    @Column(name = "ID")
    private long id;

    public VRicercaAggrCodMazzetto() {
    }

    public String getCode() {
        return code;
    }

    public void setCode(String code) {
        this.code = code;
    }

    public String getSiglavideo() {
        return siglavideo;
    }

    public void setSiglavideo(String siglavideo) {
        this.siglavideo = siglavideo;
    }

    public String getFrazionario() {
        return frazionario;
    }

    public void setFrazionario(String frazionario) {
        this.frazionario = frazionario;
    }

    public char getStatus() {
        return status;
    }

    public void setStatus(char status) {
        this.status = status;
    }

    public Date getCreationDate() {
        return creationDate;
    }

    public void setCreationDate(Date creationDate) {
        this.creationDate = creationDate;
    }

    public Date getScanDate() {
        return scanDate;
    }

    public void setScanDate(Date scanDate) {
        this.scanDate = scanDate;
    }

    public Date getCheckDate() {
        return checkDate;
    }

    public void setCheckDate(Date checkDate) {
        this.checkDate = checkDate;
    }

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }

    public long getId() {
        return id;
    }

    public void setId(long id) {
        this.id = id;
    }
   public String getDescrizoneStatus(){ 
        String descrizioneStatus;
        if(status.equalsIgnoreCase("E")) {
             descrizioneStatus = "Electonical Image" ;
                        return descrizioneStatus;
         }
    }
}

[andbin]

Originariamente inviato da n1o
scusami che significa che devo usare il campo status???

....
public class VRicercaAggrCodMazzetto implements Serializable {

   ....
    @Column(name = "STATUS")
    private char status;

    public char getStatus() {
        return status;
    }

    public String getDescrizioneStatus(){ 
        if (status.equalsIgnoreCase("E")) {
           return ".....";
        } else if ......
    ....
}

[n1o]

allora era come pensavo io, ma continua a darmi sempre lo stesso errore:

 
    public String getDescrizoneStatus(){ 
        if(status.equalsIgnoreCase("E")) {
            return "Electonical Image";
         }
    }

Non accetta equalsIgnoreCase()

come mai?

[andbin]

Originariamente inviato da n1o
Non accetta equalsIgnoreCase()

come mai?

No, dovevo già segnalartelo prima ... char non è un oggetto!

Li devi confrontare con:

if (status == 'E') { // apici singoli! Non "E" che è un String

E se vuoi puoi anche usare uno switch:

switch (status) {
case 'E': return .....
}

E tieni presente che il metodo deve comunque restituire un String, quindi devi contemplare tutti i vari casi ma anche un "default" se non è nessuno di quelli che ti aspetti.

[n1o]

Originariamente inviato da andbin
No, dovevo già segnalartelo prima ... char non è un oggetto!

Li devi confrontare con:

if (status == 'E') { // apici singoli! Non "E" che è un String

E se vuoi puoi anche usare uno switch:

switch (status) {
case 'E': return .....
}

E tieni presente che il metodo deve comunque restituire un String, quindi devi contemplare tutti i vari casi ma anche un "default" se non è nessuno di quelli che ti aspetti.

Ci sono dei controlli lato client all'inserimento del campo di ricerca perciò arriverà sempre e comunque un risultato valido.
adesso mi da un Missing return Statement all'intestazione del metodo
adesso il metodo è così:

 
    public String getDescrizoneStatus(){   //Missing return Statement
        if(status == 'E') {
            return "Electonical Image";
         } else if(status == 'K') {
            return "in Controllo";
         } else if(status == 'T') {
            return "Controllato";
         } else if(status== 'O') {
            return "Aperto";
         } else if(status =='S') {
            return "Sospeso";
         } else if(status == 'C') {
            return "Chiuso";
         } else if(status == 'X') {
            return "Expired";
         } else if(status== 'L') {
            return "Aggregato";
         }
    }

[andbin]

Originariamente inviato da n1o
adesso mi da un Missing return Statement all'intestazione del metodo

Ma l'ho detto poco fa e lo ripeto: qualunque sia il "flusso" di esecuzione del metodo, deve restituire qualcosa.
Il compilatore non "sa" che tu sei sicuro che status avrà sempre uno di quei valori. Sa solo che se tecnicamente nessuno degli if è true, il metodo resta "sguarnito" del return!

Quindi o fai restituire un default (es. stringa vuota o altro) o lasci che una di quelle descrizioni, ad esempio l'ultima, sia il "default" (quindi niente test, solo } else { return ... )