当前位置：首页 > news >正文

从代码搬运到精益求精：一次发票打印功能的优化实录

news 2026/3/27 0:35:16

从代码搬运到精益求精：一次发票打印功能的优化实录

最近在实现发票连续打印功能时，我复用了一段网上的代码来处理图片转PDF。本以为只是简单的工具类编写，没想到一次单元测试，却引发了一场关于代码质量、需求理解和用户体验的深度思考。

一段“有趣”的代码

在编写图片转PDF的单元测试时，我发现了这样一段代码：

protectedvoidscaleImage(com.itextpdf.text.Imageimage){floatimageHeight=image.getScaledHeight();floatimageWidth=image.getScaledWidth();inti=0;while(imageHeight>500||imageWidth>500){image.scalePercent(100-i);i++;imageHeight=image.getScaledHeight();imageWidth=image.getScaledWidth();}}

这段代码的逻辑很直观：如果图片的宽或高大于500像素，就循环对其进行缩放，直到尺寸符合要求。但有趣（或者说“笨拙”）的地方在于，它采用了一种暴力尝试的方式：从100%开始，每次减少1%进行尝试，直到图片尺寸降到500以下。

我的测试图片是795x528，这意味着它要一直尝试到62%才符合要求，足足循环了38次！虽然经过查看itextpdf源码，发现scalePercent方法内部主要是纯计算操作，执行几千次也无伤大雅，但这个逻辑本身确实不够优雅。

一个更高效、更简洁的方式是直接计算出缩放比例：

protectedvoidscaleImage(Imageimage){floatimageHeight=image.getScaledHeight();floatimageWidth=image.getScaledWidth();// 计算适应目标尺寸（500）所需的缩放百分比floatscalePercent=Math.min(500/imageHeight,500/imageWidth)*100;image.scalePercent(scalePercent);}

优化之后，就完事了吗？

当我将代码“优化”成直接计算缩放后，不禁反问自己：做到这一步就算完事了吗？

显然没有。一个好的功能不是代码能跑通就结束了，而是要回答更深层次的问题：

为什么要把图片缩小到500以内？这个“500”是业务需求还是历史遗留？
为什么生成的PDF是A4纸大小？如果图片本身很小，周围留白大片，不仅浪费“纸张”，与其他电子发票合并时也会显得不伦不类。
如何与系统中其他PDF文档（如OFD转PDF的发票）样式统一？

追根溯源，直击本质

带着这些问题，我重新审视了最终的用户场景——发票合并打印。

统一宽度是关键：通过与系统中其他格式（如OFD转PDF）的发票对比，我发现它们的宽度基本都是595磅（即A4纸的宽度）。为了让所有文档在合并后视觉上整齐划一，我应该只限制宽度为595，而不限制高度。如果限制高度，遇到长发票时会被压缩得内容不清。
PDF尺寸应与内容契合：生成的PDF文档尺寸，应该与缩放后的图片大小完全一致，而不是固定为A4。这样既能避免浪费“纸张”，也能让每个文档都贴合自身内容。

基于以上思考，我得到了最终的优化版本：

protectedvoidscaleImage(Imageimage){floatimageWidth=image.getScaledWidth();// 只限制宽度为目标值（595），高度自适应floatscalePercent=595/imageWidth*100;image.scalePercent(scalePercent);}protectedInputStreamimageToPdf(InputStreamis,List<File>tempFiles){FileoutFile=FileUtil.createTempFile(".pdf",true);try(FileOutputStreamout=newFileOutputStream(outFile)){Imageimage=Image.getInstance(IoUtil.readBytes(is));// 1. 缩放图片scaleImage(image);// 2. 创建与图片尺寸完全一致的PDF文档，并移除所有页边距Documentdocument=newDocument(newRectangle(image.getScaledWidth(),image.getScaledHeight()),0,0,0,0);// 关键：四周边距设为0PdfWriter.getInstance(document,out);document.open();image.setAlignment(Image.ALIGN_CENTER);document.add(image);document.close();out.flush();returnFiles.newInputStream(outFile.toPath());}catch(Exceptione){log.error("图片转换成pdf失败",e);thrownewBusinessException("图片转换成pdf失败");}finally{IoUtil.close(is);tempFiles.add(outFile);// 用于后续清理}}