Code cleaning

diff --git a/back/src/Controller/IoSocketController.ts b/back/src/Controller/IoSocketController.ts
index 9ea88991e4dc6f76ddbde0fc6327bd2b7569a290..2c79879741e81c6bb8170f726c20eeeb5598fd57 100644 (file)
--- a/back/src/Controller/IoSocketController.ts
+++ b/back/src/Controller/IoSocketController.ts
@@ -94,10 +94,11 @@ export class IoSocketController {
                         x: user x position on map
                         y: user y position on map
             */
-            socket.on(SockerIoEvent.JOIN_ROOM, (roomId: any) => {
+            socket.on(SockerIoEvent.JOIN_ROOM, (roomId: any): void => {
                 try {
                     if (typeof(roomId) !== 'string') {
-                        return socket.emit(SockerIoEvent.MESSAGE_ERROR, {message: 'Expected roomId as a string.'})
+                        socket.emit(SockerIoEvent.MESSAGE_ERROR, {message: 'Expected roomId as a string.'});
+                        return;
                     }
 
                     let Client = (socket as ExSocketInterface);
@@ -124,11 +125,12 @@ export class IoSocketController {
                 }
             });
 
-            socket.on(SockerIoEvent.USER_POSITION, (message: any) => {
+            socket.on(SockerIoEvent.USER_POSITION, (message: any): void => {
                 try {
                     let position = this.hydratePositionReceive(message);
                     if (position instanceof Error) {
-                        return socket.emit(SockerIoEvent.MESSAGE_ERROR, {message: position.message});
+                        socket.emit(SockerIoEvent.MESSAGE_ERROR, {message: position.message});
+                        return;
                     }
 
                     let Client = (socket as ExSocketInterface);

diff --git a/back/tsconfig.json b/back/tsconfig.json
index 7686fb67270854f155eda0702a0659a4182c42e9..03f8c9d8b1eb51b00ad04490455f58b8c29e1095 100644 (file)
--- a/back/tsconfig.json
+++ b/back/tsconfig.json
@@ -36,7 +36,7 @@
     /* Additional Checks */
     // "noUnusedLocals": true,                /* Report errors on unused locals. */
     // "noUnusedParameters": true,            /* Report errors on unused parameters. */
-    // "noImplicitReturns": true,             /* Report error when not all code paths in function return a value. */
+    "noImplicitReturns": true,             /* Report error when not all code paths in function return a value. */
     // "noFallthroughCasesInSwitch": true,    /* Report errors for fallthrough cases in switch statement. */
 
     /* Module Resolution Options */

diff --git a/front/src/WebRtc/SimplePeer.ts b/front/src/WebRtc/SimplePeer.ts
index 54b7e0ed51ec7127ceb9dced4fb1421db19a3214..595e83b2eca0f7da7d60c405caccd171a9d0268d 100644 (file)
--- a/front/src/WebRtc/SimplePeer.ts
+++ b/front/src/WebRtc/SimplePeer.ts
@@ -49,18 +49,12 @@ export class SimplePeer implements SimplePeerInterface{
         });
 
         //receive signal by gemer
-        this.Connexion.disconnectMessage((message: any) => {
-            let data = message;
+        this.Connexion.disconnectMessage((data: any) => {
             this.closeConnexion(data.userId);
         });
     }
 
-    /**
-     *
-     * @param message
-     */
-    private receiveWebrtcStart(message: any) {
-        let data = message;
+    private receiveWebrtcStart(data: any) {
         this.WebRtcRoomId = data.roomId;
         this.Users = data.clients;
 
@@ -193,12 +187,7 @@ export class SimplePeer implements SimplePeerInterface{
         }
     }
 
-    /**
-     *
-     * @param message
-     */
-    private receiveWebrtcSignal(message: any) {
-        let data = message;
+    private receiveWebrtcSignal(data: any) {
         try {
             //if offer type, create peer connexion
             if(data.signal.type === "offer"){