[Java] Problema con metodo

[djluis]

Ciao a tutti. Ho un problema col metodo compareTo della seguente classe.
Non riesco ad interpretare bene la richiesta.
Se qualcuno può darmi un idea...
Grazie a chi risponderà
----------------------------------------------------------------------------------------
public class SimpleID3 {

private String artist;
private String album;
private int year;

public SimpleID3(String artist, String album, int year) {
this.artist = artist;
this.album = album;
this.year = year;
}

/*
** Il seguente metodo restituisce -1 se this e` minore
** di id3 passato come parametro, 0 se uguale e 1 se
** maggiore. this e` minore di id3 se l'anno di pub-
** blicazione di this e` minore, maggiore se l'anno e`
** maggiore. Se l'anno e` uguale allora this e` minore
** uguale o maggiore a seconda se il nome dell'artista
** e` minore, uguale o maggiore (lessicograficamente).
*/
public int compareTo(SimpleID3 id3) {

///svolgere

}

public String getArtist() {
return artist;
}

public String getAlbum() {
return album;
}

public int getYear() {
return year;
}

public String toString() {
String temp = "";
temp += "Artist: " + artist + "\n";
temp += "Album: " + album + "\n";
temp += "Year: " + year;
return temp;
}

}

---------------------------------------------------------------------------------------

[Andrea1979]

E' uno di quei quesiti già risposti:basta accendere il cervello e leggere.

/*
    ** Il seguente metodo restituisce -1 se this e` minore
    ** di id3 passato come parametro, 0 se uguale  e  1 se
    ** maggiore. this e` minore di id3 se l'anno  di  pub-
    ** blicazione di this e` minore, maggiore se l'anno e`
    ** maggiore. Se l'anno e` uguale allora this e` minore
    ** uguale o maggiore a seconda se il nome dell'artista
    ** e` minore, uguale o maggiore (lessicograficamente).
    */
    public int compareTo(SimpleID3 id3) {
      //comparazione per anno
      return (this.getYear() > id3.getYear() ? 1 : (this.getYear() < id3.getYear() ? -1 : (this.getArtist() > id3.getArtist() ? 1 : (this.getArtist() < id3.getArtist() ? - 1 : 0)));

	///svolgere

    }

spero di non aver dimenticato qualche parentesi

[andbin]

Originariamente inviato da Andrea1979
spero di non aver dimenticato qualche parentesi

Ehm ... il fatto è che getArtist() ritorna un String ... e non puoi confrontarlo con < > ecc...

Così!

public int compareTo (SimpleID3 id3) {
    if (getYear() < id3.getYear())
        return -1;
    else if (getYear() > id3.getYear())
        return +1;

    return getArtist().compareTo (id3.getArtist());
}

[Andrea1979]

Originariamente inviato da andbin
Ehm ... il fatto è che getArtist() ritorna un String ... e non puoi confrontarlo con < > ecc...

Così!

public int compareTo (SimpleID3 id3) {
    if (getYear() < id3.getYear())
        return -1;
    else if (getYear() > id3.getYear())
        return +1;

    return getArtist().compareTo (id3.getArtist());
}

ooops vero:

public int compareTo(SimpleID3 id3) {
        return (this.getYear() > id3.getYear() ? 1 : 
            (this.getYear() < id3.getYear() ? -1 : 
                (this.getArtist().compareTo(id3.getArtist()) == 0 ? 0 : 
                    Math.abs(this.getArtist().compareTo(id3.getArtist()))/this.getArtist().compareTo(id3.getArtist()))));
    }

[andbin]

Originariamente inviato da Andrea1979
ooops vero:

public int compareTo(SimpleID3 id3) {
        return (this.getYear() > id3.getYear() ? 1 : 
            (this.getYear() < id3.getYear() ? -1 : 
                (this.getArtist().compareTo(id3.getArtist()) == 0 ? 0 : 
                    Math.abs(this.getArtist().compareTo(id3.getArtist()))/this.getArtist().compareTo(id3.getArtist()))));
    }

Eh???? VVoVe:
Perché tutto 'sto casino con Math.abs ecc...??? Ma scherziamo?

Guarda che se l'anno è uguale, basta far ritornare esattamente il risultato del compareTo sull'artista!

Il mio codice era già funzionante e chiarissimo .... se proprio si volesse compattare un po' si potrebbe fare:

public int compareTo (SimpleID3 id3) {
    return getYear() < id3.getYear() ? -1 : getYear() > id3.getYear() ? +1 :
           getArtist().compareTo (id3.getArtist());
}

[Andrea1979]

compareTo su stringa ritorna un positivo (non è detto che sia 1) o un negativo (non è detto che sia -1). L'ho semplicemente normalizzato.

[andbin]

Originariamente inviato da Andrea1979
compareTo su stringa ritorna un positivo (non è detto che sia 1) o un negativo (non è detto che sia -1). L'ho semplicemente normalizzato.

E cosa cambia???
La specifica di compareTo dice:
Returns:
a negative integer, zero, or a positive integer as this object is less than, equal to, or greater than the specified object.

Non dice che devi per forza far ritornare esattamente +1 o -1.

[Andrea1979]

ma la specifica dell'esercizio sì. solo -1 0 o 1

[andbin]

Originariamente inviato da Andrea1979
ma la specifica dell'esercizio sì. solo -1 0 o 1

Ah ok ... rileggendo il testo iniziale in effetti dice -1 o +1. Quindi in quel senso è corretto "normalizzare" il risultato.

Io comunque, personalmente, non avrei di certo usato divisioni o Math.abs!!!

Bastava fare:

int r = getArtist().compareTo (id3.getArtist());
return r < 0 ? -1 : r > 0 ? +1 : 0;

Con Java 5 in sù, si può anche fare in modo abbastanza "pulito" con:

return Integer.signum (getArtist().compareTo (id3.getArtist()));

Comunque chi ha scritto quella specifica dell'esercizio, dubito che conosca l'interfaccia Comparable e le sue specifiche ..... non mi stupirei più di tanto.

[Andrea1979]

non posso che darti ragione... d'altra parte chiunque sia passato per un corso di informatica di una qualsiasi facoltà di ingegneria o simile ti potrà confermare che la maggior parte dei docenti che dirigono tali insegnamenti sono in verità riciclati (sarebbero docenti di tutt'altra materia) e fondamentalmente non hanno molta voglia di documentarsi sulle nuove tecnologie e/o specifiche.

Mi piaceva farlo tutto su una riga