你们觉得编码规范化真的不重要么？

asuka2001

回复 4# OwnWaterloo

好吧，被蹭到笑点了，幸好牛奶喝完了！


   

三月廿七

//你说他考虑不全，他说代码要简短，能跑就行
  能跑说明什么？ 说明这段代码正确，
  正确+简短，就是优秀

//你说不应该用魔鬼数字，他说代码要写的直观
   char a[50]?
   你觉得这个 50 魔幻在哪里？
   傻子都看的出来 50是个数组长度
   #define a_length 50
   char a[a_length ];
    这样真的容易阅读吗？


//你说变量要初始化，他说这是多余
   变量值在使用之前只需要赋值一次，
   干吗要多余的初始化呢？
    int i = 0;
    for (i = 0; i < 8; i++)
   {}

    初始化不蛋疼？
   
//你说不要写一些晦涩难懂的代码，他说这样证明自己NB
其实，我觉得你的代码就够晦涩的了，

MMMIX

三月廿七 发表于 2012-03-30 23:58
//你说不应该用魔鬼数字，他说代码要写的直观
   char a[50]?
   你觉得这个 50 魔幻在哪里？
   傻子都看的出来 50是个数组长度

傻子能看出來為什麼是 50 而不是 25 或 26 麼？

三月廿七

回复 34# MMMIX

这东西只能靠注释，靠宏没有用， 宏在多数情况下都是个蛋疼的玩意
尤其是 函数内的 switch/case 以及 数组常量

xpm2bmp()
{
    char word[10];
    static int n = 0;
      
      //
      n++;

     switch()
    {
     case 1: // width
         break;
     case 2: // height;
         break;
      case 3:// length;
         break;
      case 4:// per_character
         break;
    }
}

我写注释都感觉比宏看的舒服，
没有人会傻到给 多维数组的每个数字定义一个宏吧？

gtkmm

三月廿七 发表于 2012-03-31 00:11
回复 34# MMMIX

这东西只能靠注释，靠宏没有用， 宏在多数情况下都是个蛋疼的玩意

我喜欢定义喜欢BT的宏，让人搞不明白有什么用。

比如这个:

原来有么一个函数：
for(int i = 0; i < N; ++i) {
     vector x;
     load_vector(x);            // 从硬盘上读一段数据
     process(x);                  // 处理这段数据
     write_vector(x);           // 处理结果写回
}

循环读一个文件，每次读一行，进行处理， 再写到另一个文件里去。 这是可以并行的， 也就是计算第N行时， 是可以读第N+1行的。



我会写个这样的东西（先在某个地方，开启全局异步）：
（当然，这些函数是我自己写的，所以能开全局异步）

load_vector(0);                                     // 先读一行

FOR(int i = 0; i < N; ++i) {
     vector x;
      
     process(x);                                       // 1. 处理这段数据
     if (i+1<N)
          GOTO(1)   load_vector(x);            //  2. GOTO(1), 意思是暂时跳到下一个线程去执行一条语句 （硬盘上读下一段数据）
     write_vector(x);                                // 3 . 处理结果写回
}

现在处理流程会变为：
1--->  (2, 3)  ---> ... 1 -->(2, 3) -->  ..
因为1, 3是在一个线程里，所以会顺序执行。
2不和1,3,在一个线程里，会异步执行， 并会和下一次的循环同步上。


GOTO(x)  意思是暂时跳到下面X个线程去启动一个函数。  （X可以是负数，即回跳）
WAIT(x)   意思是等待下面X个线程执行结束， 比如WAIT(-1)是等待上一个。

MMMIX

三月廿七 发表于 2012-03-31 00:11
回复 34# MMMIX

这东西只能靠注释，靠宏没有用， 宏在多数情况下都是个蛋疼的玩意

那你是不是准备在所有用到 50 的地方都给个注释？要是后面要把 50 改为 25 呢？某个数字，如果只用一次，当然可以通过注释说明，而不必非定义个宏，但要在要多次使用的情况下，宏或其他类似手段都是必须的。

另外，注释还有个问题就是如何保证它和代码保持一致。

AD8018

教条主义害死人，使用数字和宏，要看场合的。

很多长度之类，直接数字还是有必要的。比如局部的字符串转换，

char str[50];
sprintf(str, "msg = %d", i);

绝对要用数字，不要用宏。
用宏的话，因为信息不是集中在一起，容易修改一处，坏了另一处。顾此失彼。

jallyx

zylthinking 发表于 2012-03-30 22:51
好奇怎么比memset效率高了

个人感觉一个实现是由编译器负责初始化，一个是通过函数调用搞定的，编译器初始化效率>函数调用。

zylthinking

jallyx 发表于 2012-03-31 13:14
个人感觉一个实现是由编译器负责初始化，一个是通过函数调用搞定的，编译器初始化效率>函数调用。

(gdb) si
0x00001ec1        14            char buf[1024] = {0};
(gdb) disassemble
Dump of assembler code for function test2:
0x00001ea0 <test2+0>:        push   %ebp
0x00001ea1 <test2+1>:        mov    %esp,%ebp
0x00001ea3 <test2+3>:        push   %esi
0x00001ea4 <test2+4>:        sub    $0x414,%esp
0x00001eaa <test2+10>:        call   0x1eaf <test2+15>
0x00001eaf <test2+15>:        pop    %eax
0x00001eb0 <test2+16>:        mov    0x16d(%eax),%ecx
0x00001eb6 <test2+22>:        mov    (%ecx),%ecx
0x00001eb8 <test2+24>:        mov    %ecx,-0x8(%ebp)
0x00001ebb <test2+27>:        lea    -0x408(%ebp),%ecx
0x00001ec1 <test2+33>:        mov    %ecx,%edx
0x00001ec3 <test2+35>:        mov    %esp,%esi
0x00001ec5 <test2+37>:        mov    %edx,(%esi)
0x00001ec7 <test2+39>:        movl   $0x400,0x4(%esi)
0x00001ece <test2+46>:        mov    %eax,-0x40c(%ebp)
0x00001ed4 <test2+52>:        mov    %ecx,-0x410(%ebp)
0x00001eda <test2+58>:        call   0x1f52 <dyld_stub___bzero>

hellioncu

zylthinking 发表于 2012-03-31 14:33
(gdb) si
0x00001ec1        14            char buf[1024] = {0};
(gdb) disassemble

被初始化的变量size小的时候，如一个小的结构，可能会被优化成几条mov指令