Chinaunix

标题: 三目运算符神奇用法 [打印本页]
作者: yjh777    时间: 2016-02-25 15:44
标题: 三目运算符神奇用法
本帖最后由 yjh777 于 2016-02-25 15:45 编辑

好吧，有点标题党的味道

原来的写法:
  1. void dtype2str(unsigned int dtype, char *buf)
  2. {
  3.         switch(dtype) {
  4.         case DT_BLK: strcpy(buf, "DT_BLK");
  5.                 break;
  6.         case DT_CHR: strcpy(buf, "DT_CHR");
  7.                 break;
  8.         case DT_DIR: strcpy(buf, "DT_DIR");
  9.                 break;
  10.         case DT_FIFO: strcpy(buf, "DT_FIFO");
  11.                 break;
  12.         case DT_LNK: strcpy(buf, "DT_LNK");
  13.                 break;
  14.         case DT_REG: strcpy(buf, "DT_REG");
  15.                 break;
  16.         case DT_SOCK: strcpy(buf, "DT_SOCK");
  17.                 break;
  18.         case DT_UNKNOWN: strcpy(buf, "DT_UNKNOWN");
  19.                 break;
  20.         default: strcpy(buf, "undefine");
  21.                 break;
  22.         }
  23. }

  25. ..snip..
  26. dtype2str(d_type, buf);
  27. printf("%s\n", buf);
复制代码
改为三目运算符的写法:
..snip..
  1.                    printf("%-10s ", (d_type == DT_REG) ?  "regular" :
  2.                                     (d_type == DT_DIR) ?  "directory" :
  3.                                     (d_type == DT_FIFO) ? "FIFO" :
  4.                                     (d_type == DT_SOCK) ? "socket" :
  5.                                     (d_type == DT_LNK) ?  "symlink" :
  6.                                     (d_type == DT_BLK) ?  "block dev" :
  7.                                     (d_type == DT_CHR) ?  "char dev" : "???");
复制代码
作者: MMMIX    时间: 2016-02-25 16:11
回复 1# yjh777


    后一种写法的问题：

1. 不易读；
2. 扩展性不好；
3. 性能可能会低（编译器可能会对 switch 进行特殊处理）。
作者: MMMIX    时间: 2016-02-25 16:14
回复 1# yjh777


    针对1楼的例子，更好的做法是用 tabe-driven，设置一个 list (table)，然后用 dtype 做索引；这样可以有效的把数据和代码逻辑分开。
作者: yjh777    时间: 2016-02-25 16:38
MMMIX 发表于 2016-02-25 16:14
回复 1# yjh777


嗯，我原来是打算表驱动，计算哈希值的时候一时想不起来怎么算了，就直接写switch了

后来看 getdents 的man page的时候发现里面竟然有代码，打印类型居然吧 三目运算符 这样用，

脑洞打开！
作者: MMMIX    时间: 2016-02-25 17:33
yjh777 发表于 2016-02-25 16:38
嗯，我原来是打算表驱动，计算哈希值的时候一时想不起来怎么算了，就直接写switch了

后来看 getdent ...


这种写法其实挺方便的。
作者: hanxin83    时间: 2016-02-25 17:46
本帖最后由 hanxin83 于 2016-02-25 17:46 编辑

三目运算符本来就是if else, 改写后的代码不过就是一堆if else叠加罢了.

什么表驱动, 还要计算HASH这种高端术语的...例子中的值本来就是小数字, 直接数组下标就hold住了.

如果是跳转分枝, switch在很多情况下都会优化成跳跃表, 犯不着自己操那份闲心.
如果操作行为一致, 应该把数据作成表项, 可以省略一些跳转(其实CPU分枝预测早帮我们考虑好, 也犯不着考虑)

... 简单才是王道.
作者: yjh777    时间: 2016-02-25 19:00
hanxin83 发表于 2016-02-25 17:46
三目运算符本来就是if else, 改写后的代码不过就是一堆if else叠加罢了.

什么表驱动, 还要计算HASH这种高端术语的...例子中的值本来就是小数字, 直接数组下标就hold住了.
...


那几个值最大的 2的12 次方，还有一个未用的值2的14次方，
    直接用数组有点大 ...  
作者: hanxin83    时间: 2016-02-26 10:01
yjh777 发表于 2016-02-25 19:00
那几个值最大的 2的12 次方，还有一个未用的值2的14次方，
    直接用数组有点大 ...


嗯, 位标志, 就用位的偏移做表....不过写起来确实没有?:方便.



欢迎光临 Chinaunix (http://bbs.chinaunix.net/) Powered by Discuz! X3.2