From 1fe4cd65a8bad383e543fe5a631cd9710dd29de7 Mon Sep 17 00:00:00 2001 From: wh_zcy Date: Tue, 6 Jun 2023 10:16:59 +0800 Subject: [PATCH] =?UTF-8?q?=E4=BF=AE=E6=94=B9ErrorData?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- modules/worldtask/api_accept.go | 49 ++++++++++++++++--------- modules/worldtask/api_battlefinish.go | 53 ++++++++++++++++++--------- modules/worldtask/api_battlestart.go | 34 +++++++++++------ 3 files changed, 90 insertions(+), 46 deletions(-) diff --git a/modules/worldtask/api_accept.go b/modules/worldtask/api_accept.go index 5098bf4c9..ed251cd29 100644 --- a/modules/worldtask/api_accept.go +++ b/modules/worldtask/api_accept.go @@ -19,50 +19,60 @@ func (a *apiComp) AcceptCheck(session comm.IUserSession, req *pb.WorldtaskAccept } func (a *apiComp) Accept(session comm.IUserSession, req *pb.WorldtaskAcceptReq) (errdata *pb.ErrorData) { - if code = a.AcceptCheck(session, req); code != pb.ErrorCode_Success { + if errdata = a.AcceptCheck(session, req); errdata != nil { return } - data = &pb.ErrorData{} + uid := session.GetUserId() user := a.module.ModuleUser.GetUser(uid) if user == nil { - code = pb.ErrorCode_UserSessionNobeing - data.Title = code.ToString() - data.Datastring = uid + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_UserSessionNobeing, + Title: pb.ErrorCode_UserSessionNobeing.ToString(), + Datastring: uid, + } return } rsp := &pb.WorldtaskAcceptResp{} myWorldtask, err := a.module.modelWorldtask.getWorldtask(uid) if err != nil { - code = pb.ErrorCode_DBError - data.Title = code.ToString() - data.Datastring = uid + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_DBError, + Title: pb.ErrorCode_DBError.ToString(), + Datastring: uid, + } return } // 当前任务配置 curTaskConf, err := a.module.configure.getWorldtaskById(req.TaskId) if err != nil || curTaskConf == nil { - code = pb.ErrorCode_ConfigNoFound - data.Title = code.ToString() - data.Message = err.Error() + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_ConfigNoFound, + Title: pb.ErrorCode_ConfigNoFound.ToString(), + Message: err.Error(), + } return } // 判断玩家等级要求 if user.Lv < curTaskConf.Lock { - code = pb.ErrorCode_WorldtaskLvNotEnough - data.Title = code.ToString() - data.Message = fmt.Sprintf("实际等级:%d 期望等级:%d", user.Lv, curTaskConf.Lock) + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_WorldtaskLvNotEnough, + Title: pb.ErrorCode_WorldtaskLvNotEnough.ToString(), + Message: fmt.Sprintf("实际等级:%d 期望等级:%d", user.Lv, curTaskConf.Lock), + } return } // 前置任务ID 只有世界任务才校验前置 if !a.module.modelWorldtask.IsPreFinished(req.GroupId, myWorldtask, curTaskConf) { - code = pb.ErrorCode_WorldtaskLastUnFinished - data.Title = code.ToString() - data.Message = fmt.Sprintf("%v前置任务%v未完成", curTaskConf.Key, curTaskConf.Ontxe) + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_WorldtaskLastUnFinished, + Title: pb.ErrorCode_WorldtaskLastUnFinished.ToString(), + Message: fmt.Sprintf("%v前置任务%v未完成", curTaskConf.Key, curTaskConf.Ontxe), + } return } @@ -80,7 +90,10 @@ func (a *apiComp) Accept(session comm.IUserSession, req *pb.WorldtaskAcceptReq) } if err := a.module.modelWorldtask.Change(uid, update); err != nil { - code = pb.ErrorCode_DBError + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_DBError, + Title: pb.ErrorCode_DBError.ToString(), + } return } diff --git a/modules/worldtask/api_battlefinish.go b/modules/worldtask/api_battlefinish.go index e82df1979..4338cbba9 100644 --- a/modules/worldtask/api_battlefinish.go +++ b/modules/worldtask/api_battlefinish.go @@ -25,23 +25,29 @@ func (this *apiComp) BattlefinishCheck(session comm.IUserSession, req *pb.Worldt } func (this *apiComp) Battlefinish(session comm.IUserSession, req *pb.WorldtaskBattleFinishReq) (errdata *pb.ErrorData) { - if code = this.BattlefinishCheck(session, req); code != pb.ErrorCode_Success { + if errdata = this.BattlefinishCheck(session, req); errdata != nil { return } - data = &pb.ErrorData{} + uid := session.GetUserId() taskConf, err := this.module.configure.getWorldtaskById(req.TaskId) if err != nil || taskConf == nil { - code = pb.ErrorCode_ConfigNoFound - data.Title = code.ToString() - data.Message = err.Error() + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_ConfigNoFound, + Title: pb.ErrorCode_ConfigNoFound.ToString(), + Message: err.Error(), + } return } // 当前玩家世界任务 userTask, err := this.module.modelWorldtask.getWorldtask(uid) if err != nil { - code = pb.ErrorCode_DBError + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_DBError, + Title: pb.ErrorCode_DBError.ToString(), + Message: err.Error(), + } return } @@ -49,9 +55,11 @@ func (this *apiComp) Battlefinish(session comm.IUserSession, req *pb.WorldtaskBa if len(taskConf.Completetask) == 0 { if err := this.module.modelWorldtask.finishTask(taskConf.Group, req.TaskId, userTask); err != nil { - code = pb.ErrorCode_DBError - data.Title = code.ToString() - data.Message = err.Error() + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_DBError, + Title: pb.ErrorCode_DBError.ToString(), + Message: err.Error(), + } return } @@ -61,16 +69,20 @@ func (this *apiComp) Battlefinish(session comm.IUserSession, req *pb.WorldtaskBa battleModule, err := this.module.service.GetModule(comm.ModuleBattle) if err != nil { - code = pb.ErrorCode_SystemError + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_SystemError, + Title: pb.ErrorCode_SystemError.ToString(), + Message: err.Error(), + } return } if ibattle, ok := battleModule.(comm.IBattle); ok { var isWin bool - if code, isWin = ibattle.CheckBattleReport(session, req.Report); code == pb.ErrorCode_Success { + if errdata, isWin = ibattle.CheckBattleReport(session, req.Report); errdata == nil { if isWin { if battleConf, ok := this.module.worldBattleConf.GetDataMap()[req.BattleConfId]; ok { - if code = this.module.DispenseRes(session, []*cfg.Gameatn{battleConf.Playexp}, true); code != pb.ErrorCode_Success { + if errdata = this.module.DispenseRes(session, []*cfg.Gameatn{battleConf.Playexp}, true); errdata != nil { this.module.Error("世界任务战斗玩家经验结算", log.Field{Key: "uid", Value: uid}, log.Field{Key: "playerExp", Value: battleConf.Playexp}, @@ -91,7 +103,11 @@ func (this *apiComp) Battlefinish(session comm.IUserSession, req *pb.WorldtaskBa "currentTask": userTask.CurrentTask, } if err := this.module.modelWorldtask.Change(uid, update); err != nil { - code = pb.ErrorCode_DBError + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_DBError, + Title: pb.ErrorCode_DBError.ToString(), + Message: err.Error(), + } return } @@ -100,20 +116,23 @@ func (this *apiComp) Battlefinish(session comm.IUserSession, req *pb.WorldtaskBa } } else { - data.Title = code.ToString() - data.Message = "战报校验错误" + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_ExternalModule, + Title: pb.ErrorCode_ExternalModule.ToString(), + Message: "战报校验错误", + } return } //判断是否配置了通知module if len(taskConf.Module) == 0 { // 发奖 - if code = this.module.DispenseRes(session, taskConf.Reword, true); code != pb.ErrorCode_Success { + if errdata = this.module.DispenseRes(session, taskConf.Reword, true); errdata != nil { this.module.Error("资源发放", log.Field{Key: "uid", Value: uid}, log.Field{Key: "taskId", Value: req.TaskId}, log.Field{Key: "reword", Value: taskConf.Reword}, - log.Field{Key: "code", Value: code}, + log.Field{Key: "errdata", Value: errdata}, ) return } diff --git a/modules/worldtask/api_battlestart.go b/modules/worldtask/api_battlestart.go index c4c1b7fbf..a15d339bf 100644 --- a/modules/worldtask/api_battlestart.go +++ b/modules/worldtask/api_battlestart.go @@ -23,23 +23,28 @@ func (this *apiComp) BattlestartCheck(session comm.IUserSession, req *pb.Worldta } func (this *apiComp) Battlestart(session comm.IUserSession, req *pb.WorldtaskBattleStartReq) (errdata *pb.ErrorData) { - if code = this.BattlestartCheck(session, req); code != pb.ErrorCode_Success { + if errdata = this.BattlestartCheck(session, req); errdata != nil { return } - data = &pb.ErrorData{} resp := &pb.WorldtaskBattleStartResp{} battleConf, err := this.module.configure.getWorldtaskBattleById(req.BattleConfId) if err != nil || battleConf == nil { - code = pb.ErrorCode_ConfigNoFound - data.Title = code.String() - data.Message = fmt.Sprintf("战斗配置未找到 %v", req.BattleConfId) + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_ConfigNoFound, + Title: pb.ErrorCode_ConfigNoFound.ToString(), + Message: fmt.Sprintf("战斗配置未找到 %v", req.BattleConfId), + } return } iBattle, err := this.module.service.GetModule(comm.ModuleBattle) if err != nil { - code = pb.ErrorCode_SystemError + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_SystemError, + Title: pb.ErrorCode_SystemError.ToString(), + Message: err.Error(), + } return } @@ -47,14 +52,18 @@ func (this *apiComp) Battlestart(session comm.IUserSession, req *pb.WorldtaskBat var ( record *pb.DBBattleRecord ) - code, record = b.CreateEveBattle(session, &pb.BattleEVEReq{ + errdata, record = b.CreateEveBattle(session, &pb.BattleEVEReq{ Ptype: pb.PlayType_rtask, Format: req.Battle, Buleformat: battleConf.FormatList, }) - if code != pb.ErrorCode_Success { - data.Title = code.String() + if errdata != nil { + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_ExternalModule, + Title: pb.ErrorCode_ExternalModule.ToString(), + Message: "创建Eve战斗失败", + } return } @@ -73,8 +82,11 @@ func (this *apiComp) Battlestart(session comm.IUserSession, req *pb.WorldtaskBat }, } } else { - code = pb.ErrorCode_WorldtaskBattleCreate - data.Title = code.String() + errdata = &pb.ErrorData{ + Code: pb.ErrorCode_WorldtaskBattleCreate, + Title: pb.ErrorCode_WorldtaskBattleCreate.ToString(), + Message: "战斗记录是空", + } return }