siwon / projetBomberman

Projet de programmation objet C++
6 stars 2 forks source link

Fichier de configuration #13

Closed abisiaux closed 12 years ago

abisiaux commented 12 years ago

Je vois pas sous quelle forme stocker la configuration des joueurs sachant qu'il faut stocker le numéro du joueur, le type de contrôleur utilisé, l'action et la touche correspondante et tout ça sous forme de clé : valeur !

Jaconil commented 12 years ago

controller.player1.type: keyboard controller.player1.up: 62 (le numéro de la touche) ... controller.player2.type: gamepad controller.player2.up: y-axis1 (enfin un mot-clé que tu interprètes après... genre x-axis, y-axis, button1, ou un numéro de bouton, etc...)

idem pour la wii

abisiaux commented 12 years ago

ok merci

abisiaux commented 12 years ago

Autre petit question :

Pourquoi ConfigFileManager n'est pas un singleton car il n'y a qu'un seul fichier de config ?

Jaconil commented 12 years ago

non, il y a un fichier de configuration général, plus un fichier de conf par skin. C'est pour ça que je l'ai pas mis en singleton il me semble, et que j'ai mis le chemin d'accès en paramètre optionnel au constructeur (d'ailleurs avec le singleton on peut pas mettre de paramètre au constructeur).

abisiaux commented 12 years ago

Le 06/05/2012 17:10, GUIHAL Maxime a écrit :

non, il y a un fichier de configuration général, plus un fichier de conf par skin. C'est pour ça que je l'ai pas mis en singleton il me semble, et que j'ai mis le chemin d'accès en paramètre optionnel au constructeur (d'ailleurs avec le singleton on peut pas mettre de paramètre au constructeur).


Reply to this email directly or view it on GitHub: https://github.com/siwon/projetBomberman/issues/13#issuecomment-5536355 ok j'avais pas pigé ça

abisiaux commented 12 years ago

Le 06/05/2012 17:10, GUIHAL Maxime a écrit :

non, il y a un fichier de configuration général, plus un fichier de conf par skin. C'est pour ça que je l'ai pas mis en singleton il me semble, et que j'ai mis le chemin d'accès en paramètre optionnel au constructeur (d'ailleurs avec le singleton on peut pas mettre de paramètre au constructeur).


Reply to this email directly or view it on GitHub: https://github.com/siwon/projetBomberman/issues/13#issuecomment-5536355 Un truc vraiment bizarre un constructeur virtuel dans l'interface ConfigFile ! Ça existe ?

Jaconil commented 12 years ago

Non ça n'existe pas ^^ C'était pour forcer la classe fille à implémenter un constructeur avec un argument, mais on peut pas je crois, il faudra modifier...

abisiaux commented 12 years ago

Le 06/05/2012 17:18, GUIHAL Maxime a écrit :

Non ça n'existe pas ^^ C'était pour forcer la classe fille à implémenter un constructeur avec un argument, mais on peut pas je crois, il faudra modifier...


Reply to this email directly or view it on GitHub: https://github.com/siwon/projetBomberman/issues/13#issuecomment-5536432 Oui c'est clair qu'on peut pas ! Mais tkt je vais implémenter le constructeur du manager avec un paramètre

Jaconil commented 12 years ago

Paramètre optionnel hein, si on met rien ça redirige vers le fichier de conf principal ;)

abisiaux commented 12 years ago

Le 06/05/2012 17:21, GUIHAL Maxime a écrit :

Paramètre optionnel hein, si on met rien ça redirige vers le fichier de conf principal ;)


Reply to this email directly or view it on GitHub: https://github.com/siwon/projetBomberman/issues/13#issuecomment-5536453 Qui a pour nom polyBomber.conf

abisiaux commented 12 years ago

Le 06/05/2012 17:21, GUIHAL Maxime a écrit :

Paramètre optionnel hein, si on met rien ça redirige vers le fichier de conf principal ;)


Reply to this email directly or view it on GitHub: https://github.com/siwon/projetBomberman/issues/13#issuecomment-5536453 Mettre une valeur par défaut à un paramètre ne passe pas au compilateur !

Jaconil commented 12 years ago

Oui non excuse c'est en java que ça passe, mais surcharge le constructeur. Fais en un sans paramètres, et l'autre avec un ;)

abisiaux commented 12 years ago

Le 06/05/2012 17:40, GUIHAL Maxime a écrit :

Oui non excuse c'est en java que ça passe, mais surcharge le constructeur. Fais en un sans paramètres, et l'autre avec un ;)


Reply to this email directly or view it on GitHub: https://github.com/siwon/projetBomberman/issues/13#issuecomment-5536577 ok vive le c++

abisiaux commented 12 years ago

Juconil, tu peux me dire ce que doit faire la fonction createDefault() de la classe ConfigFileManager ?

Jaconil commented 12 years ago

Cette fonction est appelée si le fichier de configuration appelé n'existe pas. Elle crée alors un fichier minimal avec les valeurs par défaut des touches, des fenêtres, etc...

Remplis les valeurs pour les contrôleurs, j'éditerai cette fonction pour mettre les miennes ;)