2013-05-07 82 views
0

我已经在Python中编写了一个函数来取一个有符号整数,使用二进制补码来转换它,然后返回十六进制值。我知道有一个hex()函数,但我希望能够指定整数的大小。我怎样才能提高这段代码的质量,并且我错过了什么?谢谢!有符号整数到二进制补码十六进制

#!/usr/bin/python 

int8, int16, int32, int64 = 8, 16, 32, 64 

def intToHexString(value, bits): 
    def getBitmask(bits): 
    mask = 0 
    for i in range(bits): 
     mask = (mask << 1) + 1 
    return mask 

    if not isinstance(value, (int, long)): 
    raise ValueError("'%s' is not an Integer!"%str(value)) 
    if not isinstance(bits, int) or bits % 2 != 0: 
    raise ValueError("Illegal integer size," + 
     "value %s must be divisible by 2!"%str(bits)) 
    result = value 
    bitmask = getBitmask(bits) 
    halfMask = bitmask >> 1 
    minVal, maxVal = -halfMask, halfMask-1 
    if not minVal <= result <= maxVal: 
    raise ValueError('Out of range: %d <= %d <= %d' 
     %(minVal, result, maxVal)) 
    if value < 0: 
    result = ((abs(value)^bitmask) + 1) & bitmask 
    return '%0*X'%(int(float(bits)/4), result) 

if __name__ == '__main__': 
    x, y = 280, -54 
    print intToHexString(x, int16), intToHexString(y, int8) 

此代码应返回的值0118CA

回答

2

我会写更是这样的:

def int_to_hex_string(value, bits): 
    return "{0:0{1}X}".format(value & ((1<<bits) - 1), bits//4) 

if __name__ == '__main__': 
    x, y = 280, -54 
    print(int_to_hex_string(x, 16), int_to_hex_string(y, 8)) 

我不认为你的类型检查添加任何东西:如果值不是正确的类型,你会通过不检查得到更适合的'TypeError',同样,'int16'和'int8'作为16和8的别名并不真正增加太多。

+0

几乎完美,但你希望格式中不要'X',所以OP在输出中获得'CA'而不是'ca'。 – abarnert 2013-05-07 19:45:33

+0

@abarnert,谢谢。接得好。 – Duncan 2013-05-07 19:46:05

+0

对OP的评论:即使你想要一个更好的消息的错误,所以它并没有提及<<'的不受支持的类型(让用户想知道什么时候我尝试使用'<<??)? ...你可能想通过捕获一个异常并创建一个新的(理想情况下使用PEP 3134链接)而不是预先检查来实现,并且使用'TypeError'而不是'ValueError'。 – abarnert 2013-05-07 19:48:35

相关问题