【发布时间】:2014-09-16 09:16:08
【问题描述】:
在 SML 中实现算法时,我经常想知道是否有一种简单的方法可以使大量使用数组的代码更具可读性。例如,如果我定义一个 SML 函数来交换数组中的 2 个元素,则代码为 ...
local open Array in
fun exch (a, i, j) =
let
val tmp = sub (a, i)
val _ = update (a, i, sub (a, j))
val _ = update (a, j, tmp)
in () end
end
我想要的是一个更易读、更简洁的版本,就像这个 Scala-sn-p ...
def exch[T](a: Array[T], i: Int, j: Int) {
val tmp = a(i)
a(i) = a(j)
a(j) = tmp
}
对于像交换数组中的 2 个元素这样简单的事情,SML 版本是可以的。但是一旦算法变得越来越复杂,代码就会变得越来越难以理解,并且会混淆底层算法。
一个稍微复杂一点的例子是这个堆栈(实现为可调整大小的数组)...
structure ArrayStack = struct
type 'a stack = ('a option array * (int ref)) ref
exception Empty
fun mkStack () = ref (Array.array (1, NONE), ref 0)
fun isEmpty (ref (_, ref 0)) = true
| isEmpty _ = false
fun resize (array as ref (xs, n), capacity) =
let
val length = Array.length xs
in
array := (Array.tabulate (
capacity,
fn i => if i < length then Array.sub (xs, i) else NONE
), n)
end
fun push (array as ref (xs, n : int ref), x) =
if Array.length xs = !n then (
resize (array, !n*2)
; push (array, x))
else (
Array.update (xs, !n, SOME x)
; n := !n+1)
fun pop (ref (xs, ref 0)) = raise Empty
| pop (array as ref (xs, n : int ref)) = let
val _ = (n := !n-1)
val x = Array.sub (xs, !n)
val _ = Array.update (xs, !n, NONE)
val q = (Array.length xs) div 4
val _ = if !n > 0 andalso !n = q then resize (array, q) else ()
in
valOf x
end
end
与http://algs4.cs.princeton.edu/13stacks/ResizingArrayStack.java.html 的java 实现相比,该实现(尤其是push/pop)变得难以阅读。
如何使此类代码更具可读性?
【问题讨论】:
-
因为代码有效,这可能更适合codereview.stackexchange.com。
标签: sml