Frédéric Henry

pom.xml

@@ -1,20 +1,24 @@
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-	xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
-	<modelVersion>4.0.0</modelVersion>
-	<groupId>diginamic</groupId>
-	<artifactId>codePropre</artifactId>
-	<version>0.0.1-SNAPSHOT</version>
-	<build>
-		<sourceDirectory>src/main/java</sourceDirectory>
-		<plugins>
-			<plugin>
-				<artifactId>maven-compiler-plugin</artifactId>
-				<version>3.5.1</version>
-				<configuration>
-					<source>1.8</source>
-					<target>1.8</target>
-				</configuration>
-			</plugin>
-		</plugins>
-	</build>
+         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+    <modelVersion>4.0.0</modelVersion>
+    <groupId>diginamic</groupId>
+    <artifactId>codePropre</artifactId>
+    <version>0.0.1-SNAPSHOT</version>
+    <properties>
+        <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
+    </properties>
+
+    <build>
+        <sourceDirectory>src/main/java</sourceDirectory>
+        <plugins>
+            <plugin>
+                <artifactId>maven-compiler-plugin</artifactId>
+                <version>3.5.1</version>
+                <configuration>
+                    <source>1.8</source>
+                    <target>1.8</target>
+                </configuration>
+            </plugin>
+        </plugins>
+    </build>
 </project>

src/README.md

@@ -0,0 +1,54 @@
+# code-propre
+## Nettoyage de code
+
+### Exercice 2
+
+Le livret A n’était pas documenté.
+
+Le compte courant ne semble pas exister.
+Le compte bancaire devrait être une classe abstraite dont héritent le compte
+courant et le livret A.
+
+Si le livret A est le seul à avoir un taux de rémunération annuel le compte
+bancaire n’est pas sensé en avoir.
+
+Appeler la classe mère `Object` par le constructeur `super` est inutile dans
+la classe `CompteBancaire`: il est implicite.
+
+La méthode `debiterMontant` ne sera vraiment appliquée que dans les classes
+filles.
+~~Autant qu’elle soit abstraite dans `CompteBancaire` et définies dans chaque
+classe fille.~~ Elle peut être simplifiée puisque l’absence de découvert
+peut être illustrée par `decouvert=0`.
+De plus sa documentation est incorrecte.
+
+La méthode `appliquerRemunerationAnnuelle` est mal nommée et devrait être
+définie dans la classe `LivretA`.
+
+Le solde `solde` n’a pas besoin de mutateur `setSolde`.
+Les méthodes `ajouterMontant` et `debiterMontant` gèrent son évolution avec
+plus de contrôle.
+
+Autre détail : tabulation convertie en 4 espaces pour indenter.
+
+### Exercice 3
+
+« Je comprends que tu as un parc zoologique où tu accueilles toute sorte
+d’animaux.
+Du coup, je me serais attendu à croiser au moins une classe `Animal` en plus de
+la classe `Zoo`.
+Plus encore, j’aurais fait des sous-classes suivant l’arbre phylogénique des
+bêtes citées dans la classe exécutable `ZooApplication`.
+
+Après si ces bêtes ne sont qu’une source de donnée, on peut juste les
+rassembler sous forme d’un dictionnaire `HashMap<Integer, String[3]>` selon les
+attributs `types`, `noms` et `comportements`.
+La raison est que ça permet de mettre en avant la corrélation des attributs,
+mais aussi de distinguer chaque bête partageant les mêmes attributs.
+La ligne `0` peut être réservée comme en-tête d’un tableau futur.
+
+Après, je ne connais pas l’application exacte.
+
+Je fais juste une petite correction en assignant tes attributs à des objets
+`ArrayList` vides et en renommant `types` en `typeAnimal`…
+Voilà, la méthode `afficherListeAnimaux` remarche. »

src/main/java/ex1/Entreprise.java

@@ -0,0 +1,21 @@
+package ex1;
+
+import java.util.Date;
+/** Entreprise */
+public class Entreprise {
+    /** Numéro de siret */
+    public int siret;
+    /** Nom */
+    public String nom;
+    /** Adresse */
+    public String adresse;
+    /** Date de création */
+    public Date dateCreation;
+    /** Capital maximal */
+    public static final int capitalMax = 3000000;
+
+    /**
+     * Afficher le status
+     */
+    public void afficherStatut(){}
+}

src/main/java/ex1/calcul.java

@@ -1,12 +1,12 @@
 package ex1;
-
+/** Calcul */
 public class calcul {
-
-	public int get1(int a, int b){
-		return a+b;
-	}
-	
-	public int get2(int a, int b){
-		return a-b;
-	}
+    /** Addition */
+    public int addition(int a, int b){
+        return a+b;
+    }
+    /** Soustraction */
+    public int soustraction(int a, int b){
+        return a-b;
+    }
 }