[JAVA] Calcolo della maggiore età partendo dalla data di nascita

[Angroth]

Ciao a tutti, per un esame all'università devo portare un progetto per la gestione di un cinema.
Il progetto è composto da varie classi, una delle quali è la classe Spettatore che ha come campi idspettatore,nome,cognome,datanascita e biglietto (quest'ultima è un altra classe del progetto).
Il mio problema arriva quando devo implementare un metodo boolean, chiamato maggiorenne(), che restituisca true in caso che lo spettatore risulti maggiorenne o altrimenti false se minorenne.
Vi allego il codice della classe e dell'ambiente di test:

Spettatore.java

import java.util.*;

public class Spettatore {
	protected int idspettatore;
	protected String nome;
	protected String cognome;
	protected GregorianCalendar datanascita;
	protected Biglietto biglietto;
	public Spettatore(int idspettatore,String nome,String cognome,GregorianCalendar datanascita,Biglietto biglietto){
		this.idspettatore=idspettatore;
		this.nome=nome;
		this.cognome=cognome;
		this.datanascita=datanascita;
		this.biglietto=biglietto;
	}
	
	public boolean maggiorenne(){
		boolean mag=false;
		GregorianCalendar now=new GregorianCalendar();
		long diff= now.getTimeInMillis()-datanascita.getTimeInMillis();
		int anni=(int) diff/(1000*60*60*24*30*12);
		int mesi=(int) diff/(1000*60*60*24*30);
		int giorni=(int) diff/(1000*60*60*24);
		if (anni>18)
			mag=true;
		if((anni==18)&&(mesi>=0)&&(giorni>=0))
			mag=true;
		return mag;
	}
}

Prova.java:

import java.util.*;


public class Prova {
public static void main(String[] args) {
	int giorno=8;
	int mese=5;
	int anno=1992;
	Biglietto biglietto=new Biglietto("a1",7.5);
	GregorianCalendar datan=new GregorianCalendar(anno,mese,giorno);
	Spettatore a=new Spettatore(1,"Pinco","Pallino",datan,biglietto);
	System.out.println(datan.get(Calendar.DATE)+"-"+datan.get(Calendar.MONTH)+"-"+datan.get(Calendar.YEAR));
	boolean c=a.maggiorenne();
	if(c==true)
		System.out.println("Sei maggiorenne!");
	else if(c==false)
		System.out.println("Sei minorenne!");
}
}

Il problema è che in output ricevo sempre il messaggio "Sei minorenne!".
Non riesco a capire cosa non va (sicuramente non ho ben capito l'uso del gregoriancalendar)

[sottovento]

E' chiaro che hai approssimato un anno come 12 mesi da 30 giorni l'uno.
Il problema non e' banalissimo, soprattutto considerando che non vuoi calcolare esattamente l'eta' della persona, ma solo se e' maggiorenne.

A questo punto, perche' non "giri" l'algoritmo?
Hai la data di nascita dello spettatore, i.e. giorno/mese/anno.
Potresti calcolare il compleanno della maggiore eta' molto facilmente, i.e. giorno/mese/anno+18. Questa e' la data in cui lo spettatore e' diventato/diventera' maggiorenne.

Ti basta ora fare il confronto con la data attuale (usando ancora la getTimeInMillis(), mi sembra una buona idea)

[Angroth]

Seguendo il tuo consiglio ho sistemato cosi il codice:

	
public boolean maggiorenne(){
		boolean mag=false;
		GregorianCalendar now=new GregorianCalendar();
		int giorno=datanascita.get(Calendar.DATE);
		int mese=datanascita.get(Calendar.MONTH);
		int anno=datanascita.get(Calendar.YEAR)+18;
		GregorianCalendar mageta= new GregorianCalendar(anno,mese,giorno);
		long dif=now.getTimeInMillis()-mageta.getTimeInMillis();
		int diff=(int) dif;
		if(diff>=0)
			mag=true;
		return mag;

Ora funziona quasi perfettamente, l'unica pecca è che se, ad esempio setto la data di nascita a Febbraio 1995 qualsiasi giorno metto non fa differenza ma mi da sempre come esito maggiorenne, già se setto al mese di Marzo 1995 mi da correttamente che sono minorenne.

[sottovento]

Originariamente inviato da Angroth

	
		int giorno=datanascita.get(Calendar.DATE);

Penso che il problema sia quella linea. Il giorno si dovrebbe ottenere come

     int giorno = datanascita.get(Calendar.DAY_OF_MONTH);

[Angroth]

Umn il problema sussiste anche così , cmq grazie mille, almeno ora non sgarra di un anno

[sottovento]

Originariamente inviato da Angroth

Umn il problema sussiste anche così , cmq grazie mille, almeno ora non sgarra di un anno

Si, hai ragione:

int mese=datanascita.get(Calendar.MONTH);

mese sara' compreso fra 0 e 11, NON fra 1 e 12!!!
Quindi, se non ne hai tenuto conto all'inserimento della data di nascita avrai ancora lo scarto di un mese

[sottovento]

L'ho provata (modificandola leggermente) e funziona correttamente:

    public static boolean maggiorenne(GregorianCalendar datanascita)
    {
		boolean mag=false;

		GregorianCalendar now=new GregorianCalendar();

		int giorno=datanascita.get(Calendar.DAY_OF_MONTH);
		int mese=datanascita.get(Calendar.MONTH);
		int anno=datanascita.get(Calendar.YEAR)+18;
		GregorianCalendar mageta= new GregorianCalendar(anno,mese,giorno,0,0,0);
        mageta.set(Calendar.MILLISECOND, 0);
		long dif=now.getTimeInMillis()-mageta.getTimeInMillis();

        return dif > 0;

    }

[Angroth]

Originariamente inviato da sottovento

L'ho provata (modificandola leggermente) e funziona correttamente:

    public static boolean maggiorenne(GregorianCalendar datanascita)
    {
		boolean mag=false;

		GregorianCalendar now=new GregorianCalendar();

		int giorno=datanascita.get(Calendar.DAY_OF_MONTH);
		int mese=datanascita.get(Calendar.MONTH);
		int anno=datanascita.get(Calendar.YEAR)+18;
		GregorianCalendar mageta= new GregorianCalendar(anno,mese,giorno,0,0,0);
        mageta.set(Calendar.MILLISECOND, 0);
		long dif=now.getTimeInMillis()-mageta.getTimeInMillis();

        return dif > 0;

    }

Ti ringrazio, mi hai salvato la vita xD!