下面是用Scala和Clojure编写的函数,用于简单替换字符串中的模板。每个函数的输入是String
,其中包含{key}
形式的模板以及从符号/关键字到替换值的映射。
例如:
Scala的:
replaceTemplates("This is a {test}", Map('test -> "game"))
Clojure的:
(replace-templates "This is a {test}" {:test "game"})
将返回"This is a game"
。
输入映射使用符号/关键字,这样我就不必处理字符串中的模板包含大括号的极端情况。
不幸的是,算法效率不高。
这是Scala代码:
def replaceTemplates(text: String,
templates: Map[Symbol, String]): String = {
val builder = new StringBuilder(text)
@tailrec
def loop(key: String,
keyLength: Int,
value: String): StringBuilder = {
val index = builder.lastIndexOf(key)
if (index < 0) builder
else {
builder.replace(index, index + keyLength, value)
loop(key, keyLength, value)
}
}
templates.foreach {
case (key, value) =>
val template = "{" + key.name + "}"
loop(template, template.length, value)
}
builder.toString
}
这是Clojure代码:
(defn replace-templates
"Return a String with each occurrence of a substring of the form {key}
replaced with the corresponding value from a map parameter.
@param str the String in which to do the replacements
@param m a map of keyword->value"
[text m]
(let [sb (StringBuilder. text)]
(letfn [(replace-all [key key-length value]
(let [index (.lastIndexOf sb key)]
(if (< index 0)
sb
(do
(.replace sb index (+ index key-length) value)
(recur key key-length value)))))]
(doseq [[key value] m]
(let [template (str "{" (name key) "}")]
(replace-all template (count template) value))))
(.toString sb)))
这是一个测试用例(Scala代码):
replaceTemplates("""
Lorem ipsum dolor sit amet, consectetur adipiscing elit. Pellentesque
elit nisi, egestas et tincidunt eget, {foo} mattis non erat. Aenean ut
elit in odio vehicula facilisis. Vestibulum quis elit vel nulla
interdum facilisis ut eu sapien. Nullam cursus fermentum
sollicitudin. Donec non congue augue. {bar} Vestibulum et magna quis
arcu ultricies consectetur auctor vitae urna. Fusce hendrerit
facilisis volutpat. Ut lectus augue, mattis {baz} venenatis {foo}
lobortis sed, varius eu massa. Ut sit amet nunc quis velit hendrerit
bibendum in eget nibh. Cras blandit nibh in odio suscipit eget aliquet
tortor placerat. In tempor ullamcorper mi. Quisque egestas, metus eu
venenatis pulvinar, sem urna blandit mi, in lobortis augue sem ut
dolor. Sed in {bar} neque sapien, vitae lacinia arcu. Phasellus mollis
blandit commodo.
""", Map('foo -> "HELLO", 'bar -> "GOODBYE", 'baz -> "FORTY-TWO"))
和输出:
Lorem ipsum dolor sit amet, consectetur adipiscing elit. Pellentesque
elit nisi, egestas et tincidunt eget, HELLO mattis non erat. Aenean ut
elit in odio vehicula facilisis. Vestibulum quis elit vel nulla
interdum facilisis ut eu sapien. Nullam cursus fermentum
sollicitudin. Donec non congue augue. GOODBYE Vestibulum et magna quis
arcu ultricies consectetur auctor vitae urna. Fusce hendrerit
facilisis volutpat. Ut lectus augue, mattis FORTY-TWO venenatis HELLO
lobortis sed, varius eu massa. Ut sit amet nunc quis velit hendrerit
bibendum in eget nibh. Cras blandit nibh in odio suscipit eget aliquet
tortor placerat. In tempor ullamcorper mi. Quisque egestas, metus eu
venenatis pulvinar, sem urna blandit mi, in lobortis augue sem ut
dolor. Sed in GOODBYE neque sapien, vitae lacinia arcu. Phasellus mollis
blandit commodo.
算法横切输入映射,对于每对,在输入String
中进行替换,暂时保存在StringBuilder
中。对于每个键/值对,我们搜索键的最后一次出现(括在括号中)并将其替换为值,直到不再出现为止。
如果我们在StringBuilder中使用.lastIndexOf
与.indexOf
,会不会产生任何性能差异?
如何改进算法?是否有更惯用的方式来编写Scala和/或Clojure代码?
更新:查看我的follow-up。
更新2 :这是一个更好的Scala实现;字符串长度为O(n)。请注意,我已根据多人的建议将Map
修改为[String, String]
而非[Symbol, String]
。 (感谢mikera,kotarak):
/**
* Replace templates of the form {key} in the input String with values from the Map.
*
* @param text the String in which to do the replacements
* @param templates a Map from Symbol (key) to value
* @returns the String with all occurrences of the templates replaced by their values
*/
def replaceTemplates(text: String,
templates: Map[String, String]): String = {
val builder = new StringBuilder
val textLength = text.length
@tailrec
def loop(text: String): String = {
if (text.length == 0) builder.toString
else if (text.startsWith("{")) {
val brace = text.indexOf("}")
if (brace < 0) builder.append(text).toString
else {
val replacement = templates.get(text.substring(1, brace)).orNull
if (replacement != null) {
builder.append(replacement)
loop(text.substring(brace + 1))
} else {
builder.append("{")
loop(text.substring(1))
}
}
} else {
val brace = text.indexOf("{")
if (brace < 0) builder.append(text).toString
else {
builder.append(text.substring(0, brace))
loop(text.substring(brace))
}
}
}
loop(text)
}
更新3 :以下是一组Clojure测试用例(Scala版本留作练习: - )):
(use 'clojure.test)
(deftest test-replace-templates
(is (= ; No templates
(replace-templates "this is a test" {:foo "FOO"})
"this is a test"))
(is (= ; One simple template
(replace-templates "this is a {foo} test" {:foo "FOO"})
"this is a FOO test"))
(is (= ; Two templates, second at end of input string
(replace-templates "this is a {foo} test {bar}" {:foo "FOO" :bar "BAR"})
"this is a FOO test BAR"))
(is (= ; Two templates
(replace-templates "this is a {foo} test {bar} 42" {:foo "FOO" :bar "BAR"})
"this is a FOO test BAR 42"))
(is (= ; Second brace-enclosed item is NOT a template
(replace-templates "this is a {foo} test {baz} 42" {:foo "FOO" :bar "BAR"})
"this is a FOO test {baz} 42"))
(is (= ; Second item is not a template (no closing brace)
(replace-templates "this is a {foo} test {bar" {:foo "FOO" :bar "BAR"})
"this is a FOO test {bar"))
(is (= ; First item is enclosed in a non-template brace-pair
(replace-templates "this is {a {foo} test} {bar" {:foo "FOO" :bar "BAR"})
"this is {a FOO test} {bar")))
(run-tests)
答案 0 :(得分:8)
我认为您可以构建的最佳算法是输入字符串长度为O(n),并且类似于:
转换为Scala / Clojure作为练习: - )
答案 1 :(得分:7)
这是使用正则表达式进行替换的clojure实现的一个版本。它比你的版本更快(运行你的Lorum ipsum测试用例100次,进一步查看),并且维护的代码更少:
(defn replace-templates2 [text m]
(clojure.string/replace text
#"\{\w+\}"
(fn [groups]
((keyword (subs groups
1
(dec (.length groups)))) m))))
实施快速而且肮脏,但它确实有效。关键是我认为你应该使用正则表达式解决这个问题。
<强>更新强>
用一种时髦的方式进行实验,以进行子串,并获得了惊人的性能结果。这是代码:
(defn replace-templates3 [text m]
(clojure.string/replace text
#"\{\w+\}"
(fn [groups]
((->> groups
reverse
(drop 1)
reverse
(drop 1)
(apply str)
keyword) m))))
以下是我的机器上您的版本,我的第一个版本以及最终版本(100次迭代)的结果:
"Elapsed time: 77.475072 msecs"
"Elapsed time: 50.238911 msecs"
"Elapsed time: 38.109875 msecs"
答案 2 :(得分:7)
我为Clojure编写了一个字符串插值库,它被作为clojure.contrib.strint
引入clojure-contrib。我blogged about it;你会在那里找到对这种方法的描述。它的最新来源可以是viewed here on github。 clojure.contrib.strint
和这里的方法之间的巨大差异是后者都在运行时执行插值。根据我的经验,运行时插值在很大程度上是不必要的,并且使用在编译时执行插值的clojure.contrib.strint
之类的东西通常会为您的应用程序带来切实的性能优势。
请注意,clojure.contrib.strint
有望成为migrating to clojure.core.strint
under Clojure's "new-contrib" organization。
答案 3 :(得分:6)
有些人在遇到问题时会想“我会使用正则表达式!”。现在他们有两个问题。然而,其他人决定不使用正则表达式 - 现在他们有三个问题:实现和维护半正则表达式的临时实现,以及其他两个。
无论如何,请考虑一下:
import scala.util.matching.Regex
def replaceTemplates(text: String,
templates: Map[String, String]): String =
"""\{([^{}]*)\}""".r replaceSomeIn ( text, { case Regex.Groups(name) => templates get name } )
它使用字符串构建器进行搜索和替换。该地图使用String
而不是Symbol
,因为它更快,并且代码不会替换没有有效映射的匹配。使用replaceAllIn
可以避免这种情况,但需要一些类型注释,因为该方法已经过载。
您可能希望从Regex
的scaladoc API中浏览Scala的源代码,看看发生了什么。
答案 4 :(得分:6)
Torbjørns的答案非常好听。使用butlast摆脱双反转,以及字符串/连接而不是apply'ing str可能会很好。另外使用地图作为功能。 因此,clojure代码可以进一步缩短为:
(defn replace-template [text m]
(clojure.string/replace text #"\{\w+\}"
(comp m keyword clojure.string/join butlast rest)))
答案 5 :(得分:1)
我不知道Clojure,所以我只能说Scala:
foreach-loop很慢,因为你在每个循环周期中遍历整个String。这可以通过先搜索模板然后再替换它们来改进。此外,数据应始终附加到StringBuilder。这是因为每次在StringBuilder内部替换某些内容时,新内容和StringBuilder的结尾都会被复制到新的Chars数组中。
def replaceTemplates(s: String, templates: Map[String, String]): String = {
type DataList = List[(Int, String, Int)]
def matchedData(from: Int, l: DataList): DataList = {
val end = s.lastIndexOf("}", from)
if (end == -1) l
else {
val begin = s.lastIndexOf("{", end)
if (begin == -1) l
else {
val template = s.substring(begin, end+1)
matchedData(begin-1, (begin, template, end+1) :: l)
}
}
}
val sb = new StringBuilder(s.length)
var prev = 0
for ((begin, template, end) <- matchedData(s.length, Nil)) {
sb.append(s.substring(prev, begin))
val ident = template.substring(1, template.length-1)
sb.append(templates.getOrElse(ident, template))
prev = end
}
sb.append(s.substring(prev, s.length))
sb.toString
}
或使用RegEx(更短但更慢):
def replaceTemplates(s: String, templates: Map[String, String]): String = {
val sb = new StringBuilder(s.length)
var prev = 0
for (m <- """\{.+?\}""".r findAllIn s matchData) {
sb.append(s.substring(prev, m.start))
val ms = m.matched
val ident = ms.substring(1, ms.length-1)
sb.append(templates.getOrElse(ident, ms))
prev = m.end
}
sb.append(s.substring(prev, s.length))
sb.toString
}
答案 6 :(得分:1)
Regex + replaceAllIn + Fold:
val template = "Hello #{name}!"
val replacements = Map( "name" -> "Aldo" )
replacements.foldLeft(template)((s:String, x:(String,String)) => ( "#\\{" + x._1 + "\\}" ).r.replaceAllIn( s, x._2 ))