可以考虑使用build script来构建
upsuper opened this issue · comments
可以将生成字典的过程放在 build.rs 里，这样就不需要单独的 Makefile 了，也不需要将字典文件单独签入代码库。
此外使用build script以后，还可以使用phf（配合phf_codegen）构建哈希表，这样在运行时效率也可以得到保证。
@upsuper 之所以没用 phf 以及 build.rs 的原因这里有解释：#27 (comment)
我……不是很同意这个见解 :) 不过既然如此那看起来改回去的PR是不会被接受的了
@upsuper 可以讨论的，#27 (comment) 那里也提到了用 phf 是可以接受的，只是说倾向于把生成字典文件的操作放到 makefile 里，毕竟之前用 bulid.rs 的时候是有点慢，编译一次要几十分钟。 @LuoZijun 大佬咋说？ (可以参考一下 #33 的问题
编译一次要几十分钟？！这是有点夸张……
编译一次要几十分钟？！这是有点夸张……
我印象当中应该是十几分钟（确信是分钟级别的），硬件是 MacBook Pro 。
这段时间 Rust 的编译器的编译速度也得到了不少提升，所以在当前的版本下也许会快不少？我稍晚一些做个测试。
@upsuper @mozillazg 我刚才重新做了下测试，2年多过去后（耗时长的问题出现在 2017年），phf 代码生成速度已经是以秒为单位了。所以编译速度已经不再是个问题。
但是采用 phf 来替代二分查找算法并没有带来微小的性能提升。所以新的问题大概就是还有没有必要迁回 phf 这种模式 ...
测试样本是 钱钟书的《围城》: https://github.com/mozillazg/rust-pinyin/blob/add-hashmap/data/weicheng.txt
build.rs: https://github.com/mozillazg/rust-pinyin/blob/add-hashmap/build.rs
话说……围城的版权可还没过期啊……