xkcoding / spring-boot-demo

🚀一个用来深入学习并实战 Spring Boot 的项目。
https://parg.co/UZM
MIT License
32.84k stars 10.82k forks source link

[BUG] demo-orm-jdbctemplate 中,日志打印问题 #233

Open zornx5 opened 2 years ago

zornx5 commented 2 years ago

描述问题

在日志打印 SQL 参数时,基本数据类型使用 JSONUtil.toJsonStr() 无法 JSON 序列化,导致日志打印内容为空

c.x.orm.jdbctemplate.dao.base.BaseDao    : 【执行SQL】SQL:DELETE FROM `orm_user` where id = ?
c.x.orm.jdbctemplate.dao.base.BaseDao    : 【执行SQL】参数:{}
c.x.orm.jdbctemplate.dao.base.BaseDao    : 【执行SQL】SQL:SELECT * FROM `orm_user` where id = ?
c.x.orm.jdbctemplate.dao.base.BaseDao    : 【执行SQL】参数:{}

期待的结果

c.x.orm.jdbctemplate.dao.base.BaseDao    : 【执行SQL】SQL:DELETE FROM `orm_user` where id = ?
c.x.orm.jdbctemplate.dao.base.BaseDao    : 【执行SQL】参数:1
c.x.orm.jdbctemplate.dao.base.BaseDao    : 【执行SQL】SQL:SELECT * FROM `orm_user` where id = ?
c.x.orm.jdbctemplate.dao.base.BaseDao    : 【执行SQL】参数:1

截屏或录像

改进建议

此处参数数量仅为 1,打印时无需使用 JSONUtil.toJsonStr(),直接打印即可

如:log.debug("【执行SQL】参数:{}", JSONUtil.toJsonStr(pk)); -> log.debug("【执行SQL】参数:{}", pk);

zhanzhao-h commented 1 year ago

我来提一个fix 这个issues

guxiren commented 1 year ago

您的邮件已收到!

zhanzhao-h commented 1 year ago

您好,pr能否麻烦合并一下。这边比较急。谢谢

---- 回复的原邮件 ---- | 发件人 | @.> | | 日期 | 2022年12月22日 10:10 | | 收件人 | @.> | | 抄送至 | @.>@.> | | 主题 | Re: [xkcoding/spring-boot-demo] [BUG] demo-orm-jdbctemplate 中,日志打印问题 (Issue #233) |

您的邮件已收到!

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>