Composizione di classi

[bjunior]

ciao a tutti sto studiando la relazione di composizione di classi e ho dedotto questo: per evitare la rottura dell'incapsulamento passando riferimenti come parametri d'ingresso o di uscita dei metodi bisogna simulare i passaggi per valore. Ora mi sto esercitando ed ho ideato una classe Spesa che è una composizione di oggetti della classe Prodotto;il codice è questo

 public class Spesa {
    //attributi
    private ArrayList<Prodotto> ProdottiAcquistatati=new ArrayList<>(); //riferimento
    
    //costruttori
    public Spesa()
    {}
    public Spesa(ArrayList<Prodotto> s) //simulazione passaggio per valore
    {
        Iterator<Prodotto> iter=s.iterator();
        for(Prodotto p:s)
        {
          this.ProdottiAcquistatati.add(new Prodotto(iter.next().getPeso(),
                    iter.next().getPrezzoEtto(),iter.next().getScontoPercentuale()));
        }
    }
    // metodi
     public ArrayList<Prodotto> getSpesa() //simulazione passaggio per valore
     {
      ArrayList<Prodotto> s=new ArrayList<>();
      Iterator<Prodotto> iter=this.ProdottiAcquistatati.iterator();
      for(Prodotto p:this.ProdottiAcquistatati)
      {
          s.add(new Prodotto(iter.next().getPeso(),iter.next().getPrezzoEtto(),
                iter.next().getScontoPercentuale()));
      }
      return s;
     }
    public void setProdottiAcquistati(ArrayList<Prodotto> s)
    {
      this.ProdottiAcquistatati=s;
    }

dove la classe prodotto è la seguente:

public class Prodotto {
    //atributi
    private double Peso=0;//peso in kg
    private double PrezzoEtto=0; //prezzo all'etto
    private int ScontoPercentuale=0; //sconto in percentuale
    
    //costruttori
    public Prodotto()
    {}
    public Prodotto(double p,double pr,int s)
    {
        this.Peso=p;
        this.PrezzoEtto=pr;
        this.ScontoPercentuale=s;
    }
    
    //metodi
    public double getPeso()
    {
        return this.Peso;
    }
    public double getPrezzoEtto()
    {
        return this.PrezzoEtto;
    }
    public int getScontoPercentuale()
    {
        return this.ScontoPercentuale;
    }
    public void setPeso(float p)
    {
        this.Peso=p;
    }
    public void setPrezzoEtto(float pr)
    {
        this.PrezzoEtto=pr;
    }
    public void setScontoPercentuale(int s)
    {
        this.ScontoPercentuale=s;
    }
    @Override
    public String toString()
    {
        return "Peso: "+this.Peso+" Prezzo all'etto:"+this.PrezzoEtto+
                " Sconto in Percentuale: "+this.ScontoPercentuale;
    }
            
}

Ora volevo sapere se ho rispettato l'incapsulamento e quindi ho simulato bene il passaggio per valore. Grazie in anticipo per l'aiuto

[Alex'87]

Non ho guardato tutto, mi è caduto subito l'occhio su questa riga:

s.add(new Prodotto(iter.next().getPeso(),iter.next().getPrez zoEtto(),
iter.next().getScontoPercentuale()));

Non penso faccia quello che pensi faccia...
Ogni chiamata a next() porta avanti l'iteratore, in pratica stai creando un Prodotto usando valori di tre prodotti diversi.

Ah, i campi delle classi vanno scritti con l'iniziale minuscola: prezzoEtto, non PrezzoEtto, ecc

[bjunior]

Grazie mille per l'aiuto
ho corretto così:

    public Spesa(ArrayList<Prodotto> s) //simulazione passaggio per valore
    {
        Iterator<Prodotto> iter=s.iterator();
        Prodotto aus;
        for(Prodotto p:s)
        {
          aus=iter.next();
          this.ProdottiAcquistatati.add(new Prodotto(aus.getPeso(),
                    aus.getPrezzoEtto(),aus.getScontoPercentuale()));
        }
    }

Così credo vada bene giusto??

[francesco.muia]

perchè scomodare un ulteriore iterator

for(Prodotto p: s)
        {

          this.ProdottiAcquistatati.add(newProdotto(p.getPeso(),
                    p.getPrezzoEtto(),p.getScontoPercentuale()));
        }

[bjunior]

Grazie mille scusate per gli erroracci ma sono alle prime armi con java :P
nessuno mi sa dire niente per quanto riguarda la composizione e la simulazione di passaggio per "valore"??

[francesco.muia]

Grazie mille scusate per gli erroracci ma sono alle prime armi con java :P
nessuno mi sa dire niente per quanto riguarda la composizione e la simulazione di passaggio per "valore"??

A occhio, non ho letto tutto, mi sembra che sia corretta...

[bjunior]

grazie ancora un ultima cosa: per il metodo setProdottiAcquistati non devo simulare il passaggio per valore perchè non sto passando il riferimento prodottiAcquistati come parametro d'ingresso o di uscita del metodo, giusto??

[kuarl]

sinceramente non capisco perche` tu voglia fare una cosa simile. Qual e` il problema del passare la lista di prodotti per riferimento? Farne una copia poi rendere piu` difficile la manutenzione.

Piu` che altro penso tu abbia sbagliato il modello. Di solito si fa:
- una classe Ordine che rappresenta l'acquisto di una serie di prodotti
- una classe prodotto che rappresenta i prodotti acqustabili
- una classe rigaordine che lega i prodotti all'ordine specificando ulteriori parametri quali quantita`, taglia ed eventuali altre opzioni d'acquisto specifiche per il singolo ordine

L'incapsulamento non e` necessariamente un male. I concetti che bisogna tener conto in fase di progettazione sono coesione e accoppiamento. Ci sono molte risorse in rete che ne parlano.

[bjunior]

In totale sto facendo un progetto per gestire una spesa in una macelleria:
ho scelto due classi: Carne e Formaggio che rappresentano i prodotti della macelleria e sono legati dalla relazione di ereditarietà alla classe prodotto che andrà a comporre la classe spesa da cui la classe registratore_di_cassa dipenderà.
Dici che così non va bene??

[kuarl]

In totale sto facendo un progetto per gestire una spesa in una macelleria:
ho scelto due classi: Carne e Formaggio che rappresentano i prodotti della macelleria e sono legati dalla relazione di ereditarietà alla classe prodotto che andrà a comporre la classe spesa da cui la classe registratore_di_cassa dipenderà.
Dici che così non va bene??

no... segui la struttura che ti ho indicato io che e` quella tipica per i negozi. Mi sembra superfluo poi estendere Prodotto, semplicemente aggiungi come campo l'informazione sulla tipologia.