Metodo che non fa quello che dovrebbe.

[Jamie04]

Dovrei aver risolto:

        for (Offerta key : offerte.values()) {
            
            if (key.getIdAsta() == idAsta && key.getOfferta()!=null) {
                for (Offerta key1 : offerte.values()) {
                    if (key1.getIdAsta() == idAsta && key1.getOfferta() >= importo) {
                        return 0;
                    }
                }
                //se tutto corretto inserisco la nuova offerta
                ultimoId++;
                Offerta nuovaOfferta = new Offerta (ultimoId, offerente, importo, idAsta, today);
                offerte.put(ultimoId, nuovaOfferta);
                break;
            }
        }
        db.commit();
        db.close();
        return 1;

Ho fatto molte prove e mi pare vada bene tutto. Che ne pensate? Mi ha fatto penare questa cosa...

[schumy2000]

.....Che ne pensate? Mi ha fatto penare questa cosa...

Che è inutile fare 2 for fondamentalmente UGUALI quando puoi usare degli if....è spreco di risorse e cicli macchina, perché se hai due/tre aste va bene, ma se ne hai 100.000?

[Jamie04]

Che è inutile fare 2 for fondamentalmente UGUALI quando puoi usare degli if....è spreco di risorse e cicli macchina, perché se hai due/tre aste va bene, ma se ne hai 100.000?

Ma era quello che stavo facendo all'inizio, un solo ciclo for, solo che non andava bene, perché si fermava non appena incontrava la prima offerta per quell'asta e quindi faceva il confronto solo col prezzo base. Io ho bisogno che il confronto con l'importo lo faccia per tutte le offerte inserite per quell'asta. Se puoi dirmi come modificarlo per farlo con un solo ciclo, te ne sarò grata.

[schumy2000]

Il primo if, ovvero quello che controlla se l'id dell'asta (idAsta) passato dal metodo corrisponde all'id di un'asta presente in offerte, dovrebbe dirmi se per l'asta che ho selezionato esiste già un'offerta inserita. Questo dovrebbe risultare sempre vero visto che ad ogni inserimento di un'asta viene inserita anche una nuova offerta pari al prezzo base.
if (key.getIdAsta() == idAsta) { //sempre vero?!?

E allora perché lo metti?
se è sempre vero non c'è bisogno di mettere un if

[Jamie04]

E allora perché lo metti?
se è sempre vero non c'è bisogno di mettere un if

Mi sono spiegata male, è vero che è sempre vero ma mi serve principalmente per controllare se ci sono anche altre offerte per quell'asta, quindi è un valore vero che può esser vero una volta ma anche più volte.