7-Bit编码

Question

关于PDU的7—BIT编码，我在网上找到的源码
int PDUCode::gsmEncode7bit(const char *pSrc,char *pDst,int nSrcLength)
{ int nSrc;        // 源字符串的计数值
    int nDst;        // 目标编码串的计数值
    int nChar;       // 当前正在处理的组内字符字节的序号，范围是0-7
    BYTE nLeft;    // 上一字节残余的数据
    

    // 计数值初始化
    nSrc = 0;
    nDst = 0;
    
    // 将源串每8个字节分为一组，压缩成7个字节
    // 循环该处理过程，直至源串被处理完
    // 如果分组不到8字节，也能正确处理
    while(nSrc<nSrcLength)
    {
        // 取源字符串的计数值的最低3位
        nChar = nSrc & 7;

    
        // 处理源串的每个字节
        if(nChar == 0)
        {
            // 组内第一个字节，只是保存起来，待处理下一个字节时使用
            nLeft = *pSrc;
        }
        else
        {
            // 组内其它字节，将其右边部分与残余数据相加，得到一个目标编码字节
            *pDst = (*pSrc << (8-nChar)) | nLeft;
    
            // 将该字节剩下的左边部分，作为残余数据保存起来
            nLeft = *pSrc >> nChar;
            // 修改目标串的指针和计数值 pDst++;
            nDst++; 
        } 
        
        // 修改源串的指针和计数值
        pSrc++; nSrc++;
    }
    
    // 返回目标串长度
    return nDst; }然后调用
nLength = gsmEncode7bit((char *)pSrc->TP_UD, (char *)&buf[4], nLength+1) + 4; 
其中那个pSRC-TP_UD为用户信息，但是我输入“12345678”，返回的却是00000000000000死活调不出来，郁闷

Answer 1

nLength = gsmEncode7bit((char *)pSrc->TP_UD, (char *)buf[4], nLength+1) + 4; 
这样呢？第二个参数因该是个一级指针啊，你用了指针的指针了。

Answer 2

你等于在修改buf这个数组的首地址了。而实际你是需要buf数组的内容。

Answer 3

pSrc->TP_UD和buf分别是什么类型？nLength的值是多少？

Answer 4

又见发短信又见这段代码，呵呵这段代码有明显的bug，如果pSrc刚好是8的倍数没问题，如果不是则转换漏了最后一字节so if (nChar != 7)
{
*pDst++ = nLeft;
nDst++;
}
加到循环体外

Answer 5

int
Pack( char *src, unsigned char *dst, unsigned char len )
{
unsigned char a, b, k; for( a = 0, b = 0; b < len; a++, b++ )
{
k = b % 8; if ( k == 7 )
{
a--;
if( b < ( len - 1 ) )
{
dst[ a ] |= ( ( src[ b ] & 0x7f ) << 1 );
}
}
else
{
if( b < ( len - 1 ) )
{
dst[ a ] = ( ( ( src[ b ] & 0x7f ) >> k ) | ( ( src[ b + 1 ] & 0x7f ) << ( 7 - k ) ) );
}
else
{
dst[ a ] = ( ( ( src[ b ] & 0x7f ) >> k ) );
}
}
}
return a;
}这是俺写的，用了多年了，绝对符合规范，没有任何错误。