Spring Boot：@Autow的记住并保存相同的对象也用于下一个请求

Question

我正在开发一个spring boot应用程序，下面是一个API，例如：

@Autowired
GetUserIdByUserNameBase getUserIdByUserNameBase;

@PostMapping(value = "/UserName")
    public GetUserIdByUserNameOutput getUserIdByUserName(@RequestBody String userName) {
        GetUserIdByUserNameInput GetUserIdByUserNameInput = new GetUserIdByUserNameInput(userName);
    getUserIdByUserNameBase.setGetUserIdByUserNameInput(GetUserIdByUserNameInput);
    MsgStatus msgStatus = getUserIdByUserNameBase.execute();
    if (msgStatus == MsgStatus.ERROR_MSG) {
        LOG.error("UserRestController,getUserIdByUserName : Internal System Error");
    }
    return getUserIdByUserNameBase.getGetUserIdByUserNameOutput();
}

我的问题出现在具有相同API的第二个/下一个请求中（例如上面的一个）“GetUserIdByUserNameBase”仍然记得第一次命中的所有保存数据！。

public class GetUserIdByUserNameBase extends ServiceBase {

    private final Logger LOG = LogManager.getLogger(GetUserIdByUserNameBase.class);
    private GetUserIdByUserNameInput getUserIdByUserNameInput;
    private GetUserIdByUserNameOutput getUserIdByUserNameOutput;
    private ResultStatus resultStatus;
    private String userName;
    private TBUSER_INFO userInfo;

在我的例子中，“getUserIdByUserNameOutput”仍然保存来自API第一次命中的先前数据，直到我覆盖内容或将其标记为空。

1、GetUserIdByUserNameBase.setGetUserIdByUserNameOutput（新的GetUserIdByUserNameOutput）；

或

但在这里我创建了一个问题，因为我为每个新的API请求创建了一个新的“GetUserIdByUserNameOutput”实例，并且对系统来说效率不高且繁重...

如果我总结了我真正想知道的内容，我如何告诉@Autowired在API的每个新请求上跳转/静止对象，而不使用或覆盖它从第一个请求中记住的内容。？

以下是我的完整代码：

package com.usermanagementms.ims.servicesfacade.userservices;

import javax.transaction.Transactional;

import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Service;

import com.usermanagementms.ims.enums.MsgStatus;
import com.usermanagementms.ims.repositories.userdao.TBUserInfoDao;
import com.usermanagementms.ims.tables.user.TBUSER_INFO;
import com.usermanagementms.ims.types.commontypes.ResultStatus;
import com.usermanagementms.ims.types.getuseridbyusername.GetUserIdByUserNameInput;
import com.usermanagementms.ims.types.getuseridbyusername.GetUserIdByUserNameOutput;
import com.usermanagementms.ims.utils.ErrorMsg;
import com.usermanagementms.ims.utils.ErrorsController;
import com.usermanagementms.ims.utils.Utils;

@Service
public class GetUserIdByUserNameBase extends ServiceBase {

    private final Logger LOG = LogManager.getLogger(GetUserIdByUserNameBase.class);
    private GetUserIdByUserNameInput getUserIdByUserNameInput;
    private GetUserIdByUserNameOutput getUserIdByUserNameOutput;
    private ResultStatus resultStatus;
    private String userName;
        private TBUSER_INFO userInfo;

        @Autowired
        private TBUserInfoDao TBUserInfoDao;

        @Autowired
        private ErrorsController errorsController;

        @Transactional
        public TBUSER_INFO getUserIdByUserName(String username) {

            TBUSER_INFO userInfo;
            try {
                userInfo = TBUserInfoDao.findIdByUserName(username);
                if (userInfo == null) {
                    LOG.warn("UserService,getUserIdByUserName : userInfo is empty");
                    return null;
                }
                return userInfo;
            } catch (Exception e) {
                LOG.error("UserService,getUserIdByUserName : error in retrieving user from DB { " + e.getMessage() + "}");
            }
            return null;
        }

        @Override
        public MsgStatus inputMapping() {
            MsgStatus msgStatus = MsgStatus.SUCCESS_MSG;
            this.setGetUserIdByUserNameOutput(new GetUserIdByUserNameOutput());
            if (getResultStatus() == null) {
                this.setResultStatus(new ResultStatus());
            }
            msgStatus = super.inputMapping();
            if (msgStatus == MsgStatus.ERROR_MSG || this.getUserIdByUserNameInput == null) {
                LOG.error("GetUserIdByUserNameBase,inpuMapping() : Internal System Error");
                this.getErrorsController().fillResultStatus(ErrorMsg.SYSTEM_ERROR, this.resultStatus);
                this.getGetUserIdByUserNameOutput().setResultStatus(resultStatus);
                return MsgStatus.ERROR_MSG;
            }
            this.setUserName(getUserIdByUserNameInput.getUserNameX1());
            return MsgStatus.SUCCESS_MSG;
        }

        @Override
        public MsgStatus validate() {
            if (Utils.isNullorEmpty(userName)) {
                LOG.error("GetUserIdByUserNameBase,validate() : userName is null or empty");
                this.getErrorsController().fillResultStatus(ErrorMsg.REQUEST_BODY_EMPTY, this.resultStatus);
                this.getGetUserIdByUserNameOutput().setResultStatus(resultStatus);
                return MsgStatus.ERROR_MSG;
            }
            return MsgStatus.SUCCESS_MSG;
        }

        @Override
        public MsgStatus executeImpl() {
            MsgStatus msgStatus = MsgStatus.SUCCESS_MSG;
            TBUSER_INFO userInfo = getUserIdByUserName(userName);
            this.setUserInfo(userInfo);
            if (userInfo == null) {
                msgStatus = this.getErrorsController().fillResultStatus(ErrorMsg.USER_NOT_FOUND, resultStatus);
                if (msgStatus == MsgStatus.ERROR_MSG) {
                    this.getErrorsController().fillResultStatus(ErrorMsg.SYSTEM_ERROR, resultStatus);
                    this.getGetUserIdByUserNameOutput().setResultStatus(resultStatus);
                    LOG.error("GetUserIdByUserNameBase,executeImpl() : Internal System Error");
                    return MsgStatus.ERROR_MSG;
                }
                this.getGetUserIdByUserNameOutput().setResultStatus(resultStatus);
                return MsgStatus.ERROR_MSG;
            }
            this.getErrorsController().fillResultStatus(ErrorMsg.OK, resultStatus);
            this.getGetUserIdByUserNameOutput().setResultStatus(resultStatus);
            return MsgStatus.SUCCESS_MSG;
        }

        @Override
        public MsgStatus outputMapping() {
            this.getGetUserIdByUserNameOutput().setUserId(this.getUserInfo().getUid());
            return MsgStatus.SUCCESS_MSG;
        }

        public GetUserIdByUserNameInput getGetUserIdByUserNameInput() {
            return getUserIdByUserNameInput;
        }

        public void setGetUserIdByUserNameInput(GetUserIdByUserNameInput getUserIdByUserNameInput) {
            this.getUserIdByUserNameInput = getUserIdByUserNameInput;
        }

        public GetUserIdByUserNameOutput getGetUserIdByUserNameOutput() {
            return getUserIdByUserNameOutput;
        }

        public void setGetUserIdByUserNameOutput(GetUserIdByUserNameOutput getUserIdByUserNameOutput) {
            this.getUserIdByUserNameOutput = getUserIdByUserNameOutput;
        }

        public String getUserName() {
            return userName;
        }

        public void setUserName(String userName) {
            this.userName = userName;
        }

        public ErrorsController getErrorsController() {
            return errorsController;
        }

        public void setErrorsController(ErrorsController errorsController) {
            this.errorsController = errorsController;
        }

        public ResultStatus getResultStatus() {
            return resultStatus;
        }

        public void setResultStatus(ResultStatus resultStatus) {
            this.resultStatus = resultStatus;
        }

        public TBUSER_INFO getUserInfo() {
            return userInfo;
        }

        public void setUserInfo(TBUSER_INFO userInfo) {
            this.userInfo = userInfo;
        }
    }

public class GetUserIdByUserNameOutput {

    private Long UserId;
    private ResultStatus resultStatus;

    public GetUserIdByUserNameOutput() {

    }

    public GetUserIdByUserNameOutput(Long userId, ResultStatus resultStatus) {
        super();
        UserId = userId;
        this.resultStatus = resultStatus;
    }

    public GetUserIdByUserNameOutput(ResultStatus resultStatus) {
        this.resultStatus = resultStatus;
    }

    public Long getUserId() {
        return UserId;
    }

    public void setUserId(Long userId) {
        UserId = userId;
    }

    public ResultStatus getResultStatus() {
        return resultStatus;
    }

    public void setResultStatus(ResultStatus resultStatus) {
        this.resultStatus = resultStatus;
    }
}

如果您查看此方法：

public MsgStatus inputMapping() {
            MsgStatus msgStatus = MsgStatus.SUCCESS_MSG;
            this.setGetUserIdByUserNameOutput(new GetUserIdByUserNameOutput());

我开始创建“new GetUserIdByUserNameOutput（）”，每次请求相同的API，我想，对于系统来说，在经过一些时间之后创建大量GetUserIdByUserNameOutput实例是不够高效和沉重的。。。

Answer 1

在大多数情况下，春豆不应该有状态。bean是可重用的类的单个实例。不要使用setters！

而不是这样做：

getUserIdByUserNameBase。setGetUserIdByUserNameInput（GetUserIdByUserNameInput）

您需要在单个方法中进行计算，或者可能在bean中实例化一个新对象并返回新对象，例如。

return getUserIdByUserNameBase.getGetUserIdByUserName(GetUserIdByUserNameInput);

或者类似的

return getUserIdByUserNameBase.buildUsernameThing(GetUserIdByUserNameInput).execute();