ajustement et correction

README.md

@@ -20,7 +20,7 @@ C'est un projet de fin de Licence L1 en Informatique à UPEC, la création d'un
 
 - La partie **PROBLÈME ET SOLUTION RENCONTRÉES**
 
-## AVANT TOUT : Comment lancer le jeu.
+## AVANT TOUT : Comment lancer le jeu
 
 Pour lancer le jeu, nous pouvons utiliser les addons de vscode en appuyant sur ***run*** sur la methode **main**.
 
@@ -35,6 +35,7 @@ Les commandes pour le makefile :
 - `Make run` : il va uniquement lancer l'interpréteur java (Attention à ne pas supprimer le bin avant)
 
 Pour lancer le jeu en multijoueur avec Makefile (mot1 et mot2 sont des mots à changer en fonction du canal):
+
 - `Make channel=mot1 adversaire=mot2` : compile tout le programme et le lance en multijoueur.
 - `Make run channel=mot1 adversaire=mot2` : lance uniquement le programme en multijoueur.
 
@@ -42,6 +43,67 @@ Sur windows, vous avez une autre option que personnellement je vous encourage, c
 
 - `run.bat mot1 mot2` : il va uniquement lancer `Make channel=mot1 adversaire=mot2` et en même temps lancer `chcp 65001` qui va mettre l'utf-8.
 
+## Fichier de configuration
+
+- Le fichier de configuration sert à créer une partie sans recompiler en `.class` le programme. Étant en format xml, il ressemble à des balises html et donc plus facile  à comprendre.
+
+Pour que le programme comprenne le fichier xml, il doit y avoir des balises spécifiques:
+
+- `Configuration` est la racine du programme, il a comme attribut n pour la taille du snake. Exemple : 
+  
+  ```xml
+  <Configuration n="2"></Configuration>
+  ```
+
+- `Map` est la balise principale pour la gestion de la carte du jeu. Il y a comme attribut la taille x et y de la carte. Il permet aussi de placer les balises fraises et des murs. Exemple : 
+  
+  ```xml
+  <Map x="22" y="22"></Map>
+  ```
+
+- `Fraise` et `Wall`  sont des balises pour placer une fraise ou un mur avec comme attributs x et y. Exemple : 
+  
+  ```xml
+  <Wall x="3" y="4"/>
+  <Fraise x="5" y="6"/>
+  ```
+
+- `Personnage` est la balise principale pour la gestion des personnages du jeu. Il permet de placer des balises IA, Robot et Player. Exemple : 
+  
+  ```xml
+  <Personnage></Personnage>
+  ```
+
+- `Player`, `Robot` et `IA` sont des balises pour placer des personnages avec des attributs comme name, x et y. Sur la balise IA, on doit aussi mettre QTable avec le dossier ou le fichier qui sert a stocker l'apprentissage de l'IA. Exemple : 
+  
+  ```xml
+  <Player name="Philippe_Etchebest" x="2" y="2"/>
+  <Robot name="R2D2" x="19" y="19"/>
+  <IA name="incognito" x="19" y="19" QTable="res/save"/>
+  ```
+
+**Exemple de configuration :**
+
+```xml
+<?xml version="1.0" encoding="UTF-8"?>
+
+<!-- Configuration globale -->
+<Configuration n="2">
+    <!-- Configuration de la map -->
+    <Map x="22" y="22">
+        <Wall x="1" y="2"/>
+        <Fraise x="5" y="6"/> 
+    </Map>
+    
+    <!-- Configuration des personnages -->
+    <Personnage>
+        <IA name="incognito" x="19" y="19" QTable="res/save"/>
+        <Robot name="R2D2" x="19" y="19"/>
+    </Personnage>
+</Configuration>
+```
+
+``
 
 # JEU et GAMEPLAY
 
@@ -66,7 +128,6 @@ Le channel récupère les données contenues sur le channel qui y est dédié (l
 
 Nous avons testé Channel à distance sur 2 machines et la partie s'est déroulée correctement.
 
-
 # IA (Q-Learning)
 
 ## - Explications
@@ -95,8 +156,11 @@ Ce calcul sera la valeur de toutes les actions que l'IA va enregistrer dans sa b
 - $Q(s_t, a_t)$ : est la valeur de Q actuelle, il contient  $s_t$ qui l'état et $a_t$ qui est l'action de Q.
 
 - $\alpha$ : est le taux d'apprentissage, c'est lui qui détermine si on doit écraser les valeurs ou non.
+
 - $R_t$ : est la récompense de l'action, si c'est une bonne action ou non.
+
 - $\gamma$ : est l'importance des futures récompenses.
+
 - $\max(Q(s_{t+1}, a))$ : est la valeur maximale de Q du prochain tour parmi toute sa base de donnée.
 
 ## Resultat :
@@ -129,7 +193,7 @@ Cependant, grâce à lui j'ai énormément appris et son aide m'a souvent été
 
 Je trouve notre projet bien construit, utilisant des moyens adaptés à chacuns des points du jeu. La division des classes est pertinente et les commentaires apportent une grande plus-value à ceux qui voudrait l'utiliser.
 
-## - Impressions GUEZO Loïc 
+## - Impressions GUEZO Loïc:
 
 Durant ce projet, j'ai pu tester et apprendre de nouvelles choses. J'ai pu aussi "tester" à gérer une "mini équipe" (on etait que 2). Mon camarade s'est quand même bien débrouillé comparé à notre différence de penser et comment gérer les problèmes.
 

src/Main.java

@@ -1,13 +1,7 @@
-import java.io.File;
-import java.io.ObjectInputFilter.Config;
-import java.util.ArrayList;
-
 import configuration.ConfigGame;
 import game.Terminal;
 import game.environnement.*;
 import personnage.*;
-import personnage.IAQLearning.QTable;
-import tests.IATest;
 
 public class Main {
     /**

src/conf.xml

@@ -1,22 +1,13 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
 <!-- Configuration globale -->
-<Configuration>
+<Configuration n="2">
-    <Size n="2"/>
     <!-- Configuration de la map -->
-    <Map>
+    <Map x="22" y="22"></Map>
-        <Coordinate x="22" y="22"/>
-
-        <Wall x="1" y="2"/>
-        <Wall x="3" y="4"/>
-        <Fraise x="5" y="6"/> 
-    </Map>
     
     <!-- Configuration des personnages -->
     <Personnage>
         <Player name="Philippe_Etchebest" x="2" y="2"/>
-        <!-- <Player name="Luke Skywalker" x="19" y="19"/> -->
+        <Player name="Luke Skywalker" x="19" y="19"/>
-        <!-- <IA name="" x="19" y="19" QTable=""/> -->
-        <Robot name="R2D2" x="19" y="19"/>
     </Personnage>
 </Configuration>

src/configuration/ConfigGame.java

@@ -23,21 +23,27 @@ public class ConfigGame {
         try {
             FileReaderXml fileReaderXml = new FileReaderXml(path);
             this.data = fileReaderXml.getElements();
+
+            if (this.data.get("Configuration") == null || this.data == null) {
+                System.err.println("Erreur: le fichier de configuration est introuvable.");
+                System.exit(-1);
+            }
+
         } catch (IOException | SAXException | ParserConfigurationException e) {
             e.printStackTrace();
         }
     }
 
     public Map getMap() {
-        ArrayList<HashMap<String,String>> mapList = data.get("Map.Coordinate");
+        ArrayList<HashMap<String,String>> mapList = data.get("Configuration.Map");
         
-        if (data.get("Configuration.Map") == null) {
+        if (mapList == null) {
             System.err.println("Erreur: La balise Map est introuvable.");
             System.exit(-1);
         }
 
         if (mapList.size() != 1) {
-            System.err.println("Erreur: Plusieurs Coordonnées trouvées.");
+            System.err.println("Erreur: Plusieurs ou Aucune Coordonnées trouvées.");
             System.exit(-1);
         }
 
@@ -52,13 +58,13 @@ public class ConfigGame {
     }
 
     public int getN() {
-        ArrayList<HashMap<String, String>> n;
+        HashMap<String, String> n;
 
-        if ((n = data.get("Configuration.Size")) == null) {
+        if ((n = data.get("Configuration").get(0)).isEmpty()) {
             return 4;
         } 
         
-        return Integer.parseInt(n.get(0).get("n"));
+        return Integer.parseInt(n.get("n"));
     }
 
     public Personnage[] getPersonnages() {

src/configuration/FileReaderXml.java

@@ -36,36 +36,39 @@ public class FileReaderXml {
 
     protected HashMap<String, ArrayList<HashMap<String, String>>> getElements() {
         HashMap<String, ArrayList<HashMap<String, String>>> elementsMap = new HashMap<>();
-        NodeList nodeList = document.getDocumentElement().getChildNodes();
+        Node nodeList = document.getDocumentElement();
         
         readElements(elementsMap, nodeList);
         
         return elementsMap;
     }
 
-    private void readElements(HashMap<String, ArrayList<HashMap<String, String>>> elementsMap, NodeList nodeList) {
+    private void readElements(HashMap<String, ArrayList<HashMap<String, String>>> elementsMap, Node currentNode) {
-        for (int i = 0; i < nodeList.getLength(); i++) {
+        if (currentNode.getNodeType() == Node.ELEMENT_NODE) {
-            Node childNodeList = nodeList.item(i);
+            HashMap<String, String> attributeMap = new HashMap<>();
+            NamedNodeMap attributes = currentNode.getAttributes();
+            for (int j = 0; j < attributes.getLength(); j++) {
+                Node attribute = attributes.item(j);
+                attributeMap.put(attribute.getNodeName(), attribute.getNodeValue());
+            }
             
-            if (childNodeList.getNodeType() == Node.ELEMENT_NODE) {
+            String parentNodeName, fullNodeName;
-                HashMap<String, String> attributeMap = new HashMap<>();
-                NamedNodeMap attributes = childNodeList.getAttributes();
-                for (int j = 0; j < attributes.getLength(); j++) {
-                    Node attribute = attributes.item(j);
-                    attributeMap.put(attribute.getNodeName(), attribute.getNodeValue());
-                }
 
-                String fullNodeName = childNodeList.getParentNode().getNodeName() + "." + childNodeList.getNodeName();
+            if ((parentNodeName = currentNode.getParentNode().getNodeName()).charAt(0) == '#') {
+                fullNodeName = currentNode.getNodeName();
+            } else {
+                fullNodeName = parentNodeName + "." + currentNode.getNodeName();   
+            }
     
-                if (!elementsMap.containsKey(fullNodeName)) {
+            if (!elementsMap.containsKey(fullNodeName)) {
-                    elementsMap.put(fullNodeName, new ArrayList<>());
+                elementsMap.put(fullNodeName, new ArrayList<>());
-                }
+            }
     
-                elementsMap.get(fullNodeName).add(attributeMap);
+            elementsMap.get(fullNodeName).add(attributeMap);
 
-                if (childNodeList.hasChildNodes()) {
+            NodeList childNodes = currentNode.getChildNodes();
-                    readElements(elementsMap, childNodeList.getChildNodes());
+            for (int i = 0; i < childNodes.getLength(); i++) {
-                }
+                readElements(elementsMap, childNodes.item(i));
             }
         }
     }

src/tests/IATest.java

@@ -3,8 +3,6 @@ package tests;
 import java.io.File;
 import java.util.Arrays;
 
-import display.Display;
-import game.environnement.Grid;
 import game.environnement.Map;
 import personnage.IA;
 import personnage.Personnage;

src/tests/XmlReaderTest.java

@@ -5,8 +5,6 @@ import configuration.ConfigGame;
 public class XmlReaderTest {
     public static void main(String[] args) {
         ConfigGame configXml = new ConfigGame(null);
-        System.out.println(configXml.getPersonnages());
-        System.out.println(configXml.getN());
         System.out.println(configXml.getMap());
     }
 }