spring-boot-demo icon indicating copy to clipboard operation
spring-boot-demo copied to clipboard

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

Open zornx5 opened this issue 3 years ago • 5 comments

描述问题

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

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

期待的结果

  • deleteById(1)
c.x.orm.jdbctemplate.dao.base.BaseDao    : 【执行SQL】SQL:DELETE FROM `orm_user` where id = ?
c.x.orm.jdbctemplate.dao.base.BaseDao    : 【执行SQL】参数:1
  • 以及 findOneById(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);

zornx5 avatar Aug 30 '22 08:08 zornx5

我来提一个fix 这个issues

zhanzhao-h avatar Dec 22 '22 02:12 zhanzhao-h

您的邮件已收到!

guxiren avatar Dec 22 '22 02:12 guxiren

您好,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: @.***>

zhanzhao-h avatar Dec 29 '22 12:12 zhanzhao-h