Skip to content
This repository was archived by the owner on Oct 27, 2021. It is now read-only.

Conversation

@huyi0912
Copy link
Contributor

@huyi0912 huyi0912 commented May 4, 2018

針對目前專案所做的一些修改:
1.能正確處理 POI 產出含 null cell 的 Excel
2.輸出格式調整(會計格式等)
3.最後一行漏掉

測試目錄 testexcel 內放有目前的測試檔,建議附上,
若有更多的範本,可以含蓋更多的可能狀況.

huyi0912 added 8 commits May 4, 2018 10:59
1.可設定紙張大小,若單張 Excel,依據版面設定中的紙張大小作為 PDF 輸出紙張大小
2.允許設定是否自動產生頁碼
3.表格置中
1.將 POI Font 轉換到 iText Font
2.預先載入系統內字型(慢)
1.Excel 的最後一行不會處理
2.若有 null cell(程式產生的 Excel),則排版換行會出錯
3.輸出欄位寬度與 Excel 同比例
4.納入Cell 內定義的格式(會計格式等)
@caryyu
Copy link
Owner

caryyu commented May 4, 2018

@huyi0912 能够把多个 commits 进行合并吗?相应的样本文件是否可以全部采用英文并按 Maven 目录规则整理到 src/test/resources 目录中去? 另外,我看你在 testcase 中对样本文件路径获取进行了特殊处理,其实你按照 Maven 一定的规则来进行就不用那么麻烦的。

谢谢你的 PR!

@caryyu
Copy link
Owner

caryyu commented May 4, 2018

@huyi0912 另外你把你的 Master 同步一下之后再合并,我刚才提交了一些内容产生了冲突。

@huyi0912
Copy link
Contributor Author

huyi0912 commented May 4, 2018

衝突解決了!

@caryyu
Copy link
Owner

caryyu commented May 4, 2018

@huyi0912 能把这些 Commits 合并为一个吗(主要为了后续功能追溯哈)?然后描述更详细一些增加了哪些功能,我把这些改进或新增加的功能后续增加到文档中去,谢谢啦!

@huyi0912
Copy link
Contributor Author

huyi0912 commented May 5, 2018

說實在我對 Git 不太熟,不過 GitHub 有一個 Squash commit 功能,應該是您需要的.
https://blog.github.com/2016-04-01-squash-your-commits/
修改部分,總結如下:
錯誤修正(Bug fix)
1.Excel 的最後一行不會處理
2.若有 null cell(程式產生的 Excel),則排版換行會出錯
改進(Enhance)
1.可設定紙張大小,若單張 Excel,則依據版面設定中的紙張大小作為 PDF 輸出大小
2.允許設定是否自動產生頁碼
3.輸出欄位寬度與 Excel 同比例
4.納入Cell 內定義的格式(會計格式等)
5.表格置中(這個應該設計得更有彈性)
6.只要將測試Excel檔放入 src/test/resources 下,單元測試會自動納入,測試檔也放入 Git 以方便各類型 Excel 可加入測試.

@caryyu
Copy link
Owner

caryyu commented May 5, 2018

@huyi0912 主要使用 rebase ,比如由于你仓库的 Master 增加了 23 项 Commits , 为了合并这些 Commits 你可以在你的 Master 仓库这样进行处理:git rebase -i HEAD~23 这个时候会弹出一个 VI 编辑器,然后手动把除第一行的所有 pick 换成 squash ,然后 :wq 保存之后下一步弹出提交的 Comment 描述,把你上面的功能描述贴进去再 :wq 就好了。这个时候你在 Git 日志就能看到你处理的情况了。 你可以参考链接:https://blog.csdn.net/yangcs2009/article/details/47166361

@caryyu caryyu merged commit 0620510 into caryyu:master May 8, 2018
@caryyu
Copy link
Owner

caryyu commented May 8, 2018

@huyi0912 已经合并了,感谢你贡献的代码!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants