rpc优化：所有remote加上try catch

game-server/app/servers/role/remote/roleRemote.ts

@@ -12,6 +12,7 @@ import { getDefaultRoleInfo } from '../../../services/roleService';
 import { PVPConfigModel, PVPConfigType } from '../../../db/SystemConfig';
 import { treatRoleName, taflush } from '../../../services/sdkService';
 import { getServerMainten, setServerMainten, stopServerMainten } from '../../../services/gmService';
+import { errlogger } from '../../../util/logger';
 
 export default function (app: Application) {
     new HandlerService(app, {});
@@ -46,64 +47,93 @@ export class RoleRemote {
     }
 
     public setRankFirst(rank: RankFirstType) {
-        // console.log('****** setRankFirst', rank.serverId, rank.id, rank)
-        if(!this.rankFirstRewards.has(rank.serverId)) {
-            this.rankFirstRewards.set(rank.serverId, new Map());
+        try {
+            // console.log('****** setRankFirst', rank.serverId, rank.id, rank)
+            if(!this.rankFirstRewards.has(rank.serverId)) {
+                this.rankFirstRewards.set(rank.serverId, new Map());
+            }
+            this.rankFirstRewards.get(rank.serverId).set(rank.id, rank);
+            this.app.set('rankFirstRewards', this.rankFirstRewards);
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
         }
-        this.rankFirstRewards.get(rank.serverId).set(rank.id, rank);
-        this.app.set('rankFirstRewards', this.rankFirstRewards);
+
     }
 
     public getRankFirstById(serverId: number, id: number) {
-        return this.rankFirstRewards.get(serverId)?.get(id);
+        try {
+            return this.rankFirstRewards.get(serverId)?.get(id);
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
+        }
     }
 
     public setInitRole() {
-        let result = getInitRoleInfo();
-        let { role, heroes, skins, figureInfo } = result;
-        for(let hero of heroes) {
-            this.initHeroes.set(hero.hid, hero);
+        try {
+            let result = getInitRoleInfo();
+            let { role, heroes, skins, figureInfo } = result;
+            for(let hero of heroes) {
+                this.initHeroes.set(hero.hid, hero);
+            }
+            for(let skin of skins) {
+                this.initSkins.set(skin.hid, skin);
+            }
+            this.initRole = role;
+            this.figureInfo = figureInfo;
+            const initRoleInfos = getDefaultRoleInfo(this.initHeroes, this.initSkins, this.initRole, this.figureInfo);
+            this.app.set('initRoleInfos', initRoleInfos);
+            this.app.set('initHeroes', this.initHeroes);
+            this.app.set('initSkins', this.initSkins);
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
         }
-        for(let skin of skins) {
-            this.initSkins.set(skin.hid, skin);
-        }
-        this.initRole = role;
-        this.figureInfo = figureInfo;
-        const initRoleInfos = getDefaultRoleInfo(this.initHeroes, this.initSkins, this.initRole, this.figureInfo);
-        this.app.set('initRoleInfos', initRoleInfos);
-        this.app.set('initHeroes', this.initHeroes);
-        this.app.set('initSkins', this.initSkins);
     }
 
     public getInitRoleInfos() {
-        let initRoleInfos = this.app.get('initRoleInfos');
-        if (!initRoleInfos) {
-            initRoleInfos = getDefaultRoleInfo(this.initHeroes, this.initSkins, this.initRole, this.figureInfo);
-            this.app.set('initRoleInfos', initRoleInfos);
+        try {
+            let initRoleInfos = this.app.get('initRoleInfos');
+            if (!initRoleInfos) {
+                initRoleInfos = getDefaultRoleInfo(this.initHeroes, this.initSkins, this.initRole, this.figureInfo);
+                this.app.set('initRoleInfos', initRoleInfos);
+            }
+            return initRoleInfos;
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
         }
-        return initRoleInfos;
     }
 
     public getInitHeroes() {
-        let result: HeroUpdate[] = [];
-        for(let hid of DEFAULT_HEROES) {
-            result.push(this.initHeroes.get(hid));
+        try {
+            let result: HeroUpdate[] = [];
+            for(let hid of DEFAULT_HEROES) {
+                result.push(this.initHeroes.get(hid));
+            }
+            return result;
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
         }
-        return result;
     }
 
     public getInitSkins() {
-        let result: SkinUpdate[] = [];
-        for(let hid of DEFAULT_HEROES) {
-            result.push(this.initSkins.get(hid));
+        try {
+            let result: SkinUpdate[] = [];
+            for(let hid of DEFAULT_HEROES) {
+                result.push(this.initSkins.get(hid));
+            }
+            return result;
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
         }
-        return result;
     }
 
     public getInitHeroById(hid: number) {
-        return {
-            heroInfo: this.initHeroes.get(hid),
-            skinInfo: this.initSkins.get(hid)
+        try {
+            return {
+                heroInfo: this.initHeroes.get(hid),
+                skinInfo: this.initSkins.get(hid)
+            }
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
         }
     }
 
@@ -112,39 +142,71 @@ export class RoleRemote {
      * 重载json资源
      */
     public async reloadResources() {
-        reloadResources();
+        try {
+            reloadResources();
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
+        }
     }
 
 
     public setPvpSeasonNum(pvpConfig: PVPConfigType) {
-        if(pvpConfig) {
-            this.app.set('pvpSeasonNum', pvpConfig.seasonNum);
-            this.app.set('pvpSeasonEndTime', pvpConfig.seasonEndTime);
+        try {
+            if(pvpConfig) {
+                this.app.set('pvpSeasonNum', pvpConfig.seasonNum);
+                this.app.set('pvpSeasonEndTime', pvpConfig.seasonEndTime);
+            }
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
         }
     }
 
     public async initPvpSeasonNum() {
-        let pvpConfig = await PVPConfigModel.findCurPVPConfig();
-        this.setPvpSeasonNum(pvpConfig);
+        try {
+            let pvpConfig = await PVPConfigModel.findCurPVPConfig();
+            this.setPvpSeasonNum(pvpConfig);
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
+        }
     }
 
     public async treatRoleName(roleId: string) {
-        await treatRoleName(roleId);
+        try {
+            await treatRoleName(roleId);
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
+        }
     }
 
     public setServerMainten(serverIds: number[], startTime: number, endTime: number) {
-        setServerMainten(serverIds, startTime, endTime);
+        try {
+            setServerMainten(serverIds, startTime, endTime);
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
+        }
     }
 
     public stopServerMainten(serverIds: number[]) {
-        stopServerMainten(serverIds);
+        try {
+            stopServerMainten(serverIds);
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
+        }
     }
 
     public getServerMainten(serverId: number) {
-        return getServerMainten(serverId);
+        try {
+            return getServerMainten(serverId);
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
+        }
     }
 
     public taflush() {
-        return taflush();
+        try {
+            return taflush();
+        } catch(e) {
+            errlogger.error(`remote ${__filename} \n ${e.stack}`);
+        }
     }
 }