Conversation
games/ctf/src/main/java/fr/efreicraft/ludos/games/ctf/EventListener.java
Outdated
Show resolved
Hide resolved
games/ctf/src/main/java/fr/efreicraft/ludos/games/ctf/EventListener.java
Show resolved
Hide resolved
games/ctf/src/main/java/fr/efreicraft/ludos/games/ctf/GameLogic.java
Outdated
Show resolved
Hide resolved
games/ctf/src/main/java/fr/efreicraft/ludos/games/ctf/GameLogic.java
Outdated
Show resolved
Hide resolved
games/ctf/src/main/java/fr/efreicraft/ludos/games/ctf/GameLogic.java
Outdated
Show resolved
Hide resolved
games/ctf/src/main/java/fr/efreicraft/ludos/games/ctf/GameLogic.java
Outdated
Show resolved
Hide resolved
(et le nom des teams en gras bien évidemment)
|
Hésite pas à ajouter une cartouche en haut de tes classes pour expliquer l"utilité de chaque classe et indiquer ton nom + prénom pour être présenté en qualité d'auteur. |
(+ plus besoins de poser le drapeau pour marquer des points)
(+ messages globaux)
| } | ||
| } | ||
|
|
||
| public String getTeamColorCode(String teamName) { |
There was a problem hiding this comment.
Normalement, tu peux récupérer la couleur d'une team avec le getTeam du Team manager.
Aussi, tu peux renommer les teams en "Rouge" et "Bleu" si tu le souhaites.
(+ commentaires au dessus des classes)
(+ stocker le rayon des bases dans une constante)
(et ne jamais le drop en dessous de la killzone)
games/ctf/src/main/java/fr/efreicraft/ludos/games/ctf/CtfWinners.java
Outdated
Show resolved
Hide resolved
# Conflicts: # settings.gradle.kts
JiveOff
left a comment
JiveOff
left a comment
There was a problem hiding this comment.
Petite review rapide que je fais sur tel, donc j'ai peut-être raté des trucs, notamment le formatage
| private static final double BASE_RADIUS = 4; | ||
|
|
||
|
|
||
| public GameLogic(LudosGame ludosGame1) { |
There was a problem hiding this comment.
Tu peux utiliser une classe record pour ne pas avoir de constructeur comme ça
| * @param red_location position initiale du drapeau rouge | ||
| * @param blue_location position initiale du drapeau bleu | ||
| */ | ||
| public void initFlags(Location red_location, Location blue_location) { |
There was a problem hiding this comment.
Généralement, on essaye de privilégier un seul casing pour les arguments de méthodes, il faut donc éviter le snake case occasionnel.
|
|
||
| Material helmetSlotMaterial = helmetItem.getType(); | ||
| player.entity().getInventory().clear(39); //39 = slot helmet | ||
| player.entity().getInventory().clear(1); // 1 = deuxième slot hotbar |
There was a problem hiding this comment.
Attention aux clear dans les slots quand les joueurs peuvent manipuler leur inventaire.
There was a problem hiding this comment.
Au final j'ai pas fait avec les nbt, j'utilise juste inventory.remove pour retirer les drapeaux
There was a problem hiding this comment.
Je sais pas si c'est une bonne idée de faire ce que tu as fait avec le slot du helmet mais bon, ça devrait être bon.
games/ctf/src/main/java/fr/efreicraft/ludos/games/ctf/GameLogic.java
Outdated
Show resolved
Hide resolved
| } | ||
|
|
||
| public void setWorld(World world1) { | ||
| world = world1; |
There was a problem hiding this comment.
Je préfère que tu utilise un game point pour déterminer le monde de jeu
| @Override | ||
| public void beginGame() { | ||
| super.beginGame(); | ||
| GameTimer gt = new GameTimer(gameLogic::spawnParticles, -1, 10); |
There was a problem hiding this comment.
J'aurais bien vu les routines de particules dans une classe séparée avec une méthode d'initialisation des timers
There was a problem hiding this comment.
Je viens de le faire mais je suis vraiment pas sûr de si c'est vraiment ce que tu voulais
|
|
||
| @Override | ||
| public void postMapParse() { | ||
| Map<String, ArrayList<GamePoint>> game_points = Core.get().getMapManager().getCurrentMap().getGamePoints(); |
Co-authored-by: Antoine <antoine@jiveoff.fr>
(sauf le casque)
3 participants
No description provided.