approach0 / search-engine

A math-aware search engine.
http://approach0.xyz
MIT License
346 stars 50 forks source link

小建议 #17

Closed bedreamer closed 7 years ago

bedreamer commented 7 years ago

基础库中引用的C库可以使用简单函数名替换。例如: printf 的使用,如果在库中频繁使用printf功能,则建议使用类似如下的方式替换, static inline int lib_printf(const char *fmt, ...) { .... printf( some string ); .... } 这样有个好处,库被别人引用时,可以定制是否需要输出。

w32zhong commented 7 years ago

谢谢 @bedreamer 的建议。我现在更倾向于用宏直接写在 .c 文件里控制打印。当然这样看起来不是很美观。但是有一些好处:

  1. 这种情况宏不但可以控制打印语句,还可以进一步控制和打印相关的其他语句。

  2. 有些情况需要 log 的时候宏也可以控制。如果使用类似 lib_printf 和 lib_fprintf 的当然也可以,但这样增加了两个新的东西,对比使用宏配合 printf 和 fprintf 这样公共所知的东西,用宏控制说不定更有可读性。

w32zhong commented 7 years ago

暂时先关了这个 issue,下次尽量用英文在 Github 上交流。加油!