NestJS中Service内部方法调用返回undefined问题排查与解决

问题背景 #

我是NestJS和OOP的新手，已创建一个简单API。

Controller代码 #

@Controller('api')
export class UsersController {
    constructor(private readonly usersService: UsersService) { }

    @Get('user/:id')
    async create(@Body() createUserDto: CreateUserDto) {
        return this.usersService.create(createUserDto);
    }

    @Get('test/:id')
    async test(@Param('id') id: string): Promise<User> {
        return this.usersService.testMethod(id);
    }
}

UsersService代码 #

export class UsersService {
    constructor(
        @InjectModel(User.name) private readonly userModel: Model<UserDocument>,
    ) { }

    async findOne(id: string): Promise<User> {
        try {
            const data = await fetch(process.env.SOURCE_URI + 'users/' + id)
            const json = await data.json()
            return json.data

        } catch (error) { return error }
    }

    async testMethod(id: string): Promise<User> {
          const user=await this.findOne(id)
          console.log('found user', user) // return user undefined
          return user
    }
    async create(createUserDto: CreateUserDto): Promise<User> {
        try {
            let user = await this.userModel.findOne({ email: 
             createUserDto.email })
            if (!user) {
                user = await this.userModel.create(createUserDto);
            }
            return user;

        } catch (error) {

        }
    }

}

问题现象 #

调用user/:id路由可正常返回用户信息，但调用test/:id路由时，testMethod中的findOne返回undefined，无法正常工作。

问题根源 #

1. Controller路由映射错误
   @Get('user/:id')绑定的是create方法，但create方法是处理用户创建的数据库逻辑，且GET请求本身没有请求体，路由中的:id参数完全没被使用。之前觉得user/:id正常返回，实际是走了数据库查询/创建流程，和findOne无关。

2. findOne方法存在缺陷

   • 未检查HTTP响应状态：fetch仅在网络错误时抛出异常，若请求返回404、500等状态码，fetch不会报错，但此时响应的json().data可能为空或不存在。
   • URL拼接可能出错：process.env.SOURCE_URI若未配置斜杠，会导致拼接后的URL格式错误（比如http://example.comusers/1），请求失败后返回undefined。
   • catch块直接返回error：出错时返回Error对象而非合理值，可能导致后续逻辑混乱。

3. testMethod缺乏错误处理
   未捕获findOne可能抛出的异常，无法定位具体错误原因。

解决步骤 #

1. 修复Controller路由映射 #

将用户查询的路由与正确方法绑定，同时把create方法改为POST请求（符合REST规范）：

@Controller('api')
export class UsersController {
    constructor(private readonly usersService: UsersService) { }

    // 处理用户创建：POST请求
    @Post('user')
    async create(@Body() createUserDto: CreateUserDto) {
        return this.usersService.create(createUserDto);
    }

    // 处理单个用户查询：GET请求，使用路由参数id
    @Get('user/:id')
    async findOne(@Param('id') id: string): Promise<User> {
        return this.usersService.findOne(id);
    }

    @Get('test/:id')
    async test(@Param('id') id: string): Promise<User> {
        return this.usersService.testMethod(id);
    }
}

2. 完善findOne方法的错误处理与响应校验 #

async findOne(id: string): Promise<User> {
    try {
        // 用模板字符串避免斜杠拼接问题
        const fullUrl = `${process.env.SOURCE_URI}/users/${id}`;
        const response = await fetch(fullUrl);
        
        // 检查HTTP响应状态，非200则抛出错误
        if (!response.ok) {
            throw new Error(`请求用户数据失败，状态码：${response.status}`);
        }

        const json = await response.json();
        
        // 校验响应数据结构
        if (!json?.data) {
            throw new Error('第三方接口返回数据格式错误，缺少data字段');
        }

        return json.data;
    } catch (error) {
        // 记录错误日志便于排查
        console.error('findOne方法执行失败：', error.message);
        // 抛出错误让NestJS统一处理，或根据业务需求返回null
        throw error;
    }
}

3. 优化testMethod的错误处理 #

async testMethod(id: string): Promise<User> {
    try {
        const user = await this.findOne(id);
        console.log('查询到用户：', user);
        return user;
    } catch (error) {
        console.error('testMethod执行失败：', error.message);
        throw error;
    }
}

4. 检查环境变量配置 #

确认process.env.SOURCE_URI已正确配置，比如值应为http://third-party-api.com（末尾无需斜杠，模板字符串已处理），确保拼接后的URL可正常访问。

内容的提问来源于stack exchange，提问作者morteza mortezaie