Commit 83668522 authored by Glenn PLOUHINEC's avatar Glenn PLOUHINEC
Browse files

Fixes #26, #27, #28 - Code smells

parent eb4cd667
......@@ -102,10 +102,11 @@ public abstract class AbstractTache extends AbstractObjetServeur implements ITac
*/
protected List<ITache> tachesAnterieures;
/**
* Initialisation des variables.
*/ {
* Le constructeur par defaut.
*/
protected AbstractTache() {
super();
this.dansLaPoubelle = false;
this.dansArchive = false;
this.priorite = 0;
......@@ -120,59 +121,64 @@ public abstract class AbstractTache extends AbstractObjetServeur implements ITac
this.contacts = new ArrayList<IContact>();
}
/**
* Le constructeur par defaut.
*/
public AbstractTache() {
super();
}
/**
* Le constructeur avec initialisation du nom, de la priorite, du taux d'effort, du contexte et du createur.
*
* @param n Le nom.
* @param p La priorite.
* @param effort Le taux d'effort.
* @param c Le contexte.
* @param cr Le createur de la tache.
* @param nom_ Le nom.
* @param priorite_ La priorite.
* @param effort Le taux d'effort.
* @param contexte_ Le contexte.
* @param createur_ Le createur de la tache.
*/
public AbstractTache(final String n, final int p, final int effort, final IContexte c, final IParticipant cr) {
this.nom = n;
this.priorite = p;
protected AbstractTache(final String nom_, final int priorite_, final int effort, final IContexte contexte_, final IParticipant createur_) {
this.nom = nom_;
this.priorite = priorite_;
this.tauxEffort = effort;
this.contexte = c;
this.createur = cr;
this.contexte = contexte_;
this.createur = createur_;
this.dansLaPoubelle = false;
this.dansArchive = false;
this.priorite = 0;
this.tauxEffort = 0;
this.dateDebut = new Date();
this.dateFin = new Date();
this.dansLaPoubelle = false;
this.dateDerniereModif = new Date();
this.listeDesTags = new ArrayList<ITag>();
this.listeDesURLs = new ArrayList<String>();
this.tachesAnterieures = new ArrayList<ITache>();
this.contacts = new ArrayList<IContact>();
}
/**
* Constructeur de recopie d'une tache.
*
* @param t Tache a recopier
* @param tache_ Tache a recopier
*/
public AbstractTache(final ITache t) {
this.copier(t);
protected AbstractTache(final ITache tache_) {
this.copier(tache_);
}
/**
* Methode de copie.
*
* @param t La tache a copier.
* @param tache_ La tache a copier.
*/
public final void copier(final ITache t) {
this.nom = t.getNom();
this.priorite = t.getPriorite();
this.tauxEffort = t.getTauxEffort();
this.dateDebut = t.getDateDebut();
this.dateFin = t.getDateFin();
this.dansLaPoubelle = t.isDansLaPoubelle();
this.identifiantServeur = t.getIdentifiantServeur();
this.dateDerniereModif = t.getDateDerniereModif();
this.createur = t.getCreateur();
this.participant = t.getParticipant();
this.contexte = t.getContexte();
this.listeDesTags = t.getListeDeTags();
this.listeDesURLs = t.getListeDesURLs();
this.projetConteneur = t.getProjetConteneur();
public final void copier(final ITache tache_) {
this.nom = tache_.getNom();
this.priorite = tache_.getPriorite();
this.tauxEffort = tache_.getTauxEffort();
this.dateDebut = tache_.getDateDebut();
this.dateFin = tache_.getDateFin();
this.dansLaPoubelle = tache_.isDansLaPoubelle();
this.identifiantServeur = tache_.getIdentifiantServeur();
this.dateDerniereModif = tache_.getDateDerniereModif();
this.createur = tache_.getCreateur();
this.participant = tache_.getParticipant();
this.contexte = tache_.getContexte();
this.listeDesTags = tache_.getListeDeTags();
this.listeDesURLs = tache_.getListeDesURLs();
this.projetConteneur = tache_.getProjetConteneur();
}
@Override
......@@ -194,16 +200,16 @@ public abstract class AbstractTache extends AbstractObjetServeur implements ITac
public abstract String getNom();
@Override
public final void setNom(final String n) {
this.nom = n;
public final void setNom(final String nom_) {
this.nom = nom_;
}
@Override
public abstract int getPriorite();
@Override
public final void setPriorite(final int p) {
this.priorite = p;
public final void setPriorite(final int priorite_) {
this.priorite = priorite_;
}
@Override
......@@ -215,13 +221,13 @@ public abstract class AbstractTache extends AbstractObjetServeur implements ITac
}
@Override
public final void setAvancement(final Avancement a) {
this.avancement = a;
public final void setAvancement(final Avancement avancement_) {
this.avancement = avancement_;
}
@Override
public final void setFrequence(final Frequence f) {
this.frequence = f;
public final void setFrequence(final Frequence frequence_) {
this.frequence = frequence_;
}
@Override
......@@ -245,18 +251,18 @@ public abstract class AbstractTache extends AbstractObjetServeur implements ITac
}
@Override
public final void setCreateur(final IParticipant c) {
this.createur = c;
public final void setCreateur(final IParticipant createur_) {
this.createur = createur_;
}
@Override
public final void setParticipant(final IParticipant p) {
this.participant = p;
public final void setParticipant(final IParticipant participant_) {
this.participant = participant_;
}
@Override
public final void setContexte(final IContexte c) {
this.contexte = c;
public final void setContexte(final IContexte contexte_) {
this.contexte = contexte_;
}
@Override
......
......@@ -49,11 +49,11 @@ public abstract class AbstractTag implements ITag {
/**
* Constructeur d'un Tag (a partir du nom correspondant au Tag et du participant l'ayant cree).
* @param name Nom correspondant au Tag
* @param p Participant ayant creer la tache
* @param createur_ Participant ayant creer la tache
*/
public AbstractTag(final String name, final IParticipant p) {
public AbstractTag(final String name, final IParticipant createur_) {
this.nom = name;
this.createur = p;
this.createur = createur_;
}
/**
......@@ -64,7 +64,7 @@ public abstract class AbstractTag implements ITag {
this.copier(tag);
}
public void copier(final ITag tag){
public final void copier(final ITag tag){
this.nom = tag.getNom();
this.identifiantServeur = tag.getIdentifiantServeur();
this.createur = tag.getCreateur();
......@@ -72,8 +72,8 @@ public abstract class AbstractTag implements ITag {
}
@Override
public final void setNom(final String n) {
this.nom = n;
public final void setNom(final String nom_) {
this.nom = nom_;
}
@Override
......@@ -82,8 +82,8 @@ public abstract class AbstractTag implements ITag {
}
@Override
public final void setCreateur(final IParticipant c) {
this.createur = c;
public final void setCreateur(final IParticipant createur_) {
this.createur = createur_;
}
@Override
......
......@@ -13,9 +13,9 @@ public interface IContexte extends IObjetServeur {
String getNom();
/**
* @param n La nouvelle valeur du nom.
* @param nom_ La nouvelle valeur du nom.
*/
void setNom(final String n);
void setNom(final String nom_);
/**
* @return Le createur de l'idee.
......@@ -23,8 +23,8 @@ public interface IContexte extends IObjetServeur {
IParticipant getCreateur();
/**
* @param c La nouvelle valeur du createur.
* @param createur_ La nouvelle valeur du createur.
*/
void setCreateur(final IParticipant c);
void setCreateur(final IParticipant createur_);
}
......@@ -13,9 +13,9 @@ public interface IIdee extends IObjetServeur {
String getNom();
/**
* @param n La nouvelle valeur du nom.
* @param nom_ La nouvelle valeur du nom.
*/
void setNom(final String n);
void setNom(final String nom_);
/**
* @return La description.
......@@ -23,9 +23,9 @@ public interface IIdee extends IObjetServeur {
String getDescription();
/**
* @param desc La nouvelle valeur de la description.
* @param description_ La nouvelle valeur de la description.
*/
void setDescription(final String desc);
void setDescription(final String description_);
/**
* Supprime l'idee, en la mettant a la poubelle.
......@@ -45,7 +45,7 @@ public interface IIdee extends IObjetServeur {
/**
* @param estDansPoubelle La nouvelle valeur du boolean indiquant si l'idee est dans la poubelle.
*/
void setDansLaPoubelle(final boolean estDansArchive);
void setDansLaPoubelle(final boolean estDansPoubelle);
/**
* @return Le boolean indiquant si l'idee a ete archivee.
......@@ -53,7 +53,7 @@ public interface IIdee extends IObjetServeur {
boolean isDansArchive();
/**
* @param estDansPoubelle La nouvelle valeur du boolean indiquant si l'idee a ete archivee.
* @param estDansArchive La nouvelle valeur du boolean indiquant si l'idee a ete archivee.
*/
void setDansArchive(final boolean estDansArchive);
......@@ -63,8 +63,8 @@ public interface IIdee extends IObjetServeur {
IParticipant getCreateur();
/**
* @param c La nouvelle valeur du createur.
* @param createur_ La nouvelle valeur du createur.
*/
void setCreateur(final IParticipant c);
void setCreateur(final IParticipant createur_);
}
......@@ -30,9 +30,9 @@ public interface ITache extends IObjetServeur {
String getNom();
/**
* @param n La nouvelle valeur du nom.
* @param nom_ La nouvelle valeur du nom.
*/
void setNom(final String n);
void setNom(final String nom_);
/**
* @return La priorite.
......@@ -40,9 +40,9 @@ public interface ITache extends IObjetServeur {
int getPriorite();
/**
* @param p La nouvelle valeur de la priorite.
* @param priorite_ La nouvelle valeur de la priorite.
*/
void setPriorite(final int p);
void setPriorite(final int priorite_);
/**
* @return Le tauxEffort.
......@@ -60,9 +60,9 @@ public interface ITache extends IObjetServeur {
Avancement getAvancement();
/**
* @param a La nouvelle valeur de l'avancement.
* @param avancement_ La nouvelle valeur de l'avancement.
*/
void setAvancement(final Avancement a);
void setAvancement(final Avancement avancement_);
/**
* @return La frequence.
......@@ -70,9 +70,9 @@ public interface ITache extends IObjetServeur {
Frequence getFrequence();
/**
* @param f La nouvelle valeur de la frequence.
* @param frequence_ La nouvelle valeur de la frequence.
*/
void setFrequence(final Frequence f);
void setFrequence(final Frequence frequence_);
/**
* @return La dateDebut.
......@@ -121,9 +121,9 @@ public interface ITache extends IObjetServeur {
IParticipant getCreateur();
/**
* @param c La nouvelle valeur du createur.
* @param createur_ La nouvelle valeur du createur.
*/
void setCreateur(final IParticipant c);
void setCreateur(final IParticipant createur_);
/**
* @return Le participant.
......@@ -131,9 +131,9 @@ public interface ITache extends IObjetServeur {
IParticipant getParticipant();
/**
* @param p La nouvelle valeur du participant.
* @param participant_ La nouvelle valeur du participant.
*/
void setParticipant(final IParticipant p);
void setParticipant(final IParticipant participant_);
/**
* @return Le contexte.
......@@ -141,9 +141,9 @@ public interface ITache extends IObjetServeur {
IContexte getContexte();
/**
* @param c La nouvelle valeur du contexte.
* @param contexte_ La nouvelle valeur du contexte.
*/
void setContexte(final IContexte c);
void setContexte(final IContexte contexte_);
/**
* @return La listeDesTags.
......
......@@ -13,9 +13,9 @@ public interface ITag extends IObjetServeur {
String getNom();
/**
* @param n La nouvelle valeur du nom.
* @param nom_ La nouvelle valeur du nom.
*/
void setNom(final String n);
void setNom(final String nom_);
/**
* @return Le createur de l'idee.
......@@ -23,8 +23,8 @@ public interface ITag extends IObjetServeur {
IParticipant getCreateur();
/**
* @param c La nouvelle valeur du createur.
* @param createur_ La nouvelle valeur du createur.
*/
void setCreateur(final IParticipant c);
void setCreateur(final IParticipant createur_);
}
......@@ -16,66 +16,69 @@ import fr.alma.gtd.donneespartagees.IContexte;
/**
* Bean Entite representant un contexte et charge de gerer sa persistance.
*
* @author Stephane Begaudeau, Benjamin Gosset, Alex Lagarde, Christophe Renaudineau.
* @version 1.0.0
*/
@Entity
@Table(name = "CONTEXTE")
public final class Contexte extends AbstractContexte {
/**
* L'ID.
*/
private static final long serialVersionUID = -8938444349832798401L;
/**
* Le constructeur par defaut.
*/
public Contexte() {
super();
}
/**
* Le constructeur initialisant le nom.
* @param n Le nom choisi.
*/
public Contexte(final String n) {
super(n);
}
/**
* Constructeur de recopie d'un Participant.
* @param ctx Participant a recopier.
*/
public Contexte(final IContexte ctx) {
super(ctx);
}
@Id
@GeneratedValue(generator = "system-uuid")
@GenericGenerator(name = "system-uuid", strategy = "uuid")
@Override
public String getIdentifiantServeur() {
return this.identifiantServeur;
}
@Override
@ManyToOne
public Participant getCreateur() {
return (Participant) this.createur;
}
@Override
@Basic
public Date getDateDerniereModif() {
return new Date(dateDerniereModif.getTime());
}
@Override
@Basic
public String getNom() {
return nom;
}
/**
* L'ID.
*/
private static final long serialVersionUID = -8938444349832798401L;
/**
* Le constructeur par defaut.
*/
public Contexte() {
super();
}
/**
* Le constructeur initialisant le nom.
*
* @param nom_ Le nom choisi.
*/
public Contexte(final String nom_) {
super(nom_);
}
/**
* Constructeur de recopie d'un Participant.
*
* @param contexte_ Participant a recopier.
*/
public Contexte(final IContexte contexte_) {
super(contexte_);
}
@Id
@GeneratedValue(generator = "system-uuid")
@GenericGenerator(name = "system-uuid", strategy = "uuid")
@Override
public String getIdentifiantServeur() {
return this.identifiantServeur;
}
@Override
@ManyToOne
public Participant getCreateur() {
return (Participant) this.createur;
}
@Override
@Basic
public Date getDateDerniereModif() {
return new Date(dateDerniereModif.getTime());
}
@Override
@Basic
public String getNom() {
return nom;
}
}
......@@ -17,97 +17,100 @@ import fr.alma.gtd.donneespartagees.IParticipant;
/**
* Bean Entite representant une Idee et charge de gerer sa persistance.
*
* @author Stephane Begaudeau, Benjamin Gosset, Alex Lagarde, Christophe Renaudineau.
* @version 1.0.0
*/
@Entity
@Table(name = "IDEE")
public final class Idee extends AbstractIdee {
/**
* L'ID.
*/
private static final long serialVersionUID = -2348892725312472280L;
/**
* Le constructeur par defaut (utilise par Hibernate).
*/
public Idee() {
super();
}
/**
* Le constructeur permettant l'initialisation du nom et de la description.
* @param n Le nom choisi.
* @param desc Le description choisie.
* @param c Le createur de l'idee.
*/
public Idee(final String n, final String desc, final IParticipant c) {
super(n, desc, c);
}
/**
* Constructeur de recopie d'une idee.
* @param idee Idee a recopier.
*/
public Idee(final IIdee idee) {
super(idee);
}
@Id
@GeneratedValue(generator = "system-uuid")
@GenericGenerator(name = "system-uuid", strategy = "uuid")
@Override
public String getIdentifiantServeur() {
return identifiantServeur;
}
@ManyToOne
@Override
public Participant getCreateur() {
return (Participant) this.createur;
}
@Basic
@Override
public Date getDateDerniereModif() {
return new Date(dateDerniereModif.getTime());
}
@Basic
@Override