2

有没有办法重构这个:

let collide (b1 : Box) (b2 : Box) =
  if   bottom b1 > top b2
  then false
  else if   top b1 < bottom b2
       then false
       else if   right b1 < left b2
            then false
            else if   left b1 > right b2
                 then false
                 else true

以比这更易读的方式:

let collide (b1 : Box) (b2 : Box) =
  match () with
  | _ when bottom b1 > top    b2 -> false
  | _ when top    b1 < bottom b2 -> false
  | _ when right  b1 < left   b2 -> false
  | _ when left   b1 > right  b2 -> false
  | _                            -> true

?

我正在考虑类似于 GHC 7.6.1 中的多路 if 表达式:http ://www.haskell.org/ghc/docs/7.6.1/html/users_guide/syntax-extns.html#multi-万一

4

3 回答 3

4

为什么不直接使用||-

not (bottom b1>topb2 || top b1<bottom b2 || right b1<left b2 || left b1>right b2)
于 2012-09-26T03:12:48.537 回答
4
let collide (b1 : Box) (b2 : Box) = 
    if   bottom b1 > top b2 then false 
    elif top b1 < bottom b2 then false 
    elif right b1 < left b2 then false 
    elif left b1 > right b2 then false 
    else true 
于 2012-09-26T03:28:14.610 回答
4

作为对布赖恩的回答的补充,还值得指出的elif是,这只是糖else if。即你可以重新格式化你的原始代码,这样它就不会那么糟糕了:

let collide (b1 : Box) (b2 : Box) =
    if bottom b1 > top b2 then false
    else if top b1 < bottom b2 then false
    else if right b1 < left b2 then false
    else if left b1 > right b2 then false
    else true
于 2012-09-26T03:54:33.783 回答