#1 review only,请不要提交】review-250521

开启中
dengjia 请求将 2 次代码提交从 yyc3/review-250521 合并至 yyc3/review-250520
邓佳 评论于 1 月之前
  1. unit test不要使用main
  2. 路径操作使用Paths和Path库,兼容Linux/Mac/Win
  3. css的内容写到css文件中,java代码里只需要按需挂载style,class,tag就可以,马上就会面临参数化的问题,在代码里hardcode没办法维护。
  4. removeEmptyParagraphs 如之前讨论的思路,尽量不用删除元素来控制layout,考虑用控制空行size的办法。
  5. processFirstImageAsBackground理论上不需要全部设置来特殊对待第一张图片。如果图片不按设计显示,那就调整不准确的显示属性,其他的图片也是一样的处理方式。
  6. @SuppressWarnings("checkstyle:NestedIfDepth") 拆分过于复杂的方法,不要suppressWarning。
  7. 考虑建立HtmlConverter,PdfConverter类,参数化hardcode的值,把过程性代码对象化。
  8. TOC的超链在pdf转换的过程中能正确生成吗?如果能那我就不去调查怎么在flyingsaucer处理html的过程中写定制插件了。因为如果要定制渲染过程中才能计算得到的数据,这里似乎是最合理的地方。
1. unit test不要使用main 2. 路径操作使用Paths和Path库,兼容Linux/Mac/Win 3. css的内容写到css文件中,java代码里只需要按需挂载style,class,tag就可以,马上就会面临参数化的问题,在代码里hardcode没办法维护。 4. removeEmptyParagraphs 如之前讨论的思路,尽量不用删除元素来控制layout,考虑用控制空行size的办法。 5. processFirstImageAsBackground理论上不需要全部设置来特殊对待第一张图片。如果图片不按设计显示,那就调整不准确的显示属性,其他的图片也是一样的处理方式。 6. @SuppressWarnings("checkstyle:NestedIfDepth") 拆分过于复杂的方法,不要suppressWarning。 7. 考虑建立HtmlConverter,PdfConverter类,参数化hardcode的值,把过程性代码对象化。 8. TOC的超链在pdf转换的过程中能正确生成吗?如果能那我就不去调查怎么在flyingsaucer处理html的过程中写定制插件了。因为如果要定制渲染过程中才能计算得到的数据,这里似乎是最合理的地方。
该合并请求可以进行自动合并操作。
登录 并参与到对话中。
未选择标签
未选择里程碑
未指派成员
1 名参与者
正在加载...
取消
保存
这个人很懒,什么都没留下。