dernière correction par ma maman

Reviewed-on: #53
Reviewed-by: Mat_02 <diletomatteo@gmail.com>

JournalDeBord/rapport/rapport.tex

@@ -20,10 +20,10 @@
 
 \section*{Introduction}
 
-Lors de ce deuxième quadrimèstre, le projet Informatique proposé par notre université fut partie intégrante de notre emploi du temps.
+Lors de ce deuxième quadrimestre, le projet Informatique proposé par notre université fut partie intégrante de notre emploi du temps.
-Régulièrement nous nous sommes rassemblés pour nous organiser et trouver une direction dans laquelle nous voulions voir notre projet évoluer.
+Régulièrement, nous nous sommes rassemblés pour nous organiser et trouver une direction dans laquelle nous voulions voir notre projet évoluer.
-Grâce aux objectifs fixés par nos enseignants, nous sommes - je le pense - maintenant plus apte à nous confronter à ce genre d'objectifs. Tant au niveau personel qu'en tant que groupe.
+Grâce aux objectifs fixés par nos enseignants, nous sommes - nous le pensons - maintenant plus aptes à nous confronter à ce genre d'objectifs, tant au niveau personnel qu'en tant que groupe.
-Il va sans dire que comme pour tout projets, notre chemin a été semé d'embuches. En l'occurence, nous souhaitons faire part de l'abandon d'un de nos membre. Eddy Jiofak qui souhaite se réorienter.
+Il va sans dire que comme pour tout projet, notre chemin a été semé d'embûches. En l'occurrence, nous souhaitons faire part de l'abandon d'un de nos membre. Eddy Jiofak qui souhaite se réorienter.
 Nous lui souhaitons une bonne reconversion.
 
 \section*{Objectifs}
@@ -49,7 +49,7 @@ Ce blog nous sert également à garder une trace de l'évolution du projet.
 \subsection{Choix}
 
 \begin{itemize}
-	\item{le VCS git pour garder une trace de l'avancement du projet. Avec comme remote une instance privée de gitea nous permettant de vérifier les MR/PR plus efficacement.}
+	\item{le VCS \verb|git| pour garder une trace de l'avancement du projet. Avec comme remote une instance privée de gitea nous permettant de vérifier les MR/PR plus efficacement.}
 	\item{Une instance de DroneCI permettant de vérifier que le projet soit toujours compilable et que les tests ne soient pas ratés sans que nous nous en rendions compte.}
 	\item{Javafx, comme recommandé par nos enseignants.}
 	\item{Les pièces et niveaux sont stockées sous forme de matrice de booléen}
@@ -75,11 +75,11 @@ Le tout est géré par la classe \verb|Controller| qui permet de choisir entre l
 
 \subsection{Parser de fichiers}
 
-Pour la rétention des niveaux, plusieurs possibilités s'offraient à nous. Nous avons alors décidés d'accomplir une série d'objectifs propre à notre projet avec un parser de fichiers dédié.
+Pour la rétention des niveaux, plusieurs possibilités s'offraient à nous. Nous avons alors décidé d'accomplir une série d'objectifs propres à notre projet avec un parser de fichiers dédié.
 Nous voulions que ce parser accomplisse les objectifs suivants:
 \begin{itemize}
 	\item{Les données du niveau seront encapsulées dans un header/footer pour laisser la possibilité d'enregistrer plus d'informations (images/musiques) dans un seul fichier dans le futur.}
-	\item{La taille du fichier devra être aussi petite que possible, tout en gardant les informations nécessaire au bon fonctionnement du jeu.}
+	\item{La taille du fichier devra être aussi petite que possible, tout en gardant les informations nécessaires au bon fonctionnement du jeu.}
 	\item{Il sera possible d'enregistrer l'état d'une partie en cours.}
 \end{itemize}
 
@@ -90,22 +90,22 @@ Ce parser est implémenté par la classe \verb|BinaryParser|.
 \begin{description}
 	\item[Header/Footer]{ Les données du niveau commencent par les 3 \emph{caractères} 'S', 'M', 'S' (ou \verb|0x534D53|) et se terminent par les 3 \emph{caractères} 'S', 'M', 'E' (ou \verb|0x534D45|)}
 	\item[Taille de carte]{ Le premier octet des données représente la largeur de la carte, le second sa hauteur.}
-	\item[Forme de la carte]{ Chaques cellules de la carte est représenté par un 1 ou un 0. le 1 représente un emplacement libre, un 0 une cellule vide.
+	\item[Forme de la carte]{ Chaque cellule de la carte est représenté par un 1 ou un 0. Le 1 représente un emplacement libre, un 0 une cellule vide.
 		La forme de la carte peut alors être répartie sur un nombre indéterminé d'octets.
 		Nous pouvons déterminer ce nombre grace à
 		$$\frac{\text{largeur} * \text{hauteur } (+1 \text{ si multiple de } 8)}{8}$$ 
 		en division entière.}
 	\item[Nombre de pièce]{ L' (les) octet(s) qui forme(nt) la carte représente le nombre de pièce}
-	\item[Pour chaques pièces]{
+	\item[Pour chaque pièces]{
 			\
 			\begin{description}
 				\item[Taille de la pièce]{La taille est représentée sur un seul octet sous forme de nibble\footnote{https://en.wikipedia.org/wiki/Nibble}. La première partie du nibble est la largeur. La seconde partie du nibble est la hauteur }
-				\item[Forme de la pièce]{ Chaques cellules de la pièce est représentée par un 1 ou un 0. la manière de le représenter et exactement la même que pour la forme de la carte }
+				\item[Forme de la pièce]{ Chaque cellules de la pièce est représentée par un 1 ou un 0. La manière de le représenter et exactement la même que pour la forme de la carte }
 			\end{description}
-			Dans le cas où le fichier sauvegarde l'état de la partie, à la fin, et pour chaques pièces dans le même ordre que l'apparition des pièces:
+			Dans le cas où le fichier sauvegarde l'état de la partie, à la fin, et pour chaque pièce dans le même ordre que l'apparition des pièces:
 			\begin{description}
 				\item[Position de la pièce]{2 octets par pièces, 1 octet pour sa position en x et 1 octet pour sa position en y.
-					Dans le cas où la pièce est flotante (n'est pas placée dans la carte.), les octets contenant les caractères F puis L (0x464C) remplacent les coordonées}
+					Dans le cas où la pièce est flottante (n'est pas placée dans la carte.), les octets contenant les caractères F puis L (0x464C) remplacent les coordonnées}
 			\end{description}
 		}
 \end{description}
@@ -119,14 +119,14 @@ Voici le 'hexdump' du niveau 11
 b0 22 70 22 b0 11 80 53  4d 45                    |."p"...SME|
 \end{verbatim}
 
-représente une carte de la forme :
+Représente une carte de la forme :
 
 \begin{tikzpicture}
 	\filldraw[blue] (0,0) -- (0,5) -- (3,5) -- (3,4) -- (5,4) -- (5,0) -- cycle;
 	\draw[step=1cm,gray] (0, 0) grid (5,5);
 \end{tikzpicture}
 
-avec les pièces :
+Avec les pièces :
 
 \begin{tikzpicture}
 	\fill[red] (1,1) rectangle (4,4);
@@ -138,8 +138,8 @@ avec les pièces :
 	\draw[step=1cm,gray] (0.5, -2.5) grid (13.5, 4.5);
 \end{tikzpicture}
 
-En plus de ce parser, et dans le cas où ce premier parser ne serait pas capable de stocker certaine carte (par exemple si une pièce mesure plus de 15x15).
+En plus de ce parser, et dans le cas où ce premier ne serait pas capable de stocker certaine carte (par exemple si une pièce mesure plus de 15x15),
-Nous avons également implémenté un parser très simple en utilisant l'interface \verb|Serialize| de java. Ce parser est implémenté et fonctionnel,
+nous avons également implémenté un parser très simple en utilisant l'interface \verb|Serialize| de java. Ce parser est implémenté et fonctionnel,
 mais n'est pas utilisé dans le projet à l'heure actuelle.
 
 Ces deux parseurs implémentent l'interface \verb|FileParser|.
@@ -154,11 +154,11 @@ Finalement, La classe \verb|FileParserFactory| permet une utilisation simple de
 
 Dans le cas d'une sauvegarde ou d'un chargement, le parser est choisi en fonction de l'extension de fichier ('.level', '.slevel', .'serialized', '.sserialized').
 
-L'avantage de ce system est que nous pouvons facilement ajouter d'autres parser de fichier dans le futurs.
+L'avantage de ce système est que nous pouvons facilement ajouter d'autres parser de fichiers dans le futur.
 
 \subsection{Générateur de niveaux}
 
-Le générateur de niveaux a été conçu de sorte à proposer 3 difficultés différentes
+Le générateur de niveaux a été conçu de sorte à proposer 3 difficultés différentes:
 
 \begin{itemize}
 	\item{Niveau Facile}
@@ -166,52 +166,52 @@ Le générateur de niveaux a été conçu de sorte à proposer 3 difficultés di
 	\item{Niveau Difficile}
 \end{itemize}
 
-L'algorithme derrière est le même en voici le principe :
+L'algorithme derrière est le même. En voici le principe :
 
 \subsubsection*{Gestion du plateau}
 
-Le joueur choisi une difficultée. En fonction de la difficultée choisie la grandeur du plateau de jeu sera différente.
+Le joueur choisit une difficulté. En fonction de la difficulté choisie, la grandeur du plateau de jeu sera différente.
-Si la difficulté choisie est facile ou moyenne, alors un curseur parcour les extrémités du niveau.
+Si la difficulté choisie est facile ou moyenne, alors un curseur parcourt les extrémités du niveau.
-Ce curseur sélectionne aléatoirement les cellules qui seront gardés ou non.
+Ce curseur sélectionne aléatoirement les cellules qui seront gardées ou non.
 Grâce à ça, la forme du plateau n'est pas trop carrée.
 
-Nous nous sommes basé sur le même principe pour le niveau de difficulté difficile mais en plus d'une taille encore plus grande,
+Nous nous sommes basés sur le même principe pour le niveau de difficulté difficile mais en plus d'une taille encore plus grande,
-le curseur parcours les extrémités avec une profondeur de 2 afin de faire varier la carte plus grande.
+le curseur parcourt les extrémités avec une profondeur de 2 afin de faire varier la carte plus grande.
-Cela introduit le problème des cases isolés. c'est pourquoi une boucle vérifie chaques cellules et la supprime si celle si est isolée.
+Cela introduit le problème des cases isoléss. C'est pourquoi une boucle vérifie chaque cellule et la supprime si celle-ci est isolée.
 
 \subsubsection*{Gestion des pièces}
 
-Peu importe la difficulté du niveau voici le fonctionnement :
+Peu importe la difficulté du niveau, voici le fonctionnement :
 
 Une taille maximum des pièces a été fixée au préalable à 3x3.
-Par la suite, un curseur parcours des cases de la carte préalablement conçue de manière aléatoire.
+Par la suite, un curseur parcourt des cases de la carte préalablement conçue de manière aléatoire.
-Pour chaques cases, l'algorithme teste pour chaques cases de la pièce, si l'espace est disponible.
+Pour chaque case, l'algorithme teste pour chaque case de la pièce, si l'espace est disponible.
-Si ca n'est pas le cas, alors la pièce est modifiée afin de faire correspondre la pièce et la carte.
+Si ça n'est pas le cas, alors la pièce est modifiée afin de faire correspondre la pièce et la carte.
 
 L'avantage de cette méthode est que les niveaux sont tous très différents.
 Les désavantages sont que, par malchance, il est possible d'avoir énormément de piece 1x1.
-Ainsi qu'il est plus difficile d'appliquer des textures et dessins - à l'image du jeu de base - sur les pièces.
+il est aussi plus difficile d'appliquer des textures et dessins - à l'image du jeu de base - sur les pièces.
 
-Malgrés tout, avec nos nombreux test, ce générateur de niveaux nous satisfait vraiment bien et la difficultée des niveaux coorespond bien aux attentes.
+Malgré tout, avec nos nombreux tests, ce générateur de niveaux nous satisfait vraiment bien et la difficulté des niveaux correspond bien aux attentes.
 
 \subsection{Interface graphique}
 L'interface graphique du jeu tient sur 5 classes différentes.
 
 \subsubsection*{Controller}
-Classe principale. Elle s'occupe de la gestion des autres classe, et de la cohérence entre elles.
+Classe principale. Elle s'occupe de la gestion des autres classes et de la cohérence entre elles.
 Toutes les autres classes (présentes dans le package \verb|Scenes|) sont des sous classe de Parents.
-Celà permet de les afficher grace à la méthode statique \verb|switchRoot|.
+Cela permet de les afficher grâce à la méthode statique \verb|switchRoot|.
 C'est aussi le point d'entrée du programme.
 
 \subsubsection*{MenuAccueil}
 Classe s'occupant de générer la page d'accueil du jeu.
-C'est-à-dire la première page que vera l'utilisateur.
+C'est-à-dire la première page que verra l'utilisateur.
 Cette page permet d'accéder aux niveaux du jeu de base, mais également de générer les niveaux aléatoires.
-De plus un dernier boutons "Load Game" permet de revenir sur la dernière partie en cours du joueur.
+De plus un dernier bouton "Load Game" permet de revenir sur la dernière partie en cours du joueur.
 
 \subsubsection*{MenuLevel}
 Classe s'occupant d'afficher les niveaux proposés dans le jeu, et qui,
-en fonction du jour choisir, change les boutons et les niveaux disponibles.
+en fonction du jour choisi, change les boutons et les niveaux disponibles.
 
 \subsubsection*{ScreenLevelFinish}
 Classe qui s'affiche à l'écran dès que le joueur a fini le niveau.
@@ -219,12 +219,12 @@ Celle-ci propose également de réessayer le niveau ou de retourner au menu prin
 
 \subsubsection*{GameUI}
 Classe s'occupant de l'affichage d'un niveau.
-S'occupe dans un premier temps d'afficher le plateau au milieu de l'écrant.
+S'occupe dans un premier temps d'afficher le plateau au milieu de l'écran.
-Par la suite, les pièces sont dispérsées à la gauche de l'écrant et sont rendu disponible à l'utilisateurs.
+Par la suite, les pièces sont dispersées à la gauche de l'écran et sont rendues disponibles à l'utilisateur.
-Ces pièces sont déplacables à l'aide de la souris.
+Ces pièces sont déplaçables à l'aide de la souris.
-Si elles sont lachées sur un emplacement disponible du plateau, alors elle vont s'alligner avec ses cellules.
+Si elles sont lâchées sur un emplacement disponible du plateau, alors elles vont s'aligner avec ses cellules.
 Pour se faire, nous regardons si le centre d'une cellule (la plus en haut à gauche) est présente dans une cellule du plateau.
-Ensuite, on vérifie que toutes les cellules de la pièces ont bien la place dans le plateau pour se poser. si c'est le cas, alors la pièce est posée.
+Ensuite, on vérifie que toutes les cellules de la pièce ont bien la place dans le plateau pour se poser. Si c'est le cas, alors la pièce est posée.
 
 \section{Points Faibles}
 
@@ -238,29 +238,25 @@ De plus suite à la perte de notre membre nous n'avons pas su gérer la partie d
 
 \subsection{Anthony}
 
-Personellement, ce projet m'a permis de me plonger dans la conception d'un format de fichier personnalisé.
+Personnellement, ce projet m'a permis de me plonger dans la conception d'un format de fichier personnalisé.
-C'est une chôse que je n'avais pas encore fait jusqu'à maintenant.
+C’est un exercice que je n’avais pas encore fait jusqu’a maintenant. Malgré mes efforts pour prévoir un maximum de choses à l’avance afin d’éviter de devoir modifier ma spécification pendant le développement, je me suis vite rendu compte que je n’avais pas pensé à tout et que je devrais changer des étapes pour pouvoir arriver à mes fins. Je pense que ce parser de fichier est vraiment améliorable mais je suis relativement fier du résultat.
-Et malgré mes efforts pour prévoir un maximum de choses à l'avance afin d'éviter de devoir 
-modifier ma spécification pendant le dévelopement. Je me suis vite rendu compte que je n'avais pas pensé
-à tout et que je devrais changer des choses pour pouvoir arriver à mes fins.
-Je pense que ce parser de fichier est vraiment améliorable mais je suis relativement fier du résultat.
 
 J'ai pu présenter ce parser à Dr Quoitin qui a pu me conseiller sur différentes approches à ce problème.
-J'en prend bonne notes.
+J'en prends bonne note.
 
 \subsection{Matteo}
 
-Il est clair que je peux tirer plusieurs enseignement grâce à la réalisation de notre projet.
+Il est clair que je peux tirer plusieurs enseignements grâce à la réalisation de notre projet.
 Tout d'abord, j'ai pu en apprendre beaucoup plus concernant la P.O.O en java, mais aussi  j'en ai appris
-d'avantage sur l'utilisation de la bibliotheque JavaFx.
+d'avantage sur l'utilisation de la bibliothèque JavaFx.
 
-De plus, durant la réalisation du projet, comme dit précédemment nous avons utilisé plusieurs outils et dont la plus grande découverte pour moi :
+De plus, durant la réalisation du projet, comme dit précédemment, nous avons utilisé plusieurs outils dont la plus grande découverte pour moi est
-Git. Qui révèle son intérêt pour les travaux de groupes, et qui selon moi se révélera tout aussi essentiel pour mes futurs projets
+Git qui révèle son intérêt pour les travaux de groupes et qui, selon moi, se révélera tout aussi essentiel pour mes futurs projets
 scolaires ainsi que professionnels.
 
 \section{conclusion}
 
-En conclusion nous pouvons séparer notre travail en trois partie différentes
+En conclusion, nous pouvons séparer notre travail en trois parties différentes:
 
 \begin{enumerate}
 	\item Le parser de fichier (gestion sauvegarder/charger partie)
@@ -268,9 +264,9 @@ En conclusion nous pouvons séparer notre travail en trois partie différentes
 	\item Liaison à l'UI (Javafx)
 \end{enumerate}
 
-Malgré notre travail concentré sur le bon fonctionnement du jeu avec un parser suivant nos objectifs, une utilisation de la P.O.O de manière très efficace,
+Malgré notre travail concentré sur le bon fonctionnement du jeu avec un parser suivant nos objectifs, une utilisation de la P.O.O de manière très efficace
-ainsi qu'une approche correcte de l'utilisation du framework Javafx, d'autre améliorations sont toujours possible !
+ainsi qu'une approche correcte de l'utilisation du framework Javafx, d'autres améliorations sont toujours possibles!
-En effet l'idée de rajouter une histoire, des trophées, un easter egg, des pièces spéciales ou un encore un table de score basée sur le temps,
+En effet, l'idée d'ajouter une histoire, des trophées, un Easter egg, des pièces spéciales ou un encore une table de score basée sur le temps,
 reste possible afin de rendre notre jeu encore plus complet.
 
 En conclusion, notre jeu a encore plein de possibilité afin d'être encore plus complet et amusant!

app/src/main/java/school_project/Controller.java

@@ -8,7 +8,8 @@ import javafx.scene.input.KeyCode;
 import javafx.scene.input.KeyCombination;
 import javafx.stage.Screen;
 import javafx.stage.Stage;
-import school_project.Menu.MenuAccueil;
+import school_project.Scenes.GameUI;
+import school_project.Scenes.MenuAccueil;
 import school_project.Parsers.FileParserFactory;
 
 import java.io.File;
@@ -55,6 +56,13 @@ public class Controller extends Application {
                 }
             });
         }
+        else if(root instanceof MenuAccueil){
+            scene.setOnKeyPressed(event ->{
+                if(event.getCode()==KeyCode.ESCAPE){
+                    stage.close();
+                }
+            });
+        }
         stage.setScene(scene);
 
         stage.setFullScreen(true);

app/src/main/java/school_project/MapGenerator.java

@@ -93,7 +93,9 @@ public class MapGenerator {
                     }
                 }
             }
-            ret.addPiece(new Piece(shape));
+            Piece piece = new Piece(shape);
+            piece.RotateRight(rand.nextInt(4));
+            ret.addPiece(piece);
         }
 
         //generate pieces

app/src/main/java/school_project/Scenes/GameUI.java

@@ -1,10 +1,14 @@
-package school_project;
+package school_project.Scenes;
 
 import javafx.scene.Group;
 import javafx.scene.input.MouseButton;
 
-import school_project.Menu.ScreenLevelFinish;
+import school_project.Controller;
+import school_project.Map;
+import school_project.Piece;
+import school_project.Scenes.ScreenLevelFinish;
 import school_project.Utils.MatrixShape;
+import school_project.Vec2;
 
 import java.io.FileNotFoundException;
 

app/src/main/java/school_project/Scenes/MenuAccueil.java

@@ -1,4 +1,4 @@
-package school_project.Menu;
+package school_project.Scenes;
 import javafx.geometry.Insets;
 import javafx.geometry.Pos;
 import javafx.scene.control.Button;
@@ -8,7 +8,6 @@ import javafx.scene.layout.StackPane;
 import javafx.scene.paint.Color;
 import javafx.scene.text.Font;
 import school_project.Controller;
-import school_project.GameUI;
 import school_project.MapGenerator;
 import school_project.Parsers.FileParserFactory;
 
@@ -30,7 +29,6 @@ public class MenuAccueil extends StackPane {
         Label Title = new Label("Welcome to Road to Master");
             SlctDifficulty.setOnAction(event -> {
                 String choosediff = SlctDifficulty.getSelectionModel().getSelectedItem();
-                System.out.println(choosediff);
                 switch (choosediff) {
                     case "Easy":
                         try {

app/src/main/java/school_project/Scenes/MenuLevel.java

@@ -1,4 +1,4 @@
-package school_project.Menu;
+package school_project.Scenes;
 
 import javafx.geometry.HPos;
 import javafx.geometry.Insets;
@@ -7,7 +7,6 @@ import javafx.scene.layout.ColumnConstraints;
 import javafx.scene.layout.GridPane;
 import javafx.scene.layout.RowConstraints;
 import school_project.Controller;
-import school_project.GameUI;
 import school_project.Parsers.FileParserFactory;
 
 import java.io.File;
@@ -69,7 +68,6 @@ public class MenuLevel extends GridPane {
                 levelButton.setOnAction(event -> {
                     try {
                         String levelName = ((Button)event.getSource()).getText().replace(" ", "") + ".level";
-                        System.out.println(levelName);
                         GameUI level = new GameUI(FileParserFactory.loadMapFromFile(new File(Controller.class.getResource("levels/" + levelName).getFile())));
                         Controller.switchRoot(level);
                     } catch (IOException e) {

app/src/main/java/school_project/Scenes/ScreenLevelFinish.java

@@ -1,4 +1,4 @@
-package school_project.Menu;
+package school_project.Scenes;
 
 
 import javafx.geometry.Insets;
@@ -9,7 +9,6 @@ import javafx.scene.control.Label;
 import javafx.scene.layout.StackPane;
 import javafx.scene.text.Font;
 import school_project.Controller;
-import school_project.GameUI;
 import school_project.Map;
 import java.io.FileNotFoundException;
 

app/src/main/java/school_project/Utils/MatrixShape.java

@@ -8,6 +8,7 @@ import javafx.scene.layout.Pane;
 import javafx.scene.paint.Paint;
 import javafx.scene.shape.Rectangle;
 import school_project.*;
+import school_project.Scenes.GameUI;
 
 import java.io.FileInputStream;
 import java.io.FileNotFoundException;