ti incollo il mio metodo

[code type="markup"]
private class ErogaAA implements ActionListener
{
public void actionPerformed(ActionEvent arg0)
{
if (quantitaF.getText().equals("")
JOptionPane.showMessageDialog(null, " Quantità nulla!! ";
else
try
{
IDEsternoProdotto tipo = new IDEsternoProdotto(iDEsternoProdottoF.getText());
controllore.eroga(tipo,Integer.parseInt(quantitaF.getText()));
}
catch (NumberFormatException e)
{
JOptionPane.showMessageDialog(null, " Quantità non valida. ";
}
catch (IDEsternoProdottoException e)
{
JOptionPane.showMessageDialog(null, " Nome prodotto non valido. ";
}
catch (QuantityException e)
{
JOptionPane.showMessageDialog(null, " Quantità negativa o nulla. ";
}
catch (GiacenzaInsufficienteException e)
{
//Qui viene gestita l'eccezione di GiacenzaInsuff ..... e ok!
int quantita = Integer.parseInt(quantitaF.getText());
int giacenza = e.getGiacenza(); //non badarci..tu hai usato controllore.giacenza(tipo) va bene ugualmente
if (giacenza == 0)
JOptionPane.showMessageDialog(null, e.getMessage());

else if (giacenza < quantita)
{
//cioè qui ...se la giacenza è < della quantità inserita ...che succede?
IDEsternoProdotto tipo = new IDEsternoProdotto(iDEsternoProdottoF.getText());
int ritorno = JOptionPane.showConfirmDialog( null, e.getMessage(), "Attenzione!", JOptionPane.YES_NO_OPTION);
if(ritorno == JOptionPane.YES_OPTION)
{
try {

controllore.eroga(tipo, giacenza);

} catch (IDEsternoProdottoException e1) {

JOptionPane.showMessageDialog(null, e1.getMessage());

} catch (QuantityException e1) {

e1.printStackTrace();
} catch (GiacenzaInsufficienteException e1) {

JOptionPane.showMessageDialog(null, e1.getMessage(), "Attenzione!", JOptionPane.INFORMATION_MESSAGE);
}
}
else
{
JOptionPane.showMessageDialog(null, "Operazione annullata!", "Attenzione!", JOptionPane.INFORMATION_MESSAGE);
}
}

}
}
}
[/code]

così funziona perchè nel caso si verifichi l'eccezione viene eseguito quello che c'è nel catch

Forse è una domanda stupida, ma per quale motivo eroghi una seconda volta?
edit: ah ho capito, immagino volevi semplicemente usare il messaggio di conferma SI/NO, dove se premi SI ....ti ricalcola eroga. ok ok...

Quindi idee su come implementare affitto?
secondo me un'altra classe non serve. basta aggiungere i metodi adatti in prodotto e sistemare il controllore. che dite?

Lo so. Ho notato che fa eroga(tipo, giacenza). Infatti è qui la cosa strana. Mi dà valori negativi. Come può essere? Il codice è esattamente quello.

Se faccio il copia e incolla così com'è mi dà errore su e.getGiacenza(), e se al suo posto metto controllore.giacenza(tipo) mi diche che questo lo devo includere in un try/catch.

è ovvio, genera un'eccezione

Risolto il problema però con quel codice se la giacenza>0, es: 1, e si cerca di prelevare una quantità=2 allora la giacenza va in negativo...

no perchè tu stai prelevando l'intera giacenza e non la quantità richiesta (che è maggiore della giacenza) quindi non va in negativo.
cioè quando avvisi l'utente che la giacenza è inferiore alla quantità richiesta e l'utente conferma di voler erogare comunque, tu chiami il metodo eroga passandogli come parametro la giacenza e non la quantità.

Se faccio il copia e incolla così com'è mi dà errore su e.getGiacenza(), e se al suo posto metto controllore.giacenza(tipo) mi diche che questo lo devo includere in un try/catch.

è ovvio, genera un'eccezione

Risolto il problema però con quel codice se la giacenza>0, es: 1, e si cerca di prelevare una quantità=2 allora la giacenza va in negativo...

Quando faccio la classe Affitto devo anche modificare l'interfaccia e il controllore per gestire gli affitti? (non ci vuole troppo tempo?)

idem per la classe utente: gli basta la classe utente o mi devo preoccupare anche di chi crea il primo amministratore ecc?

mi autoquoto:
per gestire gli affitti non basta aggiungere i metodi gradimento_affitto e successo_affitto oltre che affitta?
che senso ha fare un'altra classe se i prodotti sono gli stessi, che possono essere sia affittati che venduti??


riuppo invece la seconda domanda.

Quando faccio la classe Affitto devo anche modificare l'interfaccia e il controllore per gestire gli affitti? (non ci vuole troppo tempo?)

idem per la classe utente: gli basta la classe utente o mi devo preoccupare anche di chi crea il primo amministratore ecc?

Se faccio il copia e incolla così com'è mi dà errore su e.getGiacenza(), e se al suo posto metto controllore.giacenza(tipo) mi diche che questo lo devo includere in un try/catch.

è ovvio, genera un'eccezione

Se faccio il copia e incolla così com'è mi dà errore su e.getGiacenza(), e se al suo posto metto controllore.giacenza(tipo) mi diche che questo lo devo includere in un try/catch.

se incolli il codice provo a dargli un'occhiata. incolla pure dichiarazioni e costruttore di prodotto

[code type="markup"]public class Prodotto implements Serializable
{
private static final long serialVersionUID = 1L;

private LogMonitorGiacenza logGiacenze;
private String descrizione;
private int giacenza;
private IDEsternoProdotto iDEsterno;
private int nBrickErogati;
private int nBrickRichiesti;
private int nBrickAffittati;
private int nBrickdaAffittare;

private ContatoreSingolo contatoreComuneProdottiAffittati = null;
private ContatoreSingolo contatoreComuneProdottiErogati = null;

private ConfineOperativo confine;

public Prodotto(IDEsternoProdotto t, String nuovaDescrizione, ConfineOperativo confine)
{
this.confine = confine;
iDEsterno = t;
logGiacenze = new LogMonitorGiacenza(iDEsterno.toString());
descrizione = nuovaDescrizione;
giacenza = 0;
logGiacenze.aggiungi("0";
nBrickErogati = 0;
nBrickRichiesti = 0;
nBrickAffittati = 0;
nBrickdaAffittare = 0;


if (contatoreComuneProdottiErogati == null)
contatoreComuneProdottiErogati = ContatoreSingolo.crea();

if (contatoreComuneProdottiAffittati == null)
contatoreComuneProdottiAffittati = ContatoreSingolo.crea();

public void affitta(int quantita) throws GiacenzaInsufficienteException, QuantityException
{
if (quantita <= 0)
throw new QuantityException();
nBrickAffittati = nBrickAffittati+quantita;
if (giacenza == 0)
{
throw new GiacenzaInsufficienteException("Giacenza insufficiente";
}
else
{
if(quantita <= giacenza){
giacenza = giacenza - quantita;
logGiacenze.aggiungi(String.valueOf(giacenza));
nBrickAffittati += quantita;
contatoreComuneProdottiAffittati.incrementa(quantita);
}
else
{
throw new GiacenzaInsufficienteException("Giacenza insufficiente";
}
}
}
}[/code]
Grazie mille, sei veramente gentile

Scusa, ma come hai fatto a far funzionare il suo erogaAA? A me non funziona. Mi dà errore.

Che tipo di errore ti da?

se incolli il codice provo a dargli un'occhiata. incolla pure dichiarazioni e costruttore di prodotto

[code type="markup"]public class Prodotto implements Serializable
{
private static final long serialVersionUID = 1L;

private LogMonitorGiacenza logGiacenze;
private String descrizione;
private int giacenza;
private IDEsternoProdotto iDEsterno;
private int nBrickErogati;
private int nBrickRichiesti;
private int nBrickAffittati;
private int nBrickdaAffittare;

private ContatoreSingolo contatoreComuneProdottiAffittati = null;
private ContatoreSingolo contatoreComuneProdottiErogati = null;

private ConfineOperativo confine;

public Prodotto(IDEsternoProdotto t, String nuovaDescrizione, ConfineOperativo confine)
{
this.confine = confine;
iDEsterno = t;
logGiacenze = new LogMonitorGiacenza(iDEsterno.toString());
descrizione = nuovaDescrizione;
giacenza = 0;
logGiacenze.aggiungi("0";
nBrickErogati = 0;
nBrickRichiesti = 0;
nBrickAffittati = 0;
nBrickdaAffittare = 0;


if (contatoreComuneProdottiErogati == null)
contatoreComuneProdottiErogati = ContatoreSingolo.crea();

if (contatoreComuneProdottiAffittati == null)
contatoreComuneProdottiAffittati = ContatoreSingolo.crea();

public void affitta(int quantita) throws GiacenzaInsufficienteException, QuantityException
{
if (quantita <= 0)
throw new QuantityException();
nBrickAffittati = nBrickAffittati+quantita;
if (giacenza == 0)
{
throw new GiacenzaInsufficienteException("Giacenza insufficiente";
}
else
{
if(quantita <= giacenza){
giacenza = giacenza - quantita;
logGiacenze.aggiungi(String.valueOf(giacenza));
nBrickAffittati += quantita;
contatoreComuneProdottiAffittati.incrementa(quantita);
}
else
{
throw new GiacenzaInsufficienteException("Giacenza insufficiente";
}
}
}
}[/code]
Grazie mille, sei veramente gentile

Scusa, ma come hai fatto a far funzionare il suo erogaAA? A me non funziona. Mi dà errore.

se incolli il codice provo a dargli un'occhiata. incolla pure dichiarazioni e costruttore di prodotto

[code type="markup"]public class Prodotto implements Serializable
{
private static final long serialVersionUID = 1L;

private LogMonitorGiacenza logGiacenze;
private String descrizione;
private int giacenza;
private IDEsternoProdotto iDEsterno;
private int nBrickErogati;
private int nBrickRichiesti;
private int nBrickAffittati;
private int nBrickdaAffittare;

private ContatoreSingolo contatoreComuneProdottiAffittati = null;
private ContatoreSingolo contatoreComuneProdottiErogati = null;

private ConfineOperativo confine;

public Prodotto(IDEsternoProdotto t, String nuovaDescrizione, ConfineOperativo confine)
{
this.confine = confine;
iDEsterno = t;
logGiacenze = new LogMonitorGiacenza(iDEsterno.toString());
descrizione = nuovaDescrizione;
giacenza = 0;
logGiacenze.aggiungi("0";
nBrickErogati = 0;
nBrickRichiesti = 0;
nBrickAffittati = 0;
nBrickdaAffittare = 0;


if (contatoreComuneProdottiErogati == null)
contatoreComuneProdottiErogati = ContatoreSingolo.crea();

if (contatoreComuneProdottiAffittati == null)
contatoreComuneProdottiAffittati = ContatoreSingolo.crea();

public void affitta(int quantita) throws GiacenzaInsufficienteException, QuantityException
{
if (quantita <= 0)
throw new QuantityException();
nBrickAffittati = nBrickAffittati+quantita;
if (giacenza == 0)
{
throw new GiacenzaInsufficienteException("Giacenza insufficiente";
}
else
{
if(quantita <= giacenza){
giacenza = giacenza - quantita;
logGiacenze.aggiungi(String.valueOf(giacenza));
nBrickAffittati += quantita;
contatoreComuneProdottiAffittati.incrementa(quantita);
}
else
{
throw new GiacenzaInsufficienteException("Giacenza insufficiente";
}
}
}
}[/code]
Grazie mille, sei veramente gentile

Guarda che non puoi utilizzare lo stesso tipo di contatore sia per gli affittati che per gli erogati. Le due variabili dei contatore puntano alla stesso oggetto in questo modo perchè crea() ti restituisce un contatore nuovo solo se la variabile statica meStesso == null..ma questa viene istanziata già prima dal contatore degli erogati. Crea un'altra classe identica ma separata per il contatore degli affitti e dovresti risolvere. Scusa ma vado un pò di fretta, prova così comunque!

Non ti preoccupare, ma che scherzi, già è tanto che me lo hai guardato! Grazie mille proverò così

se incolli il codice provo a dargli un'occhiata. incolla pure dichiarazioni e costruttore di prodotto

[code type="markup"]public class Prodotto implements Serializable
{
private static final long serialVersionUID = 1L;

private LogMonitorGiacenza logGiacenze;
private String descrizione;
private int giacenza;
private IDEsternoProdotto iDEsterno;
private int nBrickErogati;
private int nBrickRichiesti;
private int nBrickAffittati;
private int nBrickdaAffittare;

private ContatoreSingolo contatoreComuneProdottiAffittati = null;
private ContatoreSingolo contatoreComuneProdottiErogati = null;

private ConfineOperativo confine;

public Prodotto(IDEsternoProdotto t, String nuovaDescrizione, ConfineOperativo confine)
{
this.confine = confine;
iDEsterno = t;
logGiacenze = new LogMonitorGiacenza(iDEsterno.toString());
descrizione = nuovaDescrizione;
giacenza = 0;
logGiacenze.aggiungi("0";
nBrickErogati = 0;
nBrickRichiesti = 0;
nBrickAffittati = 0;
nBrickdaAffittare = 0;


if (contatoreComuneProdottiErogati == null)
contatoreComuneProdottiErogati = ContatoreSingolo.crea();

if (contatoreComuneProdottiAffittati == null)
contatoreComuneProdottiAffittati = ContatoreSingolo.crea();

public void affitta(int quantita) throws GiacenzaInsufficienteException, QuantityException
{
if (quantita <= 0)
throw new QuantityException();
nBrickAffittati = nBrickAffittati+quantita;
if (giacenza == 0)
{
throw new GiacenzaInsufficienteException("Giacenza insufficiente";
}
else
{
if(quantita <= giacenza){
giacenza = giacenza - quantita;
logGiacenze.aggiungi(String.valueOf(giacenza));
nBrickAffittati += quantita;
contatoreComuneProdottiAffittati.incrementa(quantita);
}
else
{
throw new GiacenzaInsufficienteException("Giacenza insufficiente";
}
}
}
}[/code]
Grazie mille, sei veramente gentile

Guarda che non puoi utilizzare lo stesso tipo di contatore sia per gli affittati che per gli erogati. Le due variabili dei contatore puntano alla stesso oggetto in questo modo perchè crea() ti restituisce un contatore nuovo solo se la variabile statica meStesso == null..ma questa viene istanziata già prima dal contatore degli erogati. Crea un'altra classe identica ma separata per il contatore degli affitti e dovresti risolvere. Scusa ma vado un pò di fretta, prova così comunque!

Mi puoi dire se questo codice può andar bene? Fa quello che deve fare e non dà errori ma è così che lo vorrebbe il prof? Gestisci_giacenza fa un check se giacenza>quantità & giacenza!=0 allora eroga, altrimenti dai false.

a funzionare funziona. però così l'eccezione giacenzaInsufficienteException non credo venga mai sollevata perchè il check avviene nel controllore. Diciamo che se il prof si mette a spulciare il codice si potrebbe infastidire

se incolli il codice provo a dargli un'occhiata. incolla pure dichiarazioni e costruttore di prodotto

[code type="markup"]public class Prodotto implements Serializable
{
private static final long serialVersionUID = 1L;

private LogMonitorGiacenza logGiacenze;
private String descrizione;
private int giacenza;
private IDEsternoProdotto iDEsterno;
private int nBrickErogati;
private int nBrickRichiesti;
private int nBrickAffittati;
private int nBrickdaAffittare;

private ContatoreSingolo contatoreComuneProdottiAffittati = null;
private ContatoreSingolo contatoreComuneProdottiErogati = null;

private ConfineOperativo confine;

public Prodotto(IDEsternoProdotto t, String nuovaDescrizione, ConfineOperativo confine)
{
this.confine = confine;
iDEsterno = t;
logGiacenze = new LogMonitorGiacenza(iDEsterno.toString());
descrizione = nuovaDescrizione;
giacenza = 0;
logGiacenze.aggiungi("0";
nBrickErogati = 0;
nBrickRichiesti = 0;
nBrickAffittati = 0;
nBrickdaAffittare = 0;


if (contatoreComuneProdottiErogati == null)
contatoreComuneProdottiErogati = ContatoreSingolo.crea();

if (contatoreComuneProdottiAffittati == null)
contatoreComuneProdottiAffittati = ContatoreSingolo.crea();

public void affitta(int quantita) throws GiacenzaInsufficienteException, QuantityException
{
if (quantita <= 0)
throw new QuantityException();
nBrickAffittati = nBrickAffittati+quantita;
if (giacenza == 0)
{
throw new GiacenzaInsufficienteException("Giacenza insufficiente";
}
else
{
if(quantita <= giacenza){
giacenza = giacenza - quantita;
logGiacenze.aggiungi(String.valueOf(giacenza));
nBrickAffittati += quantita;
contatoreComuneProdottiAffittati.incrementa(quantita);
}
else
{
throw new GiacenzaInsufficienteException("Giacenza insufficiente";
}
}
}
}[/code]
Grazie mille, sei veramente gentile

se incolli il codice provo a dargli un'occhiata. incolla pure dichiarazioni e costruttore di prodotto

Scusate, a me non funziona? Devo inserire i controlli dentro GiacenzaInsufficienteException? Se sì, come devo fare?

ho già risposto a seven986 alla stessa identica domanda praticamente..ho pure incollato il codice con il catch fatto. Leggi i post indietro...

[code type="markup"]private class ErogaAA implements ActionListener
{
public void actionPerformed(ActionEvent arg0)
{
try
{
if (quantitaF.getText().equals("" || Integer.parseInt(quantitaF.getText())<=0)
throw new QuantityException();

IDEsternoProdotto tipo = new IDEsternoProdotto(iDEsternoProdottoF.getText());
int newQuantity=0;

if(!controllore.gestisci_giacenza(tipo, Integer.parseInt(quantitaF.getText()))){


int ritorno= JOptionPane.showConfirmDialog( null, "Quantità disponobile = " + controllore.giacenza(tipo) + " procedere?", "Attenzione!", JOptionPane.YES_NO_OPTION);
if(ritorno == JOptionPane.YES_OPTION)
{
newQuantity = controllore.giacenza(tipo) ;
controllore.eroga(tipo,newQuantity);
}
else
{
JOptionPane.showMessageDialog(null, "Operazione annullata!", "Attenzione!", JOptionPane.INFORMATION_MESSAGE);

}
}
else
{
JOptionPane.showMessageDialog(null, "Erogate con succeso " +quantitaF.getText()+ " biciclette";
}
}
catch (NumberFormatException e)
{
JOptionPane.showMessageDialog(null, " Quantità non valida. ";
}
catch (IDEsternoProdottoException e)
{
JOptionPane.showMessageDialog(null, e.getMessage());
}
catch (QuantityException e)
{
JOptionPane.showMessageDialog(null, " Quantità negativa o nulla. ";
}
catch (GiacenzaInsufficienteException e)
{
JOptionPane.showMessageDialog(null, " Giacenza insufficiente. ";
}
}
}[/code]
Mi puoi dire se questo codice può andar bene? Fa quello che deve fare e non dà errori ma è così che lo vorrebbe il prof? Gestisci_giacenza fa un check se giacenza>quantità & giacenza!=0 allora eroga, altrimenti dai false.

se incolli il codice provo a dargli un'occhiata. incolla pure dichiarazioni e costruttore di prodotto

Scusate, a me non funziona? Devo inserire i controlli dentro GiacenzaInsufficienteException? Se sì, come devo fare?

ho già risposto a seven986 alla stessa identica domanda praticamente..ho pure incollato il codice con il catch fatto. Leggi i post indietro
Non devi inserire nessun controllo nella classe dell'eccezione..ma nel catch della classe di confine che gestisce l'eccezione

se incolli il codice provo a dargli un'occhiata. incolla pure dichiarazioni e costruttore di prodotto

Scusate, a me non funziona? Devo inserire i controlli dentro GiacenzaInsufficienteException? Se sì, come devo fare?

se incolli il codice provo a dargli un'occhiata. incolla pure dichiarazioni e costruttore di prodotto

ti incollo il mio metodo

[code type="markup"]
private class ErogaAA implements ActionListener
{
public void actionPerformed(ActionEvent arg0)
{
if (quantitaF.getText().equals("")
JOptionPane.showMessageDialog(null, " Quantità nulla!! ";
else
try
{
IDEsternoProdotto tipo = new IDEsternoProdotto(iDEsternoProdottoF.getText());
controllore.eroga(tipo,Integer.parseInt(quantitaF.getText()));
}
catch (NumberFormatException e)
{
JOptionPane.showMessageDialog(null, " Quantità non valida. ";
}
catch (IDEsternoProdottoException e)
{
JOptionPane.showMessageDialog(null, " Nome prodotto non valido. ";
}
catch (QuantityException e)
{
JOptionPane.showMessageDialog(null, " Quantità negativa o nulla. ";
}
catch (GiacenzaInsufficienteException e)
{

int quantita = Integer.parseInt(quantitaF.getText());
int giacenza = e.getGiacenza(); //non badarci..tu hai usato controllore.giacenza(tipo) va bene ugualmente

if (giacenza == 0)
JOptionPane.showMessageDialog(null, e.getMessage());

else if (giacenza < quantita)
{

IDEsternoProdotto tipo = new IDEsternoProdotto(iDEsternoProdottoF.getText());
int ritorno = JOptionPane.showConfirmDialog( null, e.getMessage(), "Attenzione!", JOptionPane.YES_NO_OPTION);
if(ritorno == JOptionPane.YES_OPTION)
{
try {

controllore.eroga(tipo, giacenza);
} catch (IDEsternoProdottoException e1) {

JOptionPane.showMessageDialog(null, e1.getMessage());

} catch (QuantityException e1) {

e1.printStackTrace();
} catch (GiacenzaInsufficienteException e1) {

JOptionPane.showMessageDialog(null, e1.getMessage(), "Attenzione!", JOptionPane.INFORMATION_MESSAGE);
}
}
else
{
JOptionPane.showMessageDialog(null, "Operazione annullata!", "Attenzione!", JOptionPane.INFORMATION_MESSAGE);
}
}

}
}
}
[/code]

così funziona perchè nel caso si verifichi l'eccezione viene eseguito quello che c'è nel catch

Ciao, senti io ho usato lo stesso metodo di eroga per affittare. Però una volta che testo il codice eroga funziona, non lanciandomi l'eccezione, mentre affitta no e lancia una nullpointerexception nella riga del contatore [code type="markup"] ComuneProdottiAffittati.incrementa(quantita);[/code] quando comunque sono stata attenta ad inizializzare il contatore singolo. Forse devo gestire diversamente l'affitto? Non so se vuoi che ti incolli il codice. Grazie in anticipo :wink:

Ah aggiungo pure che pur lanciando l'eccezione, il metodo fa quello che deve fare, nel senso che diminuisce la giacenza.