From 492196b3334ccb6765574a667c364834541d0acb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?David=20N=C3=A9grier?= Date: Fri, 8 May 2020 15:18:22 +0200 Subject: [PATCH 1/2] Cleanup: renaming "frame" to "character" The "frame" variable actually contains a string pointing to the character selected. It has nothing to do with a frame which is usually a particular image in an animation. I'm renaming the variable accross the application to avoid confusion. --- back/src/Controller/IoSocketController.ts | 2 +- back/src/Model/Websocket/ExSocketInterface.ts | 4 +- back/src/Model/Websocket/ExtRoom.ts | 4 +- back/src/Model/Websocket/Message.ts | 8 ++-- back/tests/MessageTest.ts | 4 +- front/src/Connexion.ts | 42 +++++++++---------- front/src/Phaser/Game/GameManager.ts | 16 +++---- front/src/Phaser/Game/GameScene.ts | 10 ++--- front/src/Phaser/Player/Player.ts | 4 +- 9 files changed, 47 insertions(+), 47 deletions(-) diff --git a/back/src/Controller/IoSocketController.ts b/back/src/Controller/IoSocketController.ts index 64012349..c37165bb 100644 --- a/back/src/Controller/IoSocketController.ts +++ b/back/src/Controller/IoSocketController.ts @@ -216,7 +216,7 @@ export class IoSocketController { socket.roomId = message.roomId; socket.userId = message.userId; socket.name = message.name; - socket.frame = message.frame; + socket.character = message.character; } refreshUserPosition() { diff --git a/back/src/Model/Websocket/ExSocketInterface.ts b/back/src/Model/Websocket/ExSocketInterface.ts index 7798509b..22cd29c2 100644 --- a/back/src/Model/Websocket/ExSocketInterface.ts +++ b/back/src/Model/Websocket/ExSocketInterface.ts @@ -7,6 +7,6 @@ export interface ExSocketInterface extends Socket { webRtcRoomId: string; userId: string; name: string; - frame: string; + character: string; position: PointInterface; -} \ No newline at end of file +} diff --git a/back/src/Model/Websocket/ExtRoom.ts b/back/src/Model/Websocket/ExtRoom.ts index a1b2cb65..723fff0b 100644 --- a/back/src/Model/Websocket/ExtRoom.ts +++ b/back/src/Model/Websocket/ExtRoom.ts @@ -25,7 +25,7 @@ let RefreshUserPositionFunction = function(rooms : ExtRooms, Io: socketIO.Server roomId: socket.roomId, position: socket.position, name: socket.name, - frame: socket.frame, + character: socket.character, }; let dataArray = []; if (mapPositionUserByRoom.get(data.roomId)) { @@ -41,4 +41,4 @@ let RefreshUserPositionFunction = function(rooms : ExtRooms, Io: socketIO.Server export { RefreshUserPositionFunction -} \ No newline at end of file +} diff --git a/back/src/Model/Websocket/Message.ts b/back/src/Model/Websocket/Message.ts index 16589e3b..20df9801 100644 --- a/back/src/Model/Websocket/Message.ts +++ b/back/src/Model/Websocket/Message.ts @@ -2,7 +2,7 @@ export class Message { userId: string; roomId: string; name: string; - frame: string; + character: string; constructor(data: any) { if (!data.userId || !data.roomId) { @@ -11,7 +11,7 @@ export class Message { this.userId = data.userId; this.roomId = data.roomId; this.name = data.name; - this.frame = data.frame; + this.character = data.character; } toJson() { @@ -19,7 +19,7 @@ export class Message { userId: this.userId, roomId: this.roomId, name: this.name, - frame: this.frame + character: this.character } } -} \ No newline at end of file +} diff --git a/back/tests/MessageTest.ts b/back/tests/MessageTest.ts index 47e4cac1..71a79e67 100644 --- a/back/tests/MessageTest.ts +++ b/back/tests/MessageTest.ts @@ -8,7 +8,7 @@ describe("Message Model", () => { expect(messageObject.userId).toBe("test1"); expect(messageObject.roomId).toBe("test2"); expect(messageObject.name).toBe("foo"); - expect(messageObject.frame).toBe("user"); + expect(messageObject.character).toBe("user"); }) it("should expose a toJson method", () => { @@ -37,4 +37,4 @@ describe("Message Model", () => { let messageObject = new Message(message); }).toThrow(new Error("userId or roomId cannot be null")); }); -}) \ No newline at end of file +}) diff --git a/front/src/Connexion.ts b/front/src/Connexion.ts index 4dfddee1..315c67a9 100644 --- a/front/src/Connexion.ts +++ b/front/src/Connexion.ts @@ -18,13 +18,13 @@ class Message { userId: string; roomId: string; name: string; - frame: string; + character: string; - constructor(userId : string, roomId : string, name: string, frame: string) { + constructor(userId : string, roomId : string, name: string, character: string) { this.userId = userId; this.roomId = roomId; this.name = name; - this.frame = frame; + this.character = character; } toJson() { @@ -32,7 +32,7 @@ class Message { userId: this.userId, roomId: this.roomId, name: this.name, - frame: this.frame + character: this.character } } } @@ -71,15 +71,15 @@ export interface MessageUserPositionInterface { userId: string; roomId: string; name: string; - frame: string; + character: string; position: PointInterface; } class MessageUserPosition extends Message implements MessageUserPositionInterface{ position: PointInterface; - constructor(userId : string, roomId : string, point : Point, name: string, frame: string) { - super(userId, roomId, name, frame); + constructor(userId : string, roomId : string, point : Point, name: string, character: string) { + super(userId, roomId, name, character); this.position = point; } @@ -116,7 +116,7 @@ class ListMessageUserPosition { userPosition.position.direction ), userPosition.name, - userPosition.frame + userPosition.character )); }); } @@ -129,11 +129,11 @@ export interface ConnexionInterface { userId: string; startedRoom: string; - createConnexion(frameSelected: string): Promise; + createConnexion(characterSelected: string): Promise; - joinARoom(roomId: string, frame: string): void; + joinARoom(roomId: string, character: string): void; - sharePosition(x: number, y: number, direction: string, frame: string): void; + sharePosition(x: number, y: number, direction: string, character: string): void; positionOfAllUser(): void; @@ -161,7 +161,7 @@ export class Connexion implements ConnexionInterface { this.GameManager = GameManager; } - createConnexion(frameSelected: string): Promise { + createConnexion(characterSelected: string): Promise { return Axios.post(`${API_URL}/login`, {email: this.email}) .then((res) => { this.token = res.data.token; @@ -175,10 +175,10 @@ export class Connexion implements ConnexionInterface { }); //join the room - this.joinARoom(this.startedRoom, frameSelected); + this.joinARoom(this.startedRoom, characterSelected); //share your first position - this.sharePosition(0, 0, frameSelected); + this.sharePosition(0, 0, characterSelected); this.positionOfAllUser(); @@ -195,15 +195,15 @@ export class Connexion implements ConnexionInterface { /** * * @param roomId - * @param frame + * @param character */ - joinARoom(roomId: string, frame: string): void { + joinARoom(roomId: string, character: string): void { let messageUserPosition = new MessageUserPosition( this.userId, this.startedRoom, new Point(0, 0), this.email, - frame + character ); this.socket.emit(EventMessage.JOIN_ROOM, messageUserPosition.toString()); } @@ -212,10 +212,10 @@ export class Connexion implements ConnexionInterface { * * @param x * @param y - * @param frame + * @param character * @param direction */ - sharePosition(x : number, y : number, frame : string, direction : string = "none") : void{ + sharePosition(x : number, y : number, character : string, direction : string = "none") : void{ if(!this.socket){ return; } @@ -224,7 +224,7 @@ export class Connexion implements ConnexionInterface { ROOM[0], new Point(x, y, direction), this.email, - frame + character ); this.socket.emit(EventMessage.USER_POSITION, messageUserPosition.toString()); } @@ -280,4 +280,4 @@ export class Connexion implements ConnexionInterface { disconnectMessage(callback: Function): void { this.socket.on(EventMessage.WEBRTC_DISCONNECT, callback); } -} \ No newline at end of file +} diff --git a/front/src/Phaser/Game/GameManager.ts b/front/src/Phaser/Game/GameManager.ts index 4cc95c2a..dd3d6e58 100644 --- a/front/src/Phaser/Game/GameManager.ts +++ b/front/src/Phaser/Game/GameManager.ts @@ -13,7 +13,7 @@ export interface HasMovedEvent { direction: string; x: number; y: number; - frame: string; + character: string; } export class GameManager { @@ -22,17 +22,17 @@ export class GameManager { private currentGameScene: GameScene; private playerName: string; SimplePeer : SimplePeerInterface; - private frameUserSelected: string; + private characterUserSelected: string; constructor() { this.status = StatusGameManagerEnum.IN_PROGRESS; } - connect(name: string, frameUserSelected : string) { + connect(name: string, characterUserSelected : string) { this.playerName = name; - this.frameUserSelected = frameUserSelected; + this.characterUserSelected = characterUserSelected; this.ConnexionInstance = new Connexion(name, this); - return this.ConnexionInstance.createConnexion(frameUserSelected).then(() => { + return this.ConnexionInstance.createConnexion(characterUserSelected).then(() => { this.SimplePeer = new SimplePeer(this.ConnexionInstance); }); } @@ -70,12 +70,12 @@ export class GameManager { return this.playerName; } - getFrameSelected(): string { - return this.frameUserSelected; + getCharacterSelected(): string { + return this.characterUserSelected; } pushPlayerPosition(event: HasMovedEvent) { - this.ConnexionInstance.sharePosition(event.x, event.y, event.frame, event.direction); + this.ConnexionInstance.sharePosition(event.x, event.y, event.character, event.direction); } } diff --git a/front/src/Phaser/Game/GameScene.ts b/front/src/Phaser/Game/GameScene.ts index f449f076..c506dadb 100644 --- a/front/src/Phaser/Game/GameScene.ts +++ b/front/src/Phaser/Game/GameScene.ts @@ -167,7 +167,7 @@ export class GameScene extends Phaser.Scene implements GameSceneInterface{ this.startX, this.startY, this.GameManager.getPlayerName(), - this.GameManager.getFrameSelected() + this.GameManager.getCharacterSelected() ); this.CurrentPlayer.initAnimation(); @@ -261,15 +261,15 @@ export class GameScene extends Phaser.Scene implements GameSceneInterface{ MessageUserPosition.position.x, MessageUserPosition.position.y, MessageUserPosition.name, - MessageUserPosition.frame + MessageUserPosition.character ); player.initAnimation(); this.MapPlayers.add(player); player.updatePosition(MessageUserPosition); - //init colision - this.physics.add.collider(this.CurrentPlayer, player, (CurrentPlayer: CurrentGamerInterface, MapPlayer: GamerInterface) => { + //init collision + /*this.physics.add.collider(this.CurrentPlayer, player, (CurrentPlayer: CurrentGamerInterface, MapPlayer: GamerInterface) => { CurrentPlayer.say("Hello, how are you ? "); - }); + });*/ } } diff --git a/front/src/Phaser/Player/Player.ts b/front/src/Phaser/Player/Player.ts index 4dbda5bb..572dcd8e 100644 --- a/front/src/Phaser/Player/Player.ts +++ b/front/src/Phaser/Player/Player.ts @@ -83,12 +83,12 @@ export class Player extends PlayableCaracter implements CurrentGamerInterface, G } if (x !== 0 || y !== 0) { this.move(x, y); - this.emit(hasMovedEventName, {direction, x: this.x, y: this.y, frame: this.PlayerTexture}); + this.emit(hasMovedEventName, {direction, x: this.x, y: this.y, character: this.PlayerTexture}); } else { if (this.previousMove !== PlayerAnimationNames.None) { direction = PlayerAnimationNames.None; this.stop(); - this.emit(hasMovedEventName, {direction, x: this.x, y: this.y, frame: this.PlayerTexture}); + this.emit(hasMovedEventName, {direction, x: this.x, y: this.y, character: this.PlayerTexture}); } } From 8e9c1cac1ebe20e2301de5b6631d2b1651006cff Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?David=20N=C3=A9grier?= Date: Fri, 8 May 2020 15:20:49 +0200 Subject: [PATCH 2/2] Renaming frame in tests --- back/tests/MessageTest.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/back/tests/MessageTest.ts b/back/tests/MessageTest.ts index 71a79e67..d908e12e 100644 --- a/back/tests/MessageTest.ts +++ b/back/tests/MessageTest.ts @@ -3,7 +3,7 @@ import {Message} from "../src/Model/Websocket/Message"; describe("Message Model", () => { it("should find userId and roomId", () => { - let message = {userId: "test1", roomId: "test2", name: "foo", frame: "user"}; + let message = {userId: "test1", roomId: "test2", name: "foo", character: "user"}; let messageObject = new Message(message); expect(messageObject.userId).toBe("test1"); expect(messageObject.roomId).toBe("test2"); @@ -12,9 +12,9 @@ describe("Message Model", () => { }) it("should expose a toJson method", () => { - let message = {userId: "test1", roomId: "test2", name: "foo", frame: "user"}; + let message = {userId: "test1", roomId: "test2", name: "foo", character: "user"}; let messageObject = new Message(message); - expect(messageObject.toJson()).toEqual({userId: "test1", roomId: "test2", name: "foo", frame: "user"}); + expect(messageObject.toJson()).toEqual({userId: "test1", roomId: "test2", name: "foo", character: "user"}); }); it("should find throw error when no userId", () => {