[Java] confronto date

[Alhazred]

Mi correggete questo codice?
Dovrebbe creare due oggetti di classe Data e verificare se sono uguali oppure no.
Attualmente gli oggetti creati contengono sempre 0 sia per giorno che per mese che per anno.

Codice PHP:

import javax.swing.JOptionPane;
public class Data {
    Data data1, data2;
    int giorno, mese, anno;
    public Data(int g, int m, int a) {
        int giorno = g;
        int mese = m;
        int anno = a;
    }
    public static void verificaDataUguale(Data data1, Data data2) {
        if ((data1.giorno == data2.giorno) && (data1.mese == data2.mese) && (data1.anno == data2.anno))
            System.out.println("Le date sono uguali");
            else System.out.println("Le date sono diverse");
    }
    public static Data primaData() {
        int giorno = Integer.parseInt(JOptionPane.showInputDialog("Inserisci il giorno della prima data"));
        int mese = Integer.parseInt(JOptionPane.showInputDialog("Inserisci il mese della prima data"));
        int anno = Integer.parseInt(JOptionPane.showInputDialog("Inserisci l'anno della prima data"));
        Data data1 = new Data(giorno, mese, anno);
        return data1;
    }
    public static Data secondaData() {
        int giorno = Integer.parseInt(JOptionPane.showInputDialog("Inserisci il giorno della seconda data"));
        int mese = Integer.parseInt(JOptionPane.showInputDialog("Inserisci il mese della seconda data"));
        int anno = Integer.parseInt(JOptionPane.showInputDialog("Inserisci l'anno della seconda data"));
        Data data2 = new Data(giorno, mese, anno);
        return data2;
    }
    public static void main(String[] args) {
        primaData();
        secondaData();
        verificaDataUguale(data1, data2);
        System.exit(0);
    }
} 


[cn73]

Tralasciando il discorso sulle date che sicuramente non va gestito in questo modo, ma con le apposite classi Date, DateFormat, SimpleDateFormat, come vedi hai praticamente riscritto lo stesso codice per data 1 e data2. Allora va messo in un unico metodo.

Comunque il codice non funziona perchè tu ridefinisce Data data1 e Data2 all'interno dei metodi, mentre credo che il tuo scopo è quello di valorizzare le variabili globali data1 e data2 che a questo punto non saranno mai inizializzate e conterranno sempre 0 che è il valore col quale sono inizializzati gli interi.

import javax.swing.JOptionPane; 
public class Data { 
    static Data data1, data2; 
    int giorno, mese, anno; 
    
    public Data(int g, int m, int a) { 
        giorno = g; 
        mese = m; 
        anno = a; 
    } 
    public static void verificaDataUguale(Data data1, Data data2) { 
        if ((data1.giorno == data2.giorno) && (data1.mese == data2.mese) && (data1.anno == data2.anno)) 
            System.out.println("Le date sono uguali"); 
            else System.out.println("Le date sono diverse"); 
    } 
    public static Data getData() { 
        int giorno = Integer.parseInt(JOptionPane.showInputDialog("Inserisci il giorno della  data")); 
        int mese = Integer.parseInt(JOptionPane.showInputDialog("Inserisci il mese della  data")); 
        int anno = Integer.parseInt(JOptionPane.showInputDialog("Inserisci l'anno della  data")); 
        Data data = new Data(giorno, mese, anno); 
        return data;
    } 
    
     
    public static void main(String[] args) { 
        data1 = getData();
        data2 = getData();
        verificaDataUguale(data1, data2); 
        System.exit(0); 
    } 
}

[Molz]

i tuoi return data1 e return data2 nn servono a niente.

alla funzione verificaDataUguale passi le variabili data1 e data2 ke nn sono mai state inizializzate.

In teoria basta ke nel main metti

data1=primaData();
data2=secondaData();

[Alhazred]

Originariamente inviato da cn73
Tralasciando il discorso sulle date che sicuramente non va gestito in questo modo, ma con le apposite classi Date, DateFormat, SimpleDateFormat...

So che esistono classi che gestiscono le date, ma era un esercizio proposto sul libro su cui sto cercando di imparare qualcosa di java.

Grazie

[cn73]

Ho corretto un altro errore che mi era sfuggito...lo stesso discorso della ridefinizione delle variabili lo facevi per gli attributi giorno, mese e anno

[Alhazred]

Originariamente inviato da cn73
...
Comunque il codice non funziona perchè tu ridefinisce Data data1 e Data2 all'interno dei metodi, mentre credo che il tuo scopo è quello di valorizzare le variabili globali data1 e data2 che a questo punto non saranno mai inizializzate e conterranno sempre 0 che è il valore col quale sono inizializzati gli interi.
...

Ho provato il codice che hai scritto nel tuo post, ma mi da lo stesso problema, dice sempre che le date sono uguali. Anche come hai detto tu data1 e data2 contengono sempre 0 nei 3 attributi.

Come si corregge?

[cn73]

Hai letto che ho corretto un altro errore? Il codice scritto funziona...

[Alhazred]

Originariamente inviato da cn73
Hai letto che ho corretto un altro errore? Il codice scritto funziona...

Tutto ok, grazie