自写 html 闭合的一个 Bug

galeki posted @ 2008-06-04 01:22PM in Ruby&Rails进修

前阵子拿自己写的那个 html 闭合函数，洋洋得意的替换掉了 RubyfulSoup 那个大库，结果却没有发现隐藏在其中的一个 Bug。

当初虽然找了不少 html 样本进行测试，最后却忘记测试 self-close 标签存在的情况，多亏了獨立的圓帮我找出了这个问题。

Bug 就是如果 html 中有 <br/> <img ... /> 这样的标签，闭合函数就会错误的再追加 </br> </img> 这样的标签在简介末尾，要不是导致獨立的圓的 Blog 格式混乱，不知道什么时候才能发现这个问题。

知道了问题所在，修正就很简单了，下面就是改善过的版本，不知道还有没有没有考虑到的地方，高亮部分是改动的部分：

def close_html(html)
    stream = html.scan( /<\s*[\/]*[^>^\/]+>/ ).map { |x| x[ /<\s*([^>^\s]* )[^>]*>/, 1]}
    stack = []
    stream.each do |x|
        if x =~ /^\//
            stack.pop if x[1..-1] == stack.last
        else
            stack.push x
        end
    end
    stack.reverse.each {|x| html << "</#{x}>\n"}
    html
end