Comment exprimer "jamais" avec java.util.Date?

Question

J'ai une classe avec les champs "deletionDate" et "experiationDate" qui pourraient tous deux être indéfinis, ce qui signifierait que l'objet est supprimé et n'a pas de date d'expiration.

Ma première approche était:

private Date deletionDate = null; // null means not deleted 

Avoir le livre « code propre » à l'esprit, je me souviens de mieux utiliser les noms d'expression au lieu des commentaires. Donc, mes solutions actuelles sont:

private static final Date NEVER = null; 
private Date deletionDate = NEVER; 

Je pourrais utiliser une classe wrapper autour de Date, mais cela compliquerait le mappage JPA.

Qu'en pensez-vous? Comment voulez-vous exprimer "jamais"?

Answer 1

bien n'est jamais, pas le 1/1/2999.

Je resterais avec votre 1ère solution. Une date nulle signifie que cela ne s'est pas encore produit.

peut-être vous pouvez l'envelopper avec quelque chose comme:

boolean isNeverDeleted(){ 
    return deletionDate == null; 
} 

Answer 2

Je choisirais simplement une date de futur lointain comme valeur pour la constante JAMAIS. Ensuite, pour vérifier la suppression/expiration, il suffit de comparer contre JAMAIS.

Answer 3

Laissez la valeur par défaut être traité comme « jamais »

Answer 4

Vous pouvez penser à la date nulle comme « non disponible » ou « non applicable ». Si c'est le cas "AUCUNE DATE" ne convient pour "jamais".

Ne pas sous-type Date uniquement pour une exigence de style très exquis.

Une meilleure option consiste à ajouter de la sémantique à votre objet de modèle. Si votre avez un objet chose avec une propriété deletionDate que vous pouvez faire:

class Thing 
+ deletionDate 
+ isNeverDeleted: boolean { return deletionDate == null; } 

et il sera pratique et documentative, aussi bien dans la classe et dans votre code client:

if(myThing.isNeverDeleted()) 

Answer 5

Je considère null approprié . Il indique clairement "non défini". En fonction de la complexité que vous souhaitez obtenir, cependant, vous pourriez avoir un Enum et avoir un état comme 'NeverExpires' comme 'UserState' (ou quoi que ce soit que vous représentez). Ceci est probablement préférable, mais pourrait être inutilement complexe, en fonction de ce que votre système.

Answer 6

Je n'utiliserais pas Date mais les horodatages, en utilisant -1 pour jamais et 0 pour immédiatement;

public static final long IMMEDIATE = 0; 
public static final long NEVER = -1L; 
private long expires = NEVER; 

interprétation de l'attribut doit être dans un getter, comme:

public boolean isExpired() { 

    return (NEVER == expires) ? false : (expires < System.currentTimeMillies()); 
} 

Cette suppression est le même schéma.

Mise à jour Je sais que 0 et -1 sont valides horodatages, mais que l'expiration et la suppression de fichiers et d'autres ressources rarement (jamais dire jamais :-)) se produire en 1970 ou avant, il est une constante utile, IMHO .