（转）一段代码的几种写法-怎么样写好的代码

程序员当然要说代码了，代码是程序员的一切，离了代码说其他都是假的，我这里从一段代码开始。

    int c=0, n=0,nDiff=0;
    
int height = m_imgHeight;//image height
    LPBYTE lpDsp = pSampling;
    jp2_int16 *sp, val;
    LPBYTE dest;
    //m_imgWidth–image width

    nDiff = m_lineBytes – m_imgWidth*3;
    while (height—) {
        for (c=0; c<THREE_COMPONENT; c++) {
            dest = lpDsp+c;
            sp = m_lines[c].m_pBuf;
            for (n=m_imgWidth; n>0; n—, sp++, dest+=THREE_COMPONENT) {
                *sp += 2;//virtual option
                    ……
                *dest = *sp;
            }
        }

        /*
        If the bytes isn‘t 4-bytes multiple, 1-3 bytes will not be 
        initialized. So these bytes should be initialized as 0
        */
        if(nDiff){
            dest -= 2;
            for(n = nDiff;n>0;n—){
                *dest = 0;
                dest++;
            }
        }

        lpDsp += m_lineBytes;
    }

这段代码的目的是一个与图形相关的操作，这里省略了许多代码，所以不可能编译；其基本的含义是从一个缓冲区复制到目标缓冲区pSampling。其中m_imgWidth和m_imgHeight表示图象的宽度和高度，m_lineBytes表示4个字节对齐的一行像素的字节数。

我们知道BMP文件的像素要求每行的最终字节数必须是4个倍数。如果图象宽度是4的倍数，不用仇是没有问题的；如果不是4的倍数，例如：129，而且每个像素一个字节，那么最终存储的长度是132个字节，但后面的几个字节如果不填充是未知的内容。上面if(nDiff){后面的就是为了填充这些多余的字节。

我的一个朋友认为这样写更有效一点：

            dest -= 2;
            for(n = nDiff;n>0;n—){
                *dest = 0;
                dest++;
            }