From e3f06975d30323f54637b89ef027d336ac0dba19 Mon Sep 17 00:00:00 2001
From: Henri Montjoie <henri.montjoie@insa-rennes.fr>
Date: Wed, 11 May 2016 22:03:09 +0200
Subject: [PATCH] moins d'emploi de constantes directement dans le code

---
 src/affichage.c | 40 +++++++++++++++++-----------------------
 src/affichage.h |  2 +-
 src/attaques.c  |  2 +-
 src/main.c      |  1 -
 4 files changed, 19 insertions(+), 26 deletions(-)

diff --git a/src/affichage.c b/src/affichage.c
index fea5ae5..9f9062a 100644
--- a/src/affichage.c
+++ b/src/affichage.c
@@ -58,7 +58,6 @@ int main_affichage()
                 break;
             default:
                 break;
-
         }
     }
     TTF_Quit();
@@ -227,8 +226,8 @@ int saisie_noms_joueurs()
  */
 int affichage_ecran_accueil()
 {
-    ajouterUnJoueur(nom_joueur1,&etat);
-    ajouterUnJoueur(nom_joueur2,&etat);
+    ajouterUnJoueur(nom_1,&etat);
+    ajouterUnJoueur(nom_2,&etat);
     etat=LANCEMENT;
     SDL_Surface* ecran = SDL_SetVideoMode(LARGEUR_CARTE*(LARGEUR_CASE+1), HAUTEUR_CARTE*(HAUTEUR_CASE+2), 32, SDL_HWSURFACE|SDL_DOUBLEBUF);
     SDL_WM_SetCaption("Elder Internal Ignition",NULL);
@@ -347,32 +346,25 @@ int affichage_menu_configuration()
     police=TTF_OpenFont("../resources/fonts/arial.ttf",26);
 
     char src[50], dest[50];
-    char buffer[3];
-    for(i=0;i<3;i++)
+    char buffer[2];
+    for(i=0;i<NB_PERSO;i++)
     {
         strcpy(dest,  "personnage ");
         strcpy(src, itoa(i+1,buffer,10));
         textes[i]= TTF_RenderText_Shaded(police, strcat(dest, src), couleurBlanche,couleurNoire);
-    }
-    for(i=3;i<6;i++)
-    {
-        strcpy(dest,  "personnage ");
-        strcpy(src, itoa(i-2,buffer,10));
-        textes[i]= TTF_RenderText_Shaded(police, strcat(dest, src), couleurBlanche,couleurNoire);
-    }
-    for(i=0;i<3;i++)
-    {
         posTextes[i].x=ecran->w/4-textes[i]->w/2;
         posTextes[i].y=(i+1)*ecran->h/6;
         SDL_BlitSurface(textes[i],NULL,ecran,&(posTextes[i]));
     }
-    for(i=3;i<6;i++)
+    for(i=NB_PERSO;i<NB_JOUEURS*NB_PERSO;i++)
     {
+        strcpy(dest,  "personnage ");
+        strcpy(src, itoa(i-NB_PERSO+1,buffer,10));
+        textes[i]= TTF_RenderText_Shaded(police, strcat(dest, src), couleurBlanche,couleurNoire);
         posTextes[i].x=(3*ecran->w)/4-textes[i]->w/2;
-        posTextes[i].y=posTextes[i-3].y;
+        posTextes[i].y=posTextes[i-NB_PERSO].y;
         SDL_BlitSurface(textes[i],NULL,ecran,&posTextes[i]);
     }
-    SDL_Flip(ecran);
 
     char** path_images=obtenirSkinsClasses();
     int numClassePerso[3][2]={{-1,-1},{-1,-1},{-1,-1}};
@@ -416,6 +408,7 @@ int affichage_menu_configuration()
         position_carte[i].y=(ecran->h)-(previsualisation[i]->h+100);
         position_carte[i].x=(ecran->w-previsualisation[i]->w)/2+(10+previsualisation[i]->w)*(i-2);
     }
+    SDL_Flip(ecran);
 
     c=-1; /*le numéro de la carte choisie*/
     while(etat==CONFIGURATION)
@@ -425,6 +418,7 @@ int affichage_menu_configuration()
         {
             case SDL_QUIT:
                 etat=FERMER;
+                return 0;
                 break;
             case SDL_MOUSEBUTTONDOWN:
                 for(i=0;i<5;i++)
@@ -432,14 +426,14 @@ int affichage_menu_configuration()
                     if(event.button.x>position_carte[i].x&&event.button.x<position_carte[i].x+previsualisation[0]->w&&event.button.y>position_carte[i].y&&event.button.y<position_carte[i].y+previsualisation[0]->h)
                     {
                         c=i;
-                        selectionnerCarte(obtenirNomsCartes()[c]);
+                        //selectionnerCarte(obtenirNomsCartes()[c]);
                     }
                 }
                 if(event.button.x>posTextes[0].x&&event.button.x<posTextes[0].x+textes[0]->w)
                     j=1;
-                if(event.button.x>posTextes[3].x&&event.button.x<posTextes[3].x+textes[3]->w)
+                if(event.button.x>posTextes[NB_PERSO].x&&event.button.x<posTextes[NB_PERSO].x+textes[NB_PERSO]->w)
                     j=2;
-                for(i=0;i<3;i++)
+                for(i=0;i<NB_PERSO;i++)
                 {
                     if(event.button.y>posTextes[i].y&&event.button.y<posTextes[i].y+textes[i]->h)
                         p=i+1;
@@ -457,12 +451,12 @@ int affichage_menu_configuration()
             default:
                 break;
         }
-        for(i=0;i<6;i++)
+        for(i=0;i<NB_JOUEURS*NB_PERSO;i++)
         {
             SDL_BlitSurface(textes[i],NULL,ecran,&(posTextes[i]));
         }
-        for(i=0;i<3;i++){
-            for(k=0;k<2;k++){
+        for(i=0;i<NB_PERSO;i++){
+            for(k=0;k<NB_JOUEURS;k++){
                 positions_images[i][k].y=posTextes[i].y+textes[i]->h;
                 positions_images[i][k].x=posTextes[NB_PERSO*k].x+(textes[NB_PERSO*k]->w-images_classes[0]->w)/2;
                 images_persos[i][k]=images_classes[numClassePerso[i][k]];
diff --git a/src/affichage.h b/src/affichage.h
index 3978af6..dc1209f 100644
--- a/src/affichage.h
+++ b/src/affichage.h
@@ -12,7 +12,7 @@
 #define LARGEUR_CASE 53
 #define HAUTEUR_CASE 71
 #define NB_PERSO 3
-#define NB_JOUEURS 3
+#define NB_JOUEURS 2
 #define HAUTEUR_BOUTON 20
 #define LARGEUR_BOUTON 115
 
diff --git a/src/attaques.c b/src/attaques.c
index 2d28808..70961a9 100644
--- a/src/attaques.c
+++ b/src/attaques.c
@@ -17,7 +17,7 @@ int attaquer(Personnage *attaquant, Attaque *attaque_select, Personnage *cible)
     int i=0;
     for(i=0; i<5; i++)
     {
-        ajouter_effet(attaquant, attaque_select->effets[i], cible);
+       // ajouter_effet(attaquant, attaque_select->effets[i], cible);
     }
     return 0;
 }
diff --git a/src/main.c b/src/main.c
index e441934..0e1dcd9 100644
--- a/src/main.c
+++ b/src/main.c
@@ -92,6 +92,5 @@ int main(void)
     initJeu();
     main_affichage();
 
-
     return 0;
 }
-- 
GitLab