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

Fix #15 - variable utilisateureRemote modifiée + inversé le sens des if. Fix...

Fix #15 - variable utilisateureRemote modifiée + inversé le sens des if. Fix #16 - suppression objets inutiles. Fix #17 - mise en commentaire
parent b7893510
......@@ -44,7 +44,7 @@ public final class CommandeCreerCompte implements Commande {
/**
* Le service de gestion d'utilisateurs.
*/
private IUtilisateurServiceRemote utilisateureRemote;
private IUtilisateurServiceRemote utilisateurRemote;
/**
* Le constructeur
......@@ -68,7 +68,7 @@ public final class CommandeCreerCompte implements Commande {
try {
final Context context = new InitialContext(env);
this.utilisateureRemote = (IUtilisateurServiceRemote) context.lookup("UtilisateurService/local");
this.utilisateurRemote = (IUtilisateurServiceRemote) context.lookup("UtilisateurService/local");
} catch (NamingException e) {
e.printStackTrace();
}
......@@ -76,23 +76,12 @@ public final class CommandeCreerCompte implements Commande {
@Override
public void execute() throws RemoteException {
final Utilisateur util = new Utilisateur();
util.setLogin(this.username);
util.setPassword(this.password);
final Participant par = new Participant();
par.setDateDeDerniereModification(new Date());
par.setPseudonyme(this.pseudo);
util.setParticipantAssocie(par);
if (this.utilisateureRemote.getUtilisateurByLogin(this.username) == null) {
this.utilisateureRemote.creerUtilisateur(new Utilisateur());
if (this.utilisateurRemote.getUtilisateurByLogin(this.username) == null) {
this.utilisateurRemote.creerUtilisateur(new Utilisateur());
this.callback.onSucces("Utilisateur enregistre");
} else {
this.callback.onFailure(new Exception("Utilisateur existant"));
}
}
}
......@@ -23,8 +23,8 @@ public final class CommandeCreerContexte extends AbstractCommandeContexte {
* Le constructeur.
*
* @param identification_ L'identifiant de l'utilisateur.
* @param contexte_ Le contexte.
* @param callback_ Le callback.
* @param contexte_ Le contexte.
* @param callback_ Le callback.
*/
CommandeCreerContexte(final String identification_, final IContexte contexte_, final CallBack<IContexte> callback_) {
super(identification_, contexte_);
......@@ -39,10 +39,10 @@ public final class CommandeCreerContexte extends AbstractCommandeContexte {
if ((contexte_ == null) && (user != null) && (this.identification.equalsIgnoreCase(user.getIdentifiantServeur()))) {
this.callback.onSucces(this.contexteRemote.creerContexte(this.contexte));
} else {
if (contexte_ != null) {
this.callback.onFailure(new Exception("Contexte deja existant"));
} else {
if (contexte_ == null) {
this.callback.onFailure(new Exception("Identifiant utilisateur invalide"));
} else {
this.callback.onFailure(new Exception("Contexte deja existant"));
}
}
}
......
......@@ -23,8 +23,8 @@ public final class CommandeCreerIdee extends AbstractCommandeIdee {
* Le constructeur.
*
* @param identification_ L'identifiant de l'utilisateur.
* @param idee_ L'idee a creer.
* @param callback_ Le callback.
* @param idee_ L'idee a creer.
* @param callback_ Le callback.
*/
CommandeCreerIdee(final String identification_, final IIdee idee_, final CallBack<IIdee> callback_) {
super(identification_, idee_);
......@@ -39,10 +39,10 @@ public final class CommandeCreerIdee extends AbstractCommandeIdee {
if ((idee_ == null) && (user != null) && (this.identification.equalsIgnoreCase(this.idee.getCreateur().getIdentifiantServeur()))) {
this.callback.onSucces(this.ideeRemote.creerIdee(this.idee));
} else {
if (idee_ != null) {
this.callback.onFailure(new Exception("Idee deja existante"));
} else {
if (idee_ == null) {
this.callback.onFailure(new Exception("Identifiant utilisateur invalide"));
} else {
this.callback.onFailure(new Exception("Idee deja existante"));
}
}
}
......
......@@ -39,10 +39,10 @@ public final class CommandeCreerTache extends AbstractCommandeTache {
if ((tache_ == null) && (user != null) && (this.identification.equalsIgnoreCase(this.tache.getCreateur().getIdentifiantServeur()))) {
this.callback.onSucces(this.tacheRemote.creerTache(this.tache));
} else {
if (tache_ != null) {
this.callback.onFailure(new Exception("Tache deja existante"));
} else {
if (tache_ == null) {
this.callback.onFailure(new Exception("Identifiant utilisateur invalide"));
} else {
this.callback.onFailure(new Exception("Tache deja existante"));
}
}
}
......
......@@ -23,8 +23,8 @@ public final class CommandeCreerTag extends AbstractCommandeTag {
* Le constructeur.
*
* @param identifiant_ L'identifiant de l'utilisateur.
* @param tag_ Le tag a creer.
* @param callback_ Le callback.
* @param tag_ Le tag a creer.
* @param callback_ Le callback.
*/
CommandeCreerTag(final String identifiant_, final ITag tag_, final CallBack<ITag> callback_) {
super(identifiant_, tag_);
......@@ -39,10 +39,10 @@ public final class CommandeCreerTag extends AbstractCommandeTag {
if ((tag_ == null) && (user != null) && (this.identification.equalsIgnoreCase(this.tag.getCreateur().getIdentifiantServeur()))) {
this.callback.onSucces(this.tagRemote.creerTag(this.tag));
} else {
if (tag_ != null) {
this.callback.onFailure(new Exception("Tag deja existant"));
} else {
if (tag_ == null) {
this.callback.onFailure(new Exception("Identifiant utilisateur invalide"));
} else {
this.callback.onFailure(new Exception("Tag deja existant"));
}
}
}
......
......@@ -31,11 +31,11 @@ public final class CommandeQueue {
/**
* Lance l'execution de la commande.
* @param c La commande a executer.
* @param commande La commande a executer.
*/
public synchronized void executionCommande(final Commande c) {
public synchronized void executionCommande(final Commande commande) {
try {
c.execute();
commande.execute();
} catch (RemoteException e) {
e.printStackTrace();
}
......
......@@ -11,74 +11,77 @@ import java.util.List;
/**
* Commande de telechargement d'un contexte.
*
* @author Stephane Begaudeau, Benjamin Gosset, Alex Lagarde, Christophe Renaudineau.
* @version 1.0.0
*/
public final class CommandeTelechargeContextes implements Commande {
/**
* L'identifiant de l'utilisateur.
*/
private final String identification;
/**
* Le nom d'utilisateur.
*/
private final String username;
/**
* Le callback.
*/
private final CallBack<List<IContexte>> callback;
/**
* Le service de gestion de contexte.
*/
private IContexteServiceRemote contexteServiceRemote;
/**
* Le service de gestion d'utilisateurs.
*/
private IUtilisateurServiceRemote utilisateurServiceRemote;
/**
* Le constructeur.
* @param i L'identifiant de l'utilisateur.
* @param c Le callback.
*/
public CommandeTelechargeContextes(final String i, final CallBack<List<IContexte>> c) {
super();
this.identification = i;
this.callback = c;
this.username = "";
}
/**
* Le constructeur.
* @param i L'identifiant de l'utilisateur.
* @param u Le nom d'utilisateur.
* @param c Le callback.
*/
public CommandeTelechargeContextes(final String i, final String u, final CallBack<List<IContexte>> c) {
super();
this.identification = i;
this.username = u;
this.callback = c;
}
/**
* L'identifiant de l'utilisateur.
*/
private final String identification;
@Override
public void execute() throws RemoteException {
if ("".equalsIgnoreCase(this.username)) {
//TODO
} else {
final Utilisateur uti = this.utilisateurServiceRemote.getUtilisateurById(this.identification);
if (uti != null) {
this.callback.onSucces(this.contexteServiceRemote.getContexteByCreateur(uti.getIdentifiantServeur()));
} else {
this.callback.onFailure(new Exception("Identifiant utilisateur invalide"));
}
}
}
/**
* Le nom d'utilisateur.
*/
private final String username;
/**
* Le callback.
*/
private final CallBack<List<IContexte>> callback;
/**
* Le service de gestion de contexte.
*/
//private IContexteServiceRemote contexteRemote;
/**
* Le service de gestion d'utilisateurs.
*/
//private IUtilisateurServiceRemote utilisateurRemote;
/**
* Le constructeur.
*
* @param identifiant_ L'identifiant de l'utilisateur.
* @param callback_ Le callback.
*/
public CommandeTelechargeContextes(final String identifiant_, final CallBack<List<IContexte>> callback_) {
super();
this.identification = identifiant_;
this.callback = callback_;
this.username = "";
}
/**
* Le constructeur.
*
* @param identifiant_ L'identifiant de l'utilisateur.
* @param username_ Le nom d'utilisateur.
* @param callback_ Le callback.
*/
public CommandeTelechargeContextes(final String identifiant_, final String username_, final CallBack<List<IContexte>> callback_) {
super();
this.identification = identifiant_;
this.username = username_;
this.callback = callback_;
}
@Override
public void execute() throws RemoteException {
/*if ("".equalsIgnoreCase(this.username)) {
//TODO
} else {
final Utilisateur uti = this.utilisateurRemote.getUtilisateurById(this.identification);
if (uti == null) {
this.callback.onFailure(new Exception("Identifiant utilisateur invalide"));
} else {
this.callback.onSucces(this.contexteRemote.getContexteByCreateur(uti.getIdentifiantServeur()));
}
}*/
}
}
#Thu Apr 12 22:33:42 CEST 2018
#Thu Apr 12 23:35:51 CEST 2018
/home/glenn/Documents/Fac/L3/S6/Construction_evolution_logiciel/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/reactor/ReactorCorba.java=1523562871000
configuration*?=F196B0FC5565D6D6DD4397B41111265185CEE902
This source diff could not be displayed because it is too large. You can view the blob instead.
<?xml version="1.0"?>
<javancss>
<date>2018-04-12</date>
<time>22:33:40</time>
<time>23:35:48</time>
<packages>
<package>
<name>fr.alma.gtd.commande</name>
<classes>43</classes>
<functions>149</functions>
<ncss>838</ncss>
<ncss>822</ncss>
<javadocs>98</javadocs>
<javadoc_lines>596</javadoc_lines>
<single_comment_lines>1</single_comment_lines>
<multi_comment_lines>141</multi_comment_lines>
<javadoc_lines>593</javadoc_lines>
<single_comment_lines>2</single_comment_lines>
<multi_comment_lines>158</multi_comment_lines>
</package>
<package>
<name>fr.alma.gtd.donneespartagees</name>
......@@ -86,18 +86,18 @@
<total>
<classes>94</classes>
<functions>755</functions>
<ncss>2857</ncss>
<ncss>2841</ncss>
<javadocs>401</javadocs>
<javadoc_lines>2286</javadoc_lines>
<single_comment_lines>25</single_comment_lines>
<multi_comment_lines>469</multi_comment_lines>
<javadoc_lines>2283</javadoc_lines>
<single_comment_lines>26</single_comment_lines>
<multi_comment_lines>486</multi_comment_lines>
</total>
<table>
<tr><td>Packages</td><td>Classes</td><td>Functions</td><td>NCSS</td><td>Javadocs</td><td>per</td></tr>
<tr><td>8.00</td><td>94.00</td><td>755.00</td><td>2,857.00</td><td>401.00</td><td>Project</td></tr>
<tr><td></td><td>11.75</td><td>94.38</td><td>357.13</td><td>50.13</td><td>Package</td></tr>
<tr><td></td><td></td><td>8.03</td><td>30.39</td><td>4.27</td><td>Class</td></tr>
<tr><td></td><td></td><td></td><td>3.78</td><td>0.53</td><td>Function</td></tr>
<tr><td>8.00</td><td>94.00</td><td>755.00</td><td>2,841.00</td><td>401.00</td><td>Project</td></tr>
<tr><td></td><td>11.75</td><td>94.38</td><td>355.13</td><td>50.13</td><td>Package</td></tr>
<tr><td></td><td></td><td>8.03</td><td>30.22</td><td>4.27</td><td>Class</td></tr>
<tr><td></td><td></td><td></td><td>3.76</td><td>0.53</td><td>Function</td></tr>
</table>
</packages>
......@@ -624,7 +624,7 @@
</object>
<object>
<name>fr.alma.gtd.commande.CommandeCreerCompte</name>
<ncss>33</ncss>
<ncss>26</ncss>
<functions>2</functions>
<classes>0</classes>
<javadocs>2</javadocs>
......@@ -632,16 +632,6 @@
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>12</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeTache</name>
<ncss>18</ncss>
<functions>2</functions>
<classes>0</classes>
<javadocs>1</javadocs>
<javadocs_lines>6</javadocs_lines>
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>12</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeCreerIdee</name>
<ncss>15</ncss>
......@@ -964,13 +954,23 @@
</object>
<object>
<name>fr.alma.gtd.commande.CommandeTelechargeContextes</name>
<ncss>24</ncss>
<ncss>15</ncss>
<functions>3</functions>
<classes>0</classes>
<javadocs>3</javadocs>
<javadocs_lines>22</javadocs_lines>
<single_comment_lines>1</single_comment_lines>
<implementation_comment_lines>9</implementation_comment_lines>
<javadocs_lines>19</javadocs_lines>
<single_comment_lines>2</single_comment_lines>
<implementation_comment_lines>26</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.AbstractCommandeTache</name>
<ncss>18</ncss>
<functions>2</functions>
<classes>0</classes>
<javadocs>1</javadocs>
<javadocs_lines>6</javadocs_lines>
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>12</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeTelechargeProchainesTachesParContexte</name>
......@@ -1043,15 +1043,15 @@
<implementation_comment_lines>0</implementation_comment_lines>
</object>
<averages>
<ncss>24.29</ncss>
<ncss>24.12</ncss>
<functions>8.03</functions>
<classes>0.00</classes>
<javadocs>4.27</javadocs>
<javadocs_lines>24.32</javadocs_lines>
<single_comment_lines>0.26</single_comment_lines>
<implementation_comment_lines>4.78</implementation_comment_lines>
<javadocs_lines>24.29</javadocs_lines>
<single_comment_lines>0.27</single_comment_lines>
<implementation_comment_lines>4.96</implementation_comment_lines>
</averages>
<ncss>2,857.00</ncss>
<ncss>2,841.00</ncss>
</objects>
<functions>
......@@ -4705,22 +4705,10 @@
</function>
<function>
<name>fr.alma.gtd.commande.CommandeCreerCompte.execute()</name>
<ncss>13</ncss>
<ncss>6</ncss>
<ccn>2</ccn>
<javadocs>0</javadocs>
</function>
<function>
<name>fr.alma.gtd.commande.CommandeTache.CommandeTache(String,ITache)</name>
<ncss>12</ncss>
<ccn>2</ccn>
<javadocs>1</javadocs>
</function>
<function>
<name>fr.alma.gtd.commande.CommandeTache.execute()</name>
<ncss>1</ncss>
<ccn>1</ccn>
<javadocs>0</javadocs>
</function>
<function>
<name>fr.alma.gtd.commande.CommandeCreerIdee.CommandeCreerIdee(String,IIdee,IIdee)</name>
<ncss>3</ncss>
......@@ -5467,8 +5455,20 @@
</function>
<function>
<name>fr.alma.gtd.commande.CommandeTelechargeContextes.execute()</name>
<ncss>8</ncss>
<ccn>3</ccn>
<ncss>1</ncss>
<ccn>1</ccn>
<javadocs>0</javadocs>
</function>
<function>
<name>fr.alma.gtd.commande.AbstractCommandeTache.AbstractCommandeTache(String,ITache)</name>
<ncss>12</ncss>
<ccn>2</ccn>
<javadocs>1</javadocs>
</function>
<function>
<name>fr.alma.gtd.commande.AbstractCommandeTache.execute()</name>
<ncss>1</ncss>
<ccn>1</ccn>
<javadocs>0</javadocs>
</function>
<function>
......@@ -5586,10 +5586,10 @@
<javadocs>0</javadocs>
</function>
<function_averages>
<ncss>2.68</ncss>
<ccn>1.22</ccn>
<ncss>2.66</ncss>
<ccn>1.21</ccn>
<javadocs>0.45</javadocs>
</function_averages>
<ncss>2,857.00</ncss>
<ncss>2,841.00</ncss>
</functions>
</javancss>
......@@ -25,15 +25,15 @@
<Class sourceFile="AbstractCommandeProjet.java">
fr.alma.gtd.commande.AbstractCommandeProjet
</Class>
<Class sourceFile="AbstractCommandeTache.java">
fr.alma.gtd.commande.AbstractCommandeTache
</Class>
<Class sourceFile="AbstractCommandeTag.java">
fr.alma.gtd.commande.AbstractCommandeTag
</Class>
<Class sourceFile="Commande.java">
fr.alma.gtd.commande.Commande
</Class>
<Class sourceFile="CommandeTache.java">
fr.alma.gtd.commande.CommandeTache
</Class>
</AbstractClasses>
<ConcreteClasses>
......
fr/alma/gtd/commande/AbstractCommandeIdee.class
fr/alma/gtd/commande/CommandeTache.class
fr/alma/gtd/commande/AbstractCommandeContexte.class
fr/alma/gtd/commande/AbstractCommandeTache.class
fr/alma/gtd/commande/AbstractCommandeTag.class
......@@ -81,7 +81,6 @@
/home/glenn/Documents/Fac/L3/S6/Construction_evolution_logiciel/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/services/ContexteService.java
/home/glenn/Documents/Fac/L3/S6/Construction_evolution_logiciel/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/donneesserveur/Participant.java
/home/glenn/Documents/Fac/L3/S6/Construction_evolution_logiciel/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/donneespartagees/AbstractTache.java
/home/glenn/Documents/Fac/L3/S6/Construction_evolution_logiciel/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/CommandeTache.java
/home/glenn/Documents/Fac/L3/S6/Construction_evolution_logiciel/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/donneesserveur/Contexte.java
/home/glenn/Documents/Fac/L3/S6/Construction_evolution_logiciel/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/AbstractCommandeContexte.java
/home/glenn/Documents/Fac/L3/S6/Construction_evolution_logiciel/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/services/ProjetService.java
......@@ -92,3 +91,4 @@
/home/glenn/Documents/Fac/L3/S6/Construction_evolution_logiciel/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/CommandeCreerProjet.java
/home/glenn/Documents/Fac/L3/S6/Construction_evolution_logiciel/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/CommandeSupprimerTache.java
/home/glenn/Documents/Fac/L3/S6/Construction_evolution_logiciel/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/CommandeTelechargeContextes.java
/home/glenn/Documents/Fac/L3/S6/Construction_evolution_logiciel/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/AbstractCommandeTache.java
This source diff could not be displayed because it is too large. You can view the blob instead.
This diff is collapsed.
......@@ -27,7 +27,7 @@ under the License.
<item>
<title>File: 93,
Errors: 0,
Warnings: 9154,
Warnings: 9072,
Infos: 0
</title>
<link>http://alma.univ-nantes.fr/GTDServer/GTDServerJBoss//checkstyle.html</link>
......@@ -108,7 +108,7 @@ under the License.
0
</td>
<td>
38
37
</td>
<td>
0
......@@ -164,7 +164,7 @@ under the License.
0
</td>
<td>
22
23
</td>
<td>
0
......@@ -415,20 +415,6 @@ under the License.
<td>
0
</td>
<td>
23
</td>
<td>
0
</td>
</tr>
<tr>
<td>
<a href="http://alma.univ-nantes.fr/GTDServer/GTDServerJBoss//checkstyle.html#fr.alma.gtd.commande.CommandeTache.java">fr/alma/gtd/commande/CommandeTache.java</a>
</td>
<td>
0
</td>
<td>
24
</td>
......@@ -780,7 +766,7 @@ under the License.
0
</td>
<td>
48
47
</td>
<td>
0
......@@ -808,7 +794,7 @@ under the License.
0
</td>
<td>
98
23
</td>
<td>
0
......@@ -934,7 +920,21 @@ under the License.
0
</td>
<td>
43