k*******r 发帖数: 90 | 1 免得一些初学者还被误导了:
- 不要用macro定义常量,const 或者 enum更好,类型安全
- C++ 里面有个东西叫构造函数,struct也可以用
- class + accessor 比起struct的代码可读性更好,编译器也会消除overhead
- TicketPool allocate/free 线程不安全
- read/write没有处理 EINTR, 当然整个异步io的实现就是错误的
- 测试代码不要跟主体代码写在一个文件里
没工夫仔细看里面的逻辑,也没兴趣 |
j******a 发帖数: 100 | 2 -不要用macro定义常量,const 或者 enum更好,类型安全
-class + accessor 比起struct的代码可读性更好,编译器也会消除overhead
- 测试代码不要跟主体代码写在一个文件里
其实是习惯问题,我刚毕业那会很注意,现在看别人的代码看多了觉得无所谓,很多大
牛都这么随意
-TicketPool allocate/free 线程不安全
他link了RT lib, glibc里的实现是有锁的,你可以下个glibc看
-read/write没有处理 EINTR, 当然整个异步io的实现就是错误的
这个非常对 |
j******a 发帖数: 100 | 3 不好意思,我以为你说glibc的allocate/free
他的那个是没做完,魏有讲过
【在 j******a 的大作中提到】 : -不要用macro定义常量,const 或者 enum更好,类型安全 : -class + accessor 比起struct的代码可读性更好,编译器也会消除overhead : - 测试代码不要跟主体代码写在一个文件里 : 其实是习惯问题,我刚毕业那会很注意,现在看别人的代码看多了觉得无所谓,很多大 : 牛都这么随意 : -TicketPool allocate/free 线程不安全 : 他link了RT lib, glibc里的实现是有锁的,你可以下个glibc看 : -read/write没有处理 EINTR, 当然整个异步io的实现就是错误的 : 这个非常对
|
P*******L 发帖数: 2637 | 4
推荐用 constexpr 定义常量
【在 k*******r 的大作中提到】 : 免得一些初学者还被误导了: : - 不要用macro定义常量,const 或者 enum更好,类型安全 : - C++ 里面有个东西叫构造函数,struct也可以用 : - class + accessor 比起struct的代码可读性更好,编译器也会消除overhead : - TicketPool allocate/free 线程不安全 : - read/write没有处理 EINTR, 当然整个异步io的实现就是错误的 : - 测试代码不要跟主体代码写在一个文件里 : 没工夫仔细看里面的逻辑,也没兴趣
|