Commit 46e8ec8d authored by Glenn PLOUHINEC's avatar Glenn PLOUHINEC
Browse files

Fix #6 - AJout d'une classe abstraite CommandeTag pour supprimer la redondance

parent e0fdb378
......@@ -3,72 +3,32 @@ package fr.alma.gtd.commande;
import fr.alma.gtd.donneespartagees.ITag;
import fr.alma.gtd.donneesserveur.Utilisateur;
import fr.alma.gtd.interfacedistante.CallBack;
import fr.alma.gtd.isessions.ITagServiceRemote;
import fr.alma.gtd.isessions.IUtilisateurServiceRemote;
import java.rmi.RemoteException;
import java.util.Properties;
import javax.naming.Context;
import javax.naming.InitialContext;
import javax.naming.NamingException;
/**
* Commande de creation d'un tag.
*
* @author Stephane Begaudeau, Benjamin Gosset, Alex Lagarde, Christophe Renaudineau.
* @version 1.0.0
*/
public final class CommandeCreerTag implements Commande {
/**
* L'identifiant de l'utilisateur.
*/
private final String identification;
/**
* Le tag a creer.
*/
private final ITag tag;
public final class CommandeCreerTag extends CommandeTag {
/**
* Le callback.
*/
private final CallBack<ITag> callback;
/**
* Le service de gestion de tags.
*/
private ITagServiceRemote tagServiceRemote;
/**
* Le service de gestion d'utilisateurs.
*/
private IUtilisateurServiceRemote utilisateurServiceRemote;
/**
* Le constructeur.
*
* @param i L'identifiant de l'utilisateur.
* @param t Le tag a creer.
* @param c Le callback.
*/
public CommandeCreerTag(final String i, final ITag t, final CallBack<ITag> c) {
super();
this.identification = i;
this.tag = t;
CommandeCreerTag(final String i, final ITag t, final CallBack<ITag> c) {
super(i, t);
this.callback = c;
final Properties env = new Properties();
env.setProperty("java.naming.factory.initial", "org.jnp.interfaces.NamingContextFactory");
env.setProperty("java.naming.provider.url", "localhost:1099");
env.setProperty("java.naming.factory.url.pkgs", "org.jboss.naming");
try {
final Context context = new InitialContext(env);
this.tagServiceRemote = (ITagServiceRemote) context.lookup("TagService/local");
this.utilisateurServiceRemote = (IUtilisateurServiceRemote) context.lookup("UtilisateurService/local");
} catch (NamingException e) {
e.printStackTrace();
}
}
@Override
......
......@@ -4,32 +4,16 @@ import fr.alma.gtd.donneespartagees.ITag;
import fr.alma.gtd.donneesserveur.Utilisateur;
import fr.alma.gtd.interfacedistante.CallBack;
import fr.alma.gtd.interfacedistante.ModeDeMiseAJour;
import fr.alma.gtd.isessions.ITagServiceRemote;
import fr.alma.gtd.isessions.IUtilisateurServiceRemote;
import java.rmi.RemoteException;
import java.util.Properties;
import javax.naming.Context;
import javax.naming.InitialContext;
import javax.naming.NamingException;
/**
* Commande d'envoi d'un tag.
*
* @author Stephane Begaudeau, Benjamin Gosset, Alex Lagarde, Christophe Renaudineau.
* @version 1.0.0
*/
public final class CommandeEnvoyerTag implements Commande {
/**
* L'identifiant de l'utilisateur.
*/
private final String identification;
/**
* Le tag a envoyer.
*/
private final ITag tag;
public final class CommandeEnvoyerTag extends CommandeTag {
/**
* Le mode de mise a jour.
......@@ -41,42 +25,18 @@ public final class CommandeEnvoyerTag implements Commande {
*/
private final CallBack<ITag> callback;
/**
* Le service de gestion de tags.
*/
private ITagServiceRemote tagServiceRemote;
/**
* Le service de gestion d'utilisateurs.
*/
private IUtilisateurServiceRemote utilisateurServiceRemote;
/**
* Le constructeur.
*
* @param i L'identifiant de l'utilisateur.
* @param t Le tag a envoyer.
* @param m Le mode de mise a jour.
* @param c Le callback.
*/
public CommandeEnvoyerTag(final String i, final ITag t, final ModeDeMiseAJour m, final CallBack<ITag> c) {
super();
this.identification = i;
this.tag = t;
CommandeEnvoyerTag(final String i, final ITag t, final ModeDeMiseAJour m, final CallBack<ITag> c) {
super(i, t);
this.mode = m;
this.callback = c;
final Properties env = new Properties();
env.setProperty("java.naming.factory.initial", "org.jnp.interfaces.NamingContextFactory");
env.setProperty("java.naming.provider.url", "localhost:1099");
env.setProperty("java.naming.factory.url.pkgs", "org.jboss.naming");
try {
final Context context = new InitialContext(env);
this.tagServiceRemote = (ITagServiceRemote) context.lookup("TagService/local");
this.utilisateurServiceRemote = (IUtilisateurServiceRemote) context.lookup("UtilisateurService/local");
} catch (NamingException e) {
e.printStackTrace();
}
}
@Override
......
......@@ -3,72 +3,32 @@ package fr.alma.gtd.commande;
import fr.alma.gtd.donneespartagees.ITag;
import fr.alma.gtd.donneesserveur.Utilisateur;
import fr.alma.gtd.interfacedistante.CallBack;
import fr.alma.gtd.isessions.ITagServiceRemote;
import fr.alma.gtd.isessions.IUtilisateurServiceRemote;
import java.rmi.RemoteException;
import java.util.Properties;
import javax.naming.Context;
import javax.naming.InitialContext;
import javax.naming.NamingException;
/**
* Commande de suppression d'un tag.
*
* @author Stephane Begaudeau, Benjamin Gosset, Alex Lagarde, Christophe Renaudineau.
* @version 1.0.0
*/
public final class CommandeSupprimerTag implements Commande {
/**
* L'identifiant de l'utilisateur.
*/
private final String identification;
/**
* Le tag a supprimer.
*/
private final ITag tag;
public final class CommandeSupprimerTag extends CommandeTag {
/**
* Le callback.
*/
private final CallBack<String> callback;
/**
* Le service de gestion de tags.
*/
private ITagServiceRemote tagServiceRemote;
/**
* Le service de gestion d'utilisateurs.
*/
private IUtilisateurServiceRemote utilisateurServiceRemote;
/**
* Le constructeur.
*
* @param i L'identifiant de l'utilisateur.
* @param t Le tag a supprimer.
* @param c Le callback.
*/
public CommandeSupprimerTag(final String i, final ITag t, final CallBack<String> c) {
super();
this.identification = i;
this.tag = t;
CommandeSupprimerTag(final String i, final ITag t, final CallBack<String> c) {
super(i, t);
this.callback = c;
final Properties env = new Properties();
env.setProperty("java.naming.factory.initial", "org.jnp.interfaces.NamingContextFactory");
env.setProperty("java.naming.provider.url", "localhost:1099");
env.setProperty("java.naming.factory.url.pkgs", "org.jboss.naming");
try {
final Context context = new InitialContext(env);
this.tagServiceRemote = (ITagServiceRemote) context.lookup("TagService/local");
this.utilisateurServiceRemote = (IUtilisateurServiceRemote) context.lookup("UtilisateurService/local");
} catch (NamingException e) {
e.printStackTrace();
}
}
@Override
......
package fr.alma.gtd.commande;
import fr.alma.gtd.donneespartagees.ITag;
import fr.alma.gtd.isessions.ITagServiceRemote;
import fr.alma.gtd.isessions.IUtilisateurServiceRemote;
import java.rmi.RemoteException;
import java.util.Properties;
import javax.naming.Context;
import javax.naming.InitialContext;
import javax.naming.NamingException;
public abstract class CommandeTag implements Commande {
/**
* L'identifiant de l'utilisateur.
*/
protected final String identification;
/**
* Le tag.
*/
protected final ITag tag;
/**
* Le service de gestion de tags.
*/
ITagServiceRemote tagServiceRemote;
/**
* Le service de gestion d'utilisateurs.
*/
IUtilisateurServiceRemote utilisateurServiceRemote;
/**
* Le constructeur.
*
* @param i L'identifiant de l'utilisateur.
* @param t Le tag a creer.
*/
CommandeTag(final String i, final ITag t) {
this.identification = i;
this.tag = t;
final Properties env = new Properties();
env.setProperty("java.naming.factory.initial", "org.jnp.interfaces.NamingContextFactory");
env.setProperty("java.naming.provider.url", "localhost:1099");
env.setProperty("java.naming.factory.url.pkgs", "org.jboss.naming");
try {
final Context context = new InitialContext(env);
this.tagServiceRemote = (ITagServiceRemote) context.lookup("TagService/local");
this.utilisateurServiceRemote = (IUtilisateurServiceRemote) context.lookup("UtilisateurService/local");
} catch (NamingException e) {
e.printStackTrace();
}
}
public abstract void execute() throws RemoteException;
}
#Thu Apr 12 15:17:16 CEST 2018
#Thu Apr 12 15:50:54 CEST 2018
>>>>>>>=a1dbf8c281633bc07dba6931845c1eb4990caf40
/comptes/E149741M/Documents/L3/S6/Construction_evolution_logiciel/projetv2/GTDServer/JBoss/src/main/java/fr/alma/gtd/reactor/ReactorCorba.java=1523456666000
<<<<<<<=HEAD
configuration*?=F894531AF957325040175B7EC7B8FAE5D3E12697
/comptes/E146247P/Documents/S6/CEL/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/reactor/ReactorCorba.java=1523456303000
=\=\=\=\=\=\=
/comptes/E146247P/Documents/S6/CEL/projet/GTDServer/JBoss/src/main/java/fr/alma/gtd/reactor/ReactorCorba.java=1523456303000
configuration*?=F894531AF957325040175B7EC7B8FAE5D3E12697
<<<<<<<=HEAD
......@@ -28,34 +28,6 @@
if ((tac != null) && (uti != null) && (this.identification.equalsIgnoreCase(this.tache.getCreateur().getIdentifiantServeur()))) {]]></codefragment>
</duplication>
<duplication lines="22" tokens="174">
<file line="66"
path="/comptes/E149741M/Documents/L3/S6/Construction_evolution_logiciel/projetv2/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/CommandeEnvoyerTag.java"/>
<file line="58"
path="/comptes/E149741M/Documents/L3/S6/Construction_evolution_logiciel/projetv2/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/CommandeSupprimerTag.java"/>
<codefragment><![CDATA[ this.callback = c;
final Properties env = new Properties();
env.setProperty("java.naming.factory.initial", "org.jnp.interfaces.NamingContextFactory");
env.setProperty("java.naming.provider.url", "localhost:1099");
env.setProperty("java.naming.factory.url.pkgs", "org.jboss.naming");
try {
final Context context = new InitialContext(env);
this.tagServiceRemote = (ITagServiceRemote) context.lookup("TagService/local");
this.utilisateurServiceRemote = (IUtilisateurServiceRemote) context.lookup("UtilisateurService/local");
} catch (NamingException e) {
e.printStackTrace();
}
}
@Override
public void execute() throws RemoteException {
final ITag ta = this.tagServiceRemote.getTagById(this.tag.getIdentifiantServeur());
final Utilisateur uti = this.utilisateurServiceRemote.getUtilisateurById(this.tag.getCreateur().getIdentifiantServeur());
if ((ta != null) && (uti != null) && (this.identification.equalsIgnoreCase(this.tag.getCreateur().getIdentifiantServeur()))) {]]></codefragment>
</duplication>
<duplication lines="22" tokens="168">
<file line="66"
path="/comptes/E149741M/Documents/L3/S6/Construction_evolution_logiciel/projetv2/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/CommandeEnvoyerIdee.java"/>
......@@ -116,38 +88,6 @@
if ((tac == null) && (uti != null) && (this.identification.equalsIgnoreCase(this.tache.getCreateur().getIdentifiantServeur()))) {]]></codefragment>
</duplication>
<duplication lines="26" tokens="159">
<file line="54"
path="/comptes/E149741M/Documents/L3/S6/Construction_evolution_logiciel/projetv2/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/CommandeCreerTag.java"/>
<file line="54"
path="/comptes/E149741M/Documents/L3/S6/Construction_evolution_logiciel/projetv2/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/CommandeSupprimerTag.java"/>
<codefragment><![CDATA[ public CommandeCreerTag(final String i, final ITag t, final CallBack<ITag> c) {
super();
this.identification = i;
this.tag = t;
this.callback = c;
final Properties env = new Properties();
env.setProperty("java.naming.factory.initial", "org.jnp.interfaces.NamingContextFactory");
env.setProperty("java.naming.provider.url", "localhost:1099");
env.setProperty("java.naming.factory.url.pkgs", "org.jboss.naming");
try {
final Context context = new InitialContext(env);
this.tagServiceRemote = (ITagServiceRemote) context.lookup("TagService/local");
this.utilisateurServiceRemote = (IUtilisateurServiceRemote) context.lookup("UtilisateurService/local");
} catch (NamingException e) {
e.printStackTrace();
}
}
@Override
public void execute() throws RemoteException {
final ITag ta = this.tagServiceRemote.getTagById(this.tag.getIdentifiantServeur());
final Utilisateur uti = this.utilisateurServiceRemote.getUtilisateurById(this.tag.getCreateur().getIdentifiantServeur());
if ((ta == null) && (uti != null) && (this.identification.equalsIgnoreCase(this.tag.getCreateur().getIdentifiantServeur()))) {]]></codefragment>
</duplication>
<duplication lines="22" tokens="142">
<file line="58"
path="/comptes/E149741M/Documents/L3/S6/Construction_evolution_logiciel/projetv2/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/CommandeCreerIdee.java"/>
......@@ -206,32 +146,4 @@
if ((tac == null) && (uti != null) && (this.identification.equalsIgnoreCase(this.tache.getCreateur().getIdentifiantServeur()))) {]]></codefragment>
</duplication>
<duplication lines="22" tokens="142">
<file line="58"
path="/comptes/E149741M/Documents/L3/S6/Construction_evolution_logiciel/projetv2/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/CommandeCreerTag.java"/>
<file line="66"
path="/comptes/E149741M/Documents/L3/S6/Construction_evolution_logiciel/projetv2/GTDServer/JBoss/src/main/java/fr/alma/gtd/commande/CommandeEnvoyerTag.java"/>
<codefragment><![CDATA[ this.callback = c;
final Properties env = new Properties();
env.setProperty("java.naming.factory.initial", "org.jnp.interfaces.NamingContextFactory");
env.setProperty("java.naming.provider.url", "localhost:1099");
env.setProperty("java.naming.factory.url.pkgs", "org.jboss.naming");
try {
final Context context = new InitialContext(env);
this.tagServiceRemote = (ITagServiceRemote) context.lookup("TagService/local");
this.utilisateurServiceRemote = (IUtilisateurServiceRemote) context.lookup("UtilisateurService/local");
} catch (NamingException e) {
e.printStackTrace();
}
}
@Override
public void execute() throws RemoteException {
final ITag ta = this.tagServiceRemote.getTagById(this.tag.getIdentifiantServeur());
final Utilisateur uti = this.utilisateurServiceRemote.getUtilisateurById(this.tag.getCreateur().getIdentifiantServeur());
if ((ta == null) && (uti != null) && (this.identification.equalsIgnoreCase(this.tag.getCreateur().getIdentifiantServeur()))) {]]></codefragment>
</duplication>
</pmd-cpd>
\ No newline at end of file
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>15:17:14</time>
<time>15:50:52</time>
<packages>
<package>
<name>fr.alma.gtd.commande</name>
<classes>40</classes>
<functions>143</functions>
<ncss>952</ncss>
<javadocs>95</javadocs>
<javadoc_lines>610</javadoc_lines>
<classes>41</classes>
<functions>145</functions>
<ncss>916</ncss>
<javadocs>96</javadocs>
<javadoc_lines>604</javadoc_lines>
<single_comment_lines>1</single_comment_lines>
<multi_comment_lines>159</multi_comment_lines>
<multi_comment_lines>153</multi_comment_lines>
</package>
<package>
<name>fr.alma.gtd.donneespartagees</name>
......@@ -84,20 +84,20 @@
<multi_comment_lines>155</multi_comment_lines>
</package>
<total>
<classes>91</classes>
<functions>749</functions>
<ncss>2971</ncss>
<javadocs>398</javadocs>
<javadoc_lines>2300</javadoc_lines>
<classes>92</classes>
<functions>751</functions>
<ncss>2935</ncss>
<javadocs>399</javadocs>
<javadoc_lines>2294</javadoc_lines>
<single_comment_lines>25</single_comment_lines>
<multi_comment_lines>487</multi_comment_lines>
<multi_comment_lines>481</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>91.00</td><td>749.00</td><td>2,971.00</td><td>398.00</td><td>Project</td></tr>
<tr><td></td><td>11.38</td><td>93.63</td><td>371.38</td><td>49.75</td><td>Package</td></tr>
<tr><td></td><td></td><td>8.23</td><td>32.65</td><td>4.37</td><td>Class</td></tr>
<tr><td></td><td></td><td></td><td>3.97</td><td>0.53</td><td>Function</td></tr>
<tr><td>8.00</td><td>92.00</td><td>751.00</td><td>2,935.00</td><td>399.00</td><td>Project</td></tr>
<tr><td></td><td>11.50</td><td>93.88</td><td>366.88</td><td>49.88</td><td>Package</td></tr>
<tr><td></td><td></td><td>8.16</td><td>31.90</td><td>4.34</td><td>Class</td></tr>
<tr><td></td><td></td><td></td><td>3.91</td><td>0.53</td><td>Function</td></tr>
</table>
</packages>
......@@ -142,16 +142,6 @@
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>9</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeCreerTag</name>
<ncss>30</ncss>
<functions>2</functions>
<classes>0</classes>
<javadocs>2</javadocs>
<javadocs_lines>17</javadocs_lines>
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>9</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeEnvoyerIdee</name>
<ncss>39</ncss>
......@@ -172,16 +162,6 @@
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>12</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeEnvoyerTag</name>
<ncss>39</ncss>
<functions>2</functions>
<classes>0</classes>
<javadocs>2</javadocs>
<javadocs_lines>18</javadocs_lines>
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>12</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeFactory</name>
<ncss>104</ncss>
......@@ -262,16 +242,6 @@
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>9</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeSupprimerTag</name>
<ncss>31</ncss>
<functions>2</functions>
<classes>0</classes>
<javadocs>2</javadocs>
<javadocs_lines>17</javadocs_lines>
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>9</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeTelechargeArchive</name>
<ncss>4</ncss>
......@@ -472,6 +442,16 @@
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>3</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeEnvoyerTag</name>
<ncss>24</ncss>
<functions>2</functions>
<classes>0</classes>
<javadocs>2</javadocs>
<javadocs_lines>14</javadocs_lines>
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>6</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeContexte</name>
<ncss>19</ncss>
......@@ -482,6 +462,16 @@
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>12</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeSupprimerTag</name>
<ncss>16</ncss>
<functions>2</functions>
<classes>0</classes>
<javadocs>2</javadocs>
<javadocs_lines>13</javadocs_lines>
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>3</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeEnvoyerContexte</name>
<ncss>24</ncss>
......@@ -502,6 +492,26 @@
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>3</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeCreerTag</name>
<ncss>15</ncss>
<functions>2</functions>
<classes>0</classes>
<javadocs>2</javadocs>
<javadocs_lines>13</javadocs_lines>
<single_comment_lines>0</single_comment_lines>
<implementation_comment_lines>3</implementation_comment_lines>
</object>
<object>
<name>fr.alma.gtd.commande.CommandeTag</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.donneespartagees.AbstractContact</name>
<ncss>31</ncss>
......@@ -1013,15 +1023,15 @@
<implementation_comment_lines>150</implementation_comment_lines>
</object>
<averages>
<ncss>25.98</ncss>
<functions>8.23</functions>
<ncss>25.40</ncss>
<functions>8.16</functions>
<classes>0.00</classes>
<javadocs>4.37</javadocs>
<javadocs_lines>25.27</javadocs_lines>
<javadocs>4.34</javadocs>
<javadocs_lines>24.93</javadocs_lines>
<single_comment_lines>0.26</single_comment_lines>