[Java] Classi generiche

[zanardi84]

Salve a tutti. Ho sviluppato per esercizio una classe partendo dall'oggetto istanziato da una classe generica, come nel listato che sto per postare.
Non riesco a capire perchè eclipse mi restituisce un errore di compilazione nel metodo compareTo: - This method must return a result of type int
- implements
java.lang.Comparable<Istante>.compareTo

public class Istante implements Comparable<Istante>
{
	// variabile di istanza
	private Coppia<Data, Orario> istante;
	
	// costruttore
	public Istante(Data d, Orario o)
	{
		istante = new Coppia<Data, Orario>(d, o);
	}
	
	// campi dell'oggetto a destra nella coppia
	public int getOra()
	{
		return istante.getDestro().getOre();
	}
		
	public int getMinuti()
	{
		return istante.getDestro().getMinuti();
	}
	
	// campi dell'oggetto a sinistra nella coppia
	public int getGiorno()
	{
		return istante.getSinistro().getGiorno();
	}
	
	public int getMese()
	{
		return istante.getSinistro().getMese();
	}
	
	public int getAnno()
	{
		return istante.getSinistro().getAnno();
	}
	
	
	public Orario getOrario()
	{
		return istante.getDestro();
	}
	
	public Data getData()
	{
		return istante.getSinistro();
	}
	
	public String toString()
	{
		return istante.toString();
	}
	
	public boolean equals (Istante altro)
	{
		if(this.getData().equals(altro.getData())&& this.getOrario().equals(altro.getOrario()))
			return true;
		else
			return false;
	}
	
	
	public int compareTo(Istante altro)
	{
		if(this.getData().isMaggiore(altro.getData()))
			return 1;
		if(this.getData().isMinore(altro.getData()))
			return -1;
		if(this.getData().equals(altro.getData()))
		{
			if(this.getOrario().isMaggiore(altro.getOrario()))
				return 1;
			if(this.getOrario().isMinore(altro.getOrario()))
				return -1;
		}
		else
			return 0;
			
		
	}
	
}

Secondo voi il codice è corretto o si può creare in modo diverso partendo dall'istanza della classe Coppia?
Grazie.
Grazie.

[banryu79]

Originariamente inviato da zanardi84

	
...
	public int compareTo(Istante altro)
	{
		if(this.getData().isMaggiore(altro.getData()))
			return 1;
		if(this.getData().isMinore(altro.getData()))
			return -1;
		if(this.getData().equals(altro.getData()))
		{
			if(this.getOrario().isMaggiore(altro.getOrario()))
				return 1;
			if(this.getOrario().isMinore(altro.getOrario()))
				return -1;
		}
		else
			return 0;
			
		
	}
	
}

Il compilatore si lamenta perchè c'è un flusso d'esecuzione che non termina con la restituzione di un int; in particolare quando la condizione in grassetto è verificata mentre le due in marroncino non lo sono. Devi prevedere il caso e restituire l'intero (immagino zero).

[zanardi84]

Originariamente inviato da banryu79

Il compilatore si lamenta perchè c'è un flusso d'esecuzione che non termina con la restituzione di un int; in particolare quando la condizione in grassetto è verificata mentre le due in marroncino non lo sono. Devi prevedere il caso e restituire l'intero (immagino zero).

Vero! Mi ero dimenticato che esiste anche "l'else implicito"!
Il codice come ti sembra?