看了v2ex的源码，发现一个问题

› Remote Worker

This topic created in 5624 days ago, the information mentioned may be changed or developed.

在filters.py里
def clly(value):
imgs = re.findall('(http://cl.ly/[a-zA-Z0-9]+)\s?', value)
if (len(imgs) > 0):
for img in imgs:
img_id = re.findall('http://cl.ly/([a-zA-Z0-9]+)', img)
if (img_id[0] != 'demo' and img_id[0] != 'whatever'):
value = value.replace('http://cl.ly/' + img_id[0], '<a href="http://cl.ly/' + img_id[0] + '" target="_blank"><img src="http://cl.ly/' + img_id[0] + '/content" class="imgly" border="0" /></a>')
return value
else:
return value
register.filter(clly)
对图片的解析是替换的，如果同时放两张同样的图片地址，估计会有问题。
用了一张坛子里的图片试一下

" target="_blank">

/content" class="imgly" border="0" />

" target="_blank">

/content" class="imgly" border="0" />

value

img

imgs

12 replies • 1970-01-01 08:00:00 +08:00

evenkevin

Dec 27, 2010

这里因为是replace，所以生成的img html里的链接也会被循环调用了.

reorx

Dec 27, 2010

每一个链接都是在for循环中单独处理的，个人认为不会出问题。

evenkevin

Dec 27, 2010

但事实上我只在帖子里放了两个链接，但是出来四张图片，而生成的html里有6个img的tag

vayn

Dec 27, 2010

出现两张同样的图片会反复替换，将替换过的链接里的 Url 再次替换才会变成这个样子。确实是个 bug。

Livid

MOD

PRO

Dec 27, 2010 via iPhone

用户在什么情况下会需要贴两个一模一样的 URL？

我觉得这是贴的人的问题，没有必要为了这种人为制造出来的“bug”而增加实现的复杂度。

evenkevin

Dec 27, 2010

你的逻辑也没问题，只是在无意中发现的问题，告知一下。

xinzhi

Dec 27, 2010

有一次发很多图片的时候不小心弄重复了，结果帖子一塌糊涂。

reorx

Dec 27, 2010

@evenkevin 似乎是这样的，学习了。

lepture

Dec 27, 2010

re.sub is better.

evenkevin

Dec 27, 2010

这个也挺好解决，遍历时对于重复的照片只循环一次，下面是个解决方案，可能是个笨办法...
if (len(imgs) > 0):
for img in imgs:
repeat_num = imgs.count(img)
if repeat_num > 1:
if img_repeat.has_key(img):
continue
else:
img_repeat[img] = repeat_num
value = value.replace('http://cl.ly/' + img_id[0], '<a href="http://cl.ly/' + img_id[0] + '" target="_blank"><img src="http://cl.ly/' + img_id[0] + '/content" class="imgly" border="0" /></a>')
return value
....

darcy

Dec 27, 2010

在某些文章中两个地方引用同一张图片是正常的用途，这个不是bug，是由使用者决定的。

Dec 27, 2010

http://v2ex.appspot.com/t/6401

这里写了个demo，也许可以解决楼主的问题。