对于我的家庭作业,我需要使用递归函数将十六进制字符串转换为基数为10的整数(根据需要使用尽可能多的辅助方法)。
这是我到目前为止所得到的:
-- Question 1, part (c):
hexChar :: Char -> Integer
hexChar ch
| ch == '0' = 0
| ch == '1' = 1
| ch == '2' = 2
| ch == '3' = 3
| ch == '4' = 4
| ch == '5' = 5
| ch == '6' = 6
| ch == '7' = 7
| ch == '8' = 8
| ch == '9' = 9
| ch == 'A' = 10
| ch == 'B' = 11
| ch == 'C' = 12
| ch == 'D' = 13
| ch == 'E' = 14
| ch == 'F' = 15
| otherwise = 0
parseHex :: String -> Integer
parseHex hxStr
| length hxStr /= 0 = (hexChar(last(hxStr)))+(10*parseHex(init(hxStr)))
| otherwise = 0
但是,这不会产生正确的结果。 有谁知道这样做的正确方法?
答案 0 :(得分:4)
你真的很亲密。你的错误就在这一行:
| length hxStr /= 0 = (hexChar(last(hxStr)))+(10*parseHex(init(hxStr)))
想想你为什么乘以10.记住......十六进制是16的基数。
答案 1 :(得分:4)
现在你得到了正确的答案,你应该考虑这种风格。使用模式匹配,这看起来已经更清晰了:
parseHex :: String -> Integer
parseHex [] = 0
parseHex hxStr = (hexChar(last(hxStr)))+(16*parseHex(init(hxStr)))
而且它也更有效率,因为您不需要在每次递归调用时评估length hxStr
(即O(N))来决定适用哪种情况。总运行时间从O(N ** 2)变为O(N)。
当你删除一些括号时看起来更好,正如groovy建议的那样:
parseHex :: String -> Integer
parseHex [] = 0
parseHex hxStr = hexChar (last hxStr) + 16 * parseHex (init hxStr))
很遗憾,您无法立即在hStr
上进行模式匹配,因为您需要init
和last
而不是head
和tail
。但是你可以使用reverse
和帮助器来缓解这种情况:
parseHex :: String -> Integer
parseHex hxStr = go (reverse hxStr)
where go [] = 0
go (x:xs) = hexChar x + 16 * parseHex xs
最后一个可能仅仅是品味问题。 hexChar
也会缩短:
hexChar '0' = 0
hexChar '1' = 1
... -- other cases here
hexChar _ = 0 -- 'otherwise' case; maybe throw an error instead?
答案 2 :(得分:4)
问题解决后,这里写一个更简短的方法:
import Data.List
import Data.Maybe
hexChar ch = fromMaybe (error $ "illegal char " ++ [ch]) $
elemIndex ch "0123456789ABCDEF"
parseHex hex = foldl' f 0 hex where
f n c = 16*n + hexChar c