Stranezze return metodo

[Toma81]

Salve a tutti, mi presento con questo primo topic su un problema che sto riscontrando in un programma.

Main:

if(mensola.setVolume(libro,np)==-1)
          System.out.println("Posizione non valida");
if(mensola.setVolume(libro,np)==-2)
          System.out.println("Posizione occupata");
else
          mensola.setVolume(libro,np);

Classe Mensola:

public int setVolume(Libro libro,int posizione){    
        if((posizione<0)|(posizione>NUM_MAX_VOLUMI))    //NUM_MAX_VOLUMI=15
            return -1;    //posizione non valida
        if(volumi[posizione]!=null)
            return -2;    //posizione occupata
        volumi[posizione]=libro;
        return posizione;
    }

Il problema sta nel ritorno del metodo setVolume in quanto ritorna sempre "-2" come se nella posizione data dell'array ci sia già un oggetto.
Sintatticamente il compilatore non mi dà problemi, potete aiutarmi?

[andbin]

Il problema sta nel ritorno del metodo setVolume in quanto ritorna sempre "-2" come se nella posizione data dell'array ci sia già un oggetto.

Il punto è che facendo 3 volte mensola.setVolume con lo stesso libro/np, il primo probabilmente (anzi quasi certamente, da quanto dici) ha successo e poi chiaramente il secondo fallisce con -2. Quindi il problema è l'uso sbagliato ... non tanto il metodo setVolume in sé.


Riguardo il setVolume ci sono 2 finezze:
- Usa || invece che | . Il primo è il OR logico con comportamento "short-circuit". Se posizione<0 è già true, non perde tempo a valutare l'altro operando.
- NUM_MAX_VOLUMI lo intendi come il numero massimo di volumi oppure come la posizione massima che è ancora lecita? Perché se lo si intende (e dal nome io penserei così) come il numero massimo, allora la sua posizione è esclusa (essendo zero-based), quindi sarebbe più corretto posizione >= NUM_MAX_VOLUMI

[Toma81]

Grazie per la risposta.

Per quanto riguarda i controlli ho provato a ridurre i controlli in un solo "if", così:

public int setVolume(Libro libro,int posizione){   
        if((posizione<0) || (posizione>NUM_MAX_VOLUMI) || (volumi[posizione]!=null))
            return -1;    //posizione non valida
      
         volumi[posizione]=libro;
         return posizione;
    }

Ok ho corretto con quello che mi hai detto e l'errore è risolto. Ma quindi non posso mettere più controlli sul ritorno di uno stesso metodo?

[Toma81]

Riguardo il setVolume ci sono 2 finezze:
- Usa || invece che | . Il primo è il OR logico con comportamento "short-circuit". Se posizione<0 è già true, non perde tempo a valutare l'altro operando.
- NUM_MAX_VOLUMI lo intendi come il numero massimo di volumi oppure come la posizione massima che è ancora lecita? Perché se lo si intende (e dal nome io penserei così) come il numero massimo, allora la sua posizione è esclusa (essendo zero-based), quindi sarebbe più corretto posizione >= NUM_MAX_VOLUMI

-Si l'OR logico è stato un errore di distrazione oppure nel tentativo di cambiare qualche condizione mi sono dimenticato una |.
-Per NUM_MAX_VOLUMI intendo l'indice massimo dell'array volumi[], quindi come posizione massima.

Grazie dell'aiuto!

[andbin]

Per quanto riguarda i controlli ho provato a ridurre i controlli in un solo "if"

Non era quello il problema.

Ma quindi non posso mettere più controlli sul ritorno di uno stesso metodo?

Certo che si può ma tu invocavi più volte setVolume. Quello era il problema!

-Per NUM_MAX_VOLUMI intendo l'indice massimo dell'array volumi[], quindi come posizione massima.

Allora POS_MAX_VOLUMI

[Toma81]

Ah ok perfetto, quindi metto in una variabile il ritorno del metodo è poi faccio i controlli... Ho iniziato da poco a con Java.

Grazie mille per l'aiuto

[schumy2000]

Toglimi solo una curiosità...
hai valorizzato a null tutte le posizioni dell'array volumi?
Altrimenti ti andrà sempre in questa posizione...

  if(volumi[posizione]!=null)

perché quando vai a fare:

volumi=new Libro [NUM_MAX_VOLUMI ]

tutte le posizioni dell'array non vengono messi a null in maniera automatica, viene solo allocato lo spazio per quell'array.
Poi sarai tu a fare un ciclo (for o while) che ti andrà a settare tutte le posizioni, così come assegni null in questo progetto, dovresti invocare il costruttore in casi differenti.

Ciao.

[andbin]

volumi=new Libro [NUM_MAX_VOLUMI ]

tutte le posizioni dell'array non vengono messi a null in maniera automatica

Sì invece, in un array di N reference istanziato così (quindi non parliamo della inizializzazione con valori in { } ), i reference sono inizialmente tutti null.

[Toma81]

Sì invece, in un array di N reference istanziato così (quindi non parliamo della inizializzazione con valori in { } ), i reference sono inizialmente tutti null.

Stavo giusto per scrive che sul libro su cui sto studiando c'è scritto così, appunto.

[schumy2000]

ricordavo male allora...o forse lo confondo con qualche altro linguaggio...